Re: [PATCH v4 2/5] dt-bindings: media: Document BCM283x CSI2/CCP2 receiver

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

 



On Thu, Dec 03, 2020 at 06:50:27PM +0100, Jacopo Mondi wrote:
> Hi Sakari,
> 
> On Thu, Dec 03, 2020 at 07:41:02PM +0200, Sakari Ailus wrote:
> 
> [snip]
> 
> > >
> > > Ah! maximum: 8. I was about to reply to Rob's video-interfaces
> > > conversion and this might need to be pointed out.
> > > I was also about to ask it it's an issue if the bindings validation
> > > does not catch arrays as: [1, 3] as the constraint of being
> > > monotonically increasing values is expressed in words only. I guess
> > > it's fine, the alternative syntax is awful:
> > >
> > >             anyOf:
> > >               - items:
> > >                 - const: 1
> > >                 - const: 2
> > >               - items:
> > >                 - const: 1
> > >                 - const: 2
> > >                 - const: 3
> > >                 - const: 4
> >
> > The v4l2-fwnode framework doesn't do that, what it does check for, though,
> 
> I meant the dt-validation :)

I think it'd be very nice to have that check there, too. :-)

-- 
Sakari Ailus



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux