On Wed, Mar 27, 2024 at 05:17:04PM -0600, git@xxxxxxxxxxxx wrote: > From: Dave Stevenson <dave.stevenson@xxxxxxxxxxxxxxx> > > There are a number of variants of the imx258 modules that can not > be differentiated at runtime, so add compatible strings for them. But you are only adding 1 variant. > > Signed-off-by: Dave Stevenson <dave.stevenson@xxxxxxxxxxxxxxx> > Signed-off-by: Luigi311 <git@xxxxxxxxxxxx> > --- > .../devicetree/bindings/media/i2c/sony,imx258.yaml | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx258.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx258.yaml > index bee61a443b23..c7856de15ba3 100644 > --- a/Documentation/devicetree/bindings/media/i2c/sony,imx258.yaml > +++ b/Documentation/devicetree/bindings/media/i2c/sony,imx258.yaml > @@ -14,10 +14,14 @@ description: |- > type stacked image sensor with a square pixel array of size 4208 x 3120. It > is programmable through I2C interface. Image data is sent through MIPI > CSI-2. > + There are a number of variants of the sensor which cannot be detected at > + runtime, so multiple compatible strings are required to differentiate these. That's more reasoning/why for the patch than description of the h/w. > properties: > compatible: > - const: sony,imx258 > + - enum: > + - sony,imx258 > + - sony,imx258-pdaf How do I know which one to use? Please define what PDAF means somewhere as well as perhaps what the original/default variant is or isn't. > > assigned-clocks: true > assigned-clock-parents: true > -- > 2.42.0 >
