linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] staging: r8188eu: remove two unused function prototypes
@ 2022-07-24 15:33 Martin Kaiser
  2022-07-24 15:33 ` [PATCH 1/2] staging: r8188eu: remove OnAtim prototype Martin Kaiser
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Martin Kaiser @ 2022-07-24 15:33 UTC (permalink / raw)
  To: Greg Kroah-Hartman
  Cc: Larry Finger, Phillip Potter, Michael Straube, Pavel Skripkin,
	linux-staging, linux-kernel, Martin Kaiser

I forgot to remove the prototypes of OnAtim and DoReserved when those
functions were removed. Here's two patches to fix this.

Martin Kaiser (2):
  staging: r8188eu: remove OnAtim prototype
  staging: r8188eu: remove DoReserved prototype

 drivers/staging/r8188eu/include/rtw_mlme_ext.h | 4 ----
 1 file changed, 4 deletions(-)

-- 
2.30.2


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH 1/2] staging: r8188eu: remove OnAtim prototype
  2022-07-24 15:33 [PATCH 0/2] staging: r8188eu: remove two unused function prototypes Martin Kaiser
@ 2022-07-24 15:33 ` Martin Kaiser
  2022-07-24 15:33 ` [PATCH 2/2] staging: r8188eu: remove DoReserved prototype Martin Kaiser
  2022-07-24 18:20 ` [PATCH 0/2] staging: r8188eu: remove two unused function prototypes Philipp Hortmann
  2 siblings, 0 replies; 4+ messages in thread
From: Martin Kaiser @ 2022-07-24 15:33 UTC (permalink / raw)
  To: Greg Kroah-Hartman
  Cc: Larry Finger, Phillip Potter, Michael Straube, Pavel Skripkin,
	linux-staging, linux-kernel, Martin Kaiser

The OnAtim function was removed in commit 6d933902c609 ("staging: r8188eu:
remove OnAtim"). Remove the prototype as well.

Signed-off-by: Martin Kaiser <martin@kaiser.cx>
---
 drivers/staging/r8188eu/include/rtw_mlme_ext.h | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/staging/r8188eu/include/rtw_mlme_ext.h b/drivers/staging/r8188eu/include/rtw_mlme_ext.h
index c630700ea657..83d0980b5bf6 100644
--- a/drivers/staging/r8188eu/include/rtw_mlme_ext.h
+++ b/drivers/staging/r8188eu/include/rtw_mlme_ext.h
@@ -550,8 +550,6 @@ unsigned int DoReserved(struct adapter *padapter,
 			struct recv_frame *precv_frame);
 unsigned int OnBeacon(struct adapter *padapter,
 		      struct recv_frame *precv_frame);
-unsigned int OnAtim(struct adapter *padapter,
-		    struct recv_frame *precv_frame);
 unsigned int OnDisassoc(struct adapter *padapter,
 			struct recv_frame *precv_frame);
 unsigned int OnAuth(struct adapter *padapter,
-- 
2.30.2


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH 2/2] staging: r8188eu: remove DoReserved prototype
  2022-07-24 15:33 [PATCH 0/2] staging: r8188eu: remove two unused function prototypes Martin Kaiser
  2022-07-24 15:33 ` [PATCH 1/2] staging: r8188eu: remove OnAtim prototype Martin Kaiser
@ 2022-07-24 15:33 ` Martin Kaiser
  2022-07-24 18:20 ` [PATCH 0/2] staging: r8188eu: remove two unused function prototypes Philipp Hortmann
  2 siblings, 0 replies; 4+ messages in thread
From: Martin Kaiser @ 2022-07-24 15:33 UTC (permalink / raw)
  To: Greg Kroah-Hartman
  Cc: Larry Finger, Phillip Potter, Michael Straube, Pavel Skripkin,
	linux-staging, linux-kernel, Martin Kaiser

The DoReserved function was removed in commit 869ddbfccdea ("staging:
r8188eu: remove dummy entries from OnAction_tbl"). Remove the prototype
as well.

Signed-off-by: Martin Kaiser <martin@kaiser.cx>
---
 drivers/staging/r8188eu/include/rtw_mlme_ext.h | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/staging/r8188eu/include/rtw_mlme_ext.h b/drivers/staging/r8188eu/include/rtw_mlme_ext.h
index 83d0980b5bf6..2d7856c74542 100644
--- a/drivers/staging/r8188eu/include/rtw_mlme_ext.h
+++ b/drivers/staging/r8188eu/include/rtw_mlme_ext.h
@@ -546,8 +546,6 @@ unsigned int OnProbeReq(struct adapter *padapter,
 			struct recv_frame *precv_frame);
 unsigned int OnProbeRsp(struct adapter *padapter,
 			struct recv_frame *precv_frame);
-unsigned int DoReserved(struct adapter *padapter,
-			struct recv_frame *precv_frame);
 unsigned int OnBeacon(struct adapter *padapter,
 		      struct recv_frame *precv_frame);
 unsigned int OnDisassoc(struct adapter *padapter,
-- 
2.30.2


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 0/2] staging: r8188eu: remove two unused function prototypes
  2022-07-24 15:33 [PATCH 0/2] staging: r8188eu: remove two unused function prototypes Martin Kaiser
  2022-07-24 15:33 ` [PATCH 1/2] staging: r8188eu: remove OnAtim prototype Martin Kaiser
  2022-07-24 15:33 ` [PATCH 2/2] staging: r8188eu: remove DoReserved prototype Martin Kaiser
@ 2022-07-24 18:20 ` Philipp Hortmann
  2 siblings, 0 replies; 4+ messages in thread
From: Philipp Hortmann @ 2022-07-24 18:20 UTC (permalink / raw)
  To: Martin Kaiser, Greg Kroah-Hartman
  Cc: Larry Finger, Phillip Potter, Michael Straube, Pavel Skripkin,
	linux-staging, linux-kernel

On 7/24/22 17:33, Martin Kaiser wrote:
> I forgot to remove the prototypes of OnAtim and DoReserved when those
> functions were removed. Here's two patches to fix this.
> 
> Martin Kaiser (2):
>    staging: r8188eu: remove OnAtim prototype
>    staging: r8188eu: remove DoReserved prototype
> 
>   drivers/staging/r8188eu/include/rtw_mlme_ext.h | 4 ----
>   1 file changed, 4 deletions(-)
> 

Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com> # Edimax N150

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-07-24 18:20 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-24 15:33 [PATCH 0/2] staging: r8188eu: remove two unused function prototypes Martin Kaiser
2022-07-24 15:33 ` [PATCH 1/2] staging: r8188eu: remove OnAtim prototype Martin Kaiser
2022-07-24 15:33 ` [PATCH 2/2] staging: r8188eu: remove DoReserved prototype Martin Kaiser
2022-07-24 18:20 ` [PATCH 0/2] staging: r8188eu: remove two unused function prototypes Philipp Hortmann

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).