staging: rtl8723au: core: remove redundant endianness conversion
authorDavid Decotigny <ddecotig@gmail.com>
Mon, 8 Jun 2015 00:43:02 +0000 (17:43 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 8 Jun 2015 20:23:53 +0000 (13:23 -0700)
Source and destination have the same little-endian annotation: this
patch removes incorrect byte-swap on non-LE cpus.

This addresses the following sparse warning:
drivers/staging/rtl8723au/core/rtw_mlme_ext.c:3911:56: warning: incorrect type in argument 1 (different base types)
drivers/staging/rtl8723au/core/rtw_mlme_ext.c:3911:56:    expected unsigned short [unsigned] [usertype] val
drivers/staging/rtl8723au/core/rtw_mlme_ext.c:3911:56:    got restricted __le16 [usertype] BA_timeout_value

Signed-off-by: David Decotigny <ddecotig@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723au/core/rtw_mlme_ext.c

index 7c3b5dd177d2933d02c4be5241c37a467bfb3e99..142f214108c6f87b33432a94c0020da54b971fe3 100644 (file)
@@ -3906,8 +3906,8 @@ void issue_action_BA23a(struct rtw_adapter *padapter,
                put_unaligned_le16(BA_para_set,
                                   &mgmt->u.action.u.addba_resp.capab);
 
-               put_unaligned_le16(pmlmeinfo->ADDBA_req.BA_timeout_value,
-                                  &mgmt->u.action.u.addba_resp.timeout);
+               mgmt->u.action.u.addba_resp.timeout
+                       = pmlmeinfo->ADDBA_req.BA_timeout_value;
 
                pattrib->pktlen += 8;
                break;