staging: wilc1000: define undefined operation mode
authorJohnny Kim <johnny.kim@atmel.com>
Tue, 8 Sep 2015 08:07:59 +0000 (17:07 +0900)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 13 Sep 2015 01:24:45 +0000 (18:24 -0700)
This patch adds new define, IDLE_MODE to change comparison statement which
is wrong due to typecasting to null.

Signed-off-by: Johnny Kim <johnny.kim@atmel.com>
Signed-off-by: Tony Cho <tony.cho@atmel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wilc1000/host_interface.c
drivers/staging/wilc1000/host_interface.h

index 8458d8d..8033205 100644 (file)
@@ -754,7 +754,7 @@ static s32 Handle_SetOperationMode(tstrWILC_WFIDrv *drvHandler, tstrHostIfSetOpe
                                 get_id_from_handler(pstrWFIDrv));
 
 
-       if ((pstrHostIfSetOperationMode->u32Mode) == (u32)NULL)
+       if ((pstrHostIfSetOperationMode->u32Mode) == IDLE_MODE)
                up(&hSemDeinitDrvHandle);
 
 
index 349d5f5..a107377 100644 (file)
@@ -23,6 +23,7 @@
 #define BIT1                    ((u32)(1 << 1))
 #define BIT0                    ((u32)(1 << 0))
 
+#define IDLE_MODE      0x00
 #define AP_MODE                0x01
 #define STATION_MODE   0x02
 #define GO_MODE        0x03