> -----Original Message-----
> From: Lothar Waßmann [mailto:l...@karo-electronics.de]
> Sent: 2018年3月20日 16:50
> To: Peng Fan <peng....@nxp.com>
> Cc: joe.hershber...@ni.com; u-boot@lists.denx.de
> Subject: Re: [U-Boot] [PATCH 2/5] net: fec_mxc: simplify fec_get_miibus
> 
> Hi,
> 
> On Sat, 10 Mar 2018 09:19:54 +0800 Peng Fan wrote:
> > No need to provide two prototype for this function.
> > Use ulong for the first parameter, then this function could be shared
> > for DM/non DM case.
> >
> > Signed-off-by: Peng Fan <peng....@nxp.com>
> > ---
> >  drivers/net/fec_mxc.c | 13 ++-----------
> >  include/netdev.h      |  6 +-----
> >  2 files changed, 3 insertions(+), 16 deletions(-)
> >
> > diff --git a/drivers/net/fec_mxc.c b/drivers/net/fec_mxc.c index
> > 7c396d8d95..2bd4ba4ef1 100644
> > --- a/drivers/net/fec_mxc.c
> > +++ b/drivers/net/fec_mxc.c
> > @@ -1021,18 +1021,9 @@ static void fec_free_descs(struct fec_priv *fec)
> >     free(fec->tbd_base);
> >  }
> >
> > -#ifdef CONFIG_DM_ETH
> > -struct mii_dev *fec_get_miibus(struct udevice *dev, int dev_id)
> > -#else -struct mii_dev *fec_get_miibus(uint32_t base_addr, int dev_id)
> > -#endif
> > +struct mii_dev *fec_get_miibus(ulong base_addr, int dev_id)
> >  {
> > -#ifdef CONFIG_DM_ETH
> > -   struct fec_priv *priv = dev_get_priv(dev);
> > -   struct ethernet_regs *eth = priv->eth;
> > -#else
> >     struct ethernet_regs *eth = (struct ethernet_regs
> > *)(ulong)base_addr;
> >
> The (ulong) is redundant here.
Thanks. 
Joe, would you mind help fix when you commit the patch,
or you need me send out v2?

Thanks,
Peng.
> 
> 
> 
> Lothar Waßmann
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to