staging: wilc1000: avoid 'bool' datatype in struct sent to firmware
authorAjay Singh <ajay.kathat@microchip.com>
Tue, 5 Nov 2019 09:51:22 +0000 (09:51 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 5 Nov 2019 17:08:52 +0000 (18:08 +0100)
Use 'u8' instead of 'bool' datatype for struct element sent to firmware
because storage of bool datatype is not guaranteed.

Signed-off-by: Ajay Singh <ajay.kathat@microchip.com>
Link: https://lore.kernel.org/r/20191105095058.24223-2-ajay.kathat@microchip.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wilc1000/wilc_hif.c

index e0a95c5..59eb760 100644 (file)
@@ -32,7 +32,7 @@ struct wilc_op_mode {
 };
 
 struct wilc_reg_frame {
-       bool reg;
+       u8 reg;
        u8 reg_id;
        __le16 frame_type;
 } __packed;
@@ -1784,7 +1784,9 @@ void wilc_frame_register(struct wilc_vif *vif, u16 frame_type, bool reg)
        wid.val = (u8 *)&reg_frame;
 
        memset(&reg_frame, 0x0, sizeof(reg_frame));
-       reg_frame.reg = reg;
+
+       if (reg)
+               reg_frame.reg = 1;
 
        switch (frame_type) {
        case IEEE80211_STYPE_ACTION: