Staging: rtl8192u: Use BIT macro
authorMadhumitha Prabakaran <madhumithabiw@gmail.com>
Thu, 28 Mar 2019 15:53:33 +0000 (10:53 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 29 Mar 2019 16:15:31 +0000 (17:15 +0100)
Use BIT macro, as kernel provides the macro for improving uniform
and more readable way to define these constants.

Issue suggested by Coccinelle.

Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8192u/r8180_93cx6.c

index 163daed789b464354f8118f4a3118bba7f07df7a..f98bb11a66afc939f22a6d1277170d33230c48a9 100644 (file)
@@ -125,21 +125,21 @@ int eprom_read(struct net_device *dev, u32 addr)
 
        if (priv->epromtype == EPROM_93c56) {
                addr_str[7] = addr & 1;
-               addr_str[6] = addr & (1 << 1);
-               addr_str[5] = addr & (1 << 2);
-               addr_str[4] = addr & (1 << 3);
-               addr_str[3] = addr & (1 << 4);
-               addr_str[2] = addr & (1 << 5);
-               addr_str[1] = addr & (1 << 6);
-               addr_str[0] = addr & (1 << 7);
+               addr_str[6] = addr & BIT(1);
+               addr_str[5] = addr & BIT(2);
+               addr_str[4] = addr & BIT(3);
+               addr_str[3] = addr & BIT(4);
+               addr_str[2] = addr & BIT(5);
+               addr_str[1] = addr & BIT(6);
+               addr_str[0] = addr & BIT(7);
                addr_len = 8;
        } else {
                addr_str[5] = addr & 1;
-               addr_str[4] = addr & (1 << 1);
-               addr_str[3] = addr & (1 << 2);
-               addr_str[2] = addr & (1 << 3);
-               addr_str[1] = addr & (1 << 4);
-               addr_str[0] = addr & (1 << 5);
+               addr_str[4] = addr & BIT(1);
+               addr_str[3] = addr & BIT(2);
+               addr_str[2] = addr & BIT(3);
+               addr_str[1] = addr & BIT(4);
+               addr_str[0] = addr & BIT(5);
                addr_len = 6;
        }
        eprom_cs(dev, 1);