| Message ID | 20200416170321.4216-1-jbx6244@gmail.com (mailing list archive) |
|---|---|
| State | Mainlined |
| Commit | 287e0d538fcec2f6e8eb1e565bf0749f3b90186d |
| Headers | show |
| Series | [1/2] ARM: dts: rockchip: fix phy nodename for rk3228-evb | expand |
On Thu, 16 Apr 2020 19:03:20 +0200, Johan Jonker wrote: > A test with the command below gives for example this error: > > arch/arm/boot/dts/rk3228-evb.dt.yaml: phy@0: > '#phy-cells' is a required property > > The phy nodename is normally used by a phy-handle. > This node is however compatible with > "ethernet-phy-id1234.d400", "ethernet-phy-ieee802.3-c22" > which is just been added to 'ethernet-phy.yaml'. > So change nodename to 'ethernet-phy' for which '#phy-cells' > is not a required property > > [...] Applied, thanks! [1/2] ARM: dts: rockchip: fix phy nodename for rk3228-evb commit: 806bceeea63f58bde45dbd633092bd9d14da02b0 [2/2] ARM: dts: rockchip: fix phy nodename for rk3229-xms6 commit: 5c5555f7efed273c61152da03cd395a0d06e054c Best regards,
diff --git a/arch/arm/boot/dts/rk3228-evb.dts b/arch/arm/boot/dts/rk3228-evb.dts index 5670b33fd..aed879db6 100644 --- a/arch/arm/boot/dts/rk3228-evb.dts +++ b/arch/arm/boot/dts/rk3228-evb.dts @@ -46,7 +46,7 @@ #address-cells = <1>; #size-cells = <0>; - phy: phy@0 { + phy: ethernet-phy@0 { compatible = "ethernet-phy-id1234.d400", "ethernet-phy-ieee802.3-c22"; reg = <0>; clocks = <&cru SCLK_MAC_PHY>;
A test with the command below gives for example this error: arch/arm/boot/dts/rk3228-evb.dt.yaml: phy@0: '#phy-cells' is a required property The phy nodename is normally used by a phy-handle. This node is however compatible with "ethernet-phy-id1234.d400", "ethernet-phy-ieee802.3-c22" which is just been added to 'ethernet-phy.yaml'. So change nodename to 'ethernet-phy' for which '#phy-cells' is not a required property make ARCH=arm dtbs_check DT_SCHEMA_FILES=~/.local/lib/python3.5/site-packages/dtschema/schemas/ phy/phy-provider.yaml Signed-off-by: Johan Jonker <jbx6244@gmail.com> --- arch/arm/boot/dts/rk3228-evb.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)