diff mbox series

[17/19] media: dt-bindings: ti,ds90ub960: Add "i2c-addr" link property

Message ID 20250110-ub9xx-improvements-v1-17-e0b9a1f644da@ideasonboard.com
State New
Headers show
Series media: i2c: ds90ub9xx: Error handling, UB9702 improvements | expand

Commit Message

Tomi Valkeinen Jan. 10, 2025, 9:14 a.m. UTC
From: Jai Luthra <jai.luthra@ideasonboard.com>

The serializer's I2C address on the FPD-Link bus is usually communicated
to the deserializer once the forward-channel is established. But in some
cases it might be necessary to program the serializer (over the
back-channel) before the forward-channel is established.

This can be used e.g. to correct serializer configuration which
otherwise would prevent the FC to be enabled.

Add a new optional property to specify the I2C address of the
serializer.

Signed-off-by: Jai Luthra <jai.luthra@ideasonboard.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Cc: devicetree@vger.kernel.org
Cc: Rob Herring <robh@kernel.org>
Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
---
 Documentation/devicetree/bindings/media/i2c/ti,ds90ub960.yaml | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Konstantin Ryabitsev Jan. 15, 2025, 3:53 p.m. UTC | #1
On Tue, Jan 14, 2025 at 01:50:08PM +0200, Tomi Valkeinen wrote:
> > If you need it for your own patch management purposes, keep it under the
> > --- separator.
> 
> I'm using b4. I don't know how to do that with b4, but I'll look into it.

You just put them under --- in your git commit message and b4 will preserve
it as-is when it renders the email.

-K
Tomi Valkeinen Jan. 15, 2025, 4:09 p.m. UTC | #2
Hi,

On 15/01/2025 17:53, Konstantin Ryabitsev wrote:
> On Tue, Jan 14, 2025 at 01:50:08PM +0200, Tomi Valkeinen wrote:
>>> If you need it for your own patch management purposes, keep it under the
>>> --- separator.
>>
>> I'm using b4. I don't know how to do that with b4, but I'll look into it.
> 
> You just put them under --- in your git commit message and b4 will preserve
> it as-is when it renders the email.

Alright, thanks!

  Tomi
Tomi Valkeinen Jan. 15, 2025, 5:19 p.m. UTC | #3
Hi,

On 15/01/2025 10:40, Krzysztof Kozlowski wrote:

>>> Why only these folks? Why not all of the maintainers?
>>
>> The whole series is sent to the media list and maintainers. I thought
>> this single patch doesn't warrant sending the whole series to DT list
>> and maintainers, so I cc'd them here.
> 
> 
> I was wondering why only some of the DT maintainers, not all? My usual
> assumption is: you are not using get_maintainers.pl or you are working
> on an old kernel.

It's simpler than that: a copy-paste mistake. I'm not sure how I managed 
to miss Conor there.

Any preference on cc'ing the DT maintainers and the dt-list only for 
this patch (I'll use --- this time, I promise!), or just sending the 
whole series also to DT people?

  Tomi
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/media/i2c/ti,ds90ub960.yaml b/Documentation/devicetree/bindings/media/i2c/ti,ds90ub960.yaml
index 0b71e6f911a8..e17b508b6409 100644
--- a/Documentation/devicetree/bindings/media/i2c/ti,ds90ub960.yaml
+++ b/Documentation/devicetree/bindings/media/i2c/ti,ds90ub960.yaml
@@ -75,6 +75,13 @@  properties:
               address on the I2C bus where the deserializer resides are
               forwarded to the serializer.
 
+          i2c-addr:
+            $ref: /schemas/types.yaml#/definitions/uint32
+            description:
+              The strap I2C address of the serializer. This is used by the
+              deserializer to internally communicate to the serializer over the
+              FPD-Link back-channel.
+
           ti,rx-mode:
             $ref: /schemas/types.yaml#/definitions/uint32
             enum: