Commit 8e8af97a authored by Colin Ian King's avatar Colin Ian King Committed by David S. Miller

lan743x: remove some redundant variables and assignments

Function lan743x_phy_init assigns pointer 'netdev' but this is never read
and hence it can be removed. The return error code handling can also be
cleaned up to remove the variable 'ret'.

Function lan743x_phy_link_status_change assigns pointer 'phy' twice and
this is never read, so it also can be removed.

Finally, function lan743x_tx_napi_poll initializes pointer 'adapter'
and then re-assigns the same value into this pointer a little later on
so this second assignment is redundant and can be also removed.

Cleans up clang warnings:
drivers/net/ethernet/microchip/lan743x_main.c:951:2: warning: Value
stored to 'netdev' is never read
drivers/net/ethernet/microchip/lan743x_main.c:971:3: warning: Value
stored to 'phy' is never read
drivers/net/ethernet/microchip/lan743x_main.c:1583:26: warning: Value
stored to 'adapter' during its initialization is never read
Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent bdf08fc5
...@@ -945,15 +945,7 @@ static void lan743x_phy_update_flowcontrol(struct lan743x_adapter *adapter, ...@@ -945,15 +945,7 @@ static void lan743x_phy_update_flowcontrol(struct lan743x_adapter *adapter,
static int lan743x_phy_init(struct lan743x_adapter *adapter) static int lan743x_phy_init(struct lan743x_adapter *adapter)
{ {
struct net_device *netdev; return lan743x_phy_reset(adapter);
int ret;
netdev = adapter->netdev;
ret = lan743x_phy_reset(adapter);
if (ret)
return ret;
return 0;
} }
static void lan743x_phy_link_status_change(struct net_device *netdev) static void lan743x_phy_link_status_change(struct net_device *netdev)
...@@ -964,11 +956,9 @@ static void lan743x_phy_link_status_change(struct net_device *netdev) ...@@ -964,11 +956,9 @@ static void lan743x_phy_link_status_change(struct net_device *netdev)
phy_print_status(phydev); phy_print_status(phydev);
if (phydev->state == PHY_RUNNING) { if (phydev->state == PHY_RUNNING) {
struct ethtool_link_ksettings ksettings; struct ethtool_link_ksettings ksettings;
struct lan743x_phy *phy = NULL;
int remote_advertisement = 0; int remote_advertisement = 0;
int local_advertisement = 0; int local_advertisement = 0;
phy = &adapter->phy;
memset(&ksettings, 0, sizeof(ksettings)); memset(&ksettings, 0, sizeof(ksettings));
phy_ethtool_get_link_ksettings(netdev, &ksettings); phy_ethtool_get_link_ksettings(netdev, &ksettings);
local_advertisement = phy_read(phydev, MII_ADVERTISE); local_advertisement = phy_read(phydev, MII_ADVERTISE);
...@@ -1586,7 +1576,6 @@ static int lan743x_tx_napi_poll(struct napi_struct *napi, int weight) ...@@ -1586,7 +1576,6 @@ static int lan743x_tx_napi_poll(struct napi_struct *napi, int weight)
u32 ioc_bit = 0; u32 ioc_bit = 0;
u32 int_sts = 0; u32 int_sts = 0;
adapter = tx->adapter;
ioc_bit = DMAC_INT_BIT_TX_IOC_(tx->channel_number); ioc_bit = DMAC_INT_BIT_TX_IOC_(tx->channel_number);
int_sts = lan743x_csr_read(adapter, DMAC_INT_STS); int_sts = lan743x_csr_read(adapter, DMAC_INT_STS);
if (tx->vector_flags & LAN743X_VECTOR_FLAG_SOURCE_STATUS_W2C) if (tx->vector_flags & LAN743X_VECTOR_FLAG_SOURCE_STATUS_W2C)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment