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
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 quotesquoted
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