diff mbox series

wifi: brcm80211: remove unused structs

Message ID 20240526234553.286773-1-linux@treblig.org (mailing list archive)
State Accepted
Commit 5bcd9a0a59953b5ff55ac226f903397319bf7f40
Delegated to: Kalle Valo
Headers show
Series wifi: brcm80211: remove unused structs | expand

Commit Message

Dr. David Alan Gilbert May 26, 2024, 11:45 p.m. UTC
From: "Dr. David Alan Gilbert" <linux@treblig.org>

'brcmf_pcie_core_info' was added in
commit 9e37f045d5e7 ("brcmfmac: Adding PCIe bus layer support.")
but never used.

'brcms_c_bit_desc' last use was removed in
commit cdf4352f5c59 ("brcmsmac: Improve tx trace and debug support").

Remove them.

Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
---
 drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c | 5 -----
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/main.c | 6 ------
 2 files changed, 11 deletions(-)

Comments

Arend van Spriel May 27, 2024, 12:49 p.m. UTC | #1
On 5/27/2024 1:45 AM, linux@treblig.org wrote:
> From: "Dr. David Alan Gilbert" <linux@treblig.org>
> 
> 'brcmf_pcie_core_info' was added in
> commit 9e37f045d5e7 ("brcmfmac: Adding PCIe bus layer support.")
> but never used.
> 
> 'brcms_c_bit_desc' last use was removed in
> commit cdf4352f5c59 ("brcmsmac: Improve tx trace and debug support").
> 
> Remove them.

Thanks for cleaning this up.

Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com>
> Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> ---
>   drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c | 5 -----
>   drivers/net/wireless/broadcom/brcm80211/brcmsmac/main.c | 6 ------
>   2 files changed, 11 deletions(-)
Kalle Valo June 1, 2024, 10:09 a.m. UTC | #2
linux@treblig.org wrote:

> From: "Dr. David Alan Gilbert" <linux@treblig.org>
> 
> 'brcmf_pcie_core_info' was added in
> commit 9e37f045d5e7 ("brcmfmac: Adding PCIe bus layer support.")
> but never used.
> 
> 'brcms_c_bit_desc' last use was removed in
> commit cdf4352f5c59 ("brcmsmac: Improve tx trace and debug support").
> 
> Remove them.
> 
> Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com>

Patch applied to wireless-next.git, thanks.

5bcd9a0a5995 wifi: brcm80211: remove unused structs
diff mbox series

Patch

diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c
index 06698a714b52..ce482a3877e9 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c
@@ -313,11 +313,6 @@  struct brcmf_pcie_shared_info {
 	u8 version;
 };
 
-struct brcmf_pcie_core_info {
-	u32 base;
-	u32 wrapbase;
-};
-
 #define BRCMF_OTP_MAX_PARAM_LEN 16
 
 struct brcmf_otp_params {
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmsmac/main.c b/drivers/net/wireless/broadcom/brcm80211/brcmsmac/main.c
index 34460b5815d0..4576b18fcd97 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmsmac/main.c
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmsmac/main.c
@@ -234,12 +234,6 @@ 
 /* max # tx status to process in wlc_txstatus() */
 #define TXSBND				8
 
-/* brcmu_format_flags() bit description structure */
-struct brcms_c_bit_desc {
-	u32 bit;
-	const char *name;
-};
-
 /*
  * The following table lists the buffer memory allocated to xmt fifos in HW.
  * the size is in units of 256bytes(one block), total size is HW dependent