From mboxrd@z Thu Jan  1 00:00:00 1970
From: Lothar Felten <lothar.felten@gmail.com>
Date: Mon, 23 Apr 2018 19:39:56 +0200
Subject: [U-Boot] [PATCH v2 5/5] configs: Bananapi_M2_Ultra: enable
 gigabit on the Bananapi M2U
In-Reply-To: <20180419084119.wvbrsmksvygbbm5b@flea>
References: <20180417134235.10502-1-lothar.felten@gmail.com>
 <20180417134235.10502-5-lothar.felten@gmail.com>
 <20180419084119.wvbrsmksvygbbm5b@flea>
Message-ID: <1524505196.6569.37.camel@gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Thu, 2018-04-19 at 10:41 +0200, Maxime Ripard wrote:
> On Tue, Apr 17, 2018 at 03:42:35PM +0200, Lothar Felten wrote:
> > Enable the gigabit ethernet for the Bananapi M2 Ultra board.
> > Tested on BananaPi M2 Berry (R40), custom board (V40).
> > 
> > Signed-off-by: Lothar Felten <lothar.felten@gmail.com>
> > ---
> >  arch/arm/dts/sun8i-r40-bananapi-m2-ultra.dts | 14 ++++++++++++++
> >  configs/Bananapi_M2_Ultra_defconfig          |  4 ++++
> >  2 files changed, 18 insertions(+)
> > 
> > diff --git a/arch/arm/dts/sun8i-r40-bananapi-m2-ultra.dts
> > b/arch/arm/dts/sun8i-r40-bananapi-m2-ultra.dts
> > index ab471ab0bf..25f2112fbf 100644
> > --- a/arch/arm/dts/sun8i-r40-bananapi-m2-ultra.dts
> > +++ b/arch/arm/dts/sun8i-r40-bananapi-m2-ultra.dts
> > @@ -67,3 +67,17 @@
> >  	pinctrl-0 = <&uart0_pb_pins>;
> >  	status = "okay";
> >  };
> > +
> > +&gmac {
> > +	pinctrl-names = "default";
> > +	pinctrl-0 = <&gmac_pins_rgmii>;
> > +	status = "okay";
> > +	phy-handle = <&rgmii_phy>;
> > +};
> > +
> > +&mdio {
> > +	rgmii_phy: ethernet-phy at 1 {
> > +		compatible = "ethernet-phy-ieee802.3-c22";
> > +		reg = <1>;
> > +	};
> > +};
> > diff --git a/configs/Bananapi_M2_Ultra_defconfig
> > b/configs/Bananapi_M2_Ultra_defconfig
> > index 37cc2df5dc..e4e943dc49 100644
> > --- a/configs/Bananapi_M2_Ultra_defconfig
> > +++ b/configs/Bananapi_M2_Ultra_defconfig
> > @@ -17,3 +17,7 @@ CONFIG_AXP_DLDO4_VOLT=2500
> >  CONFIG_AXP_ELDO3_VOLT=1200
> >  CONFIG_SCSI=y
> >  CONFIG_SYS_USB_EVENT_POLL_VIA_INT_QUEUE=y
> > +CONFIG_SUN8I_EMAC=y
> > +CONFIG_RGMII=y
> > +CONFIG_SUN7I_GMAC=y
> 
> I'm not sure why you'd need both?

I used the SUN8I driver and SUN7I GMAC config approach.
In the new patch v3 I cleaned this up: the config is now in the driver,
and sysconf optional for R40 SoCs.

Lothar

> 
> Maxime
>