2021-03-31 02:40:18

by Wan Jiabing

[permalink] [raw]
Subject: [PATCH 0/4] net: Remove duplicate struct declaration

This patch series tried to remove duplicate struct declaration.

Wan Jiabing (4):
net: wireless: broadcom: Remove duplicate struct declaration
net: wireless: microchip: Remove duplicate struct declaration
net: wireless: marvell: Remove duplicate struct declaration
net: ethernet: stmicro: Remove duplicate struct declaration

drivers/net/ethernet/stmicro/stmmac/hwif.h | 1 -
drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.h | 1 -
drivers/net/wireless/marvell/libertas_tf/libertas_tf.h | 1 -
drivers/net/wireless/microchip/wilc1000/wlan.h | 1 -
4 files changed, 4 deletions(-)

--
2.25.1


2021-03-31 02:40:44

by Wan Jiabing

[permalink] [raw]
Subject: [PATCH 4/4] net: ethernet: stmicro: Remove duplicate struct declaration

struct stmmac_safety_stats is declared twice. One has been
declared at 29th line. Remove the duplicate.

Signed-off-by: Wan Jiabing <[email protected]>
---
drivers/net/ethernet/stmicro/stmmac/hwif.h | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/net/ethernet/stmicro/stmmac/hwif.h b/drivers/net/ethernet/stmicro/stmmac/hwif.h
index 979ac9fca23c..9139bbcc06c0 100644
--- a/drivers/net/ethernet/stmicro/stmmac/hwif.h
+++ b/drivers/net/ethernet/stmicro/stmmac/hwif.h
@@ -280,7 +280,6 @@ struct stmmac_dma_ops {
struct mac_device_info;
struct net_device;
struct rgmii_adv;
-struct stmmac_safety_stats;
struct stmmac_tc_entry;
struct stmmac_pps_cfg;
struct stmmac_rss;
--
2.25.1

2021-03-31 02:40:44

by Wan Jiabing

[permalink] [raw]
Subject: [PATCH 2/4] net: wireless: microchip: Remove duplicate struct declaration

struct wilc is declared twice. One has been declared
at 352nd line. Remove the duplicate.

Signed-off-by: Wan Jiabing <[email protected]>
---
drivers/net/wireless/microchip/wilc1000/wlan.h | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/net/wireless/microchip/wilc1000/wlan.h b/drivers/net/wireless/microchip/wilc1000/wlan.h
index d55eb6b3a12a..0d24e711b62b 100644
--- a/drivers/net/wireless/microchip/wilc1000/wlan.h
+++ b/drivers/net/wireless/microchip/wilc1000/wlan.h
@@ -392,7 +392,6 @@ struct wilc_cfg_rsp {
u8 seq_no;
};

-struct wilc;
struct wilc_vif;

int wilc_wlan_firmware_download(struct wilc *wilc, const u8 *buffer,
--
2.25.1

2021-03-31 02:40:45

by Wan Jiabing

[permalink] [raw]
Subject: [PATCH 3/4] net: wireless: marvell: Remove duplicate struct declaration

struct lbtf_private is declared twice. One has been declared
at 157th line. Remove the duplicate.

Signed-off-by: Wan Jiabing <[email protected]>
---
drivers/net/wireless/marvell/libertas_tf/libertas_tf.h | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/net/wireless/marvell/libertas_tf/libertas_tf.h b/drivers/net/wireless/marvell/libertas_tf/libertas_tf.h
index 67bbb6a8f113..5d726545d987 100644
--- a/drivers/net/wireless/marvell/libertas_tf/libertas_tf.h
+++ b/drivers/net/wireless/marvell/libertas_tf/libertas_tf.h
@@ -453,7 +453,6 @@ struct cmd_ds_802_11_beacon_set {
u8 beacon[MRVL_MAX_BCN_SIZE];
};

-struct lbtf_private;
struct cmd_ctrl_node;

/** Function Prototype Declaration */
--
2.25.1

2021-03-31 02:40:46

by Wan Jiabing

[permalink] [raw]
Subject: [PATCH 1/4] net: wireless: broadcom: Remove duplicate struct declaration

struct brcmf_bus is declared twice. One has been declared
at 37th line. Remove the duplicate.

Signed-off-by: Wan Jiabing <[email protected]>
---
drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.h | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.h b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.h
index 4146faeed344..44ba6f389fa9 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.h
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.h
@@ -112,7 +112,6 @@ do { \

extern int brcmf_msg_level;

-struct brcmf_bus;
struct brcmf_pub;
#ifdef DEBUG
struct dentry *brcmf_debugfs_get_devdir(struct brcmf_pub *drvr);
--
2.25.1

2021-04-18 06:36:46

by Kalle Valo

[permalink] [raw]
Subject: Re: [1/4] brcmfmac: Remove duplicate struct declaration

Wan Jiabing <[email protected]> wrote:

> struct brcmf_bus is declared twice. One has been declared
> at 37th line. Remove the duplicate.
>
> Signed-off-by: Wan Jiabing <[email protected]>

2 patches applied to wireless-drivers-next.git, thanks.

d663bc3317c9 brcmfmac: Remove duplicate struct declaration
444a9af68b5c wilc1000: Remove duplicate struct declaration

--
https://patchwork.kernel.org/project/linux-wireless/patch/[email protected]/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

2021-04-21 09:39:38

by Kalle Valo

[permalink] [raw]
Subject: Re: [3/4] libertas_tf: Remove duplicate struct declaration

Wan Jiabing <[email protected]> wrote:

> struct lbtf_private is declared twice. One has been declared
> at 157th line. Remove the duplicate.
>
> Signed-off-by: Wan Jiabing <[email protected]>

Patch applied to wireless-drivers-next.git, thanks.

ff0224e97d5d libertas_tf: Remove duplicate struct declaration

--
https://patchwork.kernel.org/project/linux-wireless/patch/[email protected]/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches