linux.git/drivers/staging/rtl8188eu, branch v4.9 Linux kernel source tree staging: rtl8188eu: fix double unlock error in rtw_resume_process() 2016-10-02T15:32:17+00:00 Wei Yongjun weiyj.lk@gmail.com 2016-09-30T16:38:08+00:00 23bf40424a0f641ca7ff4225add4aa592086bdd5 Fix following static checker warning: drivers/staging/rtl8188eu/os_dep/usb_intf.c:311 rtw_resume_process() error: double unlock 'mutex:&pwrpriv->mutex_lock' Fixes: eaf47b713b60 ("staging: rtl8188eu: fix missing unlock on error in rtw_resume_process()") Reported-By: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Wei Yongjun <weiyj.lk@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
Fix following static checker warning:
    drivers/staging/rtl8188eu/os_dep/usb_intf.c:311 rtw_resume_process()
    error: double unlock 'mutex:&pwrpriv->mutex_lock'

Fixes: eaf47b713b60 ("staging: rtl8188eu: fix missing unlock on error in rtw_resume_process()")
Reported-By: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Wei Yongjun <weiyj.lk@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging:r8188eu: remove GEN_MLME_EXT_HANDLER macro 2016-10-02T15:31:16+00:00 Ivan Safonov insafonov@gmail.com 2016-09-30T14:36:21+00:00 612e1c94bfe9736cef0a9b86db792fd863be7733 GEN_MLME_EXT_HANDLER is redundant macro. Signed-off-by: Ivan Safonov <insafonov@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
GEN_MLME_EXT_HANDLER is redundant macro.

Signed-off-by: Ivan Safonov <insafonov@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging:r8188eu: remove GEN_DRV_CMD_HANDLER macro 2016-10-02T15:31:15+00:00 Ivan Safonov insafonov@gmail.com 2016-09-30T14:36:20+00:00 d88dd94e24cf45d519f93b4ad07922441ebb5cb3 This macro does not used. Signed-off-by: Ivan Safonov <insafonov@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
This macro does not used.

Signed-off-by: Ivan Safonov <insafonov@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging:r8188eu: remove GEN_EVT_CODE macro 2016-10-02T15:31:15+00:00 Ivan Safonov insafonov@gmail.com 2016-09-30T14:36:19+00:00 c1300df211b181e0ab9897c4def6b9ec320e8046 GEN_EVT_CODE is redundant macro. Signed-off-by: Ivan Safonov <insafonov@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
GEN_EVT_CODE is redundant macro.

Signed-off-by: Ivan Safonov <insafonov@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging:r8188eu: remove GEN_CMD_CODE macro 2016-10-02T15:31:15+00:00 Ivan Safonov insafonov@gmail.com 2016-09-30T14:36:18+00:00 ae7f7aa0c02bdedd4ff383d3b3fd3bc07cfef2a8 GEN_CMD_CODE is redundant macro. Signed-off-by: Ivan Safonov <insafonov@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
GEN_CMD_CODE is redundant macro.

Signed-off-by: Ivan Safonov <insafonov@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging:r8188eu: remove pkt_newalloc member of the recv_buf structure 2016-10-02T15:31:15+00:00 Ivan Safonov insafonov@gmail.com 2016-09-30T14:36:17+00:00 2e2f78d078ca8908a79155b80c16311d06f027be This member does not used. Signed-off-by: Ivan Safonov <insafonov@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
This member does not used.

Signed-off-by: Ivan Safonov <insafonov@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging:r8188eu: remove rtw_handle_dualmac declaration 2016-10-02T15:31:15+00:00 Ivan Safonov insafonov@gmail.com 2016-09-30T14:36:16+00:00 f34395c83325480cda7f8abfc03f6f52912ea314 It is a declaration of the non-existent function. Signed-off-by: Ivan Safonov <insafonov@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
It is a declaration of the non-existent function.

Signed-off-by: Ivan Safonov <insafonov@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging:r8188eu: remove (RGTRY|BSSID)_(OFT|SZ) macros 2016-10-02T15:31:15+00:00 Ivan Safonov insafonov@gmail.com 2016-09-30T14:36:15+00:00 64164d66a36dc6f95aa07c8d56c9a002c3a13f1e These macros does not used. Signed-off-by: Ivan Safonov <insafonov@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
These macros does not used.

Signed-off-by: Ivan Safonov <insafonov@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging:r8188eu: change rtl8188e_process_phy_info function argument type 2016-10-02T15:31:15+00:00 Ivan Safonov insafonov@gmail.com 2016-09-30T14:36:14+00:00 0699715183ad3a7708a72d74e2e09995d6822f1f prframe is (void *), but function used only with (struct recv_frame *). Signed-off-by: Ivan Safonov <insafonov@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
prframe is (void *), but function used only with (struct recv_frame *).

Signed-off-by: Ivan Safonov <insafonov@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: rtl8188eu: core: rtw_xmit: Use macros instead of constants 2016-10-02T15:26:57+00:00 Georgiana Rodica Chelu georgiana.chelu93@gmail.com 2016-09-28T21:07:15+00:00 7b170bacbb13089ccb1d14ca18ad4fb5dcb8b72e Replace the 0x888e with ETH_P_PAE and 0x0806 with ETH_P_ARP. These macros can be found in drivers/staging/rtl8192e/rtllib.h Hexadecimal numbers are not case sensitive, therefore 0x888e is equal with 0x888E. The modifications improve the readability of the code. Signed-off-by: Georgiana Rodica Chelu <georgiana.chelu93@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
Replace the 0x888e with ETH_P_PAE and 0x0806 with ETH_P_ARP.
These macros can be found in drivers/staging/rtl8192e/rtllib.h

Hexadecimal numbers are not case sensitive,
therefore 0x888e is equal with 0x888E.

The modifications improve the readability of the code.

Signed-off-by: Georgiana Rodica Chelu <georgiana.chelu93@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>