> -----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