[PATCH v2] staging: rtl8192u: Fix warnings about endianness

2017-02-17 Thread maomao xu
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37: warning: incorrect 
type in assignment (different base types)
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37:expected unsigned 
short [unsigned] [usertype] len
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37:got restricted 
__be16 [usertype] 

Signed-off-by: maomao xu <albert008...@gmail.com>

diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c 
b/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
index d1057b1..1e81c24 100644
--- a/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
+++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
@@ -559,10 +559,8 @@ void ieee80211_indicate_packets(struct ieee80211_device 
*ieee, struct ieee80211_
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->src, 
ETH_ALEN);
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->dst, 
ETH_ALEN);
} else {
-   u16 len;
/* Leave Ethernet header part of hdr and full payload */
-   len = htons(sub_skb->len);
-   memcpy(skb_push(sub_skb, 2), , 2);
+   put_unaligned_be16(sub_skb->len, 
skb_push(sub_skb, 2));
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->src, 
ETH_ALEN);
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->dst, 
ETH_ALEN);
}
-- 
1.7.9.5



[PATCH v2] staging: rtl8192u: Fix warnings about endianness

2017-02-17 Thread maomao xu
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37: warning: incorrect 
type in assignment (different base types)
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37:expected unsigned 
short [unsigned] [usertype] len
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37:got restricted 
__be16 [usertype] 

Signed-off-by: maomao xu 

diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c 
b/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
index d1057b1..1e81c24 100644
--- a/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
+++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
@@ -559,10 +559,8 @@ void ieee80211_indicate_packets(struct ieee80211_device 
*ieee, struct ieee80211_
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->src, 
ETH_ALEN);
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->dst, 
ETH_ALEN);
} else {
-   u16 len;
/* Leave Ethernet header part of hdr and full payload */
-   len = htons(sub_skb->len);
-   memcpy(skb_push(sub_skb, 2), , 2);
+   put_unaligned_be16(sub_skb->len, 
skb_push(sub_skb, 2));
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->src, 
ETH_ALEN);
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->dst, 
ETH_ALEN);
}
-- 
1.7.9.5



[PATCH] staging: rtl8192e: Fix endianness issue

2017-02-15 Thread maomao xu
Fix endianness issue about Mk16_le function

Signed-off-by: maomao xu <albert008...@gmail.com>

diff --git a/drivers/staging/rtl8192e/rtllib_crypt_tkip.c 
b/drivers/staging/rtl8192e/rtllib_crypt_tkip.c
index ae103b0..05b66d4 100644
--- a/drivers/staging/rtl8192e/rtllib_crypt_tkip.c
+++ b/drivers/staging/rtl8192e/rtllib_crypt_tkip.c
@@ -162,9 +162,9 @@ static inline u16 Mk16(u8 hi, u8 lo)
 }
 
 
-static inline u16 Mk16_le(u16 *v)
+static inline u16 Mk16_le(__le16 *v)
 {
-   return *v;
+   return le16_to_cpu(*v);
 }
 
 
@@ -253,15 +253,15 @@ static void tkip_mixing_phase2(u8 *WEPSeed, const u8 *TK, 
const u16 *TTAK,
PPK[5] = TTAK[4] + IV16;
 
/* Step 2 - 96-bit bijective mixing using S-box */
-   PPK[0] += _S_(PPK[5] ^ Mk16_le((u16 *) [0]));
-   PPK[1] += _S_(PPK[0] ^ Mk16_le((u16 *) [2]));
-   PPK[2] += _S_(PPK[1] ^ Mk16_le((u16 *) [4]));
-   PPK[3] += _S_(PPK[2] ^ Mk16_le((u16 *) [6]));
-   PPK[4] += _S_(PPK[3] ^ Mk16_le((u16 *) [8]));
-   PPK[5] += _S_(PPK[4] ^ Mk16_le((u16 *) [10]));
-
-   PPK[0] += RotR1(PPK[5] ^ Mk16_le((u16 *) [12]));
-   PPK[1] += RotR1(PPK[0] ^ Mk16_le((u16 *) [14]));
+   PPK[0] += _S_(PPK[5] ^ Mk16_le((__le16 *) [0]));
+   PPK[1] += _S_(PPK[0] ^ Mk16_le((__le16 *) [2]));
+   PPK[2] += _S_(PPK[1] ^ Mk16_le((__le16 *) [4]));
+   PPK[3] += _S_(PPK[2] ^ Mk16_le((__le16 *) [6]));
+   PPK[4] += _S_(PPK[3] ^ Mk16_le((__le16 *) [8]));
+   PPK[5] += _S_(PPK[4] ^ Mk16_le((__le16 *) [10]));
+
+   PPK[0] += RotR1(PPK[5] ^ Mk16_le((__le16 *) [12]));
+   PPK[1] += RotR1(PPK[0] ^ Mk16_le((__le16 *) [14]));
PPK[2] += RotR1(PPK[1]);
PPK[3] += RotR1(PPK[2]);
PPK[4] += RotR1(PPK[3]);
@@ -273,7 +273,7 @@ static void tkip_mixing_phase2(u8 *WEPSeed, const u8 *TK, 
const u16 *TTAK,
WEPSeed[0] = Hi8(IV16);
WEPSeed[1] = (Hi8(IV16) | 0x20) & 0x7F;
WEPSeed[2] = Lo8(IV16);
-   WEPSeed[3] = Lo8((PPK[5] ^ Mk16_le((u16 *) [0])) >> 1);
+   WEPSeed[3] = Lo8((PPK[5] ^ Mk16_le((__le16 *) [0])) >> 1);
 
 #ifdef __BIG_ENDIAN
{
-- 
1.7.9.5



[PATCH] staging: rtl8192e: Fix endianness issue

2017-02-15 Thread maomao xu
Fix endianness issue about Mk16_le function

Signed-off-by: maomao xu 

diff --git a/drivers/staging/rtl8192e/rtllib_crypt_tkip.c 
b/drivers/staging/rtl8192e/rtllib_crypt_tkip.c
index ae103b0..05b66d4 100644
--- a/drivers/staging/rtl8192e/rtllib_crypt_tkip.c
+++ b/drivers/staging/rtl8192e/rtllib_crypt_tkip.c
@@ -162,9 +162,9 @@ static inline u16 Mk16(u8 hi, u8 lo)
 }
 
 
-static inline u16 Mk16_le(u16 *v)
+static inline u16 Mk16_le(__le16 *v)
 {
-   return *v;
+   return le16_to_cpu(*v);
 }
 
 
@@ -253,15 +253,15 @@ static void tkip_mixing_phase2(u8 *WEPSeed, const u8 *TK, 
const u16 *TTAK,
PPK[5] = TTAK[4] + IV16;
 
/* Step 2 - 96-bit bijective mixing using S-box */
-   PPK[0] += _S_(PPK[5] ^ Mk16_le((u16 *) [0]));
-   PPK[1] += _S_(PPK[0] ^ Mk16_le((u16 *) [2]));
-   PPK[2] += _S_(PPK[1] ^ Mk16_le((u16 *) [4]));
-   PPK[3] += _S_(PPK[2] ^ Mk16_le((u16 *) [6]));
-   PPK[4] += _S_(PPK[3] ^ Mk16_le((u16 *) [8]));
-   PPK[5] += _S_(PPK[4] ^ Mk16_le((u16 *) [10]));
-
-   PPK[0] += RotR1(PPK[5] ^ Mk16_le((u16 *) [12]));
-   PPK[1] += RotR1(PPK[0] ^ Mk16_le((u16 *) [14]));
+   PPK[0] += _S_(PPK[5] ^ Mk16_le((__le16 *) [0]));
+   PPK[1] += _S_(PPK[0] ^ Mk16_le((__le16 *) [2]));
+   PPK[2] += _S_(PPK[1] ^ Mk16_le((__le16 *) [4]));
+   PPK[3] += _S_(PPK[2] ^ Mk16_le((__le16 *) [6]));
+   PPK[4] += _S_(PPK[3] ^ Mk16_le((__le16 *) [8]));
+   PPK[5] += _S_(PPK[4] ^ Mk16_le((__le16 *) [10]));
+
+   PPK[0] += RotR1(PPK[5] ^ Mk16_le((__le16 *) [12]));
+   PPK[1] += RotR1(PPK[0] ^ Mk16_le((__le16 *) [14]));
PPK[2] += RotR1(PPK[1]);
PPK[3] += RotR1(PPK[2]);
PPK[4] += RotR1(PPK[3]);
@@ -273,7 +273,7 @@ static void tkip_mixing_phase2(u8 *WEPSeed, const u8 *TK, 
const u16 *TTAK,
WEPSeed[0] = Hi8(IV16);
WEPSeed[1] = (Hi8(IV16) | 0x20) & 0x7F;
WEPSeed[2] = Lo8(IV16);
-   WEPSeed[3] = Lo8((PPK[5] ^ Mk16_le((u16 *) [0])) >> 1);
+   WEPSeed[3] = Lo8((PPK[5] ^ Mk16_le((__le16 *) [0])) >> 1);
 
 #ifdef __BIG_ENDIAN
{
-- 
1.7.9.5



[PATCH] staging: rtl8192u: Fix warnings about endianness

2017-02-15 Thread maomao xu
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37: warning: incorrect 
type in assignment (different base types)
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37:expected unsigned 
short [unsigned] [usertype] len
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37:got restricted 
__be16 [usertype] 

Signed-off-by: maomao xu <albert008...@gmail.com>

diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c 
b/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
index d1057b1..1e81c24 100644
--- a/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
+++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
@@ -559,10 +559,8 @@ void ieee80211_indicate_packets(struct ieee80211_device 
*ieee, struct ieee80211_
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->src, 
ETH_ALEN);
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->dst, 
ETH_ALEN);
} else {
-   u16 len;
/* Leave Ethernet header part of hdr and full payload */
-   len = htons(sub_skb->len);
-   memcpy(skb_push(sub_skb, 2), , 2);
+   put_unaligned_be16(sub_skb->len, 
skb_push(sub_skb, 2));
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->src, 
ETH_ALEN);
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->dst, 
ETH_ALEN);
}
-- 
1.7.9.5



[PATCH] staging: rtl8192u: Fix warnings about endianness

2017-02-15 Thread maomao xu
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37: warning: incorrect 
type in assignment (different base types)
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37:expected unsigned 
short [unsigned] [usertype] len
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37:got restricted 
__be16 [usertype] 

Signed-off-by: maomao xu 

diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c 
b/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
index d1057b1..1e81c24 100644
--- a/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
+++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
@@ -559,10 +559,8 @@ void ieee80211_indicate_packets(struct ieee80211_device 
*ieee, struct ieee80211_
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->src, 
ETH_ALEN);
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->dst, 
ETH_ALEN);
} else {
-   u16 len;
/* Leave Ethernet header part of hdr and full payload */
-   len = htons(sub_skb->len);
-   memcpy(skb_push(sub_skb, 2), , 2);
+   put_unaligned_be16(sub_skb->len, 
skb_push(sub_skb, 2));
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->src, 
ETH_ALEN);
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->dst, 
ETH_ALEN);
}
-- 
1.7.9.5



[PATCH] staging: rtl8192u: Fix warnings about endianness

2017-02-15 Thread maomao xu
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37: warning: incorrect 
type in assignment (different base types)
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37:expected unsigned 
short [unsigned] [usertype] len
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37:got restricted 
__be16 [usertype] 

Signed-off-by: maomao xu <albert008...@gmail.com>

diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c 
b/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
index d1057b1..1e81c24 100644
--- a/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
+++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
@@ -559,10 +559,8 @@ void ieee80211_indicate_packets(struct ieee80211_device 
*ieee, struct ieee80211_
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->src, 
ETH_ALEN);
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->dst, 
ETH_ALEN);
} else {
-   u16 len;
/* Leave Ethernet header part of hdr and full payload */
-   len = htons(sub_skb->len);
-   memcpy(skb_push(sub_skb, 2), , 2);
+   put_unaligned_be16(sub_skb->len, 
kb_push(sub_skb, 2));
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->src, 
ETH_ALEN);
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->dst, 
ETH_ALEN);
}
-- 
1.7.9.5



[PATCH] staging: rtl8192u: Fix warnings about endianness

2017-02-15 Thread maomao xu
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37: warning: incorrect 
type in assignment (different base types)
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37:expected unsigned 
short [unsigned] [usertype] len
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:564:37:got restricted 
__be16 [usertype] 

Signed-off-by: maomao xu 

diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c 
b/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
index d1057b1..1e81c24 100644
--- a/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
+++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
@@ -559,10 +559,8 @@ void ieee80211_indicate_packets(struct ieee80211_device 
*ieee, struct ieee80211_
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->src, 
ETH_ALEN);
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->dst, 
ETH_ALEN);
} else {
-   u16 len;
/* Leave Ethernet header part of hdr and full payload */
-   len = htons(sub_skb->len);
-   memcpy(skb_push(sub_skb, 2), , 2);
+   put_unaligned_be16(sub_skb->len, 
kb_push(sub_skb, 2));
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->src, 
ETH_ALEN);
memcpy(skb_push(sub_skb, ETH_ALEN), prxb->dst, 
ETH_ALEN);
}
-- 
1.7.9.5



[PATCH] staging: rtl8192u: Fix warnings about endianness

2017-02-14 Thread maomao xu
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16

Signed-off-by: maomao xu <albert008...@gmail.com>

diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c 
b/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c
index 2453413..fb171bd 100644
--- a/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c
+++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c
@@ -172,7 +172,7 @@ static inline u16 Mk16(u8 hi, u8 lo)
 }
 
 
-static inline u16 Mk16_le(u16 *v)
+static inline u16 Mk16_le(__le16 *v)
 {
return le16_to_cpu(*v);
 }
@@ -264,15 +264,15 @@ static void tkip_mixing_phase2(u8 *WEPSeed, const u8 *TK, 
const u16 *TTAK,
PPK[5] = TTAK[4] + IV16;
 
/* Step 2 - 96-bit bijective mixing using S-box */
-   PPK[0] += _S_(PPK[5] ^ Mk16_le((u16 *) [0]));
-   PPK[1] += _S_(PPK[0] ^ Mk16_le((u16 *) [2]));
-   PPK[2] += _S_(PPK[1] ^ Mk16_le((u16 *) [4]));
-   PPK[3] += _S_(PPK[2] ^ Mk16_le((u16 *) [6]));
-   PPK[4] += _S_(PPK[3] ^ Mk16_le((u16 *) [8]));
-   PPK[5] += _S_(PPK[4] ^ Mk16_le((u16 *) [10]));
-
-   PPK[0] += RotR1(PPK[5] ^ Mk16_le((u16 *) [12]));
-   PPK[1] += RotR1(PPK[0] ^ Mk16_le((u16 *) [14]));
+   PPK[0] += _S_(PPK[5] ^ Mk16_le((__le16 *) [0]));
+   PPK[1] += _S_(PPK[0] ^ Mk16_le((__le16 *) [2]));
+   PPK[2] += _S_(PPK[1] ^ Mk16_le((__le16 *) [4]));
+   PPK[3] += _S_(PPK[2] ^ Mk16_le((__le16 *) [6]));
+   PPK[4] += _S_(PPK[3] ^ Mk16_le((__le16 *) [8]));
+   PPK[5] += _S_(PPK[4] ^ Mk16_le((__le16 *) [10]));
+
+   PPK[0] += RotR1(PPK[5] ^ Mk16_le((__le16 *) [12]));
+   PPK[1] += RotR1(PPK[0] ^ Mk16_le((__le16 *) [14]));
PPK[2] += RotR1(PPK[1]);
PPK[3] += RotR1(PPK[2]);
PPK[4] += RotR1(PPK[3]);
@@ -285,7 +285,7 @@ static void tkip_mixing_phase2(u8 *WEPSeed, const u8 *TK, 
const u16 *TTAK,
WEPSeed[0] = Hi8(IV16);
WEPSeed[1] = (Hi8(IV16) | 0x20) & 0x7F;
WEPSeed[2] = Lo8(IV16);
-   WEPSeed[3] = Lo8((PPK[5] ^ Mk16_le((u16 *) [0])) >> 1);
+   WEPSeed[3] = Lo8((PPK[5] ^ Mk16_le((__le16 *) [0])) >> 1);
 
 #ifdef __BIG_ENDIAN
{
-- 
1.7.9.5



[PATCH] staging: rtl8192u: Fix warnings about endianness

2017-02-14 Thread maomao xu
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast 
to restricted __le16

Signed-off-by: maomao xu 

diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c 
b/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c
index 2453413..fb171bd 100644
--- a/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c
+++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c
@@ -172,7 +172,7 @@ static inline u16 Mk16(u8 hi, u8 lo)
 }
 
 
-static inline u16 Mk16_le(u16 *v)
+static inline u16 Mk16_le(__le16 *v)
 {
return le16_to_cpu(*v);
 }
@@ -264,15 +264,15 @@ static void tkip_mixing_phase2(u8 *WEPSeed, const u8 *TK, 
const u16 *TTAK,
PPK[5] = TTAK[4] + IV16;
 
/* Step 2 - 96-bit bijective mixing using S-box */
-   PPK[0] += _S_(PPK[5] ^ Mk16_le((u16 *) [0]));
-   PPK[1] += _S_(PPK[0] ^ Mk16_le((u16 *) [2]));
-   PPK[2] += _S_(PPK[1] ^ Mk16_le((u16 *) [4]));
-   PPK[3] += _S_(PPK[2] ^ Mk16_le((u16 *) [6]));
-   PPK[4] += _S_(PPK[3] ^ Mk16_le((u16 *) [8]));
-   PPK[5] += _S_(PPK[4] ^ Mk16_le((u16 *) [10]));
-
-   PPK[0] += RotR1(PPK[5] ^ Mk16_le((u16 *) [12]));
-   PPK[1] += RotR1(PPK[0] ^ Mk16_le((u16 *) [14]));
+   PPK[0] += _S_(PPK[5] ^ Mk16_le((__le16 *) [0]));
+   PPK[1] += _S_(PPK[0] ^ Mk16_le((__le16 *) [2]));
+   PPK[2] += _S_(PPK[1] ^ Mk16_le((__le16 *) [4]));
+   PPK[3] += _S_(PPK[2] ^ Mk16_le((__le16 *) [6]));
+   PPK[4] += _S_(PPK[3] ^ Mk16_le((__le16 *) [8]));
+   PPK[5] += _S_(PPK[4] ^ Mk16_le((__le16 *) [10]));
+
+   PPK[0] += RotR1(PPK[5] ^ Mk16_le((__le16 *) [12]));
+   PPK[1] += RotR1(PPK[0] ^ Mk16_le((__le16 *) [14]));
PPK[2] += RotR1(PPK[1]);
PPK[3] += RotR1(PPK[2]);
PPK[4] += RotR1(PPK[3]);
@@ -285,7 +285,7 @@ static void tkip_mixing_phase2(u8 *WEPSeed, const u8 *TK, 
const u16 *TTAK,
WEPSeed[0] = Hi8(IV16);
WEPSeed[1] = (Hi8(IV16) | 0x20) & 0x7F;
WEPSeed[2] = Lo8(IV16);
-   WEPSeed[3] = Lo8((PPK[5] ^ Mk16_le((u16 *) [0])) >> 1);
+   WEPSeed[3] = Lo8((PPK[5] ^ Mk16_le((__le16 *) [0])) >> 1);
 
 #ifdef __BIG_ENDIAN
{
-- 
1.7.9.5



[PATCH] staging: rtl8192u: Fix sparse warnings about endianness

2017-02-14 Thread maomao xu
drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c:175:36: warning: incorrect 
type in assignment (different base types)
drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c:175:36:expected unsigned 
short [unsigned] [short] [usertype] 
drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c:175:36:got restricted 
__be16 [usertype] 

Signed-off-by: maomao xu <albert008...@gmail.com>

diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c 
b/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c
index 949c496..a1da90a 100644
--- a/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c
+++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c
@@ -172,7 +172,7 @@ static inline int ieee80211_put_snap(u8 *data, u16 h_proto)
snap->oui[1] = oui[1];
snap->oui[2] = oui[2];
 
-   *(u16 *)(data + SNAP_SIZE) = htons(h_proto);
+   put_unaligned_be16(h_proto, data + SNAP_SIZE);
 
return SNAP_SIZE + sizeof(u16);
 }
-- 
1.7.9.5



[PATCH] staging: rtl8192u: Fix sparse warnings about endianness

2017-02-14 Thread maomao xu
drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c:175:36: warning: incorrect 
type in assignment (different base types)
drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c:175:36:expected unsigned 
short [unsigned] [short] [usertype] 
drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c:175:36:got restricted 
__be16 [usertype] 

Signed-off-by: maomao xu 

diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c 
b/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c
index 949c496..a1da90a 100644
--- a/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c
+++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c
@@ -172,7 +172,7 @@ static inline int ieee80211_put_snap(u8 *data, u16 h_proto)
snap->oui[1] = oui[1];
snap->oui[2] = oui[2];
 
-   *(u16 *)(data + SNAP_SIZE) = htons(h_proto);
+   put_unaligned_be16(h_proto, data + SNAP_SIZE);
 
return SNAP_SIZE + sizeof(u16);
 }
-- 
1.7.9.5



[PATCH] staging: rtl8712: Fix sparse warnings about endianness

2017-02-14 Thread maomao xu
drivers/staging/rtl8712/rtl871x_xmit.c:350:44: warning: restricted __le32 
degrades to integer

Signed-off-by: maomao xu <albert008...@gmail.com>

diff --git a/drivers/staging/rtl8712/rtl871x_xmit.c 
b/drivers/staging/rtl8712/rtl871x_xmit.c
index 4ab82ba..1c2ac28 100644
--- a/drivers/staging/rtl8712/rtl871x_xmit.c
+++ b/drivers/staging/rtl8712/rtl871x_xmit.c
@@ -347,7 +347,7 @@ sint r8712_update_attrib(struct _adapter *padapter, _pkt 
*pkt,
 * some settings above.
 */
if (check_fwstate(pmlmepriv, WIFI_MP_STATE))
-   pattrib->priority = (txdesc.txdw1 >> QSEL_SHT) & 0x1f;
+   pattrib->priority = (le32_to_cpu(txdesc.txdw1) >> QSEL_SHT) & 
0x1f;
return _SUCCESS;
 }
 
-- 
1.7.9.5



[PATCH] staging: rtl8712: Fix sparse warnings about endianness

2017-02-14 Thread maomao xu
drivers/staging/rtl8712/rtl871x_xmit.c:350:44: warning: restricted __le32 
degrades to integer

Signed-off-by: maomao xu 

diff --git a/drivers/staging/rtl8712/rtl871x_xmit.c 
b/drivers/staging/rtl8712/rtl871x_xmit.c
index 4ab82ba..1c2ac28 100644
--- a/drivers/staging/rtl8712/rtl871x_xmit.c
+++ b/drivers/staging/rtl8712/rtl871x_xmit.c
@@ -347,7 +347,7 @@ sint r8712_update_attrib(struct _adapter *padapter, _pkt 
*pkt,
 * some settings above.
 */
if (check_fwstate(pmlmepriv, WIFI_MP_STATE))
-   pattrib->priority = (txdesc.txdw1 >> QSEL_SHT) & 0x1f;
+   pattrib->priority = (le32_to_cpu(txdesc.txdw1) >> QSEL_SHT) & 
0x1f;
return _SUCCESS;
 }
 
-- 
1.7.9.5



[PATCH] staging: gdm724x: Fix sparse warnings

2017-02-14 Thread maomao xu
drivers/staging/gdm724x/gdm_lte.c:311:39: warning: incorrect type in assignment 
(different base types)
drivers/staging/gdm724x/gdm_lte.c:311:39:expected restricted __sum16 
[addressable] [assigned] [usertype] icmp6_cksum
drivers/staging/gdm724x/gdm_lte.c:311:39:got int

Signed-off-by: maomao xu <albert008...@gmail.com>

diff --git a/drivers/staging/gdm724x/gdm_lte.c 
b/drivers/staging/gdm724x/gdm_lte.c
index a3e046c..ddec6d0 100644
--- a/drivers/staging/gdm724x/gdm_lte.c
+++ b/drivers/staging/gdm724x/gdm_lte.c
@@ -308,7 +308,7 @@ static int gdm_lte_emulate_ndp(struct sk_buff *skb_in, u32 
nic_type)
memcpy(icmp_na + sizeof(struct icmp6hdr), ,
   sizeof(struct neighbour_advertisement));
 
-   icmp6_out.icmp6_cksum = icmp6_checksum(_out,
+   icmp6_out.icmp6_cksum = (__force 
__sum16)icmp6_checksum(_out,
(u16 *)icmp_na, sizeof(icmp_na));
} else {
return -1;
-- 
1.7.9.5



[PATCH] staging: gdm724x: Fix sparse warnings

2017-02-14 Thread maomao xu
drivers/staging/gdm724x/gdm_lte.c:311:39: warning: incorrect type in assignment 
(different base types)
drivers/staging/gdm724x/gdm_lte.c:311:39:expected restricted __sum16 
[addressable] [assigned] [usertype] icmp6_cksum
drivers/staging/gdm724x/gdm_lte.c:311:39:got int

Signed-off-by: maomao xu 

diff --git a/drivers/staging/gdm724x/gdm_lte.c 
b/drivers/staging/gdm724x/gdm_lte.c
index a3e046c..ddec6d0 100644
--- a/drivers/staging/gdm724x/gdm_lte.c
+++ b/drivers/staging/gdm724x/gdm_lte.c
@@ -308,7 +308,7 @@ static int gdm_lte_emulate_ndp(struct sk_buff *skb_in, u32 
nic_type)
memcpy(icmp_na + sizeof(struct icmp6hdr), ,
   sizeof(struct neighbour_advertisement));
 
-   icmp6_out.icmp6_cksum = icmp6_checksum(_out,
+   icmp6_out.icmp6_cksum = (__force 
__sum16)icmp6_checksum(_out,
(u16 *)icmp_na, sizeof(icmp_na));
} else {
return -1;
-- 
1.7.9.5



[PATCH]staging: fbtft: Fix sparse warnings about endianness

2017-02-14 Thread maomao xu
Fixed sparse warnings about endianness. E.g.:

warning: incorrect type in assignment (different base types)

Signed-off-by: maomao xu <albert008...@gmail.com>

diff --git a/drivers/staging/fbtft/fbtft-io.c b/drivers/staging/fbtft/fbtft-io.c
index d868405..ffb9a3b 100644
--- a/drivers/staging/fbtft/fbtft-io.c
+++ b/drivers/staging/fbtft/fbtft-io.c
@@ -71,7 +71,7 @@ int fbtft_write_spi_emulate_9(struct fbtft_par *par, void 
*buf, size_t len)
src++;
}
tmp |= ((*src & 0x0100) ? 1 : 0);
-   *(u64 *)dst = cpu_to_be64(tmp);
+   *(__be64 *)dst = cpu_to_be64(tmp);
dst += 8;
*dst++ = (u8)(*src++ & 0x00FF);
added++;
-- 
1.7.9.5



[PATCH]staging: fbtft: Fix sparse warnings about endianness

2017-02-14 Thread maomao xu
Fixed sparse warnings about endianness. E.g.:

warning: incorrect type in assignment (different base types)

Signed-off-by: maomao xu 

diff --git a/drivers/staging/fbtft/fbtft-io.c b/drivers/staging/fbtft/fbtft-io.c
index d868405..ffb9a3b 100644
--- a/drivers/staging/fbtft/fbtft-io.c
+++ b/drivers/staging/fbtft/fbtft-io.c
@@ -71,7 +71,7 @@ int fbtft_write_spi_emulate_9(struct fbtft_par *par, void 
*buf, size_t len)
src++;
}
tmp |= ((*src & 0x0100) ? 1 : 0);
-   *(u64 *)dst = cpu_to_be64(tmp);
+   *(__be64 *)dst = cpu_to_be64(tmp);
dst += 8;
*dst++ = (u8)(*src++ & 0x00FF);
added++;
-- 
1.7.9.5



[PATCH] Staging: vt6656: Fixed multiple commenting codig style issues.

2015-12-30 Thread maomao xu
Fixed multiple comment blocks that didn't comply with the
kernels coding style.

Signed-off-by: maomao xu 
---
 drivers/staging/vt6656/main_usb.c |   14 --
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/vt6656/main_usb.c 
b/drivers/staging/vt6656/main_usb.c
index ee8d1e1..a2f23ae 100644
--- a/drivers/staging/vt6656/main_usb.c
+++ b/drivers/staging/vt6656/main_usb.c
@@ -74,10 +74,10 @@ MODULE_PARM_DESC(tx_buffers, "Number of receive usb tx 
buffers");
 #define LONG_RETRY_DEF 4
 
 /* BasebandType[] baseband type selected
-   0: indicate 802.11a type
-   1: indicate 802.11b type
-   2: indicate 802.11g type
-*/
+ * 0: indicate 802.11a type
+ * 1: indicate 802.11b type
+ * 2: indicate 802.11g type
+ */
 
 #define BBP_TYPE_DEF 2
 
@@ -284,7 +284,8 @@ static int vnt_init_registers(struct vnt_private *priv)
calib_rx_iq = priv->eeprom[EEP_OFS_CALIB_RX_IQ];
if (calib_tx_iq || calib_tx_dc || calib_rx_iq) {
/* CR255, enable TX/RX IQ and
-  DC compensation mode */
+* DC compensation mode
+*/
vnt_control_out_u8(priv,
   MESSAGE_REQUEST_BBREG,
   0xff,
@@ -306,7 +307,8 @@ static int vnt_init_registers(struct vnt_private *priv)
   calib_rx_iq);
} else {
/* CR255, turn off
-  BB Calibration compensation */
+* BB Calibration compensation
+*/
vnt_control_out_u8(priv,
   MESSAGE_REQUEST_BBREG,
   0xff,
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] Staging: vt6656: Fixed multiple commenting codig style issues.

2015-12-30 Thread maomao xu
Fixed multiple comment blocks that didn't comply with the
kernels coding style.

Signed-off-by: maomao xu <albert008...@gmail.com>
---
 drivers/staging/vt6656/main_usb.c |   14 --
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/vt6656/main_usb.c 
b/drivers/staging/vt6656/main_usb.c
index ee8d1e1..a2f23ae 100644
--- a/drivers/staging/vt6656/main_usb.c
+++ b/drivers/staging/vt6656/main_usb.c
@@ -74,10 +74,10 @@ MODULE_PARM_DESC(tx_buffers, "Number of receive usb tx 
buffers");
 #define LONG_RETRY_DEF 4
 
 /* BasebandType[] baseband type selected
-   0: indicate 802.11a type
-   1: indicate 802.11b type
-   2: indicate 802.11g type
-*/
+ * 0: indicate 802.11a type
+ * 1: indicate 802.11b type
+ * 2: indicate 802.11g type
+ */
 
 #define BBP_TYPE_DEF 2
 
@@ -284,7 +284,8 @@ static int vnt_init_registers(struct vnt_private *priv)
calib_rx_iq = priv->eeprom[EEP_OFS_CALIB_RX_IQ];
if (calib_tx_iq || calib_tx_dc || calib_rx_iq) {
/* CR255, enable TX/RX IQ and
-  DC compensation mode */
+* DC compensation mode
+*/
vnt_control_out_u8(priv,
   MESSAGE_REQUEST_BBREG,
   0xff,
@@ -306,7 +307,8 @@ static int vnt_init_registers(struct vnt_private *priv)
   calib_rx_iq);
} else {
/* CR255, turn off
-  BB Calibration compensation */
+* BB Calibration compensation
+*/
vnt_control_out_u8(priv,
   MESSAGE_REQUEST_BBREG,
   0xff,
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] Staging: comedi: fix block comments coding style issue in comedi.h

2015-12-21 Thread maomao xu
Fix up block comments to make a trailing */ on a separate line

Signed-off-by: maomao xu 
---
 drivers/staging/comedi/comedi.h |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/comedi/comedi.h b/drivers/staging/comedi/comedi.h
index 66edda1..ec5e59c 100644
--- a/drivers/staging/comedi/comedi.h
+++ b/drivers/staging/comedi/comedi.h
@@ -895,7 +895,8 @@ enum amplc_dio_ts_clock_src {
 };
 
 /* Values for setting a gate source with INSN_CONFIG_SET_GATE_SRC for
- * 8254 counter subdevices on Amplicon DIO boards (amplc_dio200 driver). */
+ * 8254 counter subdevices on Amplicon DIO boards (amplc_dio200 driver).
+ */
 enum amplc_dio_gate_source {
AMPLC_DIO_GAT_VCC,  /* internal high logic level */
AMPLC_DIO_GAT_GND,  /* internal low logic level */
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] Staging: comedi: fix block comments coding style issue in comedi.h

2015-12-21 Thread maomao xu
Fix up block comments to make a trailing */ on a separate line

Signed-off-by: maomao xu <albert008...@gmail.com>
---
 drivers/staging/comedi/comedi.h |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/comedi/comedi.h b/drivers/staging/comedi/comedi.h
index 66edda1..ec5e59c 100644
--- a/drivers/staging/comedi/comedi.h
+++ b/drivers/staging/comedi/comedi.h
@@ -895,7 +895,8 @@ enum amplc_dio_ts_clock_src {
 };
 
 /* Values for setting a gate source with INSN_CONFIG_SET_GATE_SRC for
- * 8254 counter subdevices on Amplicon DIO boards (amplc_dio200 driver). */
+ * 8254 counter subdevices on Amplicon DIO boards (amplc_dio200 driver).
+ */
 enum amplc_dio_gate_source {
AMPLC_DIO_GAT_VCC,  /* internal high logic level */
AMPLC_DIO_GAT_GND,  /* internal low logic level */
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] Staging: comedi: fix block comments coding style issue in comedi.h

2015-12-13 Thread maomao xu
Fix up block comments to make a trailing */ on a separate line

Signed-off-by: maomao xu 
---
 drivers/staging/comedi/comedi.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/comedi/comedi.h b/drivers/staging/comedi/comedi.h
index 66edda1..ec5e59c 100644
--- a/drivers/staging/comedi/comedi.h
+++ b/drivers/staging/comedi/comedi.h
@@ -895,7 +895,8 @@ enum amplc_dio_ts_clock_src {
 };
 
 /* Values for setting a gate source with INSN_CONFIG_SET_GATE_SRC for
- * 8254 counter subdevices on Amplicon DIO boards (amplc_dio200 driver). */
+ * 8254 counter subdevices on Amplicon DIO boards (amplc_dio200 driver).
+ */
 enum amplc_dio_gate_source {
AMPLC_DIO_GAT_VCC,  /* internal high logic level */
AMPLC_DIO_GAT_GND,  /* internal low logic level */
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] Staging: comedi: fix block comments coding style issue in comedi.h

2015-12-13 Thread maomao xu
Fix up block comments to make a trailing */ on a separate line

Signed-off-by: maomao xu <albert008...@gmail.com>
---
 drivers/staging/comedi/comedi.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/comedi/comedi.h b/drivers/staging/comedi/comedi.h
index 66edda1..ec5e59c 100644
--- a/drivers/staging/comedi/comedi.h
+++ b/drivers/staging/comedi/comedi.h
@@ -895,7 +895,8 @@ enum amplc_dio_ts_clock_src {
 };
 
 /* Values for setting a gate source with INSN_CONFIG_SET_GATE_SRC for
- * 8254 counter subdevices on Amplicon DIO boards (amplc_dio200 driver). */
+ * 8254 counter subdevices on Amplicon DIO boards (amplc_dio200 driver).
+ */
 enum amplc_dio_gate_source {
AMPLC_DIO_GAT_VCC,  /* internal high logic level */
AMPLC_DIO_GAT_GND,  /* internal low logic level */
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] Staging: comedi: fix block comments coding style issue in comedi.h

2015-11-18 Thread maomao xu
Fix up block comments to make a trailing */ on a separate line

Signed-off-by: maomao xu 
---
 drivers/staging/comedi/comedi.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/comedi/comedi.h b/drivers/staging/comedi/comedi.h
index 66edda1..ec5e59c 100644
--- a/drivers/staging/comedi/comedi.h
+++ b/drivers/staging/comedi/comedi.h
@@ -895,7 +895,8 @@ enum amplc_dio_ts_clock_src {
 };
 
 /* Values for setting a gate source with INSN_CONFIG_SET_GATE_SRC for
- * 8254 counter subdevices on Amplicon DIO boards (amplc_dio200 driver). */
+ * 8254 counter subdevices on Amplicon DIO boards (amplc_dio200 driver).
+ */
 enum amplc_dio_gate_source {
AMPLC_DIO_GAT_VCC,  /* internal high logic level */
AMPLC_DIO_GAT_GND,  /* internal low logic level */
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] Staging: comedi: fix block comments coding style issue in comedi.h

2015-11-18 Thread maomao xu
Fix up block comments to make a trailing */ on a separate line

Signed-off-by: maomao xu <albert008...@gmail.com>
---
 drivers/staging/comedi/comedi.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/comedi/comedi.h b/drivers/staging/comedi/comedi.h
index 66edda1..ec5e59c 100644
--- a/drivers/staging/comedi/comedi.h
+++ b/drivers/staging/comedi/comedi.h
@@ -895,7 +895,8 @@ enum amplc_dio_ts_clock_src {
 };
 
 /* Values for setting a gate source with INSN_CONFIG_SET_GATE_SRC for
- * 8254 counter subdevices on Amplicon DIO boards (amplc_dio200 driver). */
+ * 8254 counter subdevices on Amplicon DIO boards (amplc_dio200 driver).
+ */
 enum amplc_dio_gate_source {
AMPLC_DIO_GAT_VCC,  /* internal high logic level */
AMPLC_DIO_GAT_GND,  /* internal low logic level */
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/