Thread (15 messages) 15 messages, 3 authors, 2023-02-15

Re: [PATCH v1 2/4] dt-bindings: phy: Add starfive,jh7110-dphy-rx

From: Changhuang Liang <changhuang.liang@starfivetech.com>
Date: 2023-02-15 01:28:08
Also in: linux-phy, linux-riscv, lkml


On 2023/2/14 15:49, Krzysztof Kozlowski wrote:
On 14/02/2023 04:07, Changhuang Liang wrote:
quoted
OK, will drop quotes
quoted
quoted
+    items:
+      items:
+        - description: phandle of AON SYSCON
+        - description: register offset
+    description: The register of dphy rx driver can be configured
+      by AON SYSCON in this property.
Can be? So does not have to? But you made it a required property....
Maybe I described it wrong.
I will change to:
	description: The power of dphy rx can be configured by AON SYSCON
	  in this property.
It is like AON SYSCON is the power switch of the dphy rx, it is necessary to
configure the AON SYSCON register, so I made it a required property.
Then do not use "can be". Can is optional: I can jump but I don't have
to. :)
OK, "can be" will be changed to "is".

Best Regards,
Changhuang Liang
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help