linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 00/57] Rid W=1 warnings from Staging
@ 2021-04-14 18:10 Lee Jones
  2021-04-14 18:10 ` [PATCH 25/57] staging: sm750fb: sm750_accel: Provide description for 'accel' and fix function naming Lee Jones
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Lee Jones @ 2021-04-14 18:10 UTC (permalink / raw)
  To: lee.jones
  Cc: linux-kernel, ac100, Alexander A. Klimov, Allen Pais,
	Anders Blomdell, Andrea Merello, Andres Klode, Andrey Shvetsov,
	Arve Hjønnevåg, Christian Brauner, Comedi,
	David A. Schleef, dri-devel, Fabio Aiuto, Florian Schilhabel,
	Forest Bond, Greg Kroah-Hartman, Herman.Bruyninckx,
	H Hartley Sweeten, Hridya Valsaraju, Ian Abbott, Ilya Petrov,
	Jacob Feder, Jerry chuang, Joel Fernandes, John B. Wyatt IV,
	Jonathan Cameron, J.P. Mellor, karthik alapati, Kees Cook,
	Kernel Team, Klaas.Gadeyne, Larry Finger, Lars-Peter Clausen,
	linux-fbdev, linux-iio, linux-pwm, linux-staging, linux-tegra,
	Marc Dietrich, Marco Cesati, Martijn Coenen, Michael Hennerich,
	Michael Straube, Mori Hess, Nícolas F. R. A. Prado,
	Peter Hurley, Philipp Zabel, Pierre-Hugues Husson, Robert Love,
	Romain Perier, Ross Schmidt, Spencer E. Olson, Stanley,
	Sudip Mukherjee, Sumera Priyadarsini, Suren Baghdasaryan,
	Teddy Wang, Thierry Reding, Thomas Gleixner, Todd Kjos,
	Truxton Fulton, Uwe Kleine-König, Wim.Meeussen,
	Zhansaya Bagdauletkyzy

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 (57):
  staging: r8192U_core: Remove two unused variables 'ret' and
    'reset_status'
  staging: android: ashmem: Supply description for 'new_range'
  staging: comedi_8254: Fix descriptions for 'i8254' and 'iobase'
  staging: r8192U_core: Do not use kernel-doc formatting for !kernel-doc
    headers
  staging: r819xU_phy: Remove some local variables from the stack
  staging: r819xU_cmdpkt: Remove functionless method
    'cmpk_handle_query_config_rx'
  staging: wlan-ng: cfg80211: Move large struct onto the heap
  staging: rtw_ioctl_set: Move 'channel_table' to the only place it's
    used
  staging: rtl8188eu: core: rtw_ieee80211: Fix incorrectly documented
    function
  staging: rtl8723bs: core: rtw_mlme: Remove a bunch of unused variables
  staging: rtl8723bs: core: rtw_mlme_ext: Deal with a bunch of unused
    variables
  staging: rtl8712: rtl871x_mp_ioctl: Remove a bunch of unused tables
  staging: rtl8723bs: core: rtw_recv: Mark debug variable as
    __maybe_unused
  staging: rtl8188eu: core: rtw_security: Fix some formatting and
    misdocumentation
  staging: rtl8723bs: core: rtw_security: Demote non-conformant
    kernel-doc header
  staging: rtl8723bs: core: rtw_sta_mgt: Remove unused variable 'psta'
  staging: rtl8723bs: core: rtw_sta_mgt: Return error value directly
  staging: octeon: ethernet-tx: Fix formatting issue in function header
  staging: rtl8723bs: core: rtw_wlan_util: Remove unused variable
    'start_seq'
  staging: rtl8712: rtl871x_mp_ioctl: Move a large data struct onto the
    heap
  staging: iio: frequency: ad9834: Provide missing description for
    'devid'
  staging: nvec: Fix a bunch of kernel-doc issues
  staging: ks7010: ks_hostif: Remove a bunch of unused variables
  staging: fwserial: Demote a whole host of kernel-doc abuses
  staging: sm750fb: sm750_accel: Provide description for 'accel' and fix
    function naming
  staging: most: net: Fix some kernel-doc formatting issues
  staging: vt6655: upc: Suppress set but not used warning in macro
  staging: rtl8192u: ieee80211_softmac: Move a large data struct onto
    the heap
  staging: most: dim2: Provide missing descriptions and fix doc-rot
  staging: rtl8723bs: core: rtw_ieee80211: Remove seemingly pointless
    copy
  staging: rtl8723bs: core: rtw_mlme: 'retry' is only used if REJOIN is
    set
  staging: rtl8723bs: core: rtw_mlme_ext: 'evt_seq' is only used if
    CHECK_EVENT_SEQ is set
  staging: most: i2c: Fix a little doc-rot
  staging: most: dim2: hal: Fix one kernel-doc header and demote two
    non-conforming ones
  staging: most: dim2: hal: Demote non-conformant kernel-doc headers
  staging: axis-fifo: axis-fifo: Fix some formatting issues
  staging: rtl8188eu: os_dep: ioctl_linux: Move 2 large data buffers
    into the heap
  staging: fbtft: fb_ili9320: Remove unused variable 'ret'
  staging: rtl8723bs: core: rtw_ieee80211: Fix incorrectly named
    function
  staging: rtl8723bs: hal: odm_NoiseMonitor: Remove unused variable and
    dead code
  staging: rtl8188eu: os_dep: mon: Demote non-conforming kernel-doc
    headers
  staging: rtl8188eu: os_dep: rtw_android: Demote kernel-doc abuse
  staging: rtl8723bs: hal: rtl8723b_hal_init: Remove unused variable and
    dead code
  staging: rtl8723bs: hal: rtl8723b_phycfg: Fix a bunch of misnamed
    functions
  staging: rtl8723bs: hal: sdio_halinit: 'start' is only used if debug
    is enabled
  staging: rtl8723bs: hal: sdio_ops: Mark used 'err' as __maybe_unused
    and remove another
  staging: rtl8723bs: os_dep: ioctl_cfg80211: 'ack' is used when debug
    is enabled
  staging: comedi: drivers: jr3_pci: Remove set but unused variable
    'min_full_scale'
  staging: comedi: drivers: ni_tio: Fix slightly broken kernel-doc and
    demote others
  staging: comedi: drivers: ni_routes: Demote non-conforming kernel-doc
    headers
  staging: axis-fifo: axis-fifo: Fix function naming in the
    documentation
  staging: rtl8723bs: hal: odm_NoiseMonitor: Remove unused variable
    'func_start'
  staging: rtl8723bs: core: rtw_mlme_ext: Move very large data buffer
    onto the heap
  staging: rtl8723bs: hal: rtl8723b_hal_init: Mark a bunch of debug
    variables as __maybe_unused
  staging: comedi: drivers: ni_mio_common: Move 'range_ni_E_ao_ext' to
    where it is used
  staging: comedi: drivers: comedi_isadma: Fix misspelling of
    'dma_chan1'
  staging: rtl8723bs: hal: sdio_halinit: Remove unused variable 'ret'

 drivers/staging/android/ashmem.c              |   1 +
 drivers/staging/axis-fifo/axis-fifo.c         |  18 +--
 drivers/staging/comedi/drivers/comedi_8254.c  |   3 +-
 .../staging/comedi/drivers/comedi_isadma.c    |   2 +-
 drivers/staging/comedi/drivers/jr3_pci.c      |   3 +-
 .../staging/comedi/drivers/ni_mio_common.c    |   9 --
 drivers/staging/comedi/drivers/ni_routes.c    |   6 +-
 drivers/staging/comedi/drivers/ni_stc.h       |   9 +-
 drivers/staging/comedi/drivers/ni_tio.c       |  12 +-
 drivers/staging/fbtft/fb_ili9320.c            |   3 +-
 drivers/staging/fwserial/fwserial.c           |  46 +++----
 drivers/staging/iio/frequency/ad9834.c        |   5 +-
 drivers/staging/ks7010/ks_hostif.c            |  14 +-
 drivers/staging/most/dim2/dim2.c              |  23 ++--
 drivers/staging/most/dim2/hal.c               |  10 +-
 drivers/staging/most/i2c/i2c.c                |  12 +-
 drivers/staging/most/net/net.c                |   6 +-
 drivers/staging/nvec/nvec.c                   |   7 +-
 drivers/staging/octeon/ethernet-tx.c          |   1 -
 drivers/staging/qlge/qlge_main.c              |   4 +-
 .../staging/rtl8188eu/core/rtw_ieee80211.c    |   2 +-
 .../staging/rtl8188eu/core/rtw_ioctl_set.c    |   8 ++
 drivers/staging/rtl8188eu/core/rtw_security.c |  10 +-
 .../staging/rtl8188eu/include/rtw_mlme_ext.h  |   8 --
 .../staging/rtl8188eu/os_dep/ioctl_linux.c    |  12 +-
 drivers/staging/rtl8188eu/os_dep/mon.c        |   6 +-
 .../staging/rtl8188eu/os_dep/rtw_android.c    |   2 +-
 .../rtl8192u/ieee80211/ieee80211_softmac.c    |   9 +-
 drivers/staging/rtl8192u/r8192U_core.c        |  11 +-
 drivers/staging/rtl8192u/r819xU_cmdpkt.c      |  41 ------
 drivers/staging/rtl8192u/r819xU_phy.c         |  48 +++++--
 drivers/staging/rtl8712/rtl871x_mp_ioctl.c    |  29 ++--
 drivers/staging/rtl8712/rtl871x_mp_ioctl.h    | 127 ------------------
 .../staging/rtl8723bs/core/rtw_ieee80211.c    |   6 +-
 drivers/staging/rtl8723bs/core/rtw_mlme.c     |  17 +--
 drivers/staging/rtl8723bs/core/rtw_mlme_ext.c |  19 +--
 drivers/staging/rtl8723bs/core/rtw_recv.c     |   2 +-
 drivers/staging/rtl8723bs/core/rtw_security.c |   2 +-
 drivers/staging/rtl8723bs/core/rtw_sta_mgt.c  |  10 +-
 .../staging/rtl8723bs/core/rtw_wlan_util.c    |   4 +-
 .../staging/rtl8723bs/hal/odm_NoiseMonitor.c  |   9 +-
 .../staging/rtl8723bs/hal/rtl8723b_hal_init.c |  21 +--
 .../staging/rtl8723bs/hal/rtl8723b_phycfg.c   |  10 +-
 drivers/staging/rtl8723bs/hal/sdio_halinit.c  |   8 +-
 drivers/staging/rtl8723bs/hal/sdio_ops.c      |   7 +-
 .../staging/rtl8723bs/os_dep/ioctl_cfg80211.c |   2 +-
 drivers/staging/sm750fb/sm750_accel.c         |   4 +-
 drivers/staging/vt6655/upc.h                  |   2 +-
 drivers/staging/wlan-ng/cfg80211.c            |  30 +++--
 49 files changed, 256 insertions(+), 404 deletions(-)

Cc: ac100@lists.launchpad.net
Cc: "Alexander A. Klimov" <grandmaster@al2klimov.de>
Cc: Allen Pais <apais@linux.microsoft.com>
Cc: Anders Blomdell <anders.blomdell@control.lth.se>
Cc: Andrea Merello <andrea.merello@gmail.com>
Cc: Andres Klode <jak@jak-linux.org>
Cc: Andrey Shvetsov <andrey.shvetsov@k2l.de>
Cc: "Arve Hjønnevåg" <arve@android.com>
Cc: Christian Brauner <christian@brauner.io>
Cc: Comedi <comedi@comedi.org>
Cc: "David A. Schleef" <ds@schleef.org>
Cc: dri-devel@lists.freedesktop.org
Cc: Fabio Aiuto <fabioaiuto83@gmail.com>
Cc: Florian Schilhabel <florian.c.schilhabel@googlemail.com>
Cc: Forest Bond <forest@alittletooquiet.net>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Herman.Bruyninckx@mech.kuleuven.ac.be
Cc: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Hridya Valsaraju <hridya@google.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Ilya Petrov <ilya.muromec@gmail.com>
Cc: Jacob Feder <jacobsfeder@gmail.com>
Cc: Jerry chuang <wlanfae@realtek.com>
Cc: Joel Fernandes <joel@joelfernandes.org>
Cc: "John B. Wyatt IV" <jbwyatt4@gmail.com>
Cc: Jonathan Cameron <jic23@kernel.org>
Cc: "J.P. Mellor" <jpmellor@rose-hulman.edu>
Cc: karthik alapati <mail@karthek.com>
Cc: Kees Cook <keescook@chromium.org>
Cc: Kernel Team <ac100@lists.lauchpad.net>
Cc: Klaas.Gadeyne@mech.kuleuven.ac.be
Cc: Larry Finger <Larry.Finger@lwfinger.net>
Cc: Lars-Peter Clausen <lars@metafoo.de>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: linux-fbdev@vger.kernel.org
Cc: linux-iio@vger.kernel.org
Cc: linux-pwm@vger.kernel.org
Cc: linux-staging@lists.linux.dev
Cc: linux-tegra@vger.kernel.org
Cc: Marc Dietrich <marvin24@gmx.de>
Cc: Marco Cesati <marcocesati@gmail.com>
Cc: Martijn Coenen <maco@android.com>
Cc: Michael Hennerich <Michael.Hennerich@analog.com>
Cc: Michael Straube <straube.linux@gmail.com>
Cc: Mori Hess <fmhess@users.sourceforge.net>
Cc: "Nícolas F. R. A. Prado" <nfraprado@protonmail.com>
Cc: Peter Hurley <peter@hurleysoftware.com>
Cc: Philipp Zabel <p.zabel@pengutronix.de>
Cc: Pierre-Hugues Husson <phhusson@free.fr>
Cc: Robert Love <rlove@google.com>
Cc: Romain Perier <romain.perier@gmail.com>
Cc: Ross Schmidt <ross.schm.dev@gmail.com>
Cc: "Spencer E. Olson" <olsonse@umich.edu>
Cc: Stanley@BB.SD3
Cc: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
Cc: Sumera Priyadarsini <sylphrenadin@gmail.com>
Cc: Suren Baghdasaryan <surenb@google.com>
Cc: Teddy Wang <teddy.wang@siliconmotion.com>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Todd Kjos <tkjos@android.com>
Cc: Truxton Fulton <trux@truxton.com>
Cc: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
Cc: Wim.Meeussen@mech.kuleuven.ac.be
Cc: WLAN FAE <wlanfae@realtek.com>
Cc: Zhansaya Bagdauletkyzy <zhansayabagdaulet@gmail.com>
-- 
2.27.0


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

* [PATCH 25/57] staging: sm750fb: sm750_accel: Provide description for 'accel' and fix function naming
  2021-04-14 18:10 [PATCH 00/57] Rid W=1 warnings from Staging Lee Jones
@ 2021-04-14 18:10 ` Lee Jones
  2021-04-14 18:11 ` [PATCH 38/57] staging: fbtft: fb_ili9320: Remove unused variable 'ret' Lee Jones
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 6+ messages in thread
From: Lee Jones @ 2021-04-14 18:10 UTC (permalink / raw)
  To: lee.jones
  Cc: linux-kernel, Sudip Mukherjee, Teddy Wang, Greg Kroah-Hartman,
	linux-fbdev, linux-staging

Fixes the following W=1 kernel build warning(s):

 drivers/staging/sm750fb/sm750_accel.c:155: warning: Function parameter or member 'accel' not described in 'sm750_hw_copyarea'
 drivers/staging/sm750fb/sm750_accel.c:155: warning: expecting prototype for sm750_hm_copyarea(). Prototype was for sm750_hw_copyarea() instead
 drivers/staging/sm750fb/sm750_accel.c:321: warning: Function parameter or member 'accel' not described in 'sm750_hw_imageblit'

Cc: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
Cc: Teddy Wang <teddy.wang@siliconmotion.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: linux-fbdev@vger.kernel.org
Cc: linux-staging@lists.linux.dev
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/staging/sm750fb/sm750_accel.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/sm750fb/sm750_accel.c b/drivers/staging/sm750fb/sm750_accel.c
index 8faa601c700b2..24b9077a634a6 100644
--- a/drivers/staging/sm750fb/sm750_accel.c
+++ b/drivers/staging/sm750fb/sm750_accel.c
@@ -131,7 +131,8 @@ int sm750_hw_fillrect(struct lynx_accel *accel,
 }
 
 /**
- * sm750_hm_copyarea
+ * sm750_hw_copyarea
+ * @accel: Acceleration device data
  * @sBase: Address of source: offset in frame buffer
  * @sPitch: Pitch value of source surface in BYTE
  * @sx: Starting x coordinate of source surface
@@ -298,6 +299,7 @@ static unsigned int deGetTransparency(struct lynx_accel *accel)
 
 /**
  * sm750_hw_imageblit
+ * @accel: Acceleration device data
  * @pSrcbuf: pointer to start of source buffer in system memory
  * @srcDelta: Pitch value (in bytes) of the source buffer, +ive means top down
  *	      and -ive mean button up
-- 
2.27.0


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

* [PATCH 38/57] staging: fbtft: fb_ili9320: Remove unused variable 'ret'
  2021-04-14 18:10 [PATCH 00/57] Rid W=1 warnings from Staging Lee Jones
  2021-04-14 18:10 ` [PATCH 25/57] staging: sm750fb: sm750_accel: Provide description for 'accel' and fix function naming Lee Jones
@ 2021-04-14 18:11 ` Lee Jones
  2021-04-16  7:27 ` [PATCH 00/57] Rid W=1 warnings from Staging Fabio Aiuto
  2021-04-27  9:31 ` Greg Kroah-Hartman
  3 siblings, 0 replies; 6+ messages in thread
From: Lee Jones @ 2021-04-14 18:11 UTC (permalink / raw)
  To: lee.jones
  Cc: linux-kernel, Greg Kroah-Hartman, dri-devel, linux-fbdev, linux-staging

Fixes the following W=1 kernel build warning(s):

 drivers/staging/fbtft/fb_ili9320.c: In function ‘read_devicecode’:
 drivers/staging/fbtft/fb_ili9320.c:25:6: warning: variable ‘ret’ set but not used [-Wunused-but-set-variable]

Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: dri-devel@lists.freedesktop.org
Cc: linux-fbdev@vger.kernel.org
Cc: linux-staging@lists.linux.dev
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/staging/fbtft/fb_ili9320.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/fbtft/fb_ili9320.c b/drivers/staging/fbtft/fb_ili9320.c
index f2e72d14431db..f0ebc40857b34 100644
--- a/drivers/staging/fbtft/fb_ili9320.c
+++ b/drivers/staging/fbtft/fb_ili9320.c
@@ -22,11 +22,10 @@
 
 static unsigned int read_devicecode(struct fbtft_par *par)
 {
-	int ret;
 	u8 rxbuf[8] = {0, };
 
 	write_reg(par, 0x0000);
-	ret = par->fbtftops.read(par, rxbuf, 4);
+	par->fbtftops.read(par, rxbuf, 4);
 	return (rxbuf[2] << 8) | rxbuf[3];
 }
 
-- 
2.27.0


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

* Re: [PATCH 00/57] Rid W=1 warnings from Staging
  2021-04-14 18:10 [PATCH 00/57] Rid W=1 warnings from Staging Lee Jones
  2021-04-14 18:10 ` [PATCH 25/57] staging: sm750fb: sm750_accel: Provide description for 'accel' and fix function naming Lee Jones
  2021-04-14 18:11 ` [PATCH 38/57] staging: fbtft: fb_ili9320: Remove unused variable 'ret' Lee Jones
@ 2021-04-16  7:27 ` Fabio Aiuto
  2021-04-16  7:31   ` Greg Kroah-Hartman
  2021-04-27  9:31 ` Greg Kroah-Hartman
  3 siblings, 1 reply; 6+ messages in thread
From: Fabio Aiuto @ 2021-04-16  7:27 UTC (permalink / raw)
  To: Lee Jones
  Cc: linux-kernel, ac100, Alexander A. Klimov, Allen Pais,
	Anders Blomdell, Andrea Merello, Andres Klode, Andrey Shvetsov,
	Arve Hjønnevåg, Christian Brauner, Comedi,
	David A. Schleef, dri-devel, Florian Schilhabel, Forest Bond,
	Greg Kroah-Hartman, Herman.Bruyninckx, H Hartley Sweeten,
	Hridya Valsaraju, Ian Abbott, Ilya Petrov, Jacob Feder,
	Jerry chuang, Joel Fernandes, John B. Wyatt IV, Jonathan Cameron,
	J.P. Mellor, karthik alapati, Kees Cook, Kernel Team,
	Klaas.Gadeyne, Larry Finger, Lars-Peter Clausen, linux-fbdev,
	linux-iio, linux-pwm, linux-staging, linux-tegra, Marc Dietrich,
	Marco Cesati, Martijn Coenen, Michael Hennerich, Michael Straube,
	Mori Hess, Nícolas F. R. A. Prado, Peter Hurley,
	Philipp Zabel, Pierre-Hugues Husson, Robert Love, Romain Perier,
	Ross Schmidt, Spencer E. Olson, Stanley, Sudip Mukherjee,
	Sumera Priyadarsini, Suren Baghdasaryan, Teddy Wang,
	Thierry Reding, Thomas Gleixner, Todd Kjos, Truxton Fulton,
	Uwe Kleine-König, Wim.Meeussen, Zhansaya Bagdauletkyzy

On Wed, Apr 14, 2021 at 07:10:32PM +0100, 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.
> 
> Lee Jones (57):
>   staging: r8192U_core: Remove two unused variables 'ret' and
>     'reset_status'
>   staging: android: ashmem: Supply description for 'new_range'
>   staging: comedi_8254: Fix descriptions for 'i8254' and 'iobase'
>   staging: r8192U_core: Do not use kernel-doc formatting for !kernel-doc
>     headers
>   staging: r819xU_phy: Remove some local variables from the stack
>   staging: r819xU_cmdpkt: Remove functionless method
>     'cmpk_handle_query_config_rx'
>   staging: wlan-ng: cfg80211: Move large struct onto the heap
>   staging: rtw_ioctl_set: Move 'channel_table' to the only place it's
>     used
>   staging: rtl8188eu: core: rtw_ieee80211: Fix incorrectly documented
>     function
>   staging: rtl8723bs: core: rtw_mlme: Remove a bunch of unused variables
>   staging: rtl8723bs: core: rtw_mlme_ext: Deal with a bunch of unused
>     variables
>   staging: rtl8712: rtl871x_mp_ioctl: Remove a bunch of unused tables
>   staging: rtl8723bs: core: rtw_recv: Mark debug variable as
>     __maybe_unused
>   staging: rtl8188eu: core: rtw_security: Fix some formatting and
>     misdocumentation
>   staging: rtl8723bs: core: rtw_security: Demote non-conformant
>     kernel-doc header
>   staging: rtl8723bs: core: rtw_sta_mgt: Remove unused variable 'psta'
>   staging: rtl8723bs: core: rtw_sta_mgt: Return error value directly
>   staging: octeon: ethernet-tx: Fix formatting issue in function header
>   staging: rtl8723bs: core: rtw_wlan_util: Remove unused variable
>     'start_seq'
>   staging: rtl8712: rtl871x_mp_ioctl: Move a large data struct onto the
>     heap
>   staging: iio: frequency: ad9834: Provide missing description for
>     'devid'
>   staging: nvec: Fix a bunch of kernel-doc issues
>   staging: ks7010: ks_hostif: Remove a bunch of unused variables
>   staging: fwserial: Demote a whole host of kernel-doc abuses
>   staging: sm750fb: sm750_accel: Provide description for 'accel' and fix
>     function naming
>   staging: most: net: Fix some kernel-doc formatting issues
>   staging: vt6655: upc: Suppress set but not used warning in macro
>   staging: rtl8192u: ieee80211_softmac: Move a large data struct onto
>     the heap
>   staging: most: dim2: Provide missing descriptions and fix doc-rot
>   staging: rtl8723bs: core: rtw_ieee80211: Remove seemingly pointless
>     copy
>   staging: rtl8723bs: core: rtw_mlme: 'retry' is only used if REJOIN is
>     set
>   staging: rtl8723bs: core: rtw_mlme_ext: 'evt_seq' is only used if
>     CHECK_EVENT_SEQ is set
>   staging: most: i2c: Fix a little doc-rot
>   staging: most: dim2: hal: Fix one kernel-doc header and demote two
>     non-conforming ones
>   staging: most: dim2: hal: Demote non-conformant kernel-doc headers
>   staging: axis-fifo: axis-fifo: Fix some formatting issues
>   staging: rtl8188eu: os_dep: ioctl_linux: Move 2 large data buffers
>     into the heap
>   staging: fbtft: fb_ili9320: Remove unused variable 'ret'
>   staging: rtl8723bs: core: rtw_ieee80211: Fix incorrectly named
>     function
>   staging: rtl8723bs: hal: odm_NoiseMonitor: Remove unused variable and
>     dead code
>   staging: rtl8188eu: os_dep: mon: Demote non-conforming kernel-doc
>     headers
>   staging: rtl8188eu: os_dep: rtw_android: Demote kernel-doc abuse
>   staging: rtl8723bs: hal: rtl8723b_hal_init: Remove unused variable and
>     dead code
>   staging: rtl8723bs: hal: rtl8723b_phycfg: Fix a bunch of misnamed
>     functions
>   staging: rtl8723bs: hal: sdio_halinit: 'start' is only used if debug
>     is enabled
>   staging: rtl8723bs: hal: sdio_ops: Mark used 'err' as __maybe_unused
>     and remove another
>   staging: rtl8723bs: os_dep: ioctl_cfg80211: 'ack' is used when debug
>     is enabled
>   staging: comedi: drivers: jr3_pci: Remove set but unused variable
>     'min_full_scale'
>   staging: comedi: drivers: ni_tio: Fix slightly broken kernel-doc and
>     demote others
>   staging: comedi: drivers: ni_routes: Demote non-conforming kernel-doc
>     headers
>   staging: axis-fifo: axis-fifo: Fix function naming in the
>     documentation
>   staging: rtl8723bs: hal: odm_NoiseMonitor: Remove unused variable
>     'func_start'
>   staging: rtl8723bs: core: rtw_mlme_ext: Move very large data buffer
>     onto the heap
>   staging: rtl8723bs: hal: rtl8723b_hal_init: Mark a bunch of debug
>     variables as __maybe_unused
>   staging: comedi: drivers: ni_mio_common: Move 'range_ni_E_ao_ext' to
>     where it is used
>   staging: comedi: drivers: comedi_isadma: Fix misspelling of
>     'dma_chan1'
>   staging: rtl8723bs: hal: sdio_halinit: Remove unused variable 'ret'
> 
>  drivers/staging/android/ashmem.c              |   1 +
>  drivers/staging/axis-fifo/axis-fifo.c         |  18 +--
>  drivers/staging/comedi/drivers/comedi_8254.c  |   3 +-
>  .../staging/comedi/drivers/comedi_isadma.c    |   2 +-
>  drivers/staging/comedi/drivers/jr3_pci.c      |   3 +-
>  .../staging/comedi/drivers/ni_mio_common.c    |   9 --
>  drivers/staging/comedi/drivers/ni_routes.c    |   6 +-
>  drivers/staging/comedi/drivers/ni_stc.h       |   9 +-
>  drivers/staging/comedi/drivers/ni_tio.c       |  12 +-
>  drivers/staging/fbtft/fb_ili9320.c            |   3 +-
>  drivers/staging/fwserial/fwserial.c           |  46 +++----
>  drivers/staging/iio/frequency/ad9834.c        |   5 +-
>  drivers/staging/ks7010/ks_hostif.c            |  14 +-
>  drivers/staging/most/dim2/dim2.c              |  23 ++--
>  drivers/staging/most/dim2/hal.c               |  10 +-
>  drivers/staging/most/i2c/i2c.c                |  12 +-
>  drivers/staging/most/net/net.c                |   6 +-
>  drivers/staging/nvec/nvec.c                   |   7 +-
>  drivers/staging/octeon/ethernet-tx.c          |   1 -
>  drivers/staging/qlge/qlge_main.c              |   4 +-
>  .../staging/rtl8188eu/core/rtw_ieee80211.c    |   2 +-
>  .../staging/rtl8188eu/core/rtw_ioctl_set.c    |   8 ++
>  drivers/staging/rtl8188eu/core/rtw_security.c |  10 +-
>  .../staging/rtl8188eu/include/rtw_mlme_ext.h  |   8 --
>  .../staging/rtl8188eu/os_dep/ioctl_linux.c    |  12 +-
>  drivers/staging/rtl8188eu/os_dep/mon.c        |   6 +-
>  .../staging/rtl8188eu/os_dep/rtw_android.c    |   2 +-
>  .../rtl8192u/ieee80211/ieee80211_softmac.c    |   9 +-
>  drivers/staging/rtl8192u/r8192U_core.c        |  11 +-
>  drivers/staging/rtl8192u/r819xU_cmdpkt.c      |  41 ------
>  drivers/staging/rtl8192u/r819xU_phy.c         |  48 +++++--
>  drivers/staging/rtl8712/rtl871x_mp_ioctl.c    |  29 ++--
>  drivers/staging/rtl8712/rtl871x_mp_ioctl.h    | 127 ------------------
>  .../staging/rtl8723bs/core/rtw_ieee80211.c    |   6 +-
>  drivers/staging/rtl8723bs/core/rtw_mlme.c     |  17 +--
>  drivers/staging/rtl8723bs/core/rtw_mlme_ext.c |  19 +--
>  drivers/staging/rtl8723bs/core/rtw_recv.c     |   2 +-
>  drivers/staging/rtl8723bs/core/rtw_security.c |   2 +-
>  drivers/staging/rtl8723bs/core/rtw_sta_mgt.c  |  10 +-
>  .../staging/rtl8723bs/core/rtw_wlan_util.c    |   4 +-
>  .../staging/rtl8723bs/hal/odm_NoiseMonitor.c  |   9 +-
>  .../staging/rtl8723bs/hal/rtl8723b_hal_init.c |  21 +--
>  .../staging/rtl8723bs/hal/rtl8723b_phycfg.c   |  10 +-
>  drivers/staging/rtl8723bs/hal/sdio_halinit.c  |   8 +-
>  drivers/staging/rtl8723bs/hal/sdio_ops.c      |   7 +-
>  .../staging/rtl8723bs/os_dep/ioctl_cfg80211.c |   2 +-
>  drivers/staging/sm750fb/sm750_accel.c         |   4 +-
>  drivers/staging/vt6655/upc.h                  |   2 +-
>  drivers/staging/wlan-ng/cfg80211.c            |  30 +++--
>  49 files changed, 256 insertions(+), 404 deletions(-)
> 
> Cc: ac100@lists.launchpad.net
> Cc: "Alexander A. Klimov" <grandmaster@al2klimov.de>
> Cc: Allen Pais <apais@linux.microsoft.com>
> Cc: Anders Blomdell <anders.blomdell@control.lth.se>
> Cc: Andrea Merello <andrea.merello@gmail.com>
> Cc: Andres Klode <jak@jak-linux.org>
> Cc: Andrey Shvetsov <andrey.shvetsov@k2l.de>
> Cc: "Arve Hjønnevåg" <arve@android.com>
> Cc: Christian Brauner <christian@brauner.io>
> Cc: Comedi <comedi@comedi.org>
> Cc: "David A. Schleef" <ds@schleef.org>
> Cc: dri-devel@lists.freedesktop.org
> Cc: Fabio Aiuto <fabioaiuto83@gmail.com>
> Cc: Florian Schilhabel <florian.c.schilhabel@googlemail.com>
> Cc: Forest Bond <forest@alittletooquiet.net>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Herman.Bruyninckx@mech.kuleuven.ac.be
> Cc: H Hartley Sweeten <hsweeten@visionengravers.com>
> Cc: Hridya Valsaraju <hridya@google.com>
> Cc: Ian Abbott <abbotti@mev.co.uk>
> Cc: Ilya Petrov <ilya.muromec@gmail.com>
> Cc: Jacob Feder <jacobsfeder@gmail.com>
> Cc: Jerry chuang <wlanfae@realtek.com>
> Cc: Joel Fernandes <joel@joelfernandes.org>
> Cc: "John B. Wyatt IV" <jbwyatt4@gmail.com>
> Cc: Jonathan Cameron <jic23@kernel.org>
> Cc: "J.P. Mellor" <jpmellor@rose-hulman.edu>
> Cc: karthik alapati <mail@karthek.com>
> Cc: Kees Cook <keescook@chromium.org>
> Cc: Kernel Team <ac100@lists.lauchpad.net>
> Cc: Klaas.Gadeyne@mech.kuleuven.ac.be
> Cc: Larry Finger <Larry.Finger@lwfinger.net>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: linux-fbdev@vger.kernel.org
> Cc: linux-iio@vger.kernel.org
> Cc: linux-pwm@vger.kernel.org
> Cc: linux-staging@lists.linux.dev
> Cc: linux-tegra@vger.kernel.org
> Cc: Marc Dietrich <marvin24@gmx.de>
> Cc: Marco Cesati <marcocesati@gmail.com>
> Cc: Martijn Coenen <maco@android.com>
> Cc: Michael Hennerich <Michael.Hennerich@analog.com>
> Cc: Michael Straube <straube.linux@gmail.com>
> Cc: Mori Hess <fmhess@users.sourceforge.net>
> Cc: "Nícolas F. R. A. Prado" <nfraprado@protonmail.com>
> Cc: Peter Hurley <peter@hurleysoftware.com>
> Cc: Philipp Zabel <p.zabel@pengutronix.de>
> Cc: Pierre-Hugues Husson <phhusson@free.fr>
> Cc: Robert Love <rlove@google.com>
> Cc: Romain Perier <romain.perier@gmail.com>
> Cc: Ross Schmidt <ross.schm.dev@gmail.com>
> Cc: "Spencer E. Olson" <olsonse@umich.edu>
> Cc: Stanley@BB.SD3
> Cc: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
> Cc: Sumera Priyadarsini <sylphrenadin@gmail.com>
> Cc: Suren Baghdasaryan <surenb@google.com>
> Cc: Teddy Wang <teddy.wang@siliconmotion.com>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: Todd Kjos <tkjos@android.com>
> Cc: Truxton Fulton <trux@truxton.com>
> Cc: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
> Cc: Wim.Meeussen@mech.kuleuven.ac.be
> Cc: WLAN FAE <wlanfae@realtek.com>
> Cc: Zhansaya Bagdauletkyzy <zhansayabagdaulet@gmail.com>
> -- 
> 2.27.0
> 

Hi,

what about splitting this series in smaller per driver series?

thank you,

fabio

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

* Re: [PATCH 00/57] Rid W=1 warnings from Staging
  2021-04-16  7:27 ` [PATCH 00/57] Rid W=1 warnings from Staging Fabio Aiuto
@ 2021-04-16  7:31   ` Greg Kroah-Hartman
  0 siblings, 0 replies; 6+ messages in thread
From: Greg Kroah-Hartman @ 2021-04-16  7:31 UTC (permalink / raw)
  To: Fabio Aiuto
  Cc: Lee Jones, linux-kernel, ac100, Alexander A. Klimov, Allen Pais,
	Anders Blomdell, Andrea Merello, Andres Klode, Andrey Shvetsov,
	Arve Hjønnevåg, Christian Brauner, Comedi,
	David A. Schleef, dri-devel, Florian Schilhabel, Forest Bond,
	Herman.Bruyninckx, H Hartley Sweeten, Hridya Valsaraju,
	Ian Abbott, Ilya Petrov, Jacob Feder, Jerry chuang,
	Joel Fernandes, John B. Wyatt IV, Jonathan Cameron, J.P. Mellor,
	karthik alapati, Kees Cook, Kernel Team, Klaas.Gadeyne,
	Larry Finger, Lars-Peter Clausen, linux-fbdev, linux-iio,
	linux-pwm, linux-staging, linux-tegra, Marc Dietrich,
	Marco Cesati, Martijn Coenen, Michael Hennerich, Michael Straube,
	Mori Hess, Nícolas F. R. A. Prado, Peter Hurley,
	Philipp Zabel, Pierre-Hugues Husson, Robert Love, Romain Perier,
	Ross Schmidt, Spencer E. Olson, Stanley, Sudip Mukherjee,
	Sumera Priyadarsini, Suren Baghdasaryan, Teddy Wang,
	Thierry Reding, Thomas Gleixner, Todd Kjos, Truxton Fulton,
	Uwe Kleine-König, Wim.Meeussen, Zhansaya Bagdauletkyzy

On Fri, Apr 16, 2021 at 09:27:51AM +0200, Fabio Aiuto wrote:
> On Wed, Apr 14, 2021 at 07:10:32PM +0100, 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.
> > 
> > Lee Jones (57):
> >   staging: r8192U_core: Remove two unused variables 'ret' and
> >     'reset_status'
> >   staging: android: ashmem: Supply description for 'new_range'
> >   staging: comedi_8254: Fix descriptions for 'i8254' and 'iobase'
> >   staging: r8192U_core: Do not use kernel-doc formatting for !kernel-doc
> >     headers
> >   staging: r819xU_phy: Remove some local variables from the stack
> >   staging: r819xU_cmdpkt: Remove functionless method
> >     'cmpk_handle_query_config_rx'
> >   staging: wlan-ng: cfg80211: Move large struct onto the heap
> >   staging: rtw_ioctl_set: Move 'channel_table' to the only place it's
> >     used
> >   staging: rtl8188eu: core: rtw_ieee80211: Fix incorrectly documented
> >     function
> >   staging: rtl8723bs: core: rtw_mlme: Remove a bunch of unused variables
> >   staging: rtl8723bs: core: rtw_mlme_ext: Deal with a bunch of unused
> >     variables
> >   staging: rtl8712: rtl871x_mp_ioctl: Remove a bunch of unused tables
> >   staging: rtl8723bs: core: rtw_recv: Mark debug variable as
> >     __maybe_unused
> >   staging: rtl8188eu: core: rtw_security: Fix some formatting and
> >     misdocumentation
> >   staging: rtl8723bs: core: rtw_security: Demote non-conformant
> >     kernel-doc header
> >   staging: rtl8723bs: core: rtw_sta_mgt: Remove unused variable 'psta'
> >   staging: rtl8723bs: core: rtw_sta_mgt: Return error value directly
> >   staging: octeon: ethernet-tx: Fix formatting issue in function header
> >   staging: rtl8723bs: core: rtw_wlan_util: Remove unused variable
> >     'start_seq'
> >   staging: rtl8712: rtl871x_mp_ioctl: Move a large data struct onto the
> >     heap
> >   staging: iio: frequency: ad9834: Provide missing description for
> >     'devid'
> >   staging: nvec: Fix a bunch of kernel-doc issues
> >   staging: ks7010: ks_hostif: Remove a bunch of unused variables
> >   staging: fwserial: Demote a whole host of kernel-doc abuses
> >   staging: sm750fb: sm750_accel: Provide description for 'accel' and fix
> >     function naming
> >   staging: most: net: Fix some kernel-doc formatting issues
> >   staging: vt6655: upc: Suppress set but not used warning in macro
> >   staging: rtl8192u: ieee80211_softmac: Move a large data struct onto
> >     the heap
> >   staging: most: dim2: Provide missing descriptions and fix doc-rot
> >   staging: rtl8723bs: core: rtw_ieee80211: Remove seemingly pointless
> >     copy
> >   staging: rtl8723bs: core: rtw_mlme: 'retry' is only used if REJOIN is
> >     set
> >   staging: rtl8723bs: core: rtw_mlme_ext: 'evt_seq' is only used if
> >     CHECK_EVENT_SEQ is set
> >   staging: most: i2c: Fix a little doc-rot
> >   staging: most: dim2: hal: Fix one kernel-doc header and demote two
> >     non-conforming ones
> >   staging: most: dim2: hal: Demote non-conformant kernel-doc headers
> >   staging: axis-fifo: axis-fifo: Fix some formatting issues
> >   staging: rtl8188eu: os_dep: ioctl_linux: Move 2 large data buffers
> >     into the heap
> >   staging: fbtft: fb_ili9320: Remove unused variable 'ret'
> >   staging: rtl8723bs: core: rtw_ieee80211: Fix incorrectly named
> >     function
> >   staging: rtl8723bs: hal: odm_NoiseMonitor: Remove unused variable and
> >     dead code
> >   staging: rtl8188eu: os_dep: mon: Demote non-conforming kernel-doc
> >     headers
> >   staging: rtl8188eu: os_dep: rtw_android: Demote kernel-doc abuse
> >   staging: rtl8723bs: hal: rtl8723b_hal_init: Remove unused variable and
> >     dead code
> >   staging: rtl8723bs: hal: rtl8723b_phycfg: Fix a bunch of misnamed
> >     functions
> >   staging: rtl8723bs: hal: sdio_halinit: 'start' is only used if debug
> >     is enabled
> >   staging: rtl8723bs: hal: sdio_ops: Mark used 'err' as __maybe_unused
> >     and remove another
> >   staging: rtl8723bs: os_dep: ioctl_cfg80211: 'ack' is used when debug
> >     is enabled
> >   staging: comedi: drivers: jr3_pci: Remove set but unused variable
> >     'min_full_scale'
> >   staging: comedi: drivers: ni_tio: Fix slightly broken kernel-doc and
> >     demote others
> >   staging: comedi: drivers: ni_routes: Demote non-conforming kernel-doc
> >     headers
> >   staging: axis-fifo: axis-fifo: Fix function naming in the
> >     documentation
> >   staging: rtl8723bs: hal: odm_NoiseMonitor: Remove unused variable
> >     'func_start'
> >   staging: rtl8723bs: core: rtw_mlme_ext: Move very large data buffer
> >     onto the heap
> >   staging: rtl8723bs: hal: rtl8723b_hal_init: Mark a bunch of debug
> >     variables as __maybe_unused
> >   staging: comedi: drivers: ni_mio_common: Move 'range_ni_E_ao_ext' to
> >     where it is used
> >   staging: comedi: drivers: comedi_isadma: Fix misspelling of
> >     'dma_chan1'
> >   staging: rtl8723bs: hal: sdio_halinit: Remove unused variable 'ret'
> > 
> >  drivers/staging/android/ashmem.c              |   1 +
> >  drivers/staging/axis-fifo/axis-fifo.c         |  18 +--
> >  drivers/staging/comedi/drivers/comedi_8254.c  |   3 +-
> >  .../staging/comedi/drivers/comedi_isadma.c    |   2 +-
> >  drivers/staging/comedi/drivers/jr3_pci.c      |   3 +-
> >  .../staging/comedi/drivers/ni_mio_common.c    |   9 --
> >  drivers/staging/comedi/drivers/ni_routes.c    |   6 +-
> >  drivers/staging/comedi/drivers/ni_stc.h       |   9 +-
> >  drivers/staging/comedi/drivers/ni_tio.c       |  12 +-
> >  drivers/staging/fbtft/fb_ili9320.c            |   3 +-
> >  drivers/staging/fwserial/fwserial.c           |  46 +++----
> >  drivers/staging/iio/frequency/ad9834.c        |   5 +-
> >  drivers/staging/ks7010/ks_hostif.c            |  14 +-
> >  drivers/staging/most/dim2/dim2.c              |  23 ++--
> >  drivers/staging/most/dim2/hal.c               |  10 +-
> >  drivers/staging/most/i2c/i2c.c                |  12 +-
> >  drivers/staging/most/net/net.c                |   6 +-
> >  drivers/staging/nvec/nvec.c                   |   7 +-
> >  drivers/staging/octeon/ethernet-tx.c          |   1 -
> >  drivers/staging/qlge/qlge_main.c              |   4 +-
> >  .../staging/rtl8188eu/core/rtw_ieee80211.c    |   2 +-
> >  .../staging/rtl8188eu/core/rtw_ioctl_set.c    |   8 ++
> >  drivers/staging/rtl8188eu/core/rtw_security.c |  10 +-
> >  .../staging/rtl8188eu/include/rtw_mlme_ext.h  |   8 --
> >  .../staging/rtl8188eu/os_dep/ioctl_linux.c    |  12 +-
> >  drivers/staging/rtl8188eu/os_dep/mon.c        |   6 +-
> >  .../staging/rtl8188eu/os_dep/rtw_android.c    |   2 +-
> >  .../rtl8192u/ieee80211/ieee80211_softmac.c    |   9 +-
> >  drivers/staging/rtl8192u/r8192U_core.c        |  11 +-
> >  drivers/staging/rtl8192u/r819xU_cmdpkt.c      |  41 ------
> >  drivers/staging/rtl8192u/r819xU_phy.c         |  48 +++++--
> >  drivers/staging/rtl8712/rtl871x_mp_ioctl.c    |  29 ++--
> >  drivers/staging/rtl8712/rtl871x_mp_ioctl.h    | 127 ------------------
> >  .../staging/rtl8723bs/core/rtw_ieee80211.c    |   6 +-
> >  drivers/staging/rtl8723bs/core/rtw_mlme.c     |  17 +--
> >  drivers/staging/rtl8723bs/core/rtw_mlme_ext.c |  19 +--
> >  drivers/staging/rtl8723bs/core/rtw_recv.c     |   2 +-
> >  drivers/staging/rtl8723bs/core/rtw_security.c |   2 +-
> >  drivers/staging/rtl8723bs/core/rtw_sta_mgt.c  |  10 +-
> >  .../staging/rtl8723bs/core/rtw_wlan_util.c    |   4 +-
> >  .../staging/rtl8723bs/hal/odm_NoiseMonitor.c  |   9 +-
> >  .../staging/rtl8723bs/hal/rtl8723b_hal_init.c |  21 +--
> >  .../staging/rtl8723bs/hal/rtl8723b_phycfg.c   |  10 +-
> >  drivers/staging/rtl8723bs/hal/sdio_halinit.c  |   8 +-
> >  drivers/staging/rtl8723bs/hal/sdio_ops.c      |   7 +-
> >  .../staging/rtl8723bs/os_dep/ioctl_cfg80211.c |   2 +-
> >  drivers/staging/sm750fb/sm750_accel.c         |   4 +-
> >  drivers/staging/vt6655/upc.h                  |   2 +-
> >  drivers/staging/wlan-ng/cfg80211.c            |  30 +++--
> >  49 files changed, 256 insertions(+), 404 deletions(-)
> > 
> > Cc: ac100@lists.launchpad.net
> > Cc: "Alexander A. Klimov" <grandmaster@al2klimov.de>
> > Cc: Allen Pais <apais@linux.microsoft.com>
> > Cc: Anders Blomdell <anders.blomdell@control.lth.se>
> > Cc: Andrea Merello <andrea.merello@gmail.com>
> > Cc: Andres Klode <jak@jak-linux.org>
> > Cc: Andrey Shvetsov <andrey.shvetsov@k2l.de>
> > Cc: "Arve Hjønnevåg" <arve@android.com>
> > Cc: Christian Brauner <christian@brauner.io>
> > Cc: Comedi <comedi@comedi.org>
> > Cc: "David A. Schleef" <ds@schleef.org>
> > Cc: dri-devel@lists.freedesktop.org
> > Cc: Fabio Aiuto <fabioaiuto83@gmail.com>
> > Cc: Florian Schilhabel <florian.c.schilhabel@googlemail.com>
> > Cc: Forest Bond <forest@alittletooquiet.net>
> > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > Cc: Herman.Bruyninckx@mech.kuleuven.ac.be
> > Cc: H Hartley Sweeten <hsweeten@visionengravers.com>
> > Cc: Hridya Valsaraju <hridya@google.com>
> > Cc: Ian Abbott <abbotti@mev.co.uk>
> > Cc: Ilya Petrov <ilya.muromec@gmail.com>
> > Cc: Jacob Feder <jacobsfeder@gmail.com>
> > Cc: Jerry chuang <wlanfae@realtek.com>
> > Cc: Joel Fernandes <joel@joelfernandes.org>
> > Cc: "John B. Wyatt IV" <jbwyatt4@gmail.com>
> > Cc: Jonathan Cameron <jic23@kernel.org>
> > Cc: "J.P. Mellor" <jpmellor@rose-hulman.edu>
> > Cc: karthik alapati <mail@karthek.com>
> > Cc: Kees Cook <keescook@chromium.org>
> > Cc: Kernel Team <ac100@lists.lauchpad.net>
> > Cc: Klaas.Gadeyne@mech.kuleuven.ac.be
> > Cc: Larry Finger <Larry.Finger@lwfinger.net>
> > Cc: Lars-Peter Clausen <lars@metafoo.de>
> > Cc: Lee Jones <lee.jones@linaro.org>
> > Cc: linux-fbdev@vger.kernel.org
> > Cc: linux-iio@vger.kernel.org
> > Cc: linux-pwm@vger.kernel.org
> > Cc: linux-staging@lists.linux.dev
> > Cc: linux-tegra@vger.kernel.org
> > Cc: Marc Dietrich <marvin24@gmx.de>
> > Cc: Marco Cesati <marcocesati@gmail.com>
> > Cc: Martijn Coenen <maco@android.com>
> > Cc: Michael Hennerich <Michael.Hennerich@analog.com>
> > Cc: Michael Straube <straube.linux@gmail.com>
> > Cc: Mori Hess <fmhess@users.sourceforge.net>
> > Cc: "Nícolas F. R. A. Prado" <nfraprado@protonmail.com>
> > Cc: Peter Hurley <peter@hurleysoftware.com>
> > Cc: Philipp Zabel <p.zabel@pengutronix.de>
> > Cc: Pierre-Hugues Husson <phhusson@free.fr>
> > Cc: Robert Love <rlove@google.com>
> > Cc: Romain Perier <romain.perier@gmail.com>
> > Cc: Ross Schmidt <ross.schm.dev@gmail.com>
> > Cc: "Spencer E. Olson" <olsonse@umich.edu>
> > Cc: Stanley@BB.SD3
> > Cc: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
> > Cc: Sumera Priyadarsini <sylphrenadin@gmail.com>
> > Cc: Suren Baghdasaryan <surenb@google.com>
> > Cc: Teddy Wang <teddy.wang@siliconmotion.com>
> > Cc: Thierry Reding <thierry.reding@gmail.com>
> > Cc: Thomas Gleixner <tglx@linutronix.de>
> > Cc: Todd Kjos <tkjos@android.com>
> > Cc: Truxton Fulton <trux@truxton.com>
> > Cc: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
> > Cc: Wim.Meeussen@mech.kuleuven.ac.be
> > Cc: WLAN FAE <wlanfae@realtek.com>
> > Cc: Zhansaya Bagdauletkyzy <zhansayabagdaulet@gmail.com>
> > -- 
> > 2.27.0
> > 
> 
> Hi,
> 
> what about splitting this series in smaller per driver series?

No real need to, I can handle patch series this big easily.

thanks,

greg k-h

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

* Re: [PATCH 00/57] Rid W=1 warnings from Staging
  2021-04-14 18:10 [PATCH 00/57] Rid W=1 warnings from Staging Lee Jones
                   ` (2 preceding siblings ...)
  2021-04-16  7:27 ` [PATCH 00/57] Rid W=1 warnings from Staging Fabio Aiuto
@ 2021-04-27  9:31 ` Greg Kroah-Hartman
  3 siblings, 0 replies; 6+ messages in thread
From: Greg Kroah-Hartman @ 2021-04-27  9:31 UTC (permalink / raw)
  To: Lee Jones
  Cc: linux-kernel, ac100, Alexander A. Klimov, Allen Pais,
	Anders Blomdell, Andrea Merello, Andres Klode, Andrey Shvetsov,
	Arve Hjønnevåg, Christian Brauner, Comedi,
	David A. Schleef, dri-devel, Fabio Aiuto, Florian Schilhabel,
	Forest Bond, Herman.Bruyninckx, H Hartley Sweeten,
	Hridya Valsaraju, Ian Abbott, Ilya Petrov, Jacob Feder,
	Jerry chuang, Joel Fernandes, John B. Wyatt IV, Jonathan Cameron,
	J.P. Mellor, karthik alapati, Kees Cook, Kernel Team,
	Klaas.Gadeyne, Larry Finger, Lars-Peter Clausen, linux-fbdev,
	linux-iio, linux-pwm, linux-staging, linux-tegra, Marc Dietrich,
	Marco Cesati, Martijn Coenen, Michael Hennerich, Michael Straube,
	Mori Hess, Nícolas F. R. A. Prado, Peter Hurley,
	Philipp Zabel, Pierre-Hugues Husson, Robert Love, Romain Perier,
	Ross Schmidt, Spencer E. Olson, Stanley, Sudip Mukherjee,
	Sumera Priyadarsini, Suren Baghdasaryan, Teddy Wang,
	Thierry Reding, Thomas Gleixner, Todd Kjos, Truxton Fulton,
	Uwe Kleine-König, Wim.Meeussen, Zhansaya Bagdauletkyzy

On Wed, Apr 14, 2021 at 07:10:32PM +0100, 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.
> 
> Lee Jones (57):

44 of these applied to my tree, I'll keep them in my "testing" branch
for now until -rc1 comes out.  Feel free to rebase your series on that
and fix up the remaining ones and resend.

Note, the comedi drivers have moved to drivers/comedi/ so those patches
need to be sent as a different series, if you still want to make those
changes based on the review comments.

thanks,

greg k-h

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

end of thread, other threads:[~2021-04-27  9:32 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-14 18:10 [PATCH 00/57] Rid W=1 warnings from Staging Lee Jones
2021-04-14 18:10 ` [PATCH 25/57] staging: sm750fb: sm750_accel: Provide description for 'accel' and fix function naming Lee Jones
2021-04-14 18:11 ` [PATCH 38/57] staging: fbtft: fb_ili9320: Remove unused variable 'ret' Lee Jones
2021-04-16  7:27 ` [PATCH 00/57] Rid W=1 warnings from Staging Fabio Aiuto
2021-04-16  7:31   ` Greg Kroah-Hartman
2021-04-27  9:31 ` Greg Kroah-Hartman

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).