Re: [PATCH net-next] net: natsemi: Remove set but not used variable

From: David Miller
Date: Sat Sep 19 2020 - 17:10:57 EST


From: Zheng Yongjun <zhengyongjun3@xxxxxxxxxx>
Date: Sat, 19 Sep 2020 10:46:44 +0800

> @@ -1226,8 +1226,8 @@ static int ns83820_get_link_ksettings(struct net_device *ndev,
>
> /* read current configuration */
> cfg = readl(dev->base + CFG) ^ SPDSTS_POLARITY;
> - tanar = readl(dev->base + TANAR);
> tbicr = readl(dev->base + TBICR);
> + readl(dev->base + TANAR);
>

Please do not change the order of the readl() operations, it might
matter.