From: Harish Patil <harish.pa...@qlogic.com>

Fix for the following clang build error:
drivers/net/bnx2x/elink.c:10384:41: error: shifting a
      negative signed value is undefined [-Werror,-Wshift-negative-value]
                vars->eee_status &= ~SHMEM_EEE_1G_ADV <<
                                    ~~~~~~~~~~~~~~~~~ ^
1 error generated.
make[6]: *** [elink.o] Error 1
make[5]: *** [bnx2x] Error 2
make[4]: *** [net] Error 2
make[3]: *** [drivers] Error 2
make[2]: *** [all] Error 2
make[1]: *** [x86_64-native-linuxapp-clang_install] Error 2
make: *** [install] Error 2

Signed-off-by: Harish Patil <harish.patil at qlogic.com>
---
 drivers/net/bnx2x/ecore_hsi.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/bnx2x/ecore_hsi.h b/drivers/net/bnx2x/ecore_hsi.h
index 6c11c5a..d2cb888 100644
--- a/drivers/net/bnx2x/ecore_hsi.h
+++ b/drivers/net/bnx2x/ecore_hsi.h
@@ -2529,7 +2529,7 @@ struct shmem2_region {
        #define SHMEM_EEE_SUPPORTED_SHIFT          16
        #define SHMEM_EEE_ADV_STATUS_MASK          0x00f00000
                #define SHMEM_EEE_100M_ADV         (1<<0)
-               #define SHMEM_EEE_1G_ADV           (1<<1)
+               #define SHMEM_EEE_1G_ADV           (1U<<1)
                #define SHMEM_EEE_10G_ADV          (1<<2)
        #define SHMEM_EEE_ADV_STATUS_SHIFT         20
        #define SHMEM_EEE_LP_ADV_STATUS_MASK       0x0f000000
-- 
1.7.10.3

Reply via email to