Thread (63 messages) 63 messages, 6 authors, 2024-02-29

Re: [PATCH 02/13] dt-bindings: i2c: nomadik: add mobileye,eyeq5-i2c bindings and example

From: Krzysztof Kozlowski <hidden>
Date: 2024-02-17 08:25:05
Also in: linux-devicetree, linux-i2c, linux-mips, lkml

On 16/02/2024 11:40, Théo Lebrun wrote:
Hello,

On Fri Feb 16, 2024 at 11:33 AM CET, Krzysztof Kozlowski wrote:
quoted
On 16/02/2024 11:18, Théo Lebrun wrote:
quoted
quoted
quoted
+        mobileye,id:
+          $ref: /schemas/types.yaml#/definitions/uint32
+          description: Platform-wide controller ID (integer starting from zero).
instance indexes are a NAK. You can use i2cN aliases if you must.

Why do you need it? To access OLB? If so, add cell args to the OLB 
phandle instead.
Why we do what we do: I2C controller must write a 2 bit value depending
on the bus speed. All I2C controllers write into the same register.
Which register?  Your devices do not share IO address space.
mobileye,olb is a prop with a phandle to a syscon. That syscon contains
the register we are interested in.
So exactly what Rob said... I don't understand why you have chosen to go
with alias.


Best regards,
Krzysztof


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help