[U-Boot] [PATCH v3 5/7] arm: dts: armada-cp110-*dtsi: add xmdio nodes

Ramon Fried rfried.dev at gmail.com
Wed Aug 7 17:54:07 UTC 2019


On Tue, Aug 6, 2019 at 6:52 PM <nhed+uboot at starry.com> wrote:
>
> From: Nevo Hed <nhed+github at 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 at 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 at 12a600 {
> +                               #address-cells = <1>;
> +                               #size-cells = <0>;
> +                               compatible = "marvell,xmdio";
> +                               reg = <0x12a600 0x16>;
> +                               status = "disabled";
> +                               device-name = "cpm-xmdio";
> +                       };
> +
>                         cpm_syscon0: system-controller at 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 at 12a600 {
> +                               #address-cells = <1>;
> +                               #size-cells = <0>;
> +                               compatible = "marvell,xmdio";
> +                               reg = <0x12a600 0x16>;
> +                               status = "disabled";
> +                               device-name = "cps-xmdio";
> +                       };
> +
>                         cps_syscon0: system-controller at 440000 {
>                                 compatible = "marvell,cp110-system-controller0",
>                                              "syscon";
> --
> 2.21.0
>
Reviewed-By: Ramon Fried <rfried.dev at gmail.com>


More information about the U-Boot mailing list