[PATCH net-next v3 35/47] net: dpaa: Use mac_dev variable in dpaa_netdev_init
Camelia Alexandra Groza
camelia.groza at nxp.com
Thu Jul 21 06:15:07 PDT 2022
> -----Original Message-----
> From: Sean Anderson <sean.anderson at seco.com>
> Sent: Saturday, July 16, 2022 1:00
> To: David S . Miller <davem at davemloft.net>; Jakub Kicinski
> <kuba at kernel.org>; Madalin Bucur <madalin.bucur at nxp.com>;
> netdev at vger.kernel.org
> Cc: Paolo Abeni <pabeni at redhat.com>; Eric Dumazet
> <edumazet at google.com>; linux-arm-kernel at lists.infradead.org; Russell
> King <linux at armlinux.org.uk>; linux-kernel at vger.kernel.org; Sean Anderson
> <sean.anderson at seco.com>
> Subject: [PATCH net-next v3 35/47] net: dpaa: Use mac_dev variable in
> dpaa_netdev_init
>
> There are several references to mac_dev in dpaa_netdev_init. Make things a
> bit more concise by adding a local variable for it.
>
> Signed-off-by: Sean Anderson <sean.anderson at seco.com>
> ---
>
> (no changes since v1)
>
> drivers/net/ethernet/freescale/dpaa/dpaa_eth.c | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c
> b/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c
> index d378247a6d0c..377e5513a414 100644
> --- a/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c
> +++ b/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c
> @@ -203,6 +203,7 @@ static int dpaa_netdev_init(struct net_device
> *net_dev,
> {
> struct dpaa_priv *priv = netdev_priv(net_dev);
> struct device *dev = net_dev->dev.parent;
> + struct mac_device *mac_dev = priv->mac_dev;
> struct dpaa_percpu_priv *percpu_priv;
> const u8 *mac_addr;
> int i, err;
> @@ -216,10 +217,10 @@ static int dpaa_netdev_init(struct net_device
> *net_dev,
> }
>
> net_dev->netdev_ops = dpaa_ops;
> - mac_addr = priv->mac_dev->addr;
> + mac_addr = mac_dev->addr;
>
> - net_dev->mem_start = (unsigned long)priv->mac_dev->vaddr;
> - net_dev->mem_end = (unsigned long)priv->mac_dev->vaddr_end;
> + net_dev->mem_start = (unsigned long)mac_dev->vaddr;
> + net_dev->mem_end = (unsigned long)mac_dev->vaddr_end;
>
> net_dev->min_mtu = ETH_MIN_MTU;
> net_dev->max_mtu = dpaa_get_max_mtu();
> @@ -246,7 +247,7 @@ static int dpaa_netdev_init(struct net_device
> *net_dev,
> eth_hw_addr_set(net_dev, mac_addr);
> } else {
> eth_hw_addr_random(net_dev);
> - err = priv->mac_dev->change_addr(priv->mac_dev-
> >fman_mac,
> + err = priv->mac_dev->change_addr(mac_dev->fman_mac,
> (const enet_addr_t *)net_dev->dev_addr);
You can replace priv->mac_dev->change_addr with mac_dev->change_addr as well.
> if (err) {
> dev_err(dev, "Failed to set random MAC address\n");
> --
> 2.35.1.1320.gc452695387.dirty
More information about the linux-arm-kernel
mailing list