diff options
author | Hante Meuleman <meuleman@broadcom.com> | 2014-09-30 04:23:14 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2014-09-30 13:17:47 -0400 |
commit | ff0a6230603c089fdc12a06b824c21498eb24691 (patch) | |
tree | a04ca811a36163bfaec829bfc995be4166addaf7 /drivers/net/wireless/brcm80211 | |
parent | ef8596e1f1ce06b4398db4752d04bb1a67a14a02 (diff) |
brcmfmac: Fix sign issue with IOCTL return code in msgbuf.
Need a cast to assure correct value is propagated.
Reviewed-by: Franky (Zhenhui) Lin <frankyl@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Reviewed-by: Daniel (Deognyoun) Kim <dekim@broadcom.com>
Signed-off-by: Hante Meuleman <meuleman@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/brcm80211')
-rw-r--r-- | drivers/net/wireless/brcm80211/brcmfmac/msgbuf.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/net/wireless/brcm80211/brcmfmac/msgbuf.c b/drivers/net/wireless/brcm80211/brcmfmac/msgbuf.c index 8f8b9373de95..106e6bf0a5da 100644 --- a/drivers/net/wireless/brcm80211/brcmfmac/msgbuf.c +++ b/drivers/net/wireless/brcm80211/brcmfmac/msgbuf.c | |||
@@ -230,7 +230,7 @@ struct brcmf_msgbuf { | |||
230 | dma_addr_t ioctbuf_handle; | 230 | dma_addr_t ioctbuf_handle; |
231 | u32 ioctbuf_phys_hi; | 231 | u32 ioctbuf_phys_hi; |
232 | u32 ioctbuf_phys_lo; | 232 | u32 ioctbuf_phys_lo; |
233 | u32 ioctl_resp_status; | 233 | int ioctl_resp_status; |
234 | u32 ioctl_resp_ret_len; | 234 | u32 ioctl_resp_ret_len; |
235 | u32 ioctl_resp_pktid; | 235 | u32 ioctl_resp_pktid; |
236 | 236 | ||
@@ -767,7 +767,8 @@ brcmf_msgbuf_process_ioctl_complete(struct brcmf_msgbuf *msgbuf, void *buf) | |||
767 | 767 | ||
768 | ioctl_resp = (struct msgbuf_ioctl_resp_hdr *)buf; | 768 | ioctl_resp = (struct msgbuf_ioctl_resp_hdr *)buf; |
769 | 769 | ||
770 | msgbuf->ioctl_resp_status = le16_to_cpu(ioctl_resp->compl_hdr.status); | 770 | msgbuf->ioctl_resp_status = |
771 | (s16)le16_to_cpu(ioctl_resp->compl_hdr.status); | ||
771 | msgbuf->ioctl_resp_ret_len = le16_to_cpu(ioctl_resp->resp_len); | 772 | msgbuf->ioctl_resp_ret_len = le16_to_cpu(ioctl_resp->resp_len); |
772 | msgbuf->ioctl_resp_pktid = le32_to_cpu(ioctl_resp->msg.request_id); | 773 | msgbuf->ioctl_resp_pktid = le32_to_cpu(ioctl_resp->msg.request_id); |
773 | 774 | ||