Staging: rtl8192u: Use put_unaligned_le16
authorVaishali Thakkar <vthakkar1994@gmail.com>
Fri, 31 Oct 2014 11:06:18 +0000 (16:36 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 4 Nov 2014 00:09:27 +0000 (16:09 -0800)
This patch introduces the use of function put_unaligned_le16.

This is done using Coccinelle and semantic patch used is as follows:

@@ identifier tmp; expression ptr; expression y,e; type T; @@

- tmp = cpu_to_le16(y);

  <+... when != tmp
- memcpy(ptr, (T)&tmp, ...);
+ put_unaligned_le16(y,ptr);
  ...+>
? tmp = e

@@ type T; identifier tmp; @@

- T tmp;
...when != tmp

Here, unnecessory comment after change is removed too.
This patch also fixes sparse warnings in file ieeee80211_softmac.c.

Signed-off-by: Vaishali Thakkar <vthakkar1994@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c

index 9b87408..d147187 100644 (file)
@@ -804,12 +804,11 @@ static struct sk_buff *ieee80211_probe_resp(struct ieee80211_device *ieee, u8 *d
        *(tag++) = ieee->current_network.channel;
 
        if(atim_len){
-       u16 val16;
                *(tag++) = MFIE_TYPE_IBSS_SET;
                *(tag++) = 2;
-               //*((u16*)(tag)) = cpu_to_le16(ieee->current_network.atim_window);
-                val16 = cpu_to_le16(ieee->current_network.atim_window);
-               memcpy((u8 *)tag, (u8 *)&val16, 2);
+
+               put_unaligned_le16(ieee->current_network.atim_window,
+                                  (u8 *)tag);
                tag+=2;
        }