| Message ID | 20220101182806.19311-3-laurent.pinchart+renesas@ideasonboard.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | media: i2c: max9286: Small new features | expand |
On Sat, Jan 01, 2022 at 08:27:57PM +0200, Laurent Pinchart wrote: > The I2C speed on the remote side (the I2C master bus of the connected > serializers) is configurable, and doesn't need to match the speed of the > local bus (the slave bus of the MAX9286). All remote buses must use the > same speed, and the MAX9286 needs to be programmed accordingly. Add a > new DT property to select the speed to make it configurable. > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > --- > .../devicetree/bindings/media/i2c/maxim,max9286.yaml | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml > index c20557b52e45..5d3e99027a79 100644 > --- a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml > +++ b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml > @@ -50,6 +50,13 @@ properties: > '#gpio-cells': > const: 2 > > + maxim,i2c-clock-frequency: Use '-hz'. I don't see much reason to align with 'clock-frequency'. Actually, I'd make this 'maxim,i2c-remote-bus-hz' or similar to be a bit more self-describing. > + enum: [ 8470, 28300, 84700, 105000, 173000, 339000, 533000, 837000 ] > + default: 105000 > + description: | > + The I2C clock frequency for the remote I2C buses. The value must match > + the configuration of the remote serializers. > + > maxim,reverse-channel-microvolt: > minimum: 30000 > maximum: 200000 > -- > Regards, > > Laurent Pinchart > >
diff --git a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml index c20557b52e45..5d3e99027a79 100644 --- a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml +++ b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml @@ -50,6 +50,13 @@ properties: '#gpio-cells': const: 2 + maxim,i2c-clock-frequency: + enum: [ 8470, 28300, 84700, 105000, 173000, 339000, 533000, 837000 ] + default: 105000 + description: | + The I2C clock frequency for the remote I2C buses. The value must match + the configuration of the remote serializers. + maxim,reverse-channel-microvolt: minimum: 30000 maximum: 200000
The I2C speed on the remote side (the I2C master bus of the connected serializers) is configurable, and doesn't need to match the speed of the local bus (the slave bus of the MAX9286). All remote buses must use the same speed, and the MAX9286 needs to be programmed accordingly. Add a new DT property to select the speed to make it configurable. Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> --- .../devicetree/bindings/media/i2c/maxim,max9286.yaml | 7 +++++++ 1 file changed, 7 insertions(+)