Re: [PATCH v2 00/10] Rid W=1 warnings from OF

2021-03-24 Thread Lee Jones
On Wed, 24 Mar 2021, Rob Herring wrote: > On Tue, Mar 23, 2021 at 2:36 AM Lee Jones wrote: > > > > On Mon, 22 Mar 2021, Rob Herring wrote: > > > > > On Thu, Mar 18, 2021 at 4:40 AM Lee Jones wrote: > > > > > > > > This set is part of a l

Re: [PATCH v2 00/10] Rid W=1 warnings from OF

2021-03-23 Thread Lee Jones
On Mon, 22 Mar 2021, Rob Herring wrote: > On Thu, Mar 18, 2021 at 4:40 AM Lee Jones wrote: > > > > This set is part of a larger effort attempting to clean-up W=1 > > kernel builds, which are currently overwhelmingly riddled with > > niggly little warnings. >

Re: [PATCH v3 03/15] mfd: altera: merge ARCH_SOCFPGA and ARCH_STRATIX10

2021-03-22 Thread Lee Jones
tions(+), 2 deletions(-) Acked-by: Lee Jones -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog

[PATCH v2 00/10] Rid W=1 warnings from OF

2021-03-18 Thread Lee Jones
This set is part of a larger effort attempting to clean-up W=1 kernel builds, which are currently overwhelmingly riddled with niggly little warnings. v2: - Provided some descriptions to exported functions Lee Jones (10): of: device: Fix function name in header and provide missing

[PATCH 08/10] of: of_net: Provide function name and param description

2021-03-18 Thread Lee Jones
Cc: Andrew Lunn Cc: Heiner Kallweit Cc: Russell King Cc: Rob Herring Cc: Frank Rowand Cc: netdev@vger.kernel.org Cc: devicet...@vger.kernel.org Signed-off-by: Lee Jones --- drivers/of/of_net.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/of/of_net.c b/drivers/of/of_net.c

Re: [PATCH 08/10] of: of_net: Demote non-conforming kernel-doc header

2021-03-12 Thread Lee Jones
On Fri, 12 Mar 2021, Andrew Lunn wrote: > On Fri, Mar 12, 2021 at 11:07:56AM +, Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > > > drivers/of/of_net.c:104: warning: Function parameter or member 'np' not > > described in

[PATCH 4/4] ptp: ptp_p: Demote non-conformant kernel-doc headers and supply a param description

2021-03-12 Thread Lee Jones
ameter or member 'mem_base' not described in 'pch_dev' drivers/ptp/ptp_pch.c:121: warning: Function parameter or member 'mem_size' not described in 'pch_dev' drivers/ptp/ptp_pch.c:121: warning: Function parameter or member 'irq' not described

[PATCH 3/4] ptp: ptp_clockmatrix: Demote non-kernel-doc header to standard comment

2021-03-12 Thread Lee Jones
drivers/ptp/ptp_clockmatrix.c:1408: warning: Cannot understand * @brief Maximum absolute value for write phase offset in picoseconds Cc: Richard Cochran Cc: idt-support-1...@lm.renesas.com Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/ptp/ptp_clockmatrix.c | 4 ++-- 1 file

[PATCH 2/4] ptp_pch: Move 'pch_*()' prototypes to shared header

2021-03-12 Thread Lee Jones
‘pch_set_station_address’ [-Wmissing-prototypes] Cc: Richard Cochran (maintainer:PTP HARDWARE CLOCK SUPPORT) Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Flavio Suligoi Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- .../net/ethernet/oki-semi/pch_gbe/pch_gbe.h | 8 --- .../et

[PATCH 1/4] ptp_pch: Remove unused function 'pch_ch_control_read()'

2021-03-12 Thread Lee Jones
Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/ethernet/oki-semi/pch_gbe/pch_gbe.h | 1 - drivers/ptp/ptp_pch.c | 11 --- 2 files changed, 12 deletions(-) diff --git a/drivers/net/ethernet/oki-semi/pch_gbe/pch_gbe.h b/drivers/net/etherne

[PATCH 0/4] Rid W=1 warnings from PTP

2021-03-12 Thread Lee Jones
This set is part of a larger effort attempting to clean-up W=1 kernel builds, which are currently overwhelmingly riddled with niggly little warnings. Lee Jones (4): ptp_pch: Remove unused function 'pch_ch_control_read()' ptp_pch: Move 'pch_*()' prototypes to

[PATCH 00/10] Rid W=1 warnings from OF

2021-03-12 Thread Lee Jones
This set is part of a larger effort attempting to clean-up W=1 kernel builds, which are currently overwhelmingly riddled with niggly little warnings. Lee Jones (10): of: device: Fix function name in header and demote kernel-doc abuse of: dynamic: Fix incorrect parameter name and demote kernel

[PATCH 08/10] of: of_net: Demote non-conforming kernel-doc header

2021-03-12 Thread Lee Jones
Cc: Andrew Lunn Cc: Heiner Kallweit Cc: Russell King Cc: Rob Herring Cc: Frank Rowand Cc: netdev@vger.kernel.org Cc: devicet...@vger.kernel.org Signed-off-by: Lee Jones --- drivers/of/of_net.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/of/of_net.c b/drive

Re: [RFC v2 3/5] arm64: socfpga: rename ARCH_STRATIX10 to ARCH_SOCFPGA64

2021-03-10 Thread Lee Jones
> drivers/fpga/Kconfig| 2 +- > drivers/mfd/Kconfig | 2 +- If it's okay with everyone else, it'll be okay with me: Acked-by: Lee Jones > drivers/net/ethernet/stmicro/stmmac/Kconfig | 4 ++-- > drivers/reset/Kconfig

Re: [PATCH 00/12] Rid W=1 warnings from Thunderbolt

2021-01-28 Thread Lee Jones
On Thu, 28 Jan 2021, Mika Westerberg wrote: > Hi Lee, > > On Wed, Jan 27, 2021 at 11:25:42AM +0000, Lee Jones wrote: > > This set is part of a larger effort attempting to clean-up W=1 > > kernel builds, which are currently overwhelmingly riddled with > > niggly littl

[PATCH 06/12] thunderbolt: xdomain: Fix 'tb_unregister_service_driver()'s 'drv' param

2021-01-27 Thread Lee Jones
.@vger.kernel.org Cc: netdev@vger.kernel.org Cc: b...@vger.kernel.org Signed-off-by: Lee Jones --- drivers/thunderbolt/xdomain.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/thunderbolt/xdomain.c b/drivers/thunderbolt/xdomain.c index f2d4db1cd84d0..6e8bea6a7d392 100644 --- a/drive

[PATCH 00/12] Rid W=1 warnings from Thunderbolt

2021-01-27 Thread Lee Jones
This set is part of a larger effort attempting to clean-up W=1 kernel builds, which are currently overwhelmingly riddled with niggly little warnings. Only 1 small set required for Thunderbolt. Pretty good! Lee Jones (12): thunderbolt: dma_port: Remove unused variable 'ret' thunde

[PATCH 1/7] net: ethernet: smsc: smc91x: Fix function name in kernel-doc header

2021-01-15 Thread Lee Jones
;nsdelay' not described in 'try_toggle_control_gpio' Cc: Nicolas Pitre Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Erik Stahlman Cc: Peter Cammaert Cc: Daris A Nevil Cc: Russell King Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/ethernet/smsc/smc91

[PATCH 7/7] net: ethernet: toshiba: spider_net: Document a whole bunch of function parameters

2021-01-15 Thread Lee Jones
27; drivers/net/ethernet/toshiba/spider_net.c:2149: warning: Excess function parameter 'data' description in 'spider_net_tx_timeout_task' drivers/net/ethernet/toshiba/spider_net.c:2182: warning: Function parameter or member 'txqueue' not described in 'spider_net_

[PATCH 5/7] net: ethernet: ibm: ibmvnic: Fix some kernel-doc misdemeanours

2021-01-15 Thread Lee Jones
net/ibm/ibmvnic.c:1911: warning: Function parameter or member 'reset_state' not described in 'do_reset' Cc: Dany Madden Cc: Lijun Pan Cc: Sukadev Bhattiprolu Cc: Michael Ellerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: "David S. Miller" Cc: Jakub Kicinski Cc:

[PATCH 6/7] net: ethernet: toshiba: ps3_gelic_net: Fix some kernel-doc misdemeanours

2021-01-15 Thread Lee Jones
.c:1795: warning: Function parameter or member 'dev' not described in 'ps3_gelic_driver_remove' Cc: Geoff Levand Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Michael Ellerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Utz Bacher Cc: Jens Osterkamp Cc: netd

[PATCH 4/7] net: ethernet: ti: am65-cpts: Document am65_cpts_rx_enable()'s 'en' parameter

2021-01-15 Thread Lee Jones
iption in 'am65_cpts_rx_enable' Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Grygorii Strashko Cc: Kurt Kanzenbach Cc: netdev@vger.kernel.org Reviewed-by: Andrew Lunn Signed-off-by: Lee Jones --- drivers/net/ethernet/ti/am65-cpts.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH 2/7] net: xen-netback: xenbus: Demote nonconformant kernel-doc headers

2021-01-15 Thread Lee Jones
27;netback_probe' Cc: Wei Liu Cc: Paul Durrant Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Alexei Starovoitov Cc: Daniel Borkmann Cc: Jesper Dangaard Brouer Cc: John Fastabend Cc: Rusty Russell Cc: xen-de...@lists.xenproject.org Cc: netdev@vger.kernel.org Cc: b...@vger.k

[PATCH 3/7] net: ethernet: ti: am65-cpsw-qos: Demote non-conformant function header

2021-01-15 Thread Lee Jones
est_new' not described in 'am65_cpsw_timer_set' Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Ivan Khoronzhuk Cc: "Gustavo A. R. Silva" Cc: netdev@vger.kernel.org Reviewed-by: Andrew Lunn Signed-off-by: Lee Jones --- drivers/net/ethernet/ti/am65-cpsw-qos.c | 2 +-

[RESEND v2 0/7] Rid W=1 warnings in Ethernet

2021-01-15 Thread Lee Jones
x27;s Reviewed-by tags on remainder Lee Jones (7): net: ethernet: smsc: smc91x: Fix function name in kernel-doc header net: xen-netback: xenbus: Demote nonconformant kernel-doc headers net: ethernet: ti: am65-cpsw-qos: Demote non-conformant function header net: ethernet: ti: am65-cpts: Doc

Re: [PATCH v2 0/7] Rid W=1 warnings in Ethernet

2021-01-15 Thread Lee Jones
On Fri, 15 Jan 2021, Christophe Leroy wrote: > > > Le 15/01/2021 à 12:18, Lee Jones a écrit : > > On Thu, 14 Jan 2021, Lee Jones wrote: > > > > > On Thu, 14 Jan 2021, Jakub Kicinski wrote: > > > > > > > On Thu, 14 Jan 2021 08:33:49 + Le

Re: [PATCH v2 0/7] Rid W=1 warnings in Ethernet

2021-01-15 Thread Lee Jones
On Thu, 14 Jan 2021, Lee Jones wrote: > On Thu, 14 Jan 2021, Jakub Kicinski wrote: > > > On Thu, 14 Jan 2021 08:33:49 +0000 Lee Jones wrote: > > > On Wed, 13 Jan 2021, Jakub Kicinski wrote: > > > > > > > On Wed, 13 Jan 2021 16:41:16 + Lee Jones w

Re: [PATCH v2 0/7] Rid W=1 warnings in Ethernet

2021-01-14 Thread Lee Jones
On Thu, 14 Jan 2021, Jakub Kicinski wrote: > On Thu, 14 Jan 2021 08:33:49 + Lee Jones wrote: > > On Wed, 13 Jan 2021, Jakub Kicinski wrote: > > > > > On Wed, 13 Jan 2021 16:41:16 + Lee Jones wrote: > > >

Re: [PATCH v2 0/7] Rid W=1 warnings in Ethernet

2021-01-14 Thread Lee Jones
On Wed, 13 Jan 2021, Jakub Kicinski wrote: > On Wed, 13 Jan 2021 16:41:16 + Lee Jones wrote: > > Resending the stragglers again. > > > > > > This set is part of a larger effort

[PATCH 6/7] net: ethernet: toshiba: ps3_gelic_net: Fix some kernel-doc misdemeanours

2021-01-13 Thread Lee Jones
.c:1795: warning: Function parameter or member 'dev' not described in 'ps3_gelic_driver_remove' Cc: Geoff Levand Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Michael Ellerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Utz Bacher Cc: Jens Osterkamp Cc: netd

[PATCH 7/7] net: ethernet: toshiba: spider_net: Document a whole bunch of function parameters

2021-01-13 Thread Lee Jones
27; drivers/net/ethernet/toshiba/spider_net.c:2149: warning: Excess function parameter 'data' description in 'spider_net_tx_timeout_task' drivers/net/ethernet/toshiba/spider_net.c:2182: warning: Function parameter or member 'txqueue' not described in 'spider_net_

[PATCH 5/7] net: ethernet: ibm: ibmvnic: Fix some kernel-doc misdemeanours

2021-01-13 Thread Lee Jones
net/ibm/ibmvnic.c:1911: warning: Function parameter or member 'reset_state' not described in 'do_reset' Cc: Dany Madden Cc: Lijun Pan Cc: Sukadev Bhattiprolu Cc: Michael Ellerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: "David S. Miller" Cc: Jakub Kicinski Cc:

[PATCH 4/7] net: ethernet: ti: am65-cpts: Document am65_cpts_rx_enable()'s 'en' parameter

2021-01-13 Thread Lee Jones
iption in 'am65_cpts_rx_enable' Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Grygorii Strashko Cc: Kurt Kanzenbach Cc: netdev@vger.kernel.org Reviewed-by: Andrew Lunn Signed-off-by: Lee Jones --- drivers/net/ethernet/ti/am65-cpts.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH 2/7] net: xen-netback: xenbus: Demote nonconformant kernel-doc headers

2021-01-13 Thread Lee Jones
27;netback_probe' Cc: Wei Liu Cc: Paul Durrant Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Alexei Starovoitov Cc: Daniel Borkmann Cc: Jesper Dangaard Brouer Cc: John Fastabend Cc: Rusty Russell Cc: xen-de...@lists.xenproject.org Cc: netdev@vger.kernel.org Cc: b...@vger.k

[PATCH 3/7] net: ethernet: ti: am65-cpsw-qos: Demote non-conformant function header

2021-01-13 Thread Lee Jones
est_new' not described in 'am65_cpsw_timer_set' Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Ivan Khoronzhuk Cc: "Gustavo A. R. Silva" Cc: netdev@vger.kernel.org Reviewed-by: Andrew Lunn Signed-off-by: Lee Jones --- drivers/net/ethernet/ti/am65-cpsw-qos.c | 2 +-

[PATCH 1/7] net: ethernet: smsc: smc91x: Fix function name in kernel-doc header

2021-01-13 Thread Lee Jones
;nsdelay' not described in 'try_toggle_control_gpio' Cc: Nicolas Pitre Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Erik Stahlman Cc: Peter Cammaert Cc: Daris A Nevil Cc: Russell King Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/ethernet/smsc/smc91

[PATCH v2 0/7] Rid W=1 warnings in Ethernet

2021-01-13 Thread Lee Jones
: - Squashed IBM patches - Fixed real issue in SMSC - Added Andrew's Reviewed-by tags on remainder Lee Jon

Re: [resend/standalone PATCH v4] Add auxiliary bus support

2020-12-18 Thread Lee Jones
here, somehow avoiding duplication. > > If we were to look at it with some OOP viewpoint I'd say the generic > HW resource related parts should be some shared superclass between > 'struct device' and 'struct platform/pnp/pci/acpi/mfd/etc_device'. You'r

Re: [resend/standalone PATCH v4] Add auxiliary bus support

2020-12-18 Thread Lee Jones
means? By doing so you keep things bound to a central location resulting in less chance of misuse. I turn away MFD implementation abuses all the time. Seeing as the 2 subsystems are totally disjoint, this just unwittingly opened up another back-channel opportunity for those abuses to make it

Re: [PATCH] dt-bindings: Fix JSON pointers

2020-12-18 Thread Lee Jones
tation/devicetree/bindings/ -name "*.yaml"` > > Cc: Maxime Ripard > Cc: Vinod Koul > Cc: Guenter Roeck > Cc: Jonathan Cameron > Cc: Lars-Peter Clausen > Cc: Lee Jones > Cc: Daniel Thompson > Cc: Jingoo Han > Cc: Pavel Machek > Cc: "David S. Miller&qu

Re: [patch 16/30] mfd: ab8500-debugfs: Remove the racy fiddling with irq_desc

2020-12-11 Thread Lee Jones
Signed-off-by: Thomas Gleixner > Cc: Linus Walleij > Cc: Lee Jones > Cc: linux-arm-ker...@lists.infradead.org > --- > drivers/mfd/ab8500-debugfs.c | 16 +++- > 1 file changed, 3 insertions(+), 13 deletions(-) Acked-by: Lee Jones -- Lee Jones [李琼斯] Senio

Re: [PATCH] PCI: Remove pci_try_set_mwi

2020-12-09 Thread Lee Jones
ers/dma/hsu/pci.c | 2 +- > drivers/ide/cs5530.c | 2 +- > drivers/mfd/intel-lpss-pci.c | 2 +- Acked-by: Lee Jones > drivers/net/ethernet/adaptec/starfire.c | 2 +- > drivers/net/ethernet/ala

Re: [PATCH 01/17] wil6210: wmi: Correct misnamed function parameter 'ptr_'

2020-12-08 Thread Lee Jones
On Wed, 02 Dec 2020, Kalle Valo wrote: > Lee Jones wrote: > > > Fixes the following W=1 kernel build warning(s): > > > > drivers/net/wireless/ath/wil6210/wmi.c:279: warning: Function parameter or > > member 'ptr_' not described in 'wmi_buffer

Re: [PATCH 5/8] net: ethernet: ibm: ibmvnic: Fix some kernel-doc misdemeanours

2020-11-30 Thread Lee Jones
tually correct. Right. This is a 'quirk' of my current process. I build once, then use a script to parse the output, fixing each issue in the order the compiler presents them. Then, either after fixing a reasonable collection, or all issues, I re-run the compile and fix the next batc

Re: [PATCH 8/8] net: ethernet: ibm: ibmvnic: Fix some kernel-doc issues

2020-11-30 Thread Lee Jones
On Sun, 29 Nov 2020, Andrew Lunn wrote: > On Thu, Nov 26, 2020 at 01:38:53PM +, Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > > > from drivers/net/ethernet/ibm/ibmvnic.c:35: > > inlined from ‘handle_vpd_rsp’ at drivers/net/et

Re: [PATCH 1/8] net: ethernet: smsc: smc91x: Demote non-conformant kernel function header

2020-11-30 Thread Lee Jones
On Sun, 29 Nov 2020, Andrew Lunn wrote: > On Thu, Nov 26, 2020 at 01:38:46PM +, Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > > > drivers/net/ethernet/smsc/smc91x.c:2200: warning: Function parameter or > > member 'dev' not

Re: [PATCH 17/17] realtek: rtw88: pci: Add prototypes for .probe, .remove and .shutdown

2020-11-30 Thread Lee Jones
On Mon, 30 Nov 2020, Pkshih wrote: > > > > -Original Message- > > From: Lee Jones [mailto:lee.jo...@linaro.org] > > Sent: Friday, November 27, 2020 4:57 PM > > To: Pkshih > > Cc: Tony Chuang; kv...@codeaurora.org; linux-ker...@vger.kernel.org; > &

Re: [PATCH 17/17] realtek: rtw88: pci: Add prototypes for .probe, .remove and .shutdown

2020-11-27 Thread Lee Jones
On Fri, 27 Nov 2020, Pkshih wrote: > On Fri, 2020-11-27 at 07:38 +, Lee Jones wrote: > > On Fri, 27 Nov 2020, Pkshih wrote: > > > > >  > > > The subject prefix doesn't need 'realtek:'; use 'rtw88:'. > > >  > > > On T

Re: [PATCH 17/17] realtek: rtw88: pci: Add prototypes for .probe, .remove and .shutdown

2020-11-26 Thread Lee Jones
On Fri, 27 Nov 2020, Pkshih wrote: > > The subject prefix doesn't need 'realtek:'; use 'rtw88:'. > > On Thu, 2020-11-26 at 13:31 +, Lee Jones wrote: > > Also strip out other duplicates from driver specific headers. > > > > Ensure 

[PATCH 5/8] net: ethernet: ibm: ibmvnic: Fix some kernel-doc misdemeanours

2020-11-26 Thread Lee Jones
net/ibm/ibmvnic.c:1911: warning: Function parameter or member 'reset_state' not described in 'do_reset' Cc: Dany Madden Cc: Lijun Pan Cc: Sukadev Bhattiprolu Cc: Michael Ellerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: "David S. Miller" Cc: Jakub Kicins

[PATCH 7/8] net: ethernet: toshiba: spider_net: Document a whole bunch of function parameters

2020-11-26 Thread Lee Jones
27; drivers/net/ethernet/toshiba/spider_net.c:2149: warning: Excess function parameter 'data' description in 'spider_net_tx_timeout_task' drivers/net/ethernet/toshiba/spider_net.c:2182: warning: Function parameter or member 'txqueue' not described in 'sp

[PATCH 8/8] net: ethernet: ibm: ibmvnic: Fix some kernel-doc issues

2020-11-26 Thread Lee Jones
Ellerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Dany Madden Cc: Lijun Pan Cc: Sukadev Bhattiprolu Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Santiago Leon Cc: Thomas Falcon Cc: John Allen Cc: linuxppc-...@lists.ozlabs.org Cc: netdev@vger.kernel.org Signed-off-by:

[PATCH 4/8] net: ethernet: ti: am65-cpts: Document am65_cpts_rx_enable()'s 'en' parameter

2020-11-26 Thread Lee Jones
iption in 'am65_cpts_rx_enable' Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Grygorii Strashko Cc: Kurt Kanzenbach Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/ethernet/ti/am65-cpts.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/

[PATCH 3/8] net: ethernet: ti: am65-cpsw-qos: Demote non-conformant function header

2020-11-26 Thread Lee Jones
est_new' not described in 'am65_cpsw_timer_set' Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Ivan Khoronzhuk Cc: "Gustavo A. R. Silva" Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/ethernet/ti/am65-cpsw-qos.c | 2 +- 1 file changed, 1 inser

[PATCH 6/8] net: ethernet: toshiba: ps3_gelic_net: Fix some kernel-doc misdemeanours

2020-11-26 Thread Lee Jones
.c:1795: warning: Function parameter or member 'dev' not described in 'ps3_gelic_driver_remove' Cc: Geoff Levand Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Michael Ellerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Utz Bacher Cc: Jens Osterkamp Cc: netdev

[PATCH 2/8] net: xen-netback: xenbus: Demote nonconformant kernel-doc headers

2020-11-26 Thread Lee Jones
27;netback_probe' Cc: Wei Liu Cc: Paul Durrant Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Alexei Starovoitov Cc: Daniel Borkmann Cc: Jesper Dangaard Brouer Cc: John Fastabend Cc: Rusty Russell Cc: xen-de...@lists.xenproject.org Cc: netdev@vger.kernel.org Cc: b...@vger.kernel.or

[PATCH 1/8] net: ethernet: smsc: smc91x: Demote non-conformant kernel function header

2020-11-26 Thread Lee Jones
;nsdelay' not described in 'try_toggle_control_gpio' Cc: Nicolas Pitre Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Erik Stahlman Cc: Peter Cammaert Cc: Daris A Nevil Cc: Russell King Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/ethernet/smsc/smc91

[PATCH 0/8] Rid W=1 warnings in Net

2020-11-26 Thread Lee Jones
Resending the stragglers. This set is part of a larger effort attempting to clean-up W=1 kernel builds, which are currently overwhelmingly riddled with niggly little warnings. Lee Jones (8): net: ethernet: smsc: smc91x: Demote non-conformant kernel function header net: xen-netback

[PATCH 07/17] iwlwifi: iwl-eeprom-read: Demote one nonconformant function header

2020-11-26 Thread Lee Jones
elho Cc: Intel Linux Wireless Cc: Kalle Valo Cc: "David S. Miller" Cc: Jakub Kicinski Cc: linux-wirel...@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/wireless/intel/iwlwifi/iwl-eeprom-read.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 02/17] iwlwifi: mvm: rs: Demote non-conformant function documentation headers

2020-11-26 Thread Lee Jones
iwlwifi/mvm/rs.c:3761: warning: Function parameter or member 'lq_sta' not described in 'rs_program_fix_rate' drivers/net/wireless/intel/iwlwifi/mvm/rs.c:4213: warning: Function parameter or member 'mvm' not described in 'iwl_mvm_tx_protection' Cc: Johannes B

[PATCH 06/17] ath9k: ar9003_2p2_initvals: Remove unused const variables

2020-11-26 Thread Lee Jones
: ‘ar9300PciePhy_clkreq_enable_L1_2p2’ defined but not used [-Wunused-const-variable=] Cc: QCA ath9k Development Cc: Kalle Valo Cc: "David S. Miller" Cc: Jakub Kicinski Cc: linux-wirel...@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- .../net/wireless/ath/ath9k/ar9003_2p2_

[PATCH 10/17] ath9k: ar5008_phy: Demote half completed function headers

2020-11-26 Thread Lee Jones
c:119: warning: Function parameter or member 'firstBit' not described in 'ar5008_hw_phy_modify_rx_buffer' Cc: QCA ath9k Development Cc: Kalle Valo Cc: "David S. Miller" Cc: Jakub Kicinski Cc: linux-wirel...@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by

[PATCH 09/17] ath: regd: Provide description for ath_reg_apply_ir_flags's 'reg' param

2020-11-26 Thread Lee Jones
org Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/wireless/ath/regd.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/wireless/ath/regd.c b/drivers/net/wireless/ath/regd.c index bee9110b91f38..b2400e2417a55 100644 --- a/drivers/net/wireless/ath/regd.c +++ b/d

[PATCH 13/17] iwlwifi: iwl-phy-db: Add missing struct member description for 'trans'

2020-11-26 Thread Lee Jones
e Valo Cc: "David S. Miller" Cc: Jakub Kicinski Cc: linux-wirel...@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/wireless/intel/iwlwifi/iwl-phy-db.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/intel/iwlwi

[PATCH 11/17] ath: dfs_pattern_detector: Fix some function kernel-doc headers

2020-11-26 Thread Lee Jones
rning: Function parameter or member 'freq' not described in 'channel_detector_get' Cc: Kalle Valo Cc: "David S. Miller" Cc: Jakub Kicinski Cc: linux-wirel...@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/wireless/ath/dfs_pattern_det

[PATCH 15/17] ath9k: dynack: Demote non-compliant function header

2020-11-26 Thread Lee Jones
described in 'ath_dynack_ewma' Cc: QCA ath9k Development Cc: Kalle Valo Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Lorenzo Bianconi Cc: linux-wirel...@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/wireless/ath/ath9k/dynack.c |

[PATCH 17/17] realtek: rtw88: pci: Add prototypes for .probe, .remove and .shutdown

2020-11-26 Thread Lee Jones
ller" Cc: Jakub Kicinski Cc: linux-wirel...@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/wireless/realtek/rtw88/pci.h | 8 drivers/net/wireless/realtek/rtw88/rtw8723de.c | 1 + drivers/net/wireless/realtek/rtw88/rtw8723de.h | 4 driv

[PATCH 16/17] iwlwifi: fw: acpi: Demote non-conformant function headers

2020-11-26 Thread Lee Jones
Cc: Luca Coelho Cc: Intel Linux Wireless Cc: Kalle Valo Cc: "David S. Miller" Cc: Jakub Kicinski Cc: linux-wirel...@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/wireless/intel/iwlwifi/fw/acpi.c | 12 ++-- 1 file changed, 6 insert

[PATCH 14/17] iwlwifi: fw: dbg: Fix misspelling of 'reg_data' in function header

2020-11-26 Thread Lee Jones
'reg' description in 'iwl_dump_ini_mem' Cc: Johannes Berg Cc: Emmanuel Grumbach Cc: Luca Coelho Cc: Intel Linux Wireless Cc: Kalle Valo Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Shahar S Matityahu Cc: linux-wirel...@vger.kernel.org Cc: netdev@vger.kernel.org S

[PATCH 12/17] ath: dfs_pri_detector: Demote zero/half completed kernel-doc headers

2020-11-26 Thread Lee Jones
fs_pri_detector.c:46: warning: Function parameter or member 'tolerance' not described in 'pde_get_multiple' Cc: Kalle Valo Cc: "David S. Miller" Cc: Jakub Kicinski Cc: linux-wirel...@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- dri

[PATCH 08/17] iwlwifi: iwl-eeprom-parse: Fix 'struct iwl_eeprom_enhanced_txpwr's header

2020-11-26 Thread Lee Jones
e Valo Cc: "David S. Miller" Cc: Jakub Kicinski Cc: linux-wirel...@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- .../net/wireless/intel/iwlwifi/iwl-eeprom-parse.c| 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/driver

[PATCH 05/17] ath9k: ar9485_initvals: Remove unused const variable 'ar9485_fast_clock_1_1_baseband_postamble'

2020-11-26 Thread Lee Jones
b Kicinski Cc: linux-wirel...@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/wireless/ath/ath9k/ar9485_initvals.h | 7 --- 1 file changed, 7 deletions(-) diff --git a/drivers/net/wireless/ath/ath9k/ar9485_initvals.h b/drivers/net/wireless/ath/ath9k/ar9485_

[PATCH 04/17] ath9k: ar9340_initvals: Remove unused const variable 'ar9340Modes_ub124_tx_gain_table_1p0'

2020-11-26 Thread Lee Jones
cinski Cc: linux-wirel...@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- .../net/wireless/ath/ath9k/ar9340_initvals.h | 101 -- 1 file changed, 101 deletions(-) diff --git a/drivers/net/wireless/ath/ath9k/ar9340_initvals.h b/drivers/net/wireless

[PATCH 03/17] ath9k: ar9330_1p1_initvals: Remove unused const variable 'ar9331_common_tx_gain_offset1_1'

2020-11-26 Thread Lee Jones
cinski Cc: linux-wirel...@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/wireless/ath/ath9k/ar9330_1p1_initvals.h | 7 --- 1 file changed, 7 deletions(-) diff --git a/drivers/net/wireless/ath/ath9k/ar9330_1p1_initvals.h b/drivers/net/wireless

[PATCH 01/17] wil6210: wmi: Correct misnamed function parameter 'ptr_'

2020-11-26 Thread Lee Jones
scription in 'wmi_buffer_block' Cc: Maya Erez Cc: Kalle Valo Cc: "David S. Miller" Cc: Jakub Kicinski Cc: linux-wirel...@vger.kernel.org Cc: wil6...@qti.qualcomm.com Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/wireless/ath/wil6210/wmi.c | 2 +- 1

[PATCH v2 00/17] [Set 4] Rid W=1 warnings in Wireless

2020-11-26 Thread Lee Jones
[Last batch!] This set is part of a larger effort attempting to clean-up W=1 kernel builds, which are currently overwhelmingly riddled with niggly little warnings. v2: - Unify rtw_pci_(probe,remove,shutdown}() prototypses - Suggested-by: Brian Norris Lee Jones (17): wil6210: wmi: Correct

[PATCH 6/6] staging: net: wimax: i2400m: fw: Fix incorrectly spelt function parameter in documentation

2020-11-12 Thread Lee Jones
chunk_len' description in 'i2400m_download_chunk' Cc: Greg Kroah-Hartman Cc: Inaky Perez-Gonzalez Cc: linux-wi...@intel.com Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Yanir Lubetkin Cc: netdev@vger.kernel.org Cc: de...@driverdev.osuosl.org Signed-off-by: Lee Jones --- drivers/s

[PATCH 4/6] staging: net: wimax: i2400m: netdev: Demote non-conformant function header

2020-11-12 Thread Lee Jones
ller" Cc: Jakub Kicinski Cc: Yanir Lubetkin Cc: netdev@vger.kernel.org Cc: de...@driverdev.osuosl.org Signed-off-by: Lee Jones --- drivers/staging/wimax/i2400m/netdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wimax/i2400m/netdev.c b/drivers/stagi

[PATCH 3/6] staging: net: wimax: i2400m: fw: Fix some function header misdemeanours

2020-11-12 Thread Lee Jones
rivers/net/wimax/i2400m/fw.c:1548: warning: Function parameter or member 'flags' not described in 'i2400m_dev_bootstrap' Cc: Greg Kroah-Hartman Cc: Inaky Perez-Gonzalez Cc: linux-wi...@intel.com Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Yanir Lubetkin Cc: netdev@v

[PATCH 2/6] staging: net: wimax: i2400m: driver: Demote some non-conformant kernel-docs, fix others

2020-11-12 Thread Lee Jones
not described in 'i2400m_setup' drivers/net/wimax/i2400m/driver.c:942: warning: Function parameter or member 'i2400m' not described in 'i2400m_release' Cc: Greg Kroah-Hartman Cc: Inaky Perez-Gonzalez Cc: linux-wi...@intel.com Cc: "David S. Miller" Cc: Ja

[PATCH 5/6] staging: net: wimax: i2400m: tx: Fix a few kernel-doc misdemeanours

2020-11-12 Thread Lee Jones
Jakub Kicinski Cc: Yanir Lubetkin Cc: netdev@vger.kernel.org Cc: de...@driverdev.osuosl.org Signed-off-by: Lee Jones --- drivers/staging/wimax/i2400m/tx.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/staging/wimax/i2400m/tx.c b/drivers/staging/wimax/i2400m/tx.c index 1255302e251

[PATCH 0/6] Rid i2400m driver set of W=1 issues

2020-11-12 Thread Lee Jones
This set is part of a larger effort attempting to clean-up W=1 kernel builds, which are currently overwhelmingly riddled with niggly little warnings. This is a rebased set that went to Net before the move to Staging. Lee Jones (6): staging: net: wimax: i2400m: control: Fix some misspellings in

[PATCH 1/6] staging: net: wimax: i2400m: control: Fix some misspellings in i2400m_set_init_config()'s docs

2020-11-12 Thread Lee Jones
;arg_size' description in 'i2400m_set_init_config' Cc: Greg Kroah-Hartman Cc: Inaky Perez-Gonzalez Cc: linux-wi...@intel.com Cc: "David S. Miller" Cc: Jakub Kicinski Cc: netdev@vger.kernel.org Cc: de...@driverdev.osuosl.org Signed-off-by: Lee Jones --- drivers/staging/wimax/i2400

Re: [PATCH 30/30] net: ethernet: smsc: smc91x: Mark 'pkt_len' as __maybe_unused

2020-11-12 Thread Lee Jones
On Mon, 02 Nov 2020, Lee Jones wrote: > 'pkt_len' is used to interact with a hardware register. It might not > be safe to remove it entirely. Mark it as __maybe_unused instead. > > Fixes the following W=1 kernel build warning(s): > > drivers/net/ethernet/smsc/s

Re: [PATCH 04/30] net: wimax: i2400m: control: Fix some misspellings in i2400m_set_init_config()'s docs

2020-11-12 Thread Lee Jones
On Mon, 02 Nov 2020, Lee Jones wrote: > Fixes the following W=1 kernel build warning(s): > > drivers/net/wimax/i2400m/control.c:1195: warning: Function parameter or > member 'arg' not described in 'i2400m_set_init_config' > drivers/net/wimax/i2400m/contr

Re: [PATCH 41/41] realtek: rtw88: pci: Add prototypes for .probe, .remove and .shutdown

2020-11-07 Thread Lee Jones
On Sat, 07 Nov 2020, Kalle Valo wrote: > Lee Jones writes: > > > On Mon, 02 Nov 2020, Brian Norris wrote: > > > >> On Mon, Nov 2, 2020 at 3:25 AM Lee Jones wrote: > >> > --- a/drivers/net/wireless/realtek/rtw88/pci.h > >> > +++ b/driver

Re: [PATCH 06/12] net: ethernet: ti: am65-cpsw-qos: Demote non-conformant function header

2020-11-04 Thread Lee Jones
On Wed, 04 Nov 2020, Andrew Lunn wrote: > On Wed, Nov 04, 2020 at 02:28:35PM +, Lee Jones wrote: > > On Wed, 04 Nov 2020, Andrew Lunn wrote: > > > > > On Wed, Nov 04, 2020 at 09:06:04AM +, Lee Jones wrote: > > > > Fixes the following W=1 kernel build w

Re: [PATCH 02/12] net: ethernet: smsc: smc911x: Mark 'status' as __maybe_unused

2020-11-04 Thread Lee Jones
On Wed, 04 Nov 2020, Andrew Lunn wrote: > On Wed, Nov 04, 2020 at 02:31:40PM +, Lee Jones wrote: > > On Wed, 04 Nov 2020, Andrew Lunn wrote: > > > > > On Wed, Nov 04, 2020 at 09:06:00AM +, Lee Jones wrote: > > > > 'status' is used to in

Re: [PATCH 02/12] net: ethernet: smsc: smc911x: Mark 'status' as __maybe_unused

2020-11-04 Thread Lee Jones
On Wed, 04 Nov 2020, Lee Jones wrote: > On Wed, 04 Nov 2020, Andrew Lunn wrote: > > > On Wed, Nov 04, 2020 at 09:06:00AM +0000, Lee Jones wrote: > > > 'status' is used to interact with a hardware register. It might not > > > be safe to remove it enti

Re: [PATCH 02/12] net: ethernet: smsc: smc911x: Mark 'status' as __maybe_unused

2020-11-04 Thread Lee Jones
On Wed, 04 Nov 2020, Andrew Lunn wrote: > On Wed, Nov 04, 2020 at 09:06:00AM +, Lee Jones wrote: > > 'status' is used to interact with a hardware register. It might not > > be safe to remove it entirely. Mark it as __maybe_unused instead. > > Hi Lee > >

Re: [PATCH 06/12] net: ethernet: ti: am65-cpsw-qos: Demote non-conformant function header

2020-11-04 Thread Lee Jones
On Wed, 04 Nov 2020, Andrew Lunn wrote: > On Wed, Nov 04, 2020 at 09:06:04AM +, Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > > > drivers/net/ethernet/ti/am65-cpsw-qos.c:364: warning: Function parameter > > or member 'ndev'

[PATCH 01/12] net: usb: lan78xx: Remove lots of set but unused 'ret' variables

2020-11-04 Thread Lee Jones
Cc: netdev@vger.kernel.org Cc: linux-...@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/usb/lan78xx.c | 212 ++ 1 file changed, 100 insertions(+), 112 deletions(-) diff --git a/drivers/net/usb/lan78xx.c b/drivers/net/usb/lan78xx.c index 65b315bc60abd..551

[PATCH 02/12] net: ethernet: smsc: smc911x: Mark 'status' as __maybe_unused

2020-11-04 Thread Lee Jones
ernet/smsc/smc911x.c: In function ‘smc911x_timeout’: drivers/net/ethernet/smsc/smc911x.c:1251:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable] Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Dustin McIntire Cc: netdev@vger.kernel.org Signed-off-by: Lee Jon

[PATCH 00/12] [Set 2] Rid W=1 warnings in Net

2020-11-04 Thread Lee Jones
MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This set is part of a larger effort attempting to clean-up W=1 kernel builds, which are currently overwhelmingly riddled with niggly little warnings. This is the last set. Lee Jones (12): net: usb

[PATCH 05/12] net: xen-netback: xenbus: Demote nonconformant kernel-doc headers

2020-11-04 Thread Lee Jones
27;netback_probe' Cc: Wei Liu Cc: Paul Durrant Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Alexei Starovoitov Cc: Daniel Borkmann Cc: Jesper Dangaard Brouer Cc: John Fastabend Cc: Rusty Russell Cc: xen-de...@lists.xenproject.org Cc: netdev@vger.kernel.org Cc: b...@vger.kernel.or

[PATCH 06/12] net: ethernet: ti: am65-cpsw-qos: Demote non-conformant function header

2020-11-04 Thread Lee Jones
est_new' not described in 'am65_cpsw_timer_set' Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Ivan Khoronzhuk Cc: "Gustavo A. R. Silva" Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/ethernet/ti/am65-cpsw-qos.c | 2 +- 1 file changed, 1 inser

[PATCH 08/12] net: xen-netfront: Demote non-kernel-doc headers to standard comment blocks

2020-11-04 Thread Lee Jones
7; Cc: Boris Ostrovsky Cc: Juergen Gross Cc: Stefano Stabellini Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Alexei Starovoitov Cc: Daniel Borkmann Cc: Jesper Dangaard Brouer Cc: John Fastabend Cc: Martin KaFai Lau Cc: Song Liu Cc: Yonghong Song Cc: Andrii Nakryiko Cc: KP Singh C

[PATCH 07/12] net: ethernet: ti: am65-cpts: Document am65_cpts_rx_enable()'s 'en' parameter

2020-11-04 Thread Lee Jones
iption in 'am65_cpts_rx_enable' Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Grygorii Strashko Cc: Kurt Kanzenbach Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones --- drivers/net/ethernet/ti/am65-cpts.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/

[PATCH 10/12] net: ethernet: toshiba: ps3_gelic_net: Fix some kernel-doc misdemeanours

2020-11-04 Thread Lee Jones
.c:1795: warning: Function parameter or member 'dev' not described in 'ps3_gelic_driver_remove' Cc: Geoff Levand Cc: "David S. Miller" Cc: Jakub Kicinski Cc: Michael Ellerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Utz Bacher Cc: Jens Osterkamp Cc: netdev

[PATCH 09/12] net: ethernet: ibm: ibmvnic: Fix some kernel-doc misdemeanours

2020-11-04 Thread Lee Jones
net/ibm/ibmvnic.c:1911: warning: Function parameter or member 'reset_state' not described in 'do_reset' Cc: Dany Madden Cc: Lijun Pan Cc: Sukadev Bhattiprolu Cc: Michael Ellerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: "David S. Miller" Cc: Jakub Kicinski Cc

  1   2   3   4   5   6   7   8   9   10   >