Hi Dave, Thank you for the patch. On Tue, Jan 31, 2023 at 07:06:59PM +0000, Dave Stevenson wrote: > The IMX290 module is available as either monochrome or colour and > the variant is not detectable at runtime. > > Add a new compatible string for the monochrome version. > > Signed-off-by: Dave Stevenson <dave.stevenson@xxxxxxxxxxxxxxx> > --- > .../devicetree/bindings/media/i2c/sony,imx290.yaml | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml > index 21377daae026..29ca4052591f 100644 > --- a/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml > +++ b/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml > @@ -12,15 +12,17 @@ maintainers: > > description: |- > The Sony IMX290 is a 1/2.8-Inch CMOS Solid-state image sensor with Square > - Pixel for Color Cameras. It is programmable through I2C and 4-wire > - interfaces. The sensor output is available via CMOS logic parallel SDR > - output, Low voltage LVDS DDR output and CSI-2 serial data output. The CSI-2 > + Pixel, available in either mono or colour variants. > + It is programmable through I2C and 4-wire interfaces. > + The sensor output is available via CMOS logic parallel SDR output, Low > + voltage LVDS DDR output and CSI-2 serial data output. The CSI-2 > bus is the default. No bindings have been defined for the other busses. Nitpicking, could you reflow the text to the 80 columns limit, or add a blank line between what you consider to be independent paragraphs ? > > properties: > compatible: > enum: > - sony,imx290 > + - sony,imx290-mono The monochrome version is called IMX290LLR, while the colour version is the IMX290LQR. Could we use sony,imx290llr (or possibly sony,imx290ll) as a compatible string instead of sony,imx290-mono ? We could also deprecate sony,imx290 in favour of sony,imx290lqr (or sony,imx290lq) if desired, and I can handle this on top. > > reg: -- Regards, Laurent Pinchart
