On Tue, Feb 11, 2025 at 06:46:10PM +0000, Conor Dooley wrote: > On Fri, Feb 07, 2025 at 01:29:55PM +0200, Laurentiu Palcu wrote: > > Add 'maxim,override-mode' property to allow the user to toggle the pin > > configured chip operation mode and 'maxim,fsync-config' to configure the > > chip for relaying a frame synchronization signal, received from > > deserializer, to the attached sensor. The latter is needed for > > synchronizing the images in multi-sensor setups. > > > > Signed-off-by: Laurentiu Palcu <laurentiu.palcu@xxxxxxxxxxx> > > --- > > .../bindings/media/i2c/maxim,max96717.yaml | 28 +++++++++++++++++++ > > 1 file changed, 28 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/media/i2c/maxim,max96717.yaml b/Documentation/devicetree/bindings/media/i2c/maxim,max96717.yaml > > index d1e8ba6e368ec..fae578d55fd4d 100644 > > --- a/Documentation/devicetree/bindings/media/i2c/maxim,max96717.yaml > > +++ b/Documentation/devicetree/bindings/media/i2c/maxim,max96717.yaml > > @@ -42,10 +42,35 @@ properties: > > number must be in range of [0, 10]. > > > > gpio-controller: true > > + gpio-reserved-ranges: true > > > > '#clock-cells': > > const: 0 > > > > + maxim,override-mode: > > + description: Toggle the operation mode from the pin configured one. > > + type: boolean > > type: flag Err, no. You can do as-is or: $ref: /schemas/types.yaml#/definitions/flag I am neutral as to which way. If I wasn't we'd make the meta-schema enforce one way or the other. Rob
