diff options
| author | Martin Kaiser <[email protected]> | 2022-10-30 18:33:24 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <[email protected]> | 2022-10-31 09:09:05 +0100 |
| commit | 8161a8335821bb6944eef2de8b750544d59c49c2 (patch) | |
| tree | aa99365b2229807efa4dd54357bed62858773005 | |
| parent | a399a3b05209dc759b20c6d6e4aa331f2e60bc8a (diff) | |
staging: r8188eu: remove return value from on_action_public_default
The only caller of on_action_public_default does not check the return
value. We can make it a void function.
Tested-by: Philipp Hortmann <[email protected]> # Edimax N150
Signed-off-by: Martin Kaiser <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Greg Kroah-Hartman <[email protected]>
| -rw-r--r-- | drivers/staging/r8188eu/core/rtw_mlme_ext.c | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/drivers/staging/r8188eu/core/rtw_mlme_ext.c b/drivers/staging/r8188eu/core/rtw_mlme_ext.c index b395457a6a60..f5923792f067 100644 --- a/drivers/staging/r8188eu/core/rtw_mlme_ext.c +++ b/drivers/staging/r8188eu/core/rtw_mlme_ext.c @@ -3788,22 +3788,15 @@ static void on_action_public_vendor(struct recv_frame *precv_frame) on_action_public_p2p(precv_frame); } -static unsigned int on_action_public_default(struct recv_frame *precv_frame) +static void on_action_public_default(struct recv_frame *precv_frame) { - unsigned int ret = _FAIL; u8 *pframe = precv_frame->rx_data; u8 *frame_body = pframe + sizeof(struct ieee80211_hdr_3addr); u8 token; token = frame_body[2]; - if (rtw_action_public_decache(precv_frame, token) == _FAIL) - goto exit; - - ret = _SUCCESS; - -exit: - return ret; + rtw_action_public_decache(precv_frame, token); } static void on_action_public(struct adapter *padapter, struct recv_frame *precv_frame) |