diff mbox series

[v2,1/2] dt-bindings: i2c-mux-gpmux: Add property for settle time

Message ID 20211028125341.2457171-2-horatiu.vultur@microchip.com
State Superseded
Headers show
Series i2c-mux-gpmux: Support settle-time-us property | expand

Commit Message

Horatiu Vultur Oct. 28, 2021, 12:53 p.m. UTC
Some HW requires some time for the signals to settle after the muxing is
changed. Allow this time to be specified in device tree.

Signed-off-by: Horatiu Vultur <horatiu.vultur@microchip.com>
---
 Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.yaml | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Peter Rosin Oct. 30, 2021, 10:31 p.m. UTC | #1
On 2021-10-28 14:53, Horatiu Vultur wrote:
> Some HW requires some time for the signals to settle after the muxing is
> changed. Allow this time to be specified in device tree.
> 
> Signed-off-by: Horatiu Vultur <horatiu.vultur@microchip.com>

Maybe this belongs in the i2c-mux binding? Because next week it's someone
with some other i2c-mux that needs the signals so settle. Please include
some language that maybe not all driver support this optional binding etc.
For reference, compare with the optional properties in the i2c binding.

Cheers,
Peter
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.yaml b/Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.yaml
index 9b0603a72f40..4fef5aad360b 100644
--- a/Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.yaml
+++ b/Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.yaml
@@ -67,6 +67,11 @@  properties:
       The properties of mux-locked and parent-locked multiplexers are discussed
       in more detail in Documentation/i2c/i2c-topology.rst.
 
+  settle-time-us:
+    default: 0
+    description:
+      The time required for the signals to settle.
+
 required:
   - compatible
   - i2c-parent