Re: [PATCH v3 1/3] dt-bindings: dma: arm-dma350: document generic and combined IRQ topologies

[Jun Guo]

On 3/20/2026 9:12 PM, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL
>
> On 20/03/2026 11:28, Jun Guo wrote:
> > > > +    description: |
> > > > +      The DMA controller may be configured with separate interrupts for each channel,
> > > > +      or with a single combined interrupt for all channels, depending on the SoC integration.
> > >
> > > And more important - you must review the LLM microslop output before
> > > posting and adjust it to Linux kernel coding style. Don't send
> > > unredacted tool output.
> > Actually, this part of the description was not AI-generated. However,
> > I’d like to confirm the issue you mentioned: are you saying that this
> > description is written too verbosely?
> > Then, do you think there are still issues with the revised version?
> >     interrupts:
> >       minItems: 1
> >       maxItems: 8
> >       description:
> >         Either one interrupt per channel (8 interrupts), or one
> >         combined interrupt for all channels.
>
> No, it is not wrapped according to Linux coding style.
>
> Please wrap code according to the preferred limit expressed in Kernel
> coding style (checkpatch is not a coding style description, but only a
> tool).  However don't wrap blindly (see Kernel coding style).
Thank you for your patience. I have clearly understood your points and 
will incorporate all of your feedback when revising the V3 version of 
the patch.

Best regards,
Jun