drivers/net/xen-netfront.c:2416:16: warning: variable ‘target’ set but not used 
[-Wunused-but-set-variable]
 2416 |  unsigned long target;

Remove target and just discard the return value from simple_strtoul().

This patch does give a checkpatch warning, but the warning was there
before anyway, as this file has lots of checkpatch warnings.

Signed-off-by: Andrew Lunn <and...@lunn.ch>
---
 drivers/net/xen-netfront.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/net/xen-netfront.c b/drivers/net/xen-netfront.c
index 3e9895bec15f..920cac4385bf 100644
--- a/drivers/net/xen-netfront.c
+++ b/drivers/net/xen-netfront.c
@@ -2413,12 +2413,11 @@ static ssize_t store_rxbuf(struct device *dev,
                           const char *buf, size_t len)
 {
        char *endp;
-       unsigned long target;
 
        if (!capable(CAP_NET_ADMIN))
                return -EPERM;
 
-       target = simple_strtoul(buf, &endp, 0);
+       simple_strtoul(buf, &endp, 0);
        if (endp == buf)
                return -EBADMSG;
 
-- 
2.28.0

Reply via email to