wifi: ray_cs: Drop useless status variable in parse_addr()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Fri, 3 Jun 2022 16:44:14 +0000 (19:44 +0300)
committerKalle Valo <kvalo@kernel.org>
Wed, 8 Jun 2022 08:08:34 +0000 (11:08 +0300)
The status variable assigned only once and used also only once.
Replace it's usage by actual value.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20220603164414.48436-2-andriy.shevchenko@linux.intel.com
drivers/net/wireless/ray_cs.c

index 9ac371d6cd6c8870afa0dc5ab89c7eef0a0a281e..1f57a0055bbd80ca73f60a330cc6d43594ce1e63 100644 (file)
@@ -1645,7 +1645,6 @@ static int parse_addr(char *in_str, UCHAR *out)
 {
        int i, k;
        int len;
-       int status;
 
        if (in_str == NULL)
                return 0;
@@ -1654,7 +1653,6 @@ static int parse_addr(char *in_str, UCHAR *out)
                return 0;
        memset(out, 0, ADDRLEN);
 
-       status = 1;
        i = 5;
 
        while (len > 0) {
@@ -1672,7 +1670,7 @@ static int parse_addr(char *in_str, UCHAR *out)
                if (!i--)
                        break;
        }
-       return status;
+       return 1;
 }
 
 /*===========================================================================*/