From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jagan Teki Date: Tue, 5 Mar 2019 21:46:48 +0530 Subject: [U-Boot] [PATCH v3 03/13] net: sun8i_emac: Retrieve GMAC clock via 'syscon' phandle In-Reply-To: References: <20190227185701.15545-1-jagan@amarulasolutions.com> <20190227185701.15545-4-jagan@amarulasolutions.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Sat, Mar 2, 2019 at 12:29 AM Joe Hershberger wrote: > > On Wed, Feb 27, 2019 at 12:59 PM Jagan Teki wrote: > > > > Unlike other Allwinner SoC's R40 GMAC clock control register > > is locate in CCU, but rest located via syscon itself. Since > > the phandle property for current code look for 'syscon' and > > it will grab the respective ccu or syscon base address based > > on DT property defined in respective SoC dtsi. > > > > So, use the existing 'syscon' code even for R40 for retrieving > > GMAC clock via CCU and update the register directly in > > sun8i_emac_set_syscon instead of writing it separately using > > ccm base. > > > > Cc: Joe Hershberger > > Cc: Lothar Felten > > Signed-off-by: Jagan Teki > > --- > > drivers/net/sun8i_emac.c | 55 ++++++++++++++++++++-------------------- > > 1 file changed, 27 insertions(+), 28 deletions(-) > > > > diff --git a/drivers/net/sun8i_emac.c b/drivers/net/sun8i_emac.c > > index c9798445c7..a7fb7ac405 100644 > > --- a/drivers/net/sun8i_emac.c > > +++ b/drivers/net/sun8i_emac.c > > @@ -285,10 +285,18 @@ static int sun8i_emac_set_syscon(struct sun8i_eth_pdata *pdata, > > int ret; > > u32 reg; > > > > - reg = readl(priv->sysctl_reg + 0x30); > > + if (priv->variant == R40_GMAC) { > > + /* Select RGMII for R40 */ > > + reg = readl(priv->sysctl_reg + 0x164); > > Please don't add more magic numbers. #define with register name. I do prefer the same, but kept it as same as original. and there are other place do have similar reg magic numbers. Will clean it up all in separate patch. would that be okay?