[PATCH net-next] rtlwifi: rtl8192de: Use DEFINE_SPINLOCK() for spinlock

2021-04-06 Thread Huang Guobin
From: Guobin Huang spinlock can be initialized automatically with DEFINE_SPINLOCK() rather than explicitly calling spin_lock_init(). Reported-by: Hulk Robot Signed-off-by: Guobin Huang --- .../realtek/rtlwifi/rtl8192de/sw.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions

[PATCH net-next] rfkill: use DEFINE_SPINLOCK() for spinlock

2021-04-06 Thread Huang Guobin
From: Guobin Huang spinlock can be initialized automatically with DEFINE_SPINLOCK() rather than explicitly calling spin_lock_init(). Reported-by: Hulk Robot Signed-off-by: Guobin Huang --- net/rfkill/input.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/net/rfkill/inp

[PATCH net-next v2] net: mscc: ocelot: remove redundant dev_err call in vsc9959_mdio_bus_alloc()

2021-03-28 Thread Huang Guobin
From: Guobin Huang There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Reported-by: Hulk Robot Signed-off-by: Guobin Huang --- drivers/net/dsa/ocelot/felix_vsc9959.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-

[PATCH net-next v2] net: stmmac: remove redundant dev_err call in qcom_ethqos_probe()

2021-03-28 Thread Huang Guobin
From: Guobin Huang There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Reported-by: Hulk Robot Signed-off-by: Guobin Huang --- .../net/ethernet/stmicro/stmmac/dwmac-qcom-ethqos.c | 1 - 1 file changed, 1 deletion(-) dif

[PATCH net-next v2] net: axienet: Remove redundant dev_err call in axienet_probe()

2021-03-28 Thread Huang Guobin
From: Guobin Huang There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Reported-by: Hulk Robot Signed-off-by: Guobin Huang --- drivers/net/ethernet/xilinx/xilinx_axienet_main.c | 1 - 1 file changed, 1 deletion(-) diff -

[PATCH net-next v2] net: mdio: Remove redundant dev_err call in mdio_mux_iproc_probe()

2021-03-28 Thread Huang Guobin
From: Guobin Huang There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Reported-by: Hulk Robot Signed-off-by: Guobin Huang --- drivers/net/mdio/mdio-mux-bcm-iproc.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)

[PATCH net-next v2] net: moxa: remove redundant dev_err call in moxart_mac_probe()

2021-03-27 Thread Huang Guobin
From: Guobin Huang There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Reported-by: Hulk Robot Signed-off-by: Guobin Huang --- drivers/net/ethernet/moxa/moxart_ether.c | 1 - 1 file changed, 1 deletion(-) diff --git a/dr

[PATCH net-next v2] net: lantiq: Remove redundant dev_err call in xrx200_probe()

2021-03-27 Thread Huang Guobin
From: Guobin Huang There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Reported-by: Hulk Robot Signed-off-by: Guobin Huang --- drivers/net/ethernet/lantiq_xrx200.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)

[PATCH net-next v2] net: netcp: fix PM reference leak in netcp_probe()

2021-03-27 Thread Huang Guobin
From: Guobin Huang pm_runtime_get_sync will increment pm usage counter even it failed. Forgetting to putting operation will result in reference leak here. Fix it by replacing it with pm_runtime_resume_and_get to keep usage counter balanced. Reported-by: Hulk Robot Signed-off-by: Guobin Huang -

[PATCH net-next v2] net: dsa: hellcreek: Remove redundant dev_err call in hellcreek_probe()

2021-03-27 Thread Huang Guobin
From: Guobin Huang There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Reported-by: Hulk Robot Signed-off-by: Guobin Huang --- drivers/net/dsa/hirschmann/hellcreek.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deleti

[PATCH net-next] mac80211_hwsim: use DEFINE_SPINLOCK() for spinlock

2021-03-27 Thread Huang Guobin
From: Guobin Huang spinlock can be initialized automatically with DEFINE_SPINLOCK() rather than explicitly calling spin_lock_init(). Reported-by: Hulk Robot Signed-off-by: Guobin Huang --- drivers/net/wireless/mac80211_hwsim.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) dif

[PATCH net-next] mt76: mt7615: remove redundant dev_err call in mt7622_wmac_probe()

2021-03-27 Thread Huang Guobin
From: Guobin Huang There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Reported-by: Hulk Robot Signed-off-by: Guobin Huang --- drivers/net/wireless/mediatek/mt76/mt7615/soc.c | 4 +--- 1 file changed, 1 insertion(+), 3 de

[PATCH net] net: wilc1000: clean up resource in error path of init mon interface

2020-09-17 Thread Huang Guobin
wfi_mon' static variable") Reported-by: Hulk Robot Signed-off-by: Huang Guobin --- drivers/net/wireless/microchip/wilc1000/mon.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/wireless/microchip/wilc1000/mon.c b/drivers/net/wireless/microchip/wilc1000

[PATCH net-next] tipc: Use is_broadcast_ether_addr() instead of memcmp()

2020-08-02 Thread Huang Guobin
Using is_broadcast_ether_addr() instead of directly use memcmp() to determine if the ethernet address is broadcast address. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Huang Guobin --- net/tipc/eth_media.c | 4 +--- 1 file changed, 1

[PATCH] net: ag71xx: add missed clk_disable_unprepare in error path of probe

2020-07-19 Thread Huang Guobin
The ag71xx_mdio_probe() forgets to call clk_disable_unprepare() when of_reset_control_get_exclusive() failed. Add the missed call to fix it. Fixes: d51b6ce441d3 ("net: ethernet: add ag71xx driver") Reported-by: Hulk Robot Signed-off-by: Huang Guobin --- drivers/net/ethernet/athero