arm: mvebu: dts: Armada8k enable mdio

[François Ozog]

Hi Simon,

That's typically what SystemReady want to avoid/address. The board MDIO has
always been "Okay". Linux may not have been able to deal with it.

So the right way to handle that is:
- there is a board DT "attached" to the board
- TFA adjusts the DRAM aspect (SPL, core boot...)
- OP-TEE adjust Secure DRAM aspects (Trusty, Q-SEE...)
- U-Boot adjusts to the booted OS (turn off MDIO if the booted OS does not
support it) or administrative desires. U-Boot is also in the best position
to deal with hats, capes...
- OS selects what it can handle

This way, there is a single source for DTB, no nightmare to sync up all
involved projects.



On Fri, 10 Sept 2021 at 08:56, Stefan Roese <sr at denx.de> wrote:

> On 24.08.21 10:14, sven.auhagen at voleatech.de wrote:
> > From: Sven Auhagen <Sven.Auhagen at voleatech.de>
> >
> > Since mvpp2 is using the new mdio driver and the cp110 has been
> > synced with the linux upstream, the mdio has to enabled in the
> > device tree file.
> > This is missing for some device tree files and therefore the
> > network cards do not come online.
> >
> > Signed-off-by: Sven Auhagen <sven.auhagen at voleatech.de>
>
> Reviewed-by: Stefan Roese <sr at denx.de>
>
> Thanks,
> Stefan
>
> > ---
> >   arch/arm/dts/armada-7040-db.dts             | 1 +
> >   arch/arm/dts/armada-8040-clearfog-gt-8k.dts | 1 +
> >   arch/arm/dts/armada-8040-db.dts             | 1 +
> >   arch/arm/dts/armada-8040-mcbin.dts          | 1 +
> >   4 files changed, 4 insertions(+)
> >
> > diff --git a/arch/arm/dts/armada-7040-db.dts
> b/arch/arm/dts/armada-7040-db.dts
> > index b158f92349..9104042359 100644
> > --- a/arch/arm/dts/armada-7040-db.dts
> > +++ b/arch/arm/dts/armada-7040-db.dts
> > @@ -175,6 +175,7 @@
> >   };
> >
> >   &cp0_mdio {
> > +     status = "okay";
> >       phy0: ethernet-phy at 0 {
> >               reg = <0>;
> >       };
> > diff --git a/arch/arm/dts/armada-8040-clearfog-gt-8k.dts
> b/arch/arm/dts/armada-8040-clearfog-gt-8k.dts
> > index 6a586dbbba..79ee871c5a 100644
> > --- a/arch/arm/dts/armada-8040-clearfog-gt-8k.dts
> > +++ b/arch/arm/dts/armada-8040-clearfog-gt-8k.dts
> > @@ -295,6 +295,7 @@
> >   };
> >
> >   &cp1_mdio {
> > +     status = "okay";
> >       phy0: ethernet-phy at 0 {
> >               reg = <0>;
> >       };
> > diff --git a/arch/arm/dts/armada-8040-db.dts
> b/arch/arm/dts/armada-8040-db.dts
> > index 51c2f23f4d..2686e00242 100644
> > --- a/arch/arm/dts/armada-8040-db.dts
> > +++ b/arch/arm/dts/armada-8040-db.dts
> > @@ -270,6 +270,7 @@
> >   };
> >
> >   &cp0_mdio {
> > +     status = "okay";
> >       phy1: ethernet-phy at 1 {
> >               reg = <1>;
> >       };
> > diff --git a/arch/arm/dts/armada-8040-mcbin.dts
> b/arch/arm/dts/armada-8040-mcbin.dts
> > index 2184648318..b0bed77ae6 100644
> > --- a/arch/arm/dts/armada-8040-mcbin.dts
> > +++ b/arch/arm/dts/armada-8040-mcbin.dts
> > @@ -155,6 +155,7 @@
> >   };
> >
> >   &cp0_mdio {
> > +     status = "okay";
> >       ge_phy: ethernet-phy at 0 {
> >               reg = <0>;
> >       };
> >
>
>
> Viele Grüße,
> Stefan
>
> --
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr at denx.de
>


-- 
François-Frédéric Ozog | *Director Business Development*
T: +33.67221.6485
francois.ozog at linaro.org | Skype: ffozog