From c65a5a562e3b55297767a4022543867e01a66080 Mon Sep 17 00:00:00 2001 From: Vaishali Thakkar Date: Wed, 29 Oct 2014 19:30:41 +0530 Subject: [PATCH] Staging: rtl8192e: Revert one previous commit This patch reverts commit 450246465a76 ("Staging: rtl8192e: Fix incorrect type in assignment in rtl819x_BAProc.c") as it is changing code and introducing bug to skip the endian conversion. Here, tmp variable is used to hold the endian-corrected values and network-data requires fixed endianness. Signed-off-by: Vaishali Thakkar Signed-off-by: Greg Kroah-Hartman --- drivers/staging/rtl8192e/rtl819x_BAProc.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8192e/rtl819x_BAProc.c b/drivers/staging/rtl8192e/rtl819x_BAProc.c index 6da57847a533..946c1dc25de5 100644 --- a/drivers/staging/rtl8192e/rtl819x_BAProc.c +++ b/drivers/staging/rtl8192e/rtl819x_BAProc.c @@ -115,14 +115,14 @@ static struct sk_buff *rtllib_ADDBA(struct rtllib_device *ieee, u8 *Dst, if (ACT_ADDBARSP == type) { RT_TRACE(COMP_DBG, "====>to send ADDBARSP\n"); - tmp = StatusCode; + tmp = cpu_to_le16(StatusCode); memcpy(tag, (u8 *)&tmp, 2); tag += 2; } - tmp = pBA->BaParamSet.shortData; + tmp = cpu_to_le16(pBA->BaParamSet.shortData); memcpy(tag, (u8 *)&tmp, 2); tag += 2; - tmp = pBA->BaTimeoutValue; + tmp = cpu_to_le16(pBA->BaTimeoutValue); memcpy(tag, (u8 *)&tmp, 2); tag += 2; @@ -178,10 +178,10 @@ static struct sk_buff *rtllib_DELBA(struct rtllib_device *ieee, u8 *dst, *tag++ = ACT_CAT_BA; *tag++ = ACT_DELBA; - tmp = DelbaParamSet.shortData; + tmp = cpu_to_le16(DelbaParamSet.shortData); memcpy(tag, (u8 *)&tmp, 2); tag += 2; - tmp = ReasonCode; + tmp = cpu_to_le16(ReasonCode); memcpy(tag, (u8 *)&tmp, 2); tag += 2; -- 2.20.1