On Tue, Aug 6, 2019 at 6:52 PM nhed+uboot@starry.com wrote:
From: Nevo Hed nhed+github@starry.com
Based on upstream-linux See https://github.com/torvalds/linux/commit/f66b2aff.
However made the XSMI register window 0x16 (22) bytes per my reading of the functional spec. Similar commits in Marvels own repo bump it to 0x200 (512) bytes but I did not see the reasoning for that.
https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/commit/4d932b4.
Also added device-name attributes to prevent ambiguity in the `mdio` command.
Signed-off-by: Nevo Hed nhed+github@starry.com
arch/arm/dts/armada-cp110-master.dtsi | 9 +++++++++ arch/arm/dts/armada-cp110-slave.dtsi | 9 +++++++++ 2 files changed, 18 insertions(+)
diff --git a/arch/arm/dts/armada-cp110-master.dtsi b/arch/arm/dts/armada-cp110-master.dtsi index e4c17e9f4b..cd5c974482 100644 --- a/arch/arm/dts/armada-cp110-master.dtsi +++ b/arch/arm/dts/armada-cp110-master.dtsi @@ -99,6 +99,15 @@ device-name = "cpm-mdio"; };
cpm_xmdio: mdio@12a600 {#address-cells = <1>;#size-cells = <0>;compatible = "marvell,xmdio";reg = <0x12a600 0x16>;status = "disabled";device-name = "cpm-xmdio";};cpm_syscon0: system-controller@440000 { compatible = "marvell,cp110-system-controller0", "syscon";diff --git a/arch/arm/dts/armada-cp110-slave.dtsi b/arch/arm/dts/armada-cp110-slave.dtsi index 2fbd7b5514..b426a4eb69 100644 --- a/arch/arm/dts/armada-cp110-slave.dtsi +++ b/arch/arm/dts/armada-cp110-slave.dtsi @@ -99,6 +99,15 @@ device-name = "cps-mdio"; };
cps_xmdio: mdio@12a600 {#address-cells = <1>;#size-cells = <0>;compatible = "marvell,xmdio";reg = <0x12a600 0x16>;status = "disabled";device-name = "cps-xmdio";};cps_syscon0: system-controller@440000 { compatible = "marvell,cp110-system-controller0", "syscon";-- 2.21.0
Reviewed-By: Ramon Fried rfried.dev@gmail.com