On Fri, Apr 02, 2021 at 12:01:35PM +0300, Laurent Pinchart wrote:
> Hi Mauro,
> 
> Thank you for the patch.
> 
> On Thu, Apr 01, 2021 at 02:17:44PM +0200, Mauro Carvalho Chehab wrote:
> > The file name: Documentation/devicetree/bindings/media/i2c/rdacm2x-gmsl.yaml
> > should be, instead: 
> > Documentation/devicetree/bindings/media/i2c/imi,rdacm2x-gmsl.yaml.
> 
> While at it, shouldn't we also rename the file to imi,rdacm2x.yaml ?

I spoke too fast, sorry. There are RDACM2x cameras that use FPD-Link III
(RDACM23, 24, 25, 28, 29).

> > Update its cross-reference accordingly.
> > 
> > Fixes: 34009bffc1c6 ("media: i2c: Add RDACM20 driver")
> > Fixes: e9f817689789 ("media: dt-bindings: media: i2c: Add bindings for IMI 
> > RDACM2x")
> > Signed-off-by: Mauro Carvalho Chehab <mchehab+hua...@kernel.org>

Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>

> > ---
> >  MAINTAINERS | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 1644b6e9697c..b405ee71f730 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -15258,7 +15258,7 @@ M:  Laurent Pinchart 
> > <laurent.pinchart+rene...@ideasonboard.com>
> >  M: Niklas Söderlund <niklas.soderlund+rene...@ragnatech.se>
> >  L: linux-me...@vger.kernel.org
> >  S: Maintained
> > -F: Documentation/devicetree/bindings/media/i2c/rdacm2x-gmsl.yaml
> > +F: Documentation/devicetree/bindings/media/i2c/imi,rdacm2x-gmsl.yaml
> >  F: drivers/media/i2c/max9271.c
> >  F: drivers/media/i2c/max9271.h
> >  F: drivers/media/i2c/rdacm21.c

-- 
Regards,

Laurent Pinchart

Reply via email to