[U-Boot] [PATCH] drivers: net: ldpaa_eth: check if the dpmac is enabled

[Pankaj Bansal]

Hi Joe,

Can you please check and ACK this patch?
This patch is related to http://patchwork.ozlabs.org/patch/990131/

Regards,
Pankaj Bansal

> -----Original Message-----
> From: Prabhakar Kushwaha
> Sent: Monday, 11 February, 2019 03:19 PM
> To: Pankaj Bansal <pankaj.bansal at nxp.com>; Joe Hershberger
> <joe.hershberger at ni.com>
> Cc: u-boot at lists.denx.de
> Subject: RE: [PATCH] drivers: net: ldpaa_eth: check if the dpmac is enabled
> 
> Dear Joe,
> 
> > -----Original Message-----
> > From: Pankaj Bansal
> > Sent: Friday, February 8, 2019 2:29 PM
> > To: Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>
> > Cc: u-boot at lists.denx.de; Pankaj Bansal <pankaj.bansal at nxp.com>
> > Subject: [PATCH] drivers: net: ldpaa_eth: check if the dpmac is
> > enabled
> >
> > some dpmacs in armv8a based freescale layerscape SOCs can be
> > configured via both serdes(sgmii, xfi, xlaui etc) bits and via
> > EC*_PMUX(rgmii) bits in RCW.
> > e.g. dpmac 17 and 18 in LX2160A can be configured as SGMII from serdes
> > bits and as RGMII via EC1_PMUX/EC2_PMUX bits Now if a dpmac is enabled
> > by serdes bits then it takes precedence over EC*_PMUX bits. i.e. in
> > LX2160A if we select serdes protocol that configures dpmac17 as SGMII
> > and set the EC1_PMUX as RGMII, then the dpmac is SGMII and not RGMII.
> >
> > Therefore, in fsl_rgmii_init function of SOC, we will check if the
> > dpmac is enabled or not? if it is (fsl_serdes_init has already enabled
> > the dpmac), then don't enable it.
> >
> > Signed-off-by: Pankaj Bansal <pankaj.bansal at nxp.com>
> > ---
> >
> > Notes:
> >     Dependencies:
> >     	- http://patchwork.ozlabs.org/patch/990131/
> >     	- http://patchwork.ozlabs.org/patch/990093/
> >
> >  drivers/net/ldpaa_eth/lx2160a.c | 4 ++--
> 
> Please let me know if you have any concern on this patch.
> 
> I would like pull this patch via u-boot-fsl-qoriq repo..
> 
> --pk