]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
net: macb: Remove redundant assignment to queue
authorJiapeng Chong <jiapeng.chong@linux.alibaba.com>
Thu, 29 Apr 2021 10:25:46 +0000 (18:25 +0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 29 Apr 2021 22:50:20 +0000 (15:50 -0700)
Variable queue is set to bp->queues but these values is not used as it
is overwritten later on, hence redundant assignment  can be removed.

Cleans up the following clang-analyzer warning:

drivers/net/ethernet/cadence/macb_main.c:4919:21: warning: Value stored
to 'queue' during its initialization is never read
[clang-analyzer-deadcode.DeadStores].

drivers/net/ethernet/cadence/macb_main.c:4832:21: warning: Value stored
to 'queue' during its initialization is never read
[clang-analyzer-deadcode.DeadStores].

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/cadence/macb_main.c

index 0e94db9cd45d208606f15012db8af2d36bbbdcf5..6bc7d41d519b7065f809b18be4f5b450b871c2d3 100644 (file)
@@ -4852,7 +4852,7 @@ static int __maybe_unused macb_suspend(struct device *dev)
 {
        struct net_device *netdev = dev_get_drvdata(dev);
        struct macb *bp = netdev_priv(netdev);
-       struct macb_queue *queue = bp->queues;
+       struct macb_queue *queue;
        unsigned long flags;
        unsigned int q;
        int err;
@@ -4939,7 +4939,7 @@ static int __maybe_unused macb_resume(struct device *dev)
 {
        struct net_device *netdev = dev_get_drvdata(dev);
        struct macb *bp = netdev_priv(netdev);
-       struct macb_queue *queue = bp->queues;
+       struct macb_queue *queue;
        unsigned long flags;
        unsigned int q;
        int err;