* [PATCH 01/15] input: joystick: xpad: Demote non-conformant kernel-doc header
2020-11-12 11:01 [PATCH 00/15] [Set 2/2] Rid W=1 issues from Input Lee Jones
@ 2020-11-12 11:01 ` Lee Jones
2020-11-20 3:16 ` Dmitry Torokhov
2020-11-12 11:01 ` [PATCH 02/15] input: keyboard: samsung-keypad: Remove set but unused variable 'var' Lee Jones
` (13 subsequent siblings)
14 siblings, 1 reply; 42+ messages in thread
From: Lee Jones @ 2020-11-12 11:01 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, Dmitry Torokhov, Łukasz Patron,
Cameron Gutman, Marko Friedemann, Oliver Schwartz, Steven Toth,
Franz Lehner, Ivan Hawkes, Dominic Cerquetti, Adam Buchbinder,
Jan Kratochvil, Christoph Fritz, linux-input
Fixes the following W=1 kernel build warning(s):
drivers/input/joystick/xpad.c:1361: warning: Function parameter or member 'xpad' not described in 'xpad_send_led_command'
drivers/input/joystick/xpad.c:1361: warning: Function parameter or member 'command' not described in 'xpad_send_led_command'
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: "Łukasz Patron" <priv.luk@gmail.com>
Cc: Cameron Gutman <aicommander@gmail.com>
Cc: Marko Friedemann <mfr@bmx-chemnitz.de>
Cc: Oliver Schwartz <Oliver.Schwartz@gmx.de>
Cc: Steven Toth <steve@toth.demon.co.uk>
Cc: Franz Lehner <franz@caos.at>
Cc: Ivan Hawkes <blackhawk@ivanhawkes.com>
Cc: Dominic Cerquetti <binary1230@yahoo.com>
Cc: Adam Buchbinder <adam.buchbinder@gmail.com>
Cc: Jan Kratochvil <honza@jikos.cz>
Cc: Christoph Fritz <chf.fritz@googlemail.com>
Cc: linux-input@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/input/joystick/xpad.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/input/joystick/xpad.c b/drivers/input/joystick/xpad.c
index c77cdb3b62b5b..2cd4296c2d082 100644
--- a/drivers/input/joystick/xpad.c
+++ b/drivers/input/joystick/xpad.c
@@ -1337,7 +1337,7 @@ struct xpad_led {
struct usb_xpad *xpad;
};
-/**
+/*
* set the LEDs on Xbox360 / Wireless Controllers
* @param command
* 0: off
--
2.25.1
^ permalink raw reply related [flat|nested] 42+ messages in thread
* Re: [PATCH 01/15] input: joystick: xpad: Demote non-conformant kernel-doc header
2020-11-12 11:01 ` [PATCH 01/15] input: joystick: xpad: Demote non-conformant kernel-doc header Lee Jones
@ 2020-11-20 3:16 ` Dmitry Torokhov
0 siblings, 0 replies; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-20 3:16 UTC (permalink / raw)
To: Lee Jones
Cc: linux-kernel, Łukasz Patron, Cameron Gutman,
Marko Friedemann, Oliver Schwartz, Steven Toth, Franz Lehner,
Ivan Hawkes, Dominic Cerquetti, Adam Buchbinder, Jan Kratochvil,
Christoph Fritz, linux-input
On Thu, Nov 12, 2020 at 11:01:50AM +0000, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/joystick/xpad.c:1361: warning: Function parameter or member 'xpad' not described in 'xpad_send_led_command'
> drivers/input/joystick/xpad.c:1361: warning: Function parameter or member 'command' not described in 'xpad_send_led_command'
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: "Łukasz Patron" <priv.luk@gmail.com>
> Cc: Cameron Gutman <aicommander@gmail.com>
> Cc: Marko Friedemann <mfr@bmx-chemnitz.de>
> Cc: Oliver Schwartz <Oliver.Schwartz@gmx.de>
> Cc: Steven Toth <steve@toth.demon.co.uk>
> Cc: Franz Lehner <franz@caos.at>
> Cc: Ivan Hawkes <blackhawk@ivanhawkes.com>
> Cc: Dominic Cerquetti <binary1230@yahoo.com>
> Cc: Adam Buchbinder <adam.buchbinder@gmail.com>
> Cc: Jan Kratochvil <honza@jikos.cz>
> Cc: Christoph Fritz <chf.fritz@googlemail.com>
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
Applied, thank you.
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* [PATCH 02/15] input: keyboard: samsung-keypad: Remove set but unused variable 'var'
2020-11-12 11:01 [PATCH 00/15] [Set 2/2] Rid W=1 issues from Input Lee Jones
2020-11-12 11:01 ` [PATCH 01/15] input: joystick: xpad: Demote non-conformant kernel-doc header Lee Jones
@ 2020-11-12 11:01 ` Lee Jones
2020-11-13 7:37 ` Dmitry Torokhov
2020-11-12 11:01 ` [PATCH 03/15] input: misc: mc13783-pwrbutton: File headers are not good candidates for kernel-doc Lee Jones
` (12 subsequent siblings)
14 siblings, 1 reply; 42+ messages in thread
From: Lee Jones @ 2020-11-12 11:01 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, Dmitry Torokhov, Joonyoung Shim, Donghwa Lee, linux-input
Fixes the following W=1 kernel build warning(s):
drivers/input/keyboard/samsung-keypad.c: In function ‘samsung_keypad_irq’:
drivers/input/keyboard/samsung-keypad.c:149:15: warning: variable ‘val’ set but not used [-Wunused-but-set-variable]
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Joonyoung Shim <jy0922.shim@samsung.com>
Cc: Donghwa Lee <dh09.lee@samsung.com>
Cc: linux-input@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/input/keyboard/samsung-keypad.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/input/keyboard/samsung-keypad.c b/drivers/input/keyboard/samsung-keypad.c
index 70c1d086bdd2a..1ed939d9798cf 100644
--- a/drivers/input/keyboard/samsung-keypad.c
+++ b/drivers/input/keyboard/samsung-keypad.c
@@ -146,13 +146,12 @@ static irqreturn_t samsung_keypad_irq(int irq, void *dev_id)
{
struct samsung_keypad *keypad = dev_id;
unsigned int row_state[SAMSUNG_MAX_COLS];
- unsigned int val;
bool key_down;
pm_runtime_get_sync(&keypad->pdev->dev);
do {
- val = readl(keypad->base + SAMSUNG_KEYIFSTSCLR);
+ readl(keypad->base + SAMSUNG_KEYIFSTSCLR);
/* Clear interrupt. */
writel(~0x0, keypad->base + SAMSUNG_KEYIFSTSCLR);
--
2.25.1
^ permalink raw reply related [flat|nested] 42+ messages in thread
* Re: [PATCH 02/15] input: keyboard: samsung-keypad: Remove set but unused variable 'var'
2020-11-12 11:01 ` [PATCH 02/15] input: keyboard: samsung-keypad: Remove set but unused variable 'var' Lee Jones
@ 2020-11-13 7:37 ` Dmitry Torokhov
0 siblings, 0 replies; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-13 7:37 UTC (permalink / raw)
To: Lee Jones; +Cc: linux-kernel, Joonyoung Shim, Donghwa Lee, linux-input
On Thu, Nov 12, 2020 at 11:01:51AM +0000, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/keyboard/samsung-keypad.c: In function ‘samsung_keypad_irq’:
> drivers/input/keyboard/samsung-keypad.c:149:15: warning: variable ‘val’ set but not used [-Wunused-but-set-variable]
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Joonyoung Shim <jy0922.shim@samsung.com>
> Cc: Donghwa Lee <dh09.lee@samsung.com>
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
Applied, thank you.
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* [PATCH 03/15] input: misc: mc13783-pwrbutton: File headers are not good candidates for kernel-doc
2020-11-12 11:01 [PATCH 00/15] [Set 2/2] Rid W=1 issues from Input Lee Jones
2020-11-12 11:01 ` [PATCH 01/15] input: joystick: xpad: Demote non-conformant kernel-doc header Lee Jones
2020-11-12 11:01 ` [PATCH 02/15] input: keyboard: samsung-keypad: Remove set but unused variable 'var' Lee Jones
@ 2020-11-12 11:01 ` Lee Jones
2020-11-20 3:16 ` Dmitry Torokhov
2020-11-12 11:01 ` [PATCH 04/15] input: touchscreen: goodix: Provide some missing function parameter descriptions Lee Jones
` (11 subsequent siblings)
14 siblings, 1 reply; 42+ messages in thread
From: Lee Jones @ 2020-11-12 11:01 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, Dmitry Torokhov, De Schrijver, Felipe Balbi, linux-input
Fixes the following W=1 kernel build warning(s):
drivers/input/misc/mc13783-pwrbutton.c:32: warning: cannot understand function prototype: 'struct mc13783_pwrb '
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: De Schrijver <peter.de-schrijver@nokia.com>
Cc: Felipe Balbi <felipe.balbi@nokia.com>
Cc: linux-input@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/input/misc/mc13783-pwrbutton.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/input/misc/mc13783-pwrbutton.c b/drivers/input/misc/mc13783-pwrbutton.c
index afdf8ef25ee86..0636eee4bb6ce 100644
--- a/drivers/input/misc/mc13783-pwrbutton.c
+++ b/drivers/input/misc/mc13783-pwrbutton.c
@@ -1,4 +1,4 @@
-/**
+/*
* Copyright (C) 2011 Philippe Rétornaz
*
* Based on twl4030-pwrbutton driver by:
--
2.25.1
^ permalink raw reply related [flat|nested] 42+ messages in thread
* [PATCH 04/15] input: touchscreen: goodix: Provide some missing function parameter descriptions
2020-11-12 11:01 [PATCH 00/15] [Set 2/2] Rid W=1 issues from Input Lee Jones
` (2 preceding siblings ...)
2020-11-12 11:01 ` [PATCH 03/15] input: misc: mc13783-pwrbutton: File headers are not good candidates for kernel-doc Lee Jones
@ 2020-11-12 11:01 ` Lee Jones
2020-11-20 3:16 ` Dmitry Torokhov
2020-11-12 11:01 ` [PATCH 05/15] input: touchscreen: imx6ul_tsc: Remove set but unused variable 'value' Lee Jones
` (10 subsequent siblings)
14 siblings, 1 reply; 42+ messages in thread
From: Lee Jones @ 2020-11-12 11:01 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, Bastien Nocera, Dmitry Torokhov, Henrik Rydberg,
K. Merker, authored by, Benjamin Tissoires, linux-input
Fixes the following W=1 kernel build warning(s):
drivers/input/touchscreen/goodix.c:569: warning: Function parameter or member 'len' not described in 'goodix_check_cfg'
drivers/input/touchscreen/goodix.c:587: warning: Function parameter or member 'len' not described in 'goodix_send_cfg'
drivers/input/touchscreen/goodix.c:1165: warning: Function parameter or member 'cfg' not described in 'goodix_config_cb'
drivers/input/touchscreen/goodix.c:1165: warning: Function parameter or member 'ctx' not described in 'goodix_config_cb'
drivers/input/touchscreen/goodix.c:1165: warning: Excess function parameter 'ts' description in 'goodix_config_cb'
Cc: Bastien Nocera <hadess@hadess.net>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Henrik Rydberg <rydberg@bitmath.org>
Cc: "K. Merker" <merker@debian.org>
Cc: authored by <andrew@goodix.com>
Cc: Benjamin Tissoires <benjamin.tissoires@gmail.com>
Cc: linux-input@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/input/touchscreen/goodix.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/input/touchscreen/goodix.c b/drivers/input/touchscreen/goodix.c
index 02c75ea385e08..80a82a66a0f3b 100644
--- a/drivers/input/touchscreen/goodix.c
+++ b/drivers/input/touchscreen/goodix.c
@@ -564,6 +564,7 @@ static void goodix_calc_cfg_checksum_16(struct goodix_ts_data *ts)
*
* @ts: goodix_ts_data pointer
* @cfg: firmware config data
+ * @len: config data length
*/
static int goodix_check_cfg(struct goodix_ts_data *ts, const u8 *cfg, int len)
{
@@ -582,6 +583,7 @@ static int goodix_check_cfg(struct goodix_ts_data *ts, const u8 *cfg, int len)
*
* @ts: goodix_ts_data pointer
* @cfg: config firmware to write to device
+ * @len: config data length
*/
static int goodix_send_cfg(struct goodix_ts_data *ts, const u8 *cfg, int len)
{
@@ -1156,6 +1158,7 @@ static int goodix_configure_dev(struct goodix_ts_data *ts)
/**
* goodix_config_cb - Callback to finish device init
*
+ * @cfg: firmware config
* @ts: our goodix_ts_data pointer
*
* request_firmware_wait callback that finishes
--
2.25.1
^ permalink raw reply related [flat|nested] 42+ messages in thread
* Re: [PATCH 04/15] input: touchscreen: goodix: Provide some missing function parameter descriptions
2020-11-12 11:01 ` [PATCH 04/15] input: touchscreen: goodix: Provide some missing function parameter descriptions Lee Jones
@ 2020-11-20 3:16 ` Dmitry Torokhov
0 siblings, 0 replies; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-20 3:16 UTC (permalink / raw)
To: Lee Jones
Cc: linux-kernel, Bastien Nocera, Henrik Rydberg, K. Merker,
authored by, Benjamin Tissoires, linux-input
On Thu, Nov 12, 2020 at 11:01:53AM +0000, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/touchscreen/goodix.c:569: warning: Function parameter or member 'len' not described in 'goodix_check_cfg'
> drivers/input/touchscreen/goodix.c:587: warning: Function parameter or member 'len' not described in 'goodix_send_cfg'
> drivers/input/touchscreen/goodix.c:1165: warning: Function parameter or member 'cfg' not described in 'goodix_config_cb'
> drivers/input/touchscreen/goodix.c:1165: warning: Function parameter or member 'ctx' not described in 'goodix_config_cb'
> drivers/input/touchscreen/goodix.c:1165: warning: Excess function parameter 'ts' description in 'goodix_config_cb'
>
> Cc: Bastien Nocera <hadess@hadess.net>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Henrik Rydberg <rydberg@bitmath.org>
> Cc: "K. Merker" <merker@debian.org>
> Cc: authored by <andrew@goodix.com>
> Cc: Benjamin Tissoires <benjamin.tissoires@gmail.com>
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
Applied, thank you.
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* [PATCH 05/15] input: touchscreen: imx6ul_tsc: Remove set but unused variable 'value'
2020-11-12 11:01 [PATCH 00/15] [Set 2/2] Rid W=1 issues from Input Lee Jones
` (3 preceding siblings ...)
2020-11-12 11:01 ` [PATCH 04/15] input: touchscreen: goodix: Provide some missing function parameter descriptions Lee Jones
@ 2020-11-12 11:01 ` Lee Jones
2020-11-12 12:20 ` Bough Chen
2020-11-13 7:37 ` Dmitry Torokhov
2020-11-12 11:01 ` [PATCH 06/15] input: touchscreen: melfas_mip4: Remove a bunch of unused variables Lee Jones
` (9 subsequent siblings)
14 siblings, 2 replies; 42+ messages in thread
From: Lee Jones @ 2020-11-12 11:01 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, Dmitry Torokhov, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
Haibo Chen, linux-input
Fixes the following W=1 kernel build warning(s):
drivers/input/touchscreen/imx6ul_tsc.c: In function ‘adc_irq_fn’:
drivers/input/touchscreen/imx6ul_tsc.c:307:6: warning: variable ‘value’ set but not used [-Wunused-but-set-variable]
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: NXP Linux Team <linux-imx@nxp.com>
Cc: Haibo Chen <haibo.chen@freescale.com>
Cc: linux-input@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/input/touchscreen/imx6ul_tsc.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/input/touchscreen/imx6ul_tsc.c b/drivers/input/touchscreen/imx6ul_tsc.c
index cd369f9ac5e60..e1852f7d4d31e 100644
--- a/drivers/input/touchscreen/imx6ul_tsc.c
+++ b/drivers/input/touchscreen/imx6ul_tsc.c
@@ -304,11 +304,10 @@ static irqreturn_t adc_irq_fn(int irq, void *dev_id)
{
struct imx6ul_tsc *tsc = dev_id;
u32 coco;
- u32 value;
coco = readl(tsc->adc_regs + REG_ADC_HS);
if (coco & 0x01) {
- value = readl(tsc->adc_regs + REG_ADC_R0);
+ readl(tsc->adc_regs + REG_ADC_R0);
complete(&tsc->completion);
}
--
2.25.1
^ permalink raw reply related [flat|nested] 42+ messages in thread
* RE: [PATCH 05/15] input: touchscreen: imx6ul_tsc: Remove set but unused variable 'value'
2020-11-12 11:01 ` [PATCH 05/15] input: touchscreen: imx6ul_tsc: Remove set but unused variable 'value' Lee Jones
@ 2020-11-12 12:20 ` Bough Chen
2020-11-13 7:37 ` Dmitry Torokhov
1 sibling, 0 replies; 42+ messages in thread
From: Bough Chen @ 2020-11-12 12:20 UTC (permalink / raw)
To: Lee Jones
Cc: linux-kernel, Dmitry Torokhov, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, dl-linux-imx, Haibo Chen,
linux-input
> -----Original Message-----
> From: Lee Jones [mailto:lee.jones@linaro.org]
> Sent: 2020年11月12日 19:02
> To: lee.jones@linaro.org
> Cc: linux-kernel@vger.kernel.org; Dmitry Torokhov
> <dmitry.torokhov@gmail.com>; Shawn Guo <shawnguo@kernel.org>; Sascha
> Hauer <s.hauer@pengutronix.de>; Pengutronix Kernel Team
> <kernel@pengutronix.de>; Fabio Estevam <festevam@gmail.com>;
> dl-linux-imx <linux-imx@nxp.com>; Haibo Chen <haibo.chen@freescale.com>;
> linux-input@vger.kernel.org
> Subject: [PATCH 05/15] input: touchscreen: imx6ul_tsc: Remove set but unused
> variable 'value'
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/touchscreen/imx6ul_tsc.c: In function ‘adc_irq_fn’:
> drivers/input/touchscreen/imx6ul_tsc.c:307:6: warning: variable ‘value’ set
> but not used [-Wunused-but-set-variable]
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Sascha Hauer <s.hauer@pengutronix.de>
> Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: NXP Linux Team <linux-imx@nxp.com>
> Cc: Haibo Chen <haibo.chen@freescale.com>
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> ---
> drivers/input/touchscreen/imx6ul_tsc.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/input/touchscreen/imx6ul_tsc.c
> b/drivers/input/touchscreen/imx6ul_tsc.c
> index cd369f9ac5e60..e1852f7d4d31e 100644
> --- a/drivers/input/touchscreen/imx6ul_tsc.c
> +++ b/drivers/input/touchscreen/imx6ul_tsc.c
> @@ -304,11 +304,10 @@ static irqreturn_t adc_irq_fn(int irq, void *dev_id)
> {
> struct imx6ul_tsc *tsc = dev_id;
> u32 coco;
> - u32 value;
>
> coco = readl(tsc->adc_regs + REG_ADC_HS);
> if (coco & 0x01) {
> - value = readl(tsc->adc_regs + REG_ADC_R0);
> + readl(tsc->adc_regs + REG_ADC_R0);
> complete(&tsc->completion);
> }
>
Reviewed-by: Haibo Chen <haibo.chen@nxp.com>
> --
> 2.25.1
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH 05/15] input: touchscreen: imx6ul_tsc: Remove set but unused variable 'value'
2020-11-12 11:01 ` [PATCH 05/15] input: touchscreen: imx6ul_tsc: Remove set but unused variable 'value' Lee Jones
2020-11-12 12:20 ` Bough Chen
@ 2020-11-13 7:37 ` Dmitry Torokhov
2020-11-19 9:12 ` Lee Jones
1 sibling, 1 reply; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-13 7:37 UTC (permalink / raw)
To: Lee Jones
Cc: linux-kernel, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, NXP Linux Team, Haibo Chen, linux-input
On Thu, Nov 12, 2020 at 11:01:54AM +0000, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/touchscreen/imx6ul_tsc.c: In function ‘adc_irq_fn’:
> drivers/input/touchscreen/imx6ul_tsc.c:307:6: warning: variable ‘value’ set but not used [-Wunused-but-set-variable]
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Sascha Hauer <s.hauer@pengutronix.de>
> Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: NXP Linux Team <linux-imx@nxp.com>
> Cc: Haibo Chen <haibo.chen@freescale.com>
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
Applied, thank you.
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH 05/15] input: touchscreen: imx6ul_tsc: Remove set but unused variable 'value'
2020-11-13 7:37 ` Dmitry Torokhov
@ 2020-11-19 9:12 ` Lee Jones
0 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2020-11-19 9:12 UTC (permalink / raw)
To: Dmitry Torokhov
Cc: linux-kernel, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, NXP Linux Team, Haibo Chen, linux-input
On Thu, 12 Nov 2020, Dmitry Torokhov wrote:
> On Thu, Nov 12, 2020 at 11:01:54AM +0000, Lee Jones wrote:
> > Fixes the following W=1 kernel build warning(s):
> >
> > drivers/input/touchscreen/imx6ul_tsc.c: In function ‘adc_irq_fn’:
> > drivers/input/touchscreen/imx6ul_tsc.c:307:6: warning: variable ‘value’ set but not used [-Wunused-but-set-variable]
> >
> > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> > Cc: Shawn Guo <shawnguo@kernel.org>
> > Cc: Sascha Hauer <s.hauer@pengutronix.de>
> > Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
> > Cc: Fabio Estevam <festevam@gmail.com>
> > Cc: NXP Linux Team <linux-imx@nxp.com>
> > Cc: Haibo Chen <haibo.chen@freescale.com>
> > Cc: linux-input@vger.kernel.org
> > Signed-off-by: Lee Jones <lee.jones@linaro.org>
>
> Applied, thank you.
Good morning Dmitry,
Are you planning on finishing this review?
About half of the patches are unreviewed and there are a couple of
open questions on others.
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 42+ messages in thread
* [PATCH 06/15] input: touchscreen: melfas_mip4: Remove a bunch of unused variables
2020-11-12 11:01 [PATCH 00/15] [Set 2/2] Rid W=1 issues from Input Lee Jones
` (4 preceding siblings ...)
2020-11-12 11:01 ` [PATCH 05/15] input: touchscreen: imx6ul_tsc: Remove set but unused variable 'value' Lee Jones
@ 2020-11-12 11:01 ` Lee Jones
2020-11-13 7:42 ` Dmitry Torokhov
2020-11-12 11:01 ` [PATCH 07/15] input: touchscreen: usbtouchscreen: Remove unused variable 'ret' Lee Jones
` (8 subsequent siblings)
14 siblings, 1 reply; 42+ messages in thread
From: Lee Jones @ 2020-11-12 11:01 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, Sangwon Jee, Dmitry Torokhov, Henrik Rydberg, linux-input
Fixes the following W=1 kernel build warning(s):
drivers/input/touchscreen/melfas_mip4.c: In function ‘mip4_report_touch’:
drivers/input/touchscreen/melfas_mip4.c:474:5: warning: variable ‘size’ set but not used [-Wunused-but-set-variable]
drivers/input/touchscreen/melfas_mip4.c:472:5: warning: variable ‘pressure_stage’ set but not used [-Wunused-but-set-variable]
drivers/input/touchscreen/melfas_mip4.c:469:7: warning: variable ‘palm’ set but not used [-Wunused-but-set-variable]
drivers/input/touchscreen/melfas_mip4.c:468:7: warning: variable ‘hover’ set but not used [-Wunused-but-set-variable]
Cc: Sangwon Jee <jeesw@melfas.com>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Henrik Rydberg <rydberg@bitmath.org>
Cc: linux-input@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/input/touchscreen/melfas_mip4.c | 11 -----------
1 file changed, 11 deletions(-)
diff --git a/drivers/input/touchscreen/melfas_mip4.c b/drivers/input/touchscreen/melfas_mip4.c
index f67efdd040b24..9c98759098c7a 100644
--- a/drivers/input/touchscreen/melfas_mip4.c
+++ b/drivers/input/touchscreen/melfas_mip4.c
@@ -465,13 +465,9 @@ static void mip4_report_keys(struct mip4_ts *ts, u8 *packet)
static void mip4_report_touch(struct mip4_ts *ts, u8 *packet)
{
int id;
- bool hover;
- bool palm;
bool state;
u16 x, y;
- u8 pressure_stage = 0;
u8 pressure;
- u8 size;
u8 touch_major;
u8 touch_minor;
@@ -480,14 +476,11 @@ static void mip4_report_touch(struct mip4_ts *ts, u8 *packet)
case 1:
/* Touch only */
state = packet[0] & BIT(7);
- hover = packet[0] & BIT(5);
- palm = packet[0] & BIT(4);
id = (packet[0] & 0x0F) - 1;
x = ((packet[1] & 0x0F) << 8) | packet[2];
y = (((packet[1] >> 4) & 0x0F) << 8) |
packet[3];
pressure = packet[4];
- size = packet[5];
if (ts->event_format == 0) {
touch_major = packet[5];
touch_minor = packet[5];
@@ -501,14 +494,10 @@ static void mip4_report_touch(struct mip4_ts *ts, u8 *packet)
default:
/* Touch + Force(Pressure) */
id = (packet[0] & 0x0F) - 1;
- hover = packet[1] & BIT(2);
- palm = packet[1] & BIT(1);
state = packet[1] & BIT(0);
x = ((packet[2] & 0x0F) << 8) | packet[3];
y = (((packet[2] >> 4) & 0x0F) << 8) |
packet[4];
- size = packet[6];
- pressure_stage = (packet[7] & 0xF0) >> 4;
pressure = ((packet[7] & 0x0F) << 8) |
packet[8];
touch_major = packet[9];
--
2.25.1
^ permalink raw reply related [flat|nested] 42+ messages in thread
* Re: [PATCH 06/15] input: touchscreen: melfas_mip4: Remove a bunch of unused variables
2020-11-12 11:01 ` [PATCH 06/15] input: touchscreen: melfas_mip4: Remove a bunch of unused variables Lee Jones
@ 2020-11-13 7:42 ` Dmitry Torokhov
2020-11-13 7:55 ` Lee Jones
0 siblings, 1 reply; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-13 7:42 UTC (permalink / raw)
To: Lee Jones; +Cc: linux-kernel, Sangwon Jee, Henrik Rydberg, linux-input
On Thu, Nov 12, 2020 at 11:01:55AM +0000, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/touchscreen/melfas_mip4.c: In function ‘mip4_report_touch’:
> drivers/input/touchscreen/melfas_mip4.c:474:5: warning: variable ‘size’ set but not used [-Wunused-but-set-variable]
> drivers/input/touchscreen/melfas_mip4.c:472:5: warning: variable ‘pressure_stage’ set but not used [-Wunused-but-set-variable]
> drivers/input/touchscreen/melfas_mip4.c:469:7: warning: variable ‘palm’ set but not used [-Wunused-but-set-variable]
> drivers/input/touchscreen/melfas_mip4.c:468:7: warning: variable ‘hover’ set but not used [-Wunused-but-set-variable]
>
> Cc: Sangwon Jee <jeesw@melfas.com>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Henrik Rydberg <rydberg@bitmath.org>
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> ---
> drivers/input/touchscreen/melfas_mip4.c | 11 -----------
> 1 file changed, 11 deletions(-)
>
> diff --git a/drivers/input/touchscreen/melfas_mip4.c b/drivers/input/touchscreen/melfas_mip4.c
> index f67efdd040b24..9c98759098c7a 100644
> --- a/drivers/input/touchscreen/melfas_mip4.c
> +++ b/drivers/input/touchscreen/melfas_mip4.c
> @@ -465,13 +465,9 @@ static void mip4_report_keys(struct mip4_ts *ts, u8 *packet)
> static void mip4_report_touch(struct mip4_ts *ts, u8 *packet)
> {
> int id;
> - bool hover;
> - bool palm;
> bool state;
> u16 x, y;
> - u8 pressure_stage = 0;
> u8 pressure;
> - u8 size;
> u8 touch_major;
> u8 touch_minor;
>
> @@ -480,14 +476,11 @@ static void mip4_report_touch(struct mip4_ts *ts, u8 *packet)
> case 1:
> /* Touch only */
> state = packet[0] & BIT(7);
> - hover = packet[0] & BIT(5);
> - palm = packet[0] & BIT(4);
No really happy that we'd be losing information about protocol. Is there
a better way to suppress the warning while keeping this info?
> id = (packet[0] & 0x0F) - 1;
> x = ((packet[1] & 0x0F) << 8) | packet[2];
> y = (((packet[1] >> 4) & 0x0F) << 8) |
> packet[3];
> pressure = packet[4];
> - size = packet[5];
> if (ts->event_format == 0) {
> touch_major = packet[5];
> touch_minor = packet[5];
> @@ -501,14 +494,10 @@ static void mip4_report_touch(struct mip4_ts *ts, u8 *packet)
> default:
> /* Touch + Force(Pressure) */
> id = (packet[0] & 0x0F) - 1;
> - hover = packet[1] & BIT(2);
> - palm = packet[1] & BIT(1);
> state = packet[1] & BIT(0);
> x = ((packet[2] & 0x0F) << 8) | packet[3];
> y = (((packet[2] >> 4) & 0x0F) << 8) |
> packet[4];
> - size = packet[6];
> - pressure_stage = (packet[7] & 0xF0) >> 4;
> pressure = ((packet[7] & 0x0F) << 8) |
> packet[8];
> touch_major = packet[9];
> --
> 2.25.1
>
Thanks.
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH 06/15] input: touchscreen: melfas_mip4: Remove a bunch of unused variables
2020-11-13 7:42 ` Dmitry Torokhov
@ 2020-11-13 7:55 ` Lee Jones
2020-11-20 3:09 ` Dmitry Torokhov
0 siblings, 1 reply; 42+ messages in thread
From: Lee Jones @ 2020-11-13 7:55 UTC (permalink / raw)
To: Dmitry Torokhov; +Cc: linux-kernel, Sangwon Jee, Henrik Rydberg, linux-input
On Thu, 12 Nov 2020, Dmitry Torokhov wrote:
> On Thu, Nov 12, 2020 at 11:01:55AM +0000, Lee Jones wrote:
> > Fixes the following W=1 kernel build warning(s):
> >
> > drivers/input/touchscreen/melfas_mip4.c: In function ‘mip4_report_touch’:
> > drivers/input/touchscreen/melfas_mip4.c:474:5: warning: variable ‘size’ set but not used [-Wunused-but-set-variable]
> > drivers/input/touchscreen/melfas_mip4.c:472:5: warning: variable ‘pressure_stage’ set but not used [-Wunused-but-set-variable]
> > drivers/input/touchscreen/melfas_mip4.c:469:7: warning: variable ‘palm’ set but not used [-Wunused-but-set-variable]
> > drivers/input/touchscreen/melfas_mip4.c:468:7: warning: variable ‘hover’ set but not used [-Wunused-but-set-variable]
> >
> > Cc: Sangwon Jee <jeesw@melfas.com>
> > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> > Cc: Henrik Rydberg <rydberg@bitmath.org>
> > Cc: linux-input@vger.kernel.org
> > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> > ---
> > drivers/input/touchscreen/melfas_mip4.c | 11 -----------
> > 1 file changed, 11 deletions(-)
> >
> > diff --git a/drivers/input/touchscreen/melfas_mip4.c b/drivers/input/touchscreen/melfas_mip4.c
> > index f67efdd040b24..9c98759098c7a 100644
> > --- a/drivers/input/touchscreen/melfas_mip4.c
> > +++ b/drivers/input/touchscreen/melfas_mip4.c
> > @@ -465,13 +465,9 @@ static void mip4_report_keys(struct mip4_ts *ts, u8 *packet)
> > static void mip4_report_touch(struct mip4_ts *ts, u8 *packet)
> > {
> > int id;
> > - bool hover;
> > - bool palm;
> > bool state;
> > u16 x, y;
> > - u8 pressure_stage = 0;
> > u8 pressure;
> > - u8 size;
> > u8 touch_major;
> > u8 touch_minor;
> >
> > @@ -480,14 +476,11 @@ static void mip4_report_touch(struct mip4_ts *ts, u8 *packet)
> > case 1:
> > /* Touch only */
> > state = packet[0] & BIT(7);
> > - hover = packet[0] & BIT(5);
> > - palm = packet[0] & BIT(4);
>
> No really happy that we'd be losing information about protocol. Is there
> a better way to suppress the warning while keeping this info?
Yes. We can either convert the information to comments, or mark the
variables as __always_unused.
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH 06/15] input: touchscreen: melfas_mip4: Remove a bunch of unused variables
2020-11-13 7:55 ` Lee Jones
@ 2020-11-20 3:09 ` Dmitry Torokhov
2020-11-20 7:45 ` Lee Jones
0 siblings, 1 reply; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-20 3:09 UTC (permalink / raw)
To: Lee Jones; +Cc: linux-kernel, Sangwon Jee, Henrik Rydberg, linux-input
On Fri, Nov 13, 2020 at 07:55:25AM +0000, Lee Jones wrote:
> On Thu, 12 Nov 2020, Dmitry Torokhov wrote:
>
> > On Thu, Nov 12, 2020 at 11:01:55AM +0000, Lee Jones wrote:
> > > Fixes the following W=1 kernel build warning(s):
> > >
> > > drivers/input/touchscreen/melfas_mip4.c: In function ‘mip4_report_touch’:
> > > drivers/input/touchscreen/melfas_mip4.c:474:5: warning: variable ‘size’ set but not used [-Wunused-but-set-variable]
> > > drivers/input/touchscreen/melfas_mip4.c:472:5: warning: variable ‘pressure_stage’ set but not used [-Wunused-but-set-variable]
> > > drivers/input/touchscreen/melfas_mip4.c:469:7: warning: variable ‘palm’ set but not used [-Wunused-but-set-variable]
> > > drivers/input/touchscreen/melfas_mip4.c:468:7: warning: variable ‘hover’ set but not used [-Wunused-but-set-variable]
> > >
> > > Cc: Sangwon Jee <jeesw@melfas.com>
> > > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> > > Cc: Henrik Rydberg <rydberg@bitmath.org>
> > > Cc: linux-input@vger.kernel.org
> > > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> > > ---
> > > drivers/input/touchscreen/melfas_mip4.c | 11 -----------
> > > 1 file changed, 11 deletions(-)
> > >
> > > diff --git a/drivers/input/touchscreen/melfas_mip4.c b/drivers/input/touchscreen/melfas_mip4.c
> > > index f67efdd040b24..9c98759098c7a 100644
> > > --- a/drivers/input/touchscreen/melfas_mip4.c
> > > +++ b/drivers/input/touchscreen/melfas_mip4.c
> > > @@ -465,13 +465,9 @@ static void mip4_report_keys(struct mip4_ts *ts, u8 *packet)
> > > static void mip4_report_touch(struct mip4_ts *ts, u8 *packet)
> > > {
> > > int id;
> > > - bool hover;
> > > - bool palm;
> > > bool state;
> > > u16 x, y;
> > > - u8 pressure_stage = 0;
> > > u8 pressure;
> > > - u8 size;
> > > u8 touch_major;
> > > u8 touch_minor;
> > >
> > > @@ -480,14 +476,11 @@ static void mip4_report_touch(struct mip4_ts *ts, u8 *packet)
> > > case 1:
> > > /* Touch only */
> > > state = packet[0] & BIT(7);
> > > - hover = packet[0] & BIT(5);
> > > - palm = packet[0] & BIT(4);
> >
> > No really happy that we'd be losing information about protocol. Is there
> > a better way to suppress the warning while keeping this info?
>
> Yes. We can either convert the information to comments, or mark the
> variables as __always_unused.
If __always unused suppresses this warning that would be my preference.
Thanks!
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH 06/15] input: touchscreen: melfas_mip4: Remove a bunch of unused variables
2020-11-20 3:09 ` Dmitry Torokhov
@ 2020-11-20 7:45 ` Lee Jones
0 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2020-11-20 7:45 UTC (permalink / raw)
To: Dmitry Torokhov; +Cc: linux-kernel, Sangwon Jee, Henrik Rydberg, linux-input
On Thu, 19 Nov 2020, Dmitry Torokhov wrote:
> On Fri, Nov 13, 2020 at 07:55:25AM +0000, Lee Jones wrote:
> > On Thu, 12 Nov 2020, Dmitry Torokhov wrote:
> >
> > > On Thu, Nov 12, 2020 at 11:01:55AM +0000, Lee Jones wrote:
> > > > Fixes the following W=1 kernel build warning(s):
> > > >
> > > > drivers/input/touchscreen/melfas_mip4.c: In function ‘mip4_report_touch’:
> > > > drivers/input/touchscreen/melfas_mip4.c:474:5: warning: variable ‘size’ set but not used [-Wunused-but-set-variable]
> > > > drivers/input/touchscreen/melfas_mip4.c:472:5: warning: variable ‘pressure_stage’ set but not used [-Wunused-but-set-variable]
> > > > drivers/input/touchscreen/melfas_mip4.c:469:7: warning: variable ‘palm’ set but not used [-Wunused-but-set-variable]
> > > > drivers/input/touchscreen/melfas_mip4.c:468:7: warning: variable ‘hover’ set but not used [-Wunused-but-set-variable]
> > > >
> > > > Cc: Sangwon Jee <jeesw@melfas.com>
> > > > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> > > > Cc: Henrik Rydberg <rydberg@bitmath.org>
> > > > Cc: linux-input@vger.kernel.org
> > > > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> > > > ---
> > > > drivers/input/touchscreen/melfas_mip4.c | 11 -----------
> > > > 1 file changed, 11 deletions(-)
> > > >
> > > > diff --git a/drivers/input/touchscreen/melfas_mip4.c b/drivers/input/touchscreen/melfas_mip4.c
> > > > index f67efdd040b24..9c98759098c7a 100644
> > > > --- a/drivers/input/touchscreen/melfas_mip4.c
> > > > +++ b/drivers/input/touchscreen/melfas_mip4.c
> > > > @@ -465,13 +465,9 @@ static void mip4_report_keys(struct mip4_ts *ts, u8 *packet)
> > > > static void mip4_report_touch(struct mip4_ts *ts, u8 *packet)
> > > > {
> > > > int id;
> > > > - bool hover;
> > > > - bool palm;
> > > > bool state;
> > > > u16 x, y;
> > > > - u8 pressure_stage = 0;
> > > > u8 pressure;
> > > > - u8 size;
> > > > u8 touch_major;
> > > > u8 touch_minor;
> > > >
> > > > @@ -480,14 +476,11 @@ static void mip4_report_touch(struct mip4_ts *ts, u8 *packet)
> > > > case 1:
> > > > /* Touch only */
> > > > state = packet[0] & BIT(7);
> > > > - hover = packet[0] & BIT(5);
> > > > - palm = packet[0] & BIT(4);
> > >
> > > No really happy that we'd be losing information about protocol. Is there
> > > a better way to suppress the warning while keeping this info?
> >
> > Yes. We can either convert the information to comments, or mark the
> > variables as __always_unused.
>
> If __always unused suppresses this warning that would be my preference.
No problem. Will fix.
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 42+ messages in thread
* [PATCH 07/15] input: touchscreen: usbtouchscreen: Remove unused variable 'ret'
2020-11-12 11:01 [PATCH 00/15] [Set 2/2] Rid W=1 issues from Input Lee Jones
` (5 preceding siblings ...)
2020-11-12 11:01 ` [PATCH 06/15] input: touchscreen: melfas_mip4: Remove a bunch of unused variables Lee Jones
@ 2020-11-12 11:01 ` Lee Jones
2020-11-13 7:36 ` Dmitry Torokhov
2020-11-12 11:01 ` [PATCH 08/15] input: touchscreen: surface3_spi: Remove set but unused variable 'timestamp' Lee Jones
` (7 subsequent siblings)
14 siblings, 1 reply; 42+ messages in thread
From: Lee Jones @ 2020-11-12 11:01 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, Dmitry Torokhov, Henrik Rydberg, James Hilliard,
Daniel Ritz, linux-input
Fixes the following W=1 kernel build warning(s):
drivers/input/touchscreen/usbtouchscreen.c: In function ‘nexio_read_data’:
drivers/input/touchscreen/usbtouchscreen.c:1052:50: warning: variable ‘ret’ set but not used [-Wunused-but-set-variable]
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Henrik Rydberg <rydberg@bitmath.org>
Cc: James Hilliard <james.hilliard1@gmail.com>
Cc: Daniel Ritz <daniel.ritz@gmx.ch>
Cc: linux-input@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/input/touchscreen/usbtouchscreen.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/input/touchscreen/usbtouchscreen.c b/drivers/input/touchscreen/usbtouchscreen.c
index 397cb1d3f481b..c3b7130cd9033 100644
--- a/drivers/input/touchscreen/usbtouchscreen.c
+++ b/drivers/input/touchscreen/usbtouchscreen.c
@@ -1049,7 +1049,7 @@ static int nexio_read_data(struct usbtouch_usb *usbtouch, unsigned char *pkt)
unsigned int data_len = be16_to_cpu(packet->data_len);
unsigned int x_len = be16_to_cpu(packet->x_len);
unsigned int y_len = be16_to_cpu(packet->y_len);
- int x, y, begin_x, begin_y, end_x, end_y, w, h, ret;
+ int x, y, begin_x, begin_y, end_x, end_y, w, h;
/* got touch data? */
if ((pkt[0] & 0xe0) != 0xe0)
@@ -1061,7 +1061,7 @@ static int nexio_read_data(struct usbtouch_usb *usbtouch, unsigned char *pkt)
x_len -= 0x80;
/* send ACK */
- ret = usb_submit_urb(priv->ack, GFP_ATOMIC);
+ usb_submit_urb(priv->ack, GFP_ATOMIC);
if (!usbtouch->type->max_xc) {
usbtouch->type->max_xc = 2 * x_len;
--
2.25.1
^ permalink raw reply related [flat|nested] 42+ messages in thread
* Re: [PATCH 07/15] input: touchscreen: usbtouchscreen: Remove unused variable 'ret'
2020-11-12 11:01 ` [PATCH 07/15] input: touchscreen: usbtouchscreen: Remove unused variable 'ret' Lee Jones
@ 2020-11-13 7:36 ` Dmitry Torokhov
2020-11-13 7:56 ` Lee Jones
0 siblings, 1 reply; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-13 7:36 UTC (permalink / raw)
To: Lee Jones
Cc: linux-kernel, Henrik Rydberg, James Hilliard, Daniel Ritz, linux-input
On Thu, Nov 12, 2020 at 11:01:56AM +0000, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/touchscreen/usbtouchscreen.c: In function ‘nexio_read_data’:
> drivers/input/touchscreen/usbtouchscreen.c:1052:50: warning: variable ‘ret’ set but not used [-Wunused-but-set-variable]
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Henrik Rydberg <rydberg@bitmath.org>
> Cc: James Hilliard <james.hilliard1@gmail.com>
> Cc: Daniel Ritz <daniel.ritz@gmx.ch>
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> ---
> drivers/input/touchscreen/usbtouchscreen.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/input/touchscreen/usbtouchscreen.c b/drivers/input/touchscreen/usbtouchscreen.c
> index 397cb1d3f481b..c3b7130cd9033 100644
> --- a/drivers/input/touchscreen/usbtouchscreen.c
> +++ b/drivers/input/touchscreen/usbtouchscreen.c
> @@ -1049,7 +1049,7 @@ static int nexio_read_data(struct usbtouch_usb *usbtouch, unsigned char *pkt)
> unsigned int data_len = be16_to_cpu(packet->data_len);
> unsigned int x_len = be16_to_cpu(packet->x_len);
> unsigned int y_len = be16_to_cpu(packet->y_len);
> - int x, y, begin_x, begin_y, end_x, end_y, w, h, ret;
> + int x, y, begin_x, begin_y, end_x, end_y, w, h;
>
> /* got touch data? */
> if ((pkt[0] & 0xe0) != 0xe0)
> @@ -1061,7 +1061,7 @@ static int nexio_read_data(struct usbtouch_usb *usbtouch, unsigned char *pkt)
> x_len -= 0x80;
>
> /* send ACK */
> - ret = usb_submit_urb(priv->ack, GFP_ATOMIC);
> + usb_submit_urb(priv->ack, GFP_ATOMIC);
I wonder if we should handle potential errors instead.
>
> if (!usbtouch->type->max_xc) {
> usbtouch->type->max_xc = 2 * x_len;
> --
> 2.25.1
>
Thanks.
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH 07/15] input: touchscreen: usbtouchscreen: Remove unused variable 'ret'
2020-11-13 7:36 ` Dmitry Torokhov
@ 2020-11-13 7:56 ` Lee Jones
2020-11-20 3:09 ` Dmitry Torokhov
0 siblings, 1 reply; 42+ messages in thread
From: Lee Jones @ 2020-11-13 7:56 UTC (permalink / raw)
To: Dmitry Torokhov
Cc: linux-kernel, Henrik Rydberg, James Hilliard, Daniel Ritz, linux-input
On Thu, 12 Nov 2020, Dmitry Torokhov wrote:
> On Thu, Nov 12, 2020 at 11:01:56AM +0000, Lee Jones wrote:
> > Fixes the following W=1 kernel build warning(s):
> >
> > drivers/input/touchscreen/usbtouchscreen.c: In function ‘nexio_read_data’:
> > drivers/input/touchscreen/usbtouchscreen.c:1052:50: warning: variable ‘ret’ set but not used [-Wunused-but-set-variable]
> >
> > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> > Cc: Henrik Rydberg <rydberg@bitmath.org>
> > Cc: James Hilliard <james.hilliard1@gmail.com>
> > Cc: Daniel Ritz <daniel.ritz@gmx.ch>
> > Cc: linux-input@vger.kernel.org
> > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> > ---
> > drivers/input/touchscreen/usbtouchscreen.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/input/touchscreen/usbtouchscreen.c b/drivers/input/touchscreen/usbtouchscreen.c
> > index 397cb1d3f481b..c3b7130cd9033 100644
> > --- a/drivers/input/touchscreen/usbtouchscreen.c
> > +++ b/drivers/input/touchscreen/usbtouchscreen.c
> > @@ -1049,7 +1049,7 @@ static int nexio_read_data(struct usbtouch_usb *usbtouch, unsigned char *pkt)
> > unsigned int data_len = be16_to_cpu(packet->data_len);
> > unsigned int x_len = be16_to_cpu(packet->x_len);
> > unsigned int y_len = be16_to_cpu(packet->y_len);
> > - int x, y, begin_x, begin_y, end_x, end_y, w, h, ret;
> > + int x, y, begin_x, begin_y, end_x, end_y, w, h;
> >
> > /* got touch data? */
> > if ((pkt[0] & 0xe0) != 0xe0)
> > @@ -1061,7 +1061,7 @@ static int nexio_read_data(struct usbtouch_usb *usbtouch, unsigned char *pkt)
> > x_len -= 0x80;
> >
> > /* send ACK */
> > - ret = usb_submit_urb(priv->ack, GFP_ATOMIC);
> > + usb_submit_urb(priv->ack, GFP_ATOMIC);
>
> I wonder if we should handle potential errors instead.
Your call. I'll do whatever you decide.
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH 07/15] input: touchscreen: usbtouchscreen: Remove unused variable 'ret'
2020-11-13 7:56 ` Lee Jones
@ 2020-11-20 3:09 ` Dmitry Torokhov
0 siblings, 0 replies; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-20 3:09 UTC (permalink / raw)
To: Lee Jones
Cc: linux-kernel, Henrik Rydberg, James Hilliard, Daniel Ritz, linux-input
On Fri, Nov 13, 2020 at 07:56:31AM +0000, Lee Jones wrote:
> On Thu, 12 Nov 2020, Dmitry Torokhov wrote:
>
> > On Thu, Nov 12, 2020 at 11:01:56AM +0000, Lee Jones wrote:
> > > Fixes the following W=1 kernel build warning(s):
> > >
> > > drivers/input/touchscreen/usbtouchscreen.c: In function ‘nexio_read_data’:
> > > drivers/input/touchscreen/usbtouchscreen.c:1052:50: warning: variable ‘ret’ set but not used [-Wunused-but-set-variable]
> > >
> > > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> > > Cc: Henrik Rydberg <rydberg@bitmath.org>
> > > Cc: James Hilliard <james.hilliard1@gmail.com>
> > > Cc: Daniel Ritz <daniel.ritz@gmx.ch>
> > > Cc: linux-input@vger.kernel.org
> > > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> > > ---
> > > drivers/input/touchscreen/usbtouchscreen.c | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/input/touchscreen/usbtouchscreen.c b/drivers/input/touchscreen/usbtouchscreen.c
> > > index 397cb1d3f481b..c3b7130cd9033 100644
> > > --- a/drivers/input/touchscreen/usbtouchscreen.c
> > > +++ b/drivers/input/touchscreen/usbtouchscreen.c
> > > @@ -1049,7 +1049,7 @@ static int nexio_read_data(struct usbtouch_usb *usbtouch, unsigned char *pkt)
> > > unsigned int data_len = be16_to_cpu(packet->data_len);
> > > unsigned int x_len = be16_to_cpu(packet->x_len);
> > > unsigned int y_len = be16_to_cpu(packet->y_len);
> > > - int x, y, begin_x, begin_y, end_x, end_y, w, h, ret;
> > > + int x, y, begin_x, begin_y, end_x, end_y, w, h;
> > >
> > > /* got touch data? */
> > > if ((pkt[0] & 0xe0) != 0xe0)
> > > @@ -1061,7 +1061,7 @@ static int nexio_read_data(struct usbtouch_usb *usbtouch, unsigned char *pkt)
> > > x_len -= 0x80;
> > >
> > > /* send ACK */
> > > - ret = usb_submit_urb(priv->ack, GFP_ATOMIC);
> > > + usb_submit_urb(priv->ack, GFP_ATOMIC);
> >
> > I wonder if we should handle potential errors instead.
>
> Your call. I'll do whatever you decide.
OK, please add error handling here.
Thank you.
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* [PATCH 08/15] input: touchscreen: surface3_spi: Remove set but unused variable 'timestamp'
2020-11-12 11:01 [PATCH 00/15] [Set 2/2] Rid W=1 issues from Input Lee Jones
` (6 preceding siblings ...)
2020-11-12 11:01 ` [PATCH 07/15] input: touchscreen: usbtouchscreen: Remove unused variable 'ret' Lee Jones
@ 2020-11-12 11:01 ` Lee Jones
2020-11-13 7:40 ` Dmitry Torokhov
2020-11-12 11:01 ` [PATCH 09/15] input: misc: wm831x-on: Source file headers are not good candidates for kernel-doc Lee Jones
` (6 subsequent siblings)
14 siblings, 1 reply; 42+ messages in thread
From: Lee Jones @ 2020-11-12 11:01 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, Dmitry Torokhov, Henrik Rydberg,
Benjamin Tissoires, linux-input
Fixes the following W=1 kernel build warning(s):
drivers/input/touchscreen/surface3_spi.c: In function ‘surface3_spi_process_touch’:
drivers/input/touchscreen/surface3_spi.c:97:6: warning: variable ‘timestamp’ set but not used [-Wunused-but-set-variable]
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Henrik Rydberg <rydberg@bitmath.org>
Cc: Benjamin Tissoires <benjamin.tissoires@gmail.com>
Cc: linux-input@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/input/touchscreen/surface3_spi.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/input/touchscreen/surface3_spi.c b/drivers/input/touchscreen/surface3_spi.c
index ce4828b1415a8..72dc4c562a4e1 100644
--- a/drivers/input/touchscreen/surface3_spi.c
+++ b/drivers/input/touchscreen/surface3_spi.c
@@ -94,9 +94,7 @@ static void surface3_spi_report_touch(struct surface3_ts_data *ts_data,
static void surface3_spi_process_touch(struct surface3_ts_data *ts_data, u8 *data)
{
- u16 timestamp;
unsigned int i;
- timestamp = get_unaligned_le16(&data[15]);
for (i = 0; i < 13; i++) {
struct surface3_ts_data_finger *finger;
--
2.25.1
^ permalink raw reply related [flat|nested] 42+ messages in thread
* Re: [PATCH 08/15] input: touchscreen: surface3_spi: Remove set but unused variable 'timestamp'
2020-11-12 11:01 ` [PATCH 08/15] input: touchscreen: surface3_spi: Remove set but unused variable 'timestamp' Lee Jones
@ 2020-11-13 7:40 ` Dmitry Torokhov
2020-11-13 7:42 ` Benjamin Tissoires
0 siblings, 1 reply; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-13 7:40 UTC (permalink / raw)
To: Lee Jones, Benjamin Tissoires; +Cc: linux-kernel, Henrik Rydberg, linux-input
On Thu, Nov 12, 2020 at 11:01:57AM +0000, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/touchscreen/surface3_spi.c: In function ‘surface3_spi_process_touch’:
> drivers/input/touchscreen/surface3_spi.c:97:6: warning: variable ‘timestamp’ set but not used [-Wunused-but-set-variable]
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Henrik Rydberg <rydberg@bitmath.org>
> Cc: Benjamin Tissoires <benjamin.tissoires@gmail.com>
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> ---
> drivers/input/touchscreen/surface3_spi.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/input/touchscreen/surface3_spi.c b/drivers/input/touchscreen/surface3_spi.c
> index ce4828b1415a8..72dc4c562a4e1 100644
> --- a/drivers/input/touchscreen/surface3_spi.c
> +++ b/drivers/input/touchscreen/surface3_spi.c
> @@ -94,9 +94,7 @@ static void surface3_spi_report_touch(struct surface3_ts_data *ts_data,
>
> static void surface3_spi_process_touch(struct surface3_ts_data *ts_data, u8 *data)
> {
> - u16 timestamp;
> unsigned int i;
> - timestamp = get_unaligned_le16(&data[15]);
Benjamin, should we pass timing data on to userspace instead?
>
> for (i = 0; i < 13; i++) {
> struct surface3_ts_data_finger *finger;
> --
> 2.25.1
>
Thanks.
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH 08/15] input: touchscreen: surface3_spi: Remove set but unused variable 'timestamp'
2020-11-13 7:40 ` Dmitry Torokhov
@ 2020-11-13 7:42 ` Benjamin Tissoires
2020-11-15 22:02 ` Peter Hutterer
0 siblings, 1 reply; 42+ messages in thread
From: Benjamin Tissoires @ 2020-11-13 7:42 UTC (permalink / raw)
To: Dmitry Torokhov, Peter Hutterer
Cc: Lee Jones, lkml, Henrik Rydberg, linux-input
On Fri, Nov 13, 2020 at 8:40 AM Dmitry Torokhov
<dmitry.torokhov@gmail.com> wrote:
>
> On Thu, Nov 12, 2020 at 11:01:57AM +0000, Lee Jones wrote:
> > Fixes the following W=1 kernel build warning(s):
> >
> > drivers/input/touchscreen/surface3_spi.c: In function ‘surface3_spi_process_touch’:
> > drivers/input/touchscreen/surface3_spi.c:97:6: warning: variable ‘timestamp’ set but not used [-Wunused-but-set-variable]
> >
> > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> > Cc: Henrik Rydberg <rydberg@bitmath.org>
> > Cc: Benjamin Tissoires <benjamin.tissoires@gmail.com>
> > Cc: linux-input@vger.kernel.org
> > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> > ---
> > drivers/input/touchscreen/surface3_spi.c | 2 --
> > 1 file changed, 2 deletions(-)
> >
> > diff --git a/drivers/input/touchscreen/surface3_spi.c b/drivers/input/touchscreen/surface3_spi.c
> > index ce4828b1415a8..72dc4c562a4e1 100644
> > --- a/drivers/input/touchscreen/surface3_spi.c
> > +++ b/drivers/input/touchscreen/surface3_spi.c
> > @@ -94,9 +94,7 @@ static void surface3_spi_report_touch(struct surface3_ts_data *ts_data,
> >
> > static void surface3_spi_process_touch(struct surface3_ts_data *ts_data, u8 *data)
> > {
> > - u16 timestamp;
> > unsigned int i;
> > - timestamp = get_unaligned_le16(&data[15]);
>
> Benjamin, should we pass timing data on to userspace instead?
Last time I checked, libinput was not using the HW timestamp. So I
don't mind dropping it.
Not sure if chrome/android uses it.
Cheers,
Benjamin
>
> >
> > for (i = 0; i < 13; i++) {
> > struct surface3_ts_data_finger *finger;
> > --
> > 2.25.1
> >
>
> Thanks.
>
> --
> Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH 08/15] input: touchscreen: surface3_spi: Remove set but unused variable 'timestamp'
2020-11-13 7:42 ` Benjamin Tissoires
@ 2020-11-15 22:02 ` Peter Hutterer
0 siblings, 0 replies; 42+ messages in thread
From: Peter Hutterer @ 2020-11-15 22:02 UTC (permalink / raw)
To: Benjamin Tissoires
Cc: Dmitry Torokhov, Lee Jones, lkml, Henrik Rydberg, linux-input
On Fri, Nov 13, 2020 at 08:42:37AM +0100, Benjamin Tissoires wrote:
> On Fri, Nov 13, 2020 at 8:40 AM Dmitry Torokhov
> <dmitry.torokhov@gmail.com> wrote:
> >
> > On Thu, Nov 12, 2020 at 11:01:57AM +0000, Lee Jones wrote:
> > > Fixes the following W=1 kernel build warning(s):
> > >
> > > drivers/input/touchscreen/surface3_spi.c: In function ‘surface3_spi_process_touch’:
> > > drivers/input/touchscreen/surface3_spi.c:97:6: warning: variable ‘timestamp’ set but not used [-Wunused-but-set-variable]
> > >
> > > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> > > Cc: Henrik Rydberg <rydberg@bitmath.org>
> > > Cc: Benjamin Tissoires <benjamin.tissoires@gmail.com>
> > > Cc: linux-input@vger.kernel.org
> > > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> > > ---
> > > drivers/input/touchscreen/surface3_spi.c | 2 --
> > > 1 file changed, 2 deletions(-)
> > >
> > > diff --git a/drivers/input/touchscreen/surface3_spi.c b/drivers/input/touchscreen/surface3_spi.c
> > > index ce4828b1415a8..72dc4c562a4e1 100644
> > > --- a/drivers/input/touchscreen/surface3_spi.c
> > > +++ b/drivers/input/touchscreen/surface3_spi.c
> > > @@ -94,9 +94,7 @@ static void surface3_spi_report_touch(struct surface3_ts_data *ts_data,
> > >
> > > static void surface3_spi_process_touch(struct surface3_ts_data *ts_data, u8 *data)
> > > {
> > > - u16 timestamp;
> > > unsigned int i;
> > > - timestamp = get_unaligned_le16(&data[15]);
> >
> > Benjamin, should we pass timing data on to userspace instead?
>
> Last time I checked, libinput was not using the HW timestamp. So I
> don't mind dropping it.
I'm assuming this would be passed on as MSC_TIMESTAMP?
I never found the time implementing this and I mask MSC_TIMESTAMP on most
devices anyway (except for Dell's i2c touchpads where I need it to work
around a fw bug). so at least from libinput's POV it wouldn't have any
effect either way.
Cheers,
Peter
>
> Not sure if chrome/android uses it.
>
> Cheers,
> Benjamin
>
> >
> > >
> > > for (i = 0; i < 13; i++) {
> > > struct surface3_ts_data_finger *finger;
> > > --
> > > 2.25.1
> > >
> >
> > Thanks.
> >
> > --
> > Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* [PATCH 09/15] input: misc: wm831x-on: Source file headers are not good candidates for kernel-doc
2020-11-12 11:01 [PATCH 00/15] [Set 2/2] Rid W=1 issues from Input Lee Jones
` (7 preceding siblings ...)
2020-11-12 11:01 ` [PATCH 08/15] input: touchscreen: surface3_spi: Remove set but unused variable 'timestamp' Lee Jones
@ 2020-11-12 11:01 ` Lee Jones
2020-11-12 11:20 ` Richard Fitzgerald
2020-11-20 3:17 ` Dmitry Torokhov
2020-11-12 11:01 ` [PATCH 10/15] input: touchscreen: surface3_spi: Fix naming issue with 'surface3_spi_get_gpio_config's header Lee Jones
` (5 subsequent siblings)
14 siblings, 2 replies; 42+ messages in thread
From: Lee Jones @ 2020-11-12 11:01 UTC (permalink / raw)
To: lee.jones; +Cc: linux-kernel, Dmitry Torokhov, Mark Brown, patches, linux-input
Fixes the following W=1 kernel build warning(s):
drivers/input/misc/wm831x-on.c:30: warning: cannot understand function prototype: 'struct wm831x_on '
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: patches@opensource.cirrus.com
Cc: linux-input@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/input/misc/wm831x-on.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/input/misc/wm831x-on.c b/drivers/input/misc/wm831x-on.c
index 1b44de265a0ad..a42fe041b73c6 100644
--- a/drivers/input/misc/wm831x-on.c
+++ b/drivers/input/misc/wm831x-on.c
@@ -1,4 +1,4 @@
-/**
+/*
* wm831x-on.c - WM831X ON pin driver
*
* Copyright (C) 2009 Wolfson Microelectronics plc
--
2.25.1
^ permalink raw reply related [flat|nested] 42+ messages in thread
* Re: [PATCH 09/15] input: misc: wm831x-on: Source file headers are not good candidates for kernel-doc
2020-11-12 11:01 ` [PATCH 09/15] input: misc: wm831x-on: Source file headers are not good candidates for kernel-doc Lee Jones
@ 2020-11-12 11:20 ` Richard Fitzgerald
2020-11-20 3:17 ` Dmitry Torokhov
1 sibling, 0 replies; 42+ messages in thread
From: Richard Fitzgerald @ 2020-11-12 11:20 UTC (permalink / raw)
To: Lee Jones; +Cc: linux-kernel, Dmitry Torokhov, Mark Brown, patches, linux-input
On 12/11/2020 11:01, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/misc/wm831x-on.c:30: warning: cannot understand function prototype: 'struct wm831x_on '
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Mark Brown <broonie@opensource.wolfsonmicro.com>
> Cc: patches@opensource.cirrus.com
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> ---
> drivers/input/misc/wm831x-on.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/input/misc/wm831x-on.c b/drivers/input/misc/wm831x-on.c
> index 1b44de265a0ad..a42fe041b73c6 100644
> --- a/drivers/input/misc/wm831x-on.c
> +++ b/drivers/input/misc/wm831x-on.c
> @@ -1,4 +1,4 @@
> -/**
> +/*
> * wm831x-on.c - WM831X ON pin driver
> *
> * Copyright (C) 2009 Wolfson Microelectronics plc
>
Acked-by: Richard Fitzgerald <rf@opensource.cirrus.com>
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH 09/15] input: misc: wm831x-on: Source file headers are not good candidates for kernel-doc
2020-11-12 11:01 ` [PATCH 09/15] input: misc: wm831x-on: Source file headers are not good candidates for kernel-doc Lee Jones
2020-11-12 11:20 ` Richard Fitzgerald
@ 2020-11-20 3:17 ` Dmitry Torokhov
1 sibling, 0 replies; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-20 3:17 UTC (permalink / raw)
To: Lee Jones; +Cc: linux-kernel, Mark Brown, patches, linux-input
On Thu, Nov 12, 2020 at 11:01:58AM +0000, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/misc/wm831x-on.c:30: warning: cannot understand function prototype: 'struct wm831x_on '
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Mark Brown <broonie@opensource.wolfsonmicro.com>
> Cc: patches@opensource.cirrus.com
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
Applied, thank you.
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* [PATCH 10/15] input: touchscreen: surface3_spi: Fix naming issue with 'surface3_spi_get_gpio_config's header
2020-11-12 11:01 [PATCH 00/15] [Set 2/2] Rid W=1 issues from Input Lee Jones
` (8 preceding siblings ...)
2020-11-12 11:01 ` [PATCH 09/15] input: misc: wm831x-on: Source file headers are not good candidates for kernel-doc Lee Jones
@ 2020-11-12 11:01 ` Lee Jones
2020-11-20 3:16 ` Dmitry Torokhov
2020-11-12 11:02 ` [PATCH 11/15] input: touchscreen: wm97xx-core: Provide missing description for 'status' Lee Jones
` (4 subsequent siblings)
14 siblings, 1 reply; 42+ messages in thread
From: Lee Jones @ 2020-11-12 11:01 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, Dmitry Torokhov, Henrik Rydberg,
Benjamin Tissoires, linux-input
Fixes the following W=1 kernel build warning(s):
drivers/input/touchscreen/surface3_spi.c: In function ‘surface3_spi_process_touch’:
drivers/input/touchscreen/surface3_spi.c:97:6: warning: variable ‘timestamp’ set but not used [-Wunused-but-set-variable]
drivers/input/touchscreen/surface3_spi.c:225: warning: Function parameter or member 'data' not described in 'surface3_spi_get_gpio_config'
drivers/input/touchscreen/surface3_spi.c:225: warning: Excess function parameter 'ts' description in 'surface3_spi_get_gpio_config'
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Henrik Rydberg <rydberg@bitmath.org>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Benjamin Tissoires <benjamin.tissoires@gmail.com>
Cc: linux-input@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/input/touchscreen/surface3_spi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/input/touchscreen/surface3_spi.c b/drivers/input/touchscreen/surface3_spi.c
index 72dc4c562a4e1..1da23e5585a0d 100644
--- a/drivers/input/touchscreen/surface3_spi.c
+++ b/drivers/input/touchscreen/surface3_spi.c
@@ -217,7 +217,7 @@ static void surface3_spi_power(struct surface3_ts_data *data, bool on)
/**
* surface3_spi_get_gpio_config - Get GPIO config from ACPI/DT
*
- * @ts: surface3_spi_ts_data pointer
+ * @data: surface3_spi_ts_data pointer
*/
static int surface3_spi_get_gpio_config(struct surface3_ts_data *data)
{
--
2.25.1
^ permalink raw reply related [flat|nested] 42+ messages in thread
* Re: [PATCH 10/15] input: touchscreen: surface3_spi: Fix naming issue with 'surface3_spi_get_gpio_config's header
2020-11-12 11:01 ` [PATCH 10/15] input: touchscreen: surface3_spi: Fix naming issue with 'surface3_spi_get_gpio_config's header Lee Jones
@ 2020-11-20 3:16 ` Dmitry Torokhov
0 siblings, 0 replies; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-20 3:16 UTC (permalink / raw)
To: Lee Jones; +Cc: linux-kernel, Henrik Rydberg, Benjamin Tissoires, linux-input
On Thu, Nov 12, 2020 at 11:01:59AM +0000, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/touchscreen/surface3_spi.c: In function ‘surface3_spi_process_touch’:
> drivers/input/touchscreen/surface3_spi.c:97:6: warning: variable ‘timestamp’ set but not used [-Wunused-but-set-variable]
> drivers/input/touchscreen/surface3_spi.c:225: warning: Function parameter or member 'data' not described in 'surface3_spi_get_gpio_config'
> drivers/input/touchscreen/surface3_spi.c:225: warning: Excess function parameter 'ts' description in 'surface3_spi_get_gpio_config'
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Henrik Rydberg <rydberg@bitmath.org>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Benjamin Tissoires <benjamin.tissoires@gmail.com>
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
Applied, thank you.
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* [PATCH 11/15] input: touchscreen: wm97xx-core: Provide missing description for 'status'
2020-11-12 11:01 [PATCH 00/15] [Set 2/2] Rid W=1 issues from Input Lee Jones
` (9 preceding siblings ...)
2020-11-12 11:01 ` [PATCH 10/15] input: touchscreen: surface3_spi: Fix naming issue with 'surface3_spi_get_gpio_config's header Lee Jones
@ 2020-11-12 11:02 ` Lee Jones
2020-11-12 11:22 ` Richard Fitzgerald
2020-11-20 3:17 ` Dmitry Torokhov
2020-11-12 11:02 ` [PATCH 12/15] input: mouse: synaptics: Demote non-conformant kernel-doc header Lee Jones
` (3 subsequent siblings)
14 siblings, 2 replies; 42+ messages in thread
From: Lee Jones @ 2020-11-12 11:02 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, Dmitry Torokhov, Liam Girdwood, Ian Molton,
Andrew Zabolotny, Russell King, patches, linux-input
Fixes the following W=1 kernel build warning(s):
drivers/input/touchscreen/wm97xx-core.c:204: warning: Function parameter or member 'status' not described in 'wm97xx_set_gpio'
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Liam Girdwood <lrg@slimlogic.co.uk>
Cc: Ian Molton <spyro@f2s.com>
Cc: Andrew Zabolotny <zap@homelink.ru>
Cc: Russell King <rmk@arm.linux.org.uk>
Cc: patches@opensource.cirrus.com
Cc: linux-input@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/input/touchscreen/wm97xx-core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/input/touchscreen/wm97xx-core.c b/drivers/input/touchscreen/wm97xx-core.c
index 0a174bd829152..45ce89467c167 100644
--- a/drivers/input/touchscreen/wm97xx-core.c
+++ b/drivers/input/touchscreen/wm97xx-core.c
@@ -194,7 +194,7 @@ EXPORT_SYMBOL_GPL(wm97xx_get_gpio);
* wm97xx_set_gpio - Set the status of a codec GPIO.
* @wm: wm97xx device.
* @gpio: gpio
- *
+ * @status: status
*
* Set the status of a codec GPIO pin
*/
--
2.25.1
^ permalink raw reply related [flat|nested] 42+ messages in thread
* Re: [PATCH 11/15] input: touchscreen: wm97xx-core: Provide missing description for 'status'
2020-11-12 11:02 ` [PATCH 11/15] input: touchscreen: wm97xx-core: Provide missing description for 'status' Lee Jones
@ 2020-11-12 11:22 ` Richard Fitzgerald
2020-11-20 3:17 ` Dmitry Torokhov
1 sibling, 0 replies; 42+ messages in thread
From: Richard Fitzgerald @ 2020-11-12 11:22 UTC (permalink / raw)
To: Lee Jones
Cc: linux-kernel, Dmitry Torokhov, Liam Girdwood, Ian Molton,
Andrew Zabolotny, Russell King, patches, linux-input
On 12/11/2020 11:02, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/touchscreen/wm97xx-core.c:204: warning: Function parameter or member 'status' not described in 'wm97xx_set_gpio'
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Liam Girdwood <lrg@slimlogic.co.uk>
> Cc: Ian Molton <spyro@f2s.com>
> Cc: Andrew Zabolotny <zap@homelink.ru>
> Cc: Russell King <rmk@arm.linux.org.uk>
> Cc: patches@opensource.cirrus.com
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> ---
> drivers/input/touchscreen/wm97xx-core.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/input/touchscreen/wm97xx-core.c b/drivers/input/touchscreen/wm97xx-core.c
> index 0a174bd829152..45ce89467c167 100644
> --- a/drivers/input/touchscreen/wm97xx-core.c
> +++ b/drivers/input/touchscreen/wm97xx-core.c
> @@ -194,7 +194,7 @@ EXPORT_SYMBOL_GPL(wm97xx_get_gpio);
> * wm97xx_set_gpio - Set the status of a codec GPIO.
> * @wm: wm97xx device.
> * @gpio: gpio
> - *
> + * @status: status
> *
> * Set the status of a codec GPIO pin
> */
>
Acked-by: Richard Fitzgerald <rf@opensource.cirrus.com>
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH 11/15] input: touchscreen: wm97xx-core: Provide missing description for 'status'
2020-11-12 11:02 ` [PATCH 11/15] input: touchscreen: wm97xx-core: Provide missing description for 'status' Lee Jones
2020-11-12 11:22 ` Richard Fitzgerald
@ 2020-11-20 3:17 ` Dmitry Torokhov
1 sibling, 0 replies; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-20 3:17 UTC (permalink / raw)
To: Lee Jones
Cc: linux-kernel, Liam Girdwood, Ian Molton, Andrew Zabolotny,
Russell King, patches, linux-input
On Thu, Nov 12, 2020 at 11:02:00AM +0000, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/touchscreen/wm97xx-core.c:204: warning: Function parameter or member 'status' not described in 'wm97xx_set_gpio'
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Liam Girdwood <lrg@slimlogic.co.uk>
> Cc: Ian Molton <spyro@f2s.com>
> Cc: Andrew Zabolotny <zap@homelink.ru>
> Cc: Russell King <rmk@arm.linux.org.uk>
> Cc: patches@opensource.cirrus.com
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
Applied, thank you.
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* [PATCH 12/15] input: mouse: synaptics: Demote non-conformant kernel-doc header
2020-11-12 11:01 [PATCH 00/15] [Set 2/2] Rid W=1 issues from Input Lee Jones
` (10 preceding siblings ...)
2020-11-12 11:02 ` [PATCH 11/15] input: touchscreen: wm97xx-core: Provide missing description for 'status' Lee Jones
@ 2020-11-12 11:02 ` Lee Jones
2020-11-20 3:18 ` Dmitry Torokhov
2020-11-12 11:02 ` [PATCH 13/15] input: touchscreen: goodix: Fix misspelling of 'ctx' Lee Jones
` (2 subsequent siblings)
14 siblings, 1 reply; 42+ messages in thread
From: Lee Jones @ 2020-11-12 11:02 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, Dmitry Torokhov, Henrik Rydberg, Peter Osterlund,
Stefan Gmeiner, C. Scott Ananian, Bruce Kalk, this to
Fixes the following W=1 kernel build warning(s):
drivers/input/mouse/synaptics.c:1781: warning: Function parameter or member 'psmouse' not described in 'synaptics_setup_intertouch'
drivers/input/mouse/synaptics.c:1781: warning: Function parameter or member 'info' not described in 'synaptics_setup_intertouch'
drivers/input/mouse/synaptics.c:1781: warning: Function parameter or member 'leave_breadcrumbs' not described in 'synaptics_setup_intertouch'
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Henrik Rydberg <rydberg@bitmath.org>
Cc: Peter Osterlund <petero2@telia.com>
Cc: Stefan Gmeiner <riddlebox@freesurf.ch>
Cc: "C. Scott Ananian" <cananian@alumni.priceton.edu>
Cc: Bruce Kalk <kall@compass.com>
Cc: this to <linux-input@vger.kernel.org>
Cc: linux-input@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/input/mouse/synaptics.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/input/mouse/synaptics.c b/drivers/input/mouse/synaptics.c
index 07835939d83b9..403d0ce0e8bd2 100644
--- a/drivers/input/mouse/synaptics.c
+++ b/drivers/input/mouse/synaptics.c
@@ -1771,7 +1771,7 @@ static int synaptics_create_intertouch(struct psmouse *psmouse,
leave_breadcrumbs);
}
-/**
+/*
* synaptics_setup_intertouch - called once the PS/2 devices are enumerated
* and decides to instantiate a SMBus InterTouch device.
*/
--
2.25.1
^ permalink raw reply related [flat|nested] 42+ messages in thread
* Re: [PATCH 12/15] input: mouse: synaptics: Demote non-conformant kernel-doc header
2020-11-12 11:02 ` [PATCH 12/15] input: mouse: synaptics: Demote non-conformant kernel-doc header Lee Jones
@ 2020-11-20 3:18 ` Dmitry Torokhov
0 siblings, 0 replies; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-20 3:18 UTC (permalink / raw)
To: Lee Jones
Cc: linux-kernel, Henrik Rydberg, Peter Osterlund, Stefan Gmeiner,
C. Scott Ananian, Bruce Kalk, this to
On Thu, Nov 12, 2020 at 11:02:01AM +0000, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/mouse/synaptics.c:1781: warning: Function parameter or member 'psmouse' not described in 'synaptics_setup_intertouch'
> drivers/input/mouse/synaptics.c:1781: warning: Function parameter or member 'info' not described in 'synaptics_setup_intertouch'
> drivers/input/mouse/synaptics.c:1781: warning: Function parameter or member 'leave_breadcrumbs' not described in 'synaptics_setup_intertouch'
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Henrik Rydberg <rydberg@bitmath.org>
> Cc: Peter Osterlund <petero2@telia.com>
> Cc: Stefan Gmeiner <riddlebox@freesurf.ch>
> Cc: "C. Scott Ananian" <cananian@alumni.priceton.edu>
> Cc: Bruce Kalk <kall@compass.com>
> Cc: this to <linux-input@vger.kernel.org>
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
Applied, thank you.
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* [PATCH 13/15] input: touchscreen: goodix: Fix misspelling of 'ctx'
2020-11-12 11:01 [PATCH 00/15] [Set 2/2] Rid W=1 issues from Input Lee Jones
` (11 preceding siblings ...)
2020-11-12 11:02 ` [PATCH 12/15] input: mouse: synaptics: Demote non-conformant kernel-doc header Lee Jones
@ 2020-11-12 11:02 ` Lee Jones
2020-11-20 3:20 ` Dmitry Torokhov
2020-11-12 11:02 ` [PATCH 14/15] input: keyboard: applespi: Provide missing struct 'message' descriptions Lee Jones
2020-11-12 11:02 ` [PATCH 15/15] input: mouse: vmmouse: Demote obvious abuse of kernel-doc header Lee Jones
14 siblings, 1 reply; 42+ messages in thread
From: Lee Jones @ 2020-11-12 11:02 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, Bastien Nocera, Dmitry Torokhov, Henrik Rydberg,
K. Merker, authored by, Benjamin Tissoires, linux-input
Fixes the following W=1 kernel build warning(s):
drivers/input/touchscreen/goodix.c:1168: warning: Function parameter or member 'ctx' not described in 'goodix_config_cb'
drivers/input/touchscreen/goodix.c:1168: warning: Excess function parameter 'ts' description in 'goodix_config_cb'
Cc: Bastien Nocera <hadess@hadess.net>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Henrik Rydberg <rydberg@bitmath.org>
Cc: "K. Merker" <merker@debian.org>
Cc: authored by <andrew@goodix.com>
Cc: Benjamin Tissoires <benjamin.tissoires@gmail.com>
Cc: linux-input@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/input/touchscreen/goodix.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/input/touchscreen/goodix.c b/drivers/input/touchscreen/goodix.c
index 80a82a66a0f3b..c1b728e47827f 100644
--- a/drivers/input/touchscreen/goodix.c
+++ b/drivers/input/touchscreen/goodix.c
@@ -1159,7 +1159,7 @@ static int goodix_configure_dev(struct goodix_ts_data *ts)
* goodix_config_cb - Callback to finish device init
*
* @cfg: firmware config
- * @ts: our goodix_ts_data pointer
+ * @ctx: our goodix_ts_data pointer
*
* request_firmware_wait callback that finishes
* initialization of the device.
--
2.25.1
^ permalink raw reply related [flat|nested] 42+ messages in thread
* Re: [PATCH 13/15] input: touchscreen: goodix: Fix misspelling of 'ctx'
2020-11-12 11:02 ` [PATCH 13/15] input: touchscreen: goodix: Fix misspelling of 'ctx' Lee Jones
@ 2020-11-20 3:20 ` Dmitry Torokhov
0 siblings, 0 replies; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-20 3:20 UTC (permalink / raw)
To: Lee Jones
Cc: linux-kernel, Bastien Nocera, Henrik Rydberg, K. Merker,
authored by, Benjamin Tissoires, linux-input
On Thu, Nov 12, 2020 at 11:02:02AM +0000, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/touchscreen/goodix.c:1168: warning: Function parameter or member 'ctx' not described in 'goodix_config_cb'
> drivers/input/touchscreen/goodix.c:1168: warning: Excess function parameter 'ts' description in 'goodix_config_cb'
>
> Cc: Bastien Nocera <hadess@hadess.net>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Henrik Rydberg <rydberg@bitmath.org>
> Cc: "K. Merker" <merker@debian.org>
> Cc: authored by <andrew@goodix.com>
> Cc: Benjamin Tissoires <benjamin.tissoires@gmail.com>
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
Applied, thank you.
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* [PATCH 14/15] input: keyboard: applespi: Provide missing struct 'message' descriptions
2020-11-12 11:01 [PATCH 00/15] [Set 2/2] Rid W=1 issues from Input Lee Jones
` (12 preceding siblings ...)
2020-11-12 11:02 ` [PATCH 13/15] input: touchscreen: goodix: Fix misspelling of 'ctx' Lee Jones
@ 2020-11-12 11:02 ` Lee Jones
2020-11-20 3:21 ` Dmitry Torokhov
2020-11-12 11:02 ` [PATCH 15/15] input: mouse: vmmouse: Demote obvious abuse of kernel-doc header Lee Jones
14 siblings, 1 reply; 42+ messages in thread
From: Lee Jones @ 2020-11-12 11:02 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, Dmitry Torokhov, Henrik Rydberg,
Gustavo A. R. Silva, linux-input
Fixes the following W=1 kernel build warning(s):
drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'keyboard' not described in 'message'
drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'touchpad' not described in 'message'
drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'tp_info' not described in 'message'
drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'tp_info_command' not described in 'message'
drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'init_mt_command' not described in 'message'
drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'capsl_command' not described in 'message'
drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'bl_command' not described in 'message'
drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'data' not described in 'message'
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Henrik Rydberg <rydberg@bitmath.org>
Cc: "Gustavo A. R. Silva" <gustavoars@kernel.org>
Cc: linux-input@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/input/keyboard/applespi.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/drivers/input/keyboard/applespi.c b/drivers/input/keyboard/applespi.c
index 14362ebab9a9d..8053a3d2ff635 100644
--- a/drivers/input/keyboard/applespi.c
+++ b/drivers/input/keyboard/applespi.c
@@ -286,6 +286,15 @@ struct command_protocol_bl {
* structure (after re-assembly in case of being split over
* multiple spi-packets), minus the trailing crc. The total size
* of the message struct is therefore @length + 10.
+ *
+ * @keyboard: Keyboard message
+ * @touchpad: Touchpad message
+ * @tp_info: Touchpad info (response)
+ * @tp_info_command: Touchpad info (CRC)
+ * @init_mt_command: Initialise Multitouch
+ * @capsl_command: Toggle caps-lock LED
+ * @bl_command: Keyboard brightness
+ * @data: Buffer data
*/
struct message {
__le16 type;
--
2.25.1
^ permalink raw reply related [flat|nested] 42+ messages in thread
* Re: [PATCH 14/15] input: keyboard: applespi: Provide missing struct 'message' descriptions
2020-11-12 11:02 ` [PATCH 14/15] input: keyboard: applespi: Provide missing struct 'message' descriptions Lee Jones
@ 2020-11-20 3:21 ` Dmitry Torokhov
0 siblings, 0 replies; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-20 3:21 UTC (permalink / raw)
To: Lee Jones; +Cc: linux-kernel, Henrik Rydberg, Gustavo A. R. Silva, linux-input
On Thu, Nov 12, 2020 at 11:02:03AM +0000, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'keyboard' not described in 'message'
> drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'touchpad' not described in 'message'
> drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'tp_info' not described in 'message'
> drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'tp_info_command' not described in 'message'
> drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'init_mt_command' not described in 'message'
> drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'capsl_command' not described in 'message'
> drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'bl_command' not described in 'message'
> drivers/input/keyboard/applespi.c:306: warning: Function parameter or member 'data' not described in 'message'
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Henrik Rydberg <rydberg@bitmath.org>
> Cc: "Gustavo A. R. Silva" <gustavoars@kernel.org>
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
Applied, thank you.
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread
* [PATCH 15/15] input: mouse: vmmouse: Demote obvious abuse of kernel-doc header
2020-11-12 11:01 [PATCH 00/15] [Set 2/2] Rid W=1 issues from Input Lee Jones
` (13 preceding siblings ...)
2020-11-12 11:02 ` [PATCH 14/15] input: keyboard: applespi: Provide missing struct 'message' descriptions Lee Jones
@ 2020-11-12 11:02 ` Lee Jones
2020-11-20 3:21 ` Dmitry Torokhov
14 siblings, 1 reply; 42+ messages in thread
From: Lee Jones @ 2020-11-12 11:02 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, VMware Graphics, VMware, Inc.,
Dmitry Torokhov, Thomas Hellstrom, linux-input
Fixes the following W=1 kernel build warning(s):
drivers/input/mouse/vmmouse.c:99: warning: Function parameter or member 'cmd' not described in 'VMMOUSE_CMD'
drivers/input/mouse/vmmouse.c:99: warning: Function parameter or member 'in1' not described in 'VMMOUSE_CMD'
drivers/input/mouse/vmmouse.c:99: warning: Function parameter or member 'out1' not described in 'VMMOUSE_CMD'
drivers/input/mouse/vmmouse.c:99: warning: Function parameter or member 'out2' not described in 'VMMOUSE_CMD'
drivers/input/mouse/vmmouse.c:99: warning: Function parameter or member 'out3' not described in 'VMMOUSE_CMD'
drivers/input/mouse/vmmouse.c:99: warning: Function parameter or member 'out4' not described in 'VMMOUSE_CMD'
Cc: VMware Graphics <linux-graphics-maintainer@vmware.com>
Cc: "VMware, Inc." <pv-drivers@vmware.com>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: linux-input@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/input/mouse/vmmouse.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/input/mouse/vmmouse.c b/drivers/input/mouse/vmmouse.c
index 148245c69be75..42443ffba7c4d 100644
--- a/drivers/input/mouse/vmmouse.c
+++ b/drivers/input/mouse/vmmouse.c
@@ -76,7 +76,7 @@ struct vmmouse_data {
char dev_name[128];
};
-/**
+/*
* Hypervisor-specific bi-directional communication channel
* implementing the vmmouse protocol. Should never execute on
* bare metal hardware.
--
2.25.1
^ permalink raw reply related [flat|nested] 42+ messages in thread
* Re: [PATCH 15/15] input: mouse: vmmouse: Demote obvious abuse of kernel-doc header
2020-11-12 11:02 ` [PATCH 15/15] input: mouse: vmmouse: Demote obvious abuse of kernel-doc header Lee Jones
@ 2020-11-20 3:21 ` Dmitry Torokhov
0 siblings, 0 replies; 42+ messages in thread
From: Dmitry Torokhov @ 2020-11-20 3:21 UTC (permalink / raw)
To: Lee Jones
Cc: linux-kernel, VMware Graphics, VMware, Inc.,
Thomas Hellstrom, linux-input
On Thu, Nov 12, 2020 at 11:02:04AM +0000, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/input/mouse/vmmouse.c:99: warning: Function parameter or member 'cmd' not described in 'VMMOUSE_CMD'
> drivers/input/mouse/vmmouse.c:99: warning: Function parameter or member 'in1' not described in 'VMMOUSE_CMD'
> drivers/input/mouse/vmmouse.c:99: warning: Function parameter or member 'out1' not described in 'VMMOUSE_CMD'
> drivers/input/mouse/vmmouse.c:99: warning: Function parameter or member 'out2' not described in 'VMMOUSE_CMD'
> drivers/input/mouse/vmmouse.c:99: warning: Function parameter or member 'out3' not described in 'VMMOUSE_CMD'
> drivers/input/mouse/vmmouse.c:99: warning: Function parameter or member 'out4' not described in 'VMMOUSE_CMD'
>
> Cc: VMware Graphics <linux-graphics-maintainer@vmware.com>
> Cc: "VMware, Inc." <pv-drivers@vmware.com>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Thomas Hellstrom <thellstrom@vmware.com>
> Cc: linux-input@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
Applied, thank you.
--
Dmitry
^ permalink raw reply [flat|nested] 42+ messages in thread