On Wed, Oct 11, 2023 at 04:53:34PM +0100, Conor Dooley wrote: > Hey, > > On Tue, Oct 10, 2023 at 05:12:07PM +0200, Jacopo Mondi wrote: > > Add bindings for OmniVision OV64A40. > > > > Signed-off-by: Jacopo Mondi <jacopo.mondi@xxxxxxxxxxxxxxxx> > > Signed-off-by: Lee Jackson <lee.jackson@xxxxxxxxxxx> > > What does Lee's SoB indicate here? > Lee has contributed to the development of the driver and validation of bindings. > > diff --git a/MAINTAINERS b/MAINTAINERS > > index b19995690904..df089d68b58c 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -15821,6 +15821,13 @@ S: Maintained > > T: git git://linuxtv.org/media_tree.git > > F: drivers/media/i2c/ov5695.c > > > > +OMNIVISION OV64A40 SENSOR DRIVER > > +M: Jacopo Mondi <jacopo.mondi@xxxxxxxxxxxxxxxx> > > +L: linux-media@xxxxxxxxxxxxxxx > > +S: Maintained > > +T: git git://linuxtv.org/media_tree.git > > Binding looks fine to me, my question is here. Usually having a tree > here means that you apply the patches yourself. Do you? > No, and only Mauro has commit rights on the media tree. All i2c sensor drivers have a tree listed, regardless who commits there. What should I put there ? > Cheers, > Conor. > > > +F: Documentation/devicetree/bindings/media/i2c/ovti,ov64a40.yaml > > + > > OMNIVISION OV7670 SENSOR DRIVER > > L: linux-media@xxxxxxxxxxxxxxx > > S: Orphan > > -- > > 2.42.0 > >
Attachment:
signature.asc
Description: PGP signature
