From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jagan Teki Date: Wed, 9 Nov 2016 17:20:00 +0530 Subject: [U-Boot] [PATCH v2 4/7] net: fec: Zap local mac variable In-Reply-To: <1478692203-4442-1-git-send-email-jagan@openedev.com> References: <1478692203-4442-1-git-send-email-jagan@openedev.com> Message-ID: <1478692203-4442-5-git-send-email-jagan@openedev.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de From: Jagan Teki Call dev->enetaddr or pdata->enetaddr directly in eth_ops instead of local mac variable. Cc: Stefano Babic Cc: Matteo Lisi Cc: Michael Trimarchi Acked-by: Joe Hershberger Signed-off-by: Jagan Teki --- drivers/net/fec_mxc.c | 18 ++++++------------ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/drivers/net/fec_mxc.c b/drivers/net/fec_mxc.c index 09433df..787823c 100644 --- a/drivers/net/fec_mxc.c +++ b/drivers/net/fec_mxc.c @@ -934,9 +934,8 @@ struct mii_dev *fec_get_miibus(uint32_t base_addr, int dev_id) static int fec_recv(struct eth_device *dev) { struct fec_priv *fec = (struct fec_priv *)dev->priv; - uchar *mac = dev->enetaddr; - return _fec_recv(fec, mac); + return _fec_recv(fec, dev->enetaddr); } static int fec_send(struct eth_device *dev, void *packet, int length) @@ -956,17 +955,15 @@ static void fec_halt(struct eth_device *dev) static int fec_set_hwaddr(struct eth_device *dev) { struct fec_priv *fec = (struct fec_priv *)dev->priv; - uchar *mac = dev->enetaddr; - return _fec_set_hwaddr(fec, mac); + return _fec_set_hwaddr(fec, dev->enetaddr); } static int fec_init(struct eth_device *dev, bd_t *bd) { struct fec_priv *fec = (struct fec_priv *)dev->priv; - uchar *mac = dev->enetaddr; - return _fec_init(fec, mac); + return _fec_init(fec, dev->enetaddr); } static int fec_get_hwaddr(int dev_id, unsigned char *mac) @@ -1142,9 +1139,8 @@ static int fec_set_hwaddr(struct udevice *dev) { struct fec_priv *fec = dev_get_priv(dev); struct eth_pdata *pdata = dev_get_platdata(dev); - uchar *mac = pdata->enetaddr; - return _fec_set_hwaddr(fec, mac); + return _fec_set_hwaddr(fec, pdata->enetaddr); } static void fec_halt(struct udevice *dev) @@ -1158,9 +1154,8 @@ static int fec_recv(struct udevice *dev, int flags, uchar **packetp) { struct fec_priv *fec = dev_get_priv(dev); struct eth_pdata *pdata = dev_get_platdata(dev); - uchar *mac = pdata->enetaddr; - return _fec_recv(fec, mac); + return _fec_recv(fec, pdata->enetaddr); } static int fec_send(struct udevice *dev, void *packet, int length) @@ -1174,9 +1169,8 @@ static int fec_init(struct udevice *dev) { struct fec_priv *fec = dev_get_priv(dev); struct eth_pdata *pdata = dev_get_platdata(dev); - uchar *mac = pdata->enetaddr; - return _fec_init(fec, mac); + return _fec_init(fec, pdata->enetaddr); } static const struct eth_ops fecmxc_ops = { -- 1.9.1