linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v3 0/5] (Set 2/2) Rid W=1 issues from Input
@ 2021-01-14 15:23 Lee Jones
  2021-01-14 15:23 ` [PATCH 5/5] input: touchscreen: stmpe-ts: Add description for 'prop' struct member Lee Jones
  0 siblings, 1 reply; 3+ messages in thread
From: Lee Jones @ 2021-01-14 15:23 UTC (permalink / raw)
  To: lee.jones
  Cc: Henrik Rydberg, C. Scott Ananian, Luotao Fu, Peter Osterlund,
	Bruce Kalk, Dmitry Torokhov, Stefan Riedmueller, linux-kernel,
	Stefan Gmeiner, Alexandre Torgue, Sangwon Jee, linux-arm-kernel,
	Maxime Coquelin, linux-input, James Hilliard, linux-stm32,
	Benjamin Tissoires, Daniel Ritz

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 second (and last) of 2 sets.

v3:
 - Make use of 'ret' instead of removing it
 - Use __always_unused instead of removing unused variables
 - Fix recent/new warning regarding undocumented 'prop' param
 
v2:
 - Replace empty if() with commentary
   - Suggested-by: Joe Perches


Lee Jones (5):
  input: mouse: synaptics: Replace NOOP with suitable commentary
  input: touchscreen: melfas_mip4: Mark a bunch of variables as
    __always_unused
  input: touchscreen: usbtouchscreen: Actually check return value of
    usb_submit_urb()
  input: touchscreen: surface3_spi: Remove set but unused variable
    'timestamp'
  input: touchscreen: stmpe-ts: Add description for 'prop' struct member

 drivers/input/mouse/synaptics.c            | 7 +++++--
 drivers/input/touchscreen/melfas_mip4.c    | 8 ++++----
 drivers/input/touchscreen/stmpe-ts.c       | 1 +
 drivers/input/touchscreen/surface3_spi.c   | 2 --
 drivers/input/touchscreen/usbtouchscreen.c | 3 +++
 5 files changed, 13 insertions(+), 8 deletions(-)

Cc: Alexandre Torgue <alexandre.torgue@st.com>
Cc: Benjamin Tissoires <benjamin.tissoires@gmail.com>
Cc: Bruce Kalk <kall@compass.com>
Cc: "C. Scott Ananian" <cananian@alumni.priceton.edu>
Cc: Daniel Ritz <daniel.ritz@gmx.ch>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Henrik Rydberg <rydberg@bitmath.org>
Cc: James Hilliard <james.hilliard1@gmail.com>
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-input@vger.kernel.org
Cc: linux-stm32@st-md-mailman.stormreply.com
Cc: Luotao Fu <l.fu@pengutronix.de>
Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
Cc: Peter Osterlund <petero2@telia.com>
Cc: Sangwon Jee <jeesw@melfas.com>
Cc: Stefan Gmeiner <riddlebox@freesurf.ch>
Cc: Stefan Riedmueller <s.riedmueller@phytec.de>
Cc: this to <linux-input@vger.kernel.org>
-- 
2.25.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* [PATCH 5/5] input: touchscreen: stmpe-ts: Add description for 'prop' struct member
  2021-01-14 15:23 [PATCH v3 0/5] (Set 2/2) Rid W=1 issues from Input Lee Jones
@ 2021-01-14 15:23 ` Lee Jones
  2021-01-20  2:23   ` Dmitry Torokhov
  0 siblings, 1 reply; 3+ messages in thread
From: Lee Jones @ 2021-01-14 15:23 UTC (permalink / raw)
  To: lee.jones
  Cc: Luotao Fu, Stefan Riedmueller, Dmitry Torokhov, linux-kernel,
	Maxime Coquelin, linux-input, linux-stm32, linux-arm-kernel,
	Alexandre Torgue

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

 drivers/input/touchscreen/stmpe-ts.c:82: warning: Function parameter or member 'prop' not described in 'stmpe_touch'

Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
Cc: Alexandre Torgue <alexandre.torgue@st.com>
Cc: Stefan Riedmueller <s.riedmueller@phytec.de>
Cc: Luotao Fu <l.fu@pengutronix.de>
Cc: linux-input@vger.kernel.org
Cc: linux-stm32@st-md-mailman.stormreply.com
Cc: linux-arm-kernel@lists.infradead.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/input/touchscreen/stmpe-ts.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/input/touchscreen/stmpe-ts.c b/drivers/input/touchscreen/stmpe-ts.c
index cd747725589b1..25c45c3a35615 100644
--- a/drivers/input/touchscreen/stmpe-ts.c
+++ b/drivers/input/touchscreen/stmpe-ts.c
@@ -52,6 +52,7 @@
  * @idev: registered input device
  * @work: a work item used to scan the device
  * @dev: a pointer back to the MFD cell struct device*
+ * @prop: Touchscreen properties
  * @ave_ctrl: Sample average control
  * (0 -> 1 sample, 1 -> 2 samples, 2 -> 4 samples, 3 -> 8 samples)
  * @touch_det_delay: Touch detect interrupt delay
-- 
2.25.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH 5/5] input: touchscreen: stmpe-ts: Add description for 'prop' struct member
  2021-01-14 15:23 ` [PATCH 5/5] input: touchscreen: stmpe-ts: Add description for 'prop' struct member Lee Jones
@ 2021-01-20  2:23   ` Dmitry Torokhov
  0 siblings, 0 replies; 3+ messages in thread
From: Dmitry Torokhov @ 2021-01-20  2:23 UTC (permalink / raw)
  To: Lee Jones
  Cc: Stefan Riedmueller, Luotao Fu, linux-kernel, Maxime Coquelin,
	linux-input, linux-stm32, linux-arm-kernel, Alexandre Torgue

On Thu, Jan 14, 2021 at 03:23:23PM +0000, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
> 
>  drivers/input/touchscreen/stmpe-ts.c:82: warning: Function parameter or member 'prop' not described in 'stmpe_touch'
> 
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
> Cc: Alexandre Torgue <alexandre.torgue@st.com>
> Cc: Stefan Riedmueller <s.riedmueller@phytec.de>
> Cc: Luotao Fu <l.fu@pengutronix.de>
> Cc: linux-input@vger.kernel.org
> Cc: linux-stm32@st-md-mailman.stormreply.com
> Cc: linux-arm-kernel@lists.infradead.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>

Applied, thank you.

-- 
Dmitry

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

end of thread, other threads:[~2021-01-20  2:26 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-14 15:23 [PATCH v3 0/5] (Set 2/2) Rid W=1 issues from Input Lee Jones
2021-01-14 15:23 ` [PATCH 5/5] input: touchscreen: stmpe-ts: Add description for 'prop' struct member Lee Jones
2021-01-20  2:23   ` Dmitry Torokhov

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