RE: [PATCH 2/2] dt-bindings: dma: renesas,usb-dmac: convert bindings to json-schema

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Geert-san,

> From: Geert Uytterhoeven, Sent: Thursday, April 16, 2020 4:50 PM
> 
> Hi Shimoda-san,
> 
> On Thu, Apr 16, 2020 at 4:04 AM Yoshihiro Shimoda
> <yoshihiro.shimoda.uh@xxxxxxxxxxx> wrote:
> > > From: Geert Uytterhoeven, Sent: Wednesday, April 15, 2020 10:56 PM
> > <snip>
> > > > --- /dev/null
> > > > +++ b/Documentation/devicetree/bindings/dma/renesas,usb-dmac.yaml
> > > > +  interrupts:
> > > > +    maxItems: 2
> > >
> > > Is there a use case for specifying a single interrupt?
> >
> > No. All USB-DMAC of R-Car Gen2/3 and RZ/Gn has 2 channels.
> > In case of R-Car Gen3, please refer to the Figure 75.1 USB-DMAC Block Diagram.
> > These USB-DMACn have CH0 and CH1
> 
> Thanks for confirming!
> 
> > > > +  dma-channels:
> > > > +    maximum: 2
> > >
> > > Is there a use case for specifying a single channel?
> > >
> > > > +
> > > > +  iommus:
> > > > +    maxItems: 2
> > >
> > > Likewise?
> >
> > As I mentioned above, there is not a use case for specifying a single channel.
> 
> Hence I think all of these should have "const: 2" or "minItems: 2".

Thank you for the proposal. I got it. I'll fix it.

Best regards,
Yoshihiro Shimoda





[Index of Archives]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux PCI]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux