Re: [PATCH 03/22] dt-bindings: sun6i-a31-mipi-dphy: Add optional direction property

From
Paul Kocialkowski <hidden>
Date
2021-09-14 07:40:24
Also in
linux-arm-kernel, linux-clk, linux-media, linux-phy, linux-staging, linux-sunxi, lkml
Reply to
Maxime Ripard, 2021-09-13
Hi,

On Mon 13 Sep 21, 10:00, Maxime Ripard wrote:
On Fri, Sep 10, 2021 at 08:41:28PM +0200, Paul Kocialkowski wrote:
quoted
The Allwinner A31 MIPI D-PHY block supports both tx and rx directions,
although each instance of the block is meant to be used in one
direction only. There will typically be one instance for MIPI DSI and
one for MIPI CSI-2 (it seems unlikely to ever see a shared instance).

Describe the direction with a new allwinner,direction property.
For backwards compatibility, the property is optional and tx mode
should be assumed by default.

Signed-off-by: Paul Kocialkowski <redacted>
---
 .../bindings/phy/allwinner,sun6i-a31-mipi-dphy.yaml  | 12 ++++++++++++
 1 file changed, 12 insertions(+)
diff --git a/Documentation/devicetree/bindings/phy/allwinner,sun6i-a31-mipi-dphy.yaml b/Documentation/devicetree/bindings/phy/allwinner,sun6i-a31-mipi-dphy.yaml
index d0b541a461f3..303bbaf3b915 100644
--- a/Documentation/devicetree/bindings/phy/allwinner,sun6i-a31-mipi-dphy.yaml
+++ b/Documentation/devicetree/bindings/phy/allwinner,sun6i-a31-mipi-dphy.yaml
@@ -37,6 +37,18 @@ properties:
   resets:
     maxItems: 1
 
+  allwinner,direction:
+    $ref: '/schemas/types.yaml#/definitions/string'
+    description: |
+      Direction of the D-PHY:
+      - "rx" for receiving (e.g. when used with MIPI CSI-2);
+      - "tx" for transmitting (e.g. when used with MIPI DSI).
+
+      When the property is missing, "tx" direction is assumed.
You can provide this using default
Understood.
quoted
+    oneOf:
+      - const: tx
+      - const: rx
+
enum?
Ah yes, enum feels like a better fit!

Thanks,

Paul

-- 
Paul Kocialkowski, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com

Attachments

Thread (48 messages) 48 messages, 6 authors, last reply 2021-09-16
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help