diff options
| author | Martin Kaiser <martin@kaiser.cx> | 2022-10-30 18:33:23 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2022-10-31 09:09:04 +0100 |
| commit | a399a3b05209dc759b20c6d6e4aa331f2e60bc8a (patch) | |
| tree | 043d8a76afb7620b04c056549d8795e0a6d6a7d3 | |
| parent | a05159da5a4e49fd4b34cb00fb33badeb4dc9ec3 (diff) | |
| download | linux-a399a3b05209dc759b20c6d6e4aa331f2e60bc8a.tar.gz linux-a399a3b05209dc759b20c6d6e4aa331f2e60bc8a.zip | |
staging: r8188eu: remove return value from on_action_public_vendor
The only caller of on_action_public_vendor does not check the return
value. We can make it a void function.
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com> # Edimax N150
Signed-off-by: Martin Kaiser <martin@kaiser.cx>
Link: https://lore.kernel.org/r/20221030173326.1588647-11-martin@kaiser.cx
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/r8188eu/core/rtw_mlme_ext.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/drivers/staging/r8188eu/core/rtw_mlme_ext.c b/drivers/staging/r8188eu/core/rtw_mlme_ext.c index b3cef3504ad3..b395457a6a60 100644 --- a/drivers/staging/r8188eu/core/rtw_mlme_ext.c +++ b/drivers/staging/r8188eu/core/rtw_mlme_ext.c @@ -3779,17 +3779,13 @@ static unsigned int on_action_public_p2p(struct recv_frame *precv_frame) return _SUCCESS; } -static unsigned int on_action_public_vendor(struct recv_frame *precv_frame) +static void on_action_public_vendor(struct recv_frame *precv_frame) { - unsigned int ret = _FAIL; u8 *pframe = precv_frame->rx_data; u8 *frame_body = pframe + sizeof(struct ieee80211_hdr_3addr); - if (!memcmp(frame_body + 2, P2P_OUI, 4)) { - ret = on_action_public_p2p(precv_frame); - } - - return ret; + if (!memcmp(frame_body + 2, P2P_OUI, 4)) + on_action_public_p2p(precv_frame); } static unsigned int on_action_public_default(struct recv_frame *precv_frame) |