Commit 0f24cd70 authored by Vaishali Thakkar's avatar Vaishali Thakkar Committed by Greg Kroah-Hartman

Staging: rtl8192e: Use put_unaligned_le16

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, to be compatible with the change header file is added too.
Signed-off-by: default avatarVaishali Thakkar <vthakkar1994@gmail.com>
Reviewed-by: default avatarArnd Bergmann <arnd@arndb.de>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent f5333f89
...@@ -16,6 +16,7 @@ ...@@ -16,6 +16,7 @@
* Contact Information: * Contact Information:
* wlanfae <wlanfae@realtek.com> * wlanfae <wlanfae@realtek.com>
******************************************************************************/ ******************************************************************************/
#include <linux/unaligned/access_ok.h>
#include "rtllib.h" #include "rtllib.h"
#include "rtl819x_BA.h" #include "rtl819x_BA.h"
...@@ -79,7 +80,6 @@ static struct sk_buff *rtllib_ADDBA(struct rtllib_device *ieee, u8 *Dst, ...@@ -79,7 +80,6 @@ static struct sk_buff *rtllib_ADDBA(struct rtllib_device *ieee, u8 *Dst,
struct sk_buff *skb = NULL; struct sk_buff *skb = NULL;
struct rtllib_hdr_3addr *BAReq = NULL; struct rtllib_hdr_3addr *BAReq = NULL;
u8 *tag = NULL; u8 *tag = NULL;
__le16 tmp = 0;
u16 len = ieee->tx_headroom + 9; u16 len = ieee->tx_headroom + 9;
RTLLIB_DEBUG(RTLLIB_DL_TRACE | RTLLIB_DL_BA, "========>%s(), frame(%d)" RTLLIB_DEBUG(RTLLIB_DL_TRACE | RTLLIB_DL_BA, "========>%s(), frame(%d)"
...@@ -115,15 +115,15 @@ static struct sk_buff *rtllib_ADDBA(struct rtllib_device *ieee, u8 *Dst, ...@@ -115,15 +115,15 @@ static struct sk_buff *rtllib_ADDBA(struct rtllib_device *ieee, u8 *Dst,
if (ACT_ADDBARSP == type) { if (ACT_ADDBARSP == type) {
RT_TRACE(COMP_DBG, "====>to send ADDBARSP\n"); RT_TRACE(COMP_DBG, "====>to send ADDBARSP\n");
tmp = cpu_to_le16(StatusCode);
memcpy(tag, (u8 *)&tmp, 2); put_unaligned_le16(StatusCode, tag);
tag += 2; tag += 2;
} }
tmp = cpu_to_le16(pBA->BaParamSet.shortData);
memcpy(tag, (u8 *)&tmp, 2); put_unaligned_le16(pBA->BaParamSet.shortData, tag);
tag += 2; tag += 2;
tmp = cpu_to_le16(pBA->BaTimeoutValue);
memcpy(tag, (u8 *)&tmp, 2); put_unaligned_le16(pBA->BaTimeoutValue, tag);
tag += 2; tag += 2;
if (ACT_ADDBAREQ == type) { if (ACT_ADDBAREQ == type) {
...@@ -143,7 +143,6 @@ static struct sk_buff *rtllib_DELBA(struct rtllib_device *ieee, u8 *dst, ...@@ -143,7 +143,6 @@ static struct sk_buff *rtllib_DELBA(struct rtllib_device *ieee, u8 *dst,
struct sk_buff *skb = NULL; struct sk_buff *skb = NULL;
struct rtllib_hdr_3addr *Delba = NULL; struct rtllib_hdr_3addr *Delba = NULL;
u8 *tag = NULL; u8 *tag = NULL;
__le16 tmp = 0;
u16 len = 6 + ieee->tx_headroom; u16 len = 6 + ieee->tx_headroom;
if (net_ratelimit()) if (net_ratelimit())
...@@ -178,11 +177,11 @@ static struct sk_buff *rtllib_DELBA(struct rtllib_device *ieee, u8 *dst, ...@@ -178,11 +177,11 @@ static struct sk_buff *rtllib_DELBA(struct rtllib_device *ieee, u8 *dst,
*tag++ = ACT_CAT_BA; *tag++ = ACT_CAT_BA;
*tag++ = ACT_DELBA; *tag++ = ACT_DELBA;
tmp = cpu_to_le16(DelbaParamSet.shortData);
memcpy(tag, (u8 *)&tmp, 2); put_unaligned_le16(DelbaParamSet.shortData, tag);
tag += 2; tag += 2;
tmp = cpu_to_le16(ReasonCode);
memcpy(tag, (u8 *)&tmp, 2); put_unaligned_le16(ReasonCode, tag);
tag += 2; tag += 2;
RTLLIB_DEBUG_DATA(RTLLIB_DL_DATA|RTLLIB_DL_BA, skb->data, skb->len); RTLLIB_DEBUG_DATA(RTLLIB_DL_DATA|RTLLIB_DL_BA, skb->data, skb->len);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment