Re: [PATCH v2 1/2] dt-bindings: phy: ti,tcan104x-can: Document mux-states property
From: Rob Herring <robh@kernel.org>
Date: 2021-12-15 20:40:47
Also in:
linux-devicetree, linux-phy, lkml
On Tue, Dec 14, 2021 at 07:59:07PM +0530, Aswath Govindraju wrote:
quoted hunk ↗ jump to hunk
On some boards, for routing CAN signals from controller to transceivers, muxes might need to be set. This can be implemented using mux-states property. Therefore, document the same in the respective bindings. Signed-off-by: Aswath Govindraju <redacted> --- .../devicetree/bindings/phy/ti,tcan104x-can.yaml | 10 ++++++++++ 1 file changed, 10 insertions(+)diff --git a/Documentation/devicetree/bindings/phy/ti,tcan104x-can.yaml b/Documentation/devicetree/bindings/phy/ti,tcan104x-can.yaml index 6107880e5246..7b9216e43b58 100644 --- a/Documentation/devicetree/bindings/phy/ti,tcan104x-can.yaml +++ b/Documentation/devicetree/bindings/phy/ti,tcan104x-can.yaml@@ -37,6 +37,15 @@ properties: max bit rate supported in bps minimum: 1 + mux-states: + description: + mux controller node to route the signals from controller to + transceiver. Two arguments can be present depending on the + mux chip. If one argument is used then it represents the state + to be set on the mux-chip. If there are two arguments then the + first argument is the control line and the second argument is + its corresponding state to be set, on the mux-chip. +
You are still describing how the mux-states works. What the cells contain and how many are opaque to this binding. Here you need to describe how many muxes you have and what they are controlling as that is what is specific to this binding. If there is only one, this boils down to 'maxItems: 1'. It's just like reg, interrupts, clocks, etc.
quoted hunk ↗ jump to hunk
required: - compatible - '#phy-cells'@@ -53,4 +62,5 @@ examples: max-bitrate = <5000000>; standby-gpios = <&wakeup_gpio1 16 GPIO_ACTIVE_LOW>; enable-gpios = <&main_gpio1 67 GPIO_ACTIVE_HIGH>; + mux-states = <&mux0 1>; };-- 2.17.1