2017-08-17 11:39:22

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 00/30] constify input serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Arvind Yadav (30):
[PATCH 01/30] Input: iforce: constify serio_device_id
[PATCH 02/30] Input: magellan: constify serio_device_id
[PATCH 03/30] Input: spaceball: constify serio_device_id
[PATCH 04/30] Input: spaceorb: constify serio_device_id
[PATCH 05/30] Input: stinger: constify serio_device_id
[PATCH 06/30] Input: twidjoy: constify serio_device_id
[PATCH 07/30] Input: warrior: constify serio_device_id
[PATCH 08/30] Input: zhenhua: constify serio_device_id
[PATCH 09/30] Input: iatkbd: constify serio_device_id
[PATCH 10/30] Input: hil_kbd: constify serio_device_id
[PATCH 11/30] Input: lkkbd: constify serio_device_id
[PATCH 12/30] Input: newtonkbd: constify serio_device_id
[PATCH 13/30] Input: stowaway: constify serio_device_id
[PATCH 14/30] Input: sunkbd: constify serio_device_id
[PATCH 15/30] Input: xtkbd: constify serio_device_id
[PATCH 16/30] Input: serio: constify serio_device_id
[PATCH 17/30] Input: wacom_serial4: constify serio_device_id
[PATCH 18/30] Input: dynapro: constify serio_device_id
[PATCH 19/30] Input: elo: constify serio_device_id
[PATCH 20/30] Input: fujitsu_ts: constify serio_device_id
[PATCH 21/30] Input: gunze: constify serio_device_id
[PATCH 22/30] Input: hampshire: constify serio_device_id
[PATCH 23/30] Input: inexio: constify serio_device_id
[PATCH 24/30] Input: mtouch: constify serio_device_id
[PATCH 25/30] Input: penmount: constify serio_device_id
[PATCH 26/30] Input: touchit213: constify serio_device_id
[PATCH 27/30] Input: touchright: constify serio_device_id
[PATCH 28/30] Input: touchwin: constify serio_device_id
[PATCH 29/30] Input: tsc40: constify serio_device_id
[PATCH 30/30] Input: wacom_w8001: constify serio_device_id

drivers/input/joystick/iforce/iforce-serio.c | 2 +-
drivers/input/joystick/magellan.c | 2 +-
drivers/input/joystick/spaceball.c | 2 +-
drivers/input/joystick/spaceorb.c | 2 +-
drivers/input/joystick/stinger.c | 2 +-
drivers/input/joystick/twidjoy.c | 2 +-
drivers/input/joystick/warrior.c | 2 +-
drivers/input/joystick/zhenhua.c | 2 +-
drivers/input/keyboard/atkbd.c | 2 +-
drivers/input/keyboard/hil_kbd.c | 2 +-
drivers/input/keyboard/lkkbd.c | 2 +-
drivers/input/keyboard/newtonkbd.c | 2 +-
drivers/input/keyboard/stowaway.c | 2 +-
drivers/input/keyboard/sunkbd.c | 2 +-
drivers/input/keyboard/xtkbd.c | 2 +-
drivers/input/serio/serio_raw.c | 2 +-
drivers/input/tablet/wacom_serial4.c | 2 +-
drivers/input/touchscreen/dynapro.c | 2 +-
drivers/input/touchscreen/elo.c | 2 +-
drivers/input/touchscreen/fujitsu_ts.c | 2 +-
drivers/input/touchscreen/gunze.c | 2 +-
drivers/input/touchscreen/hampshire.c | 2 +-
drivers/input/touchscreen/inexio.c | 2 +-
drivers/input/touchscreen/mtouch.c | 2 +-
drivers/input/touchscreen/penmount.c | 2 +-
drivers/input/touchscreen/touchit213.c | 2 +-
drivers/input/touchscreen/touchright.c | 2 +-
drivers/input/touchscreen/touchwin.c | 2 +-
drivers/input/touchscreen/tsc40.c | 2 +-
drivers/input/touchscreen/wacom_w8001.c | 2 +-
30 files changed, 30 insertions(+), 30 deletions(-)

--
2.7.4


2017-08-17 11:39:26

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 02/30] Input: magellan: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/joystick/magellan.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/joystick/magellan.c b/drivers/input/joystick/magellan.c
index c5358ba..a9d0e3e 100644
--- a/drivers/input/joystick/magellan.c
+++ b/drivers/input/joystick/magellan.c
@@ -198,7 +198,7 @@ static int magellan_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id magellan_serio_ids[] = {
+static const struct serio_device_id magellan_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_MAGELLAN,
--
2.7.4

2017-08-17 11:39:31

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 04/30] Input: spaceorb: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/joystick/spaceorb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/joystick/spaceorb.c b/drivers/input/joystick/spaceorb.c
index f266782..05da0ed 100644
--- a/drivers/input/joystick/spaceorb.c
+++ b/drivers/input/joystick/spaceorb.c
@@ -213,7 +213,7 @@ static int spaceorb_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id spaceorb_serio_ids[] = {
+static const struct serio_device_id spaceorb_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_SPACEORB,
--
2.7.4

2017-08-17 11:39:36

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 06/30] Input: twidjoy: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/joystick/twidjoy.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/joystick/twidjoy.c b/drivers/input/joystick/twidjoy.c
index 7f7e5ab..e60cb00 100644
--- a/drivers/input/joystick/twidjoy.c
+++ b/drivers/input/joystick/twidjoy.c
@@ -233,7 +233,7 @@ static int twidjoy_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id twidjoy_serio_ids[] = {
+static const struct serio_device_id twidjoy_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_TWIDJOY,
--
2.7.4

2017-08-17 11:39:53

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 15/30] Input: xtkbd: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/keyboard/xtkbd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/keyboard/xtkbd.c b/drivers/input/keyboard/xtkbd.c
index 7c2325b..8f64b9d 100644
--- a/drivers/input/keyboard/xtkbd.c
+++ b/drivers/input/keyboard/xtkbd.c
@@ -145,7 +145,7 @@ static void xtkbd_disconnect(struct serio *serio)
kfree(xtkbd);
}

-static struct serio_device_id xtkbd_serio_ids[] = {
+static const struct serio_device_id xtkbd_serio_ids[] = {
{
.type = SERIO_XT,
.proto = SERIO_ANY,
--
2.7.4

2017-08-17 11:39:58

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 17/30] Input: wacom_serial4: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/tablet/wacom_serial4.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/tablet/wacom_serial4.c b/drivers/input/tablet/wacom_serial4.c
index 20ab802..38bfaca 100644
--- a/drivers/input/tablet/wacom_serial4.c
+++ b/drivers/input/tablet/wacom_serial4.c
@@ -594,7 +594,7 @@ static int wacom_connect(struct serio *serio, struct serio_driver *drv)
return err;
}

-static struct serio_device_id wacom_serio_ids[] = {
+static const struct serio_device_id wacom_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_WACOM_IV,
--
2.7.4

2017-08-17 11:40:05

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 21/30] Input: gunze: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/touchscreen/gunze.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/touchscreen/gunze.c b/drivers/input/touchscreen/gunze.c
index e2ee626..4815869 100644
--- a/drivers/input/touchscreen/gunze.c
+++ b/drivers/input/touchscreen/gunze.c
@@ -162,7 +162,7 @@ static int gunze_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id gunze_serio_ids[] = {
+static const struct serio_device_id gunze_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_GUNZE,
--
2.7.4

2017-08-17 11:40:07

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 20/30] Input: fujitsu_ts: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/touchscreen/fujitsu_ts.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/touchscreen/fujitsu_ts.c b/drivers/input/touchscreen/fujitsu_ts.c
index d0e46a7..a0fbb45 100644
--- a/drivers/input/touchscreen/fujitsu_ts.c
+++ b/drivers/input/touchscreen/fujitsu_ts.c
@@ -151,7 +151,7 @@ static int fujitsu_connect(struct serio *serio, struct serio_driver *drv)
/*
* The serio driver structure.
*/
-static struct serio_device_id fujitsu_serio_ids[] = {
+static const struct serio_device_id fujitsu_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_FUJITSU,
--
2.7.4

2017-08-17 11:40:11

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 25/30] Input: penmount: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/touchscreen/penmount.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/touchscreen/penmount.c b/drivers/input/touchscreen/penmount.c
index 417d873..6e6d7fd 100644
--- a/drivers/input/touchscreen/penmount.c
+++ b/drivers/input/touchscreen/penmount.c
@@ -293,7 +293,7 @@ static int pm_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id pm_serio_ids[] = {
+static const struct serio_device_id pm_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_PENMOUNT,
--
2.7.4

2017-08-17 11:40:17

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 27/30] Input: touchright: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/touchscreen/touchright.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/touchscreen/touchright.c b/drivers/input/touchscreen/touchright.c
index 4000e52..45c325c 100644
--- a/drivers/input/touchscreen/touchright.c
+++ b/drivers/input/touchscreen/touchright.c
@@ -152,7 +152,7 @@ static int tr_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id tr_serio_ids[] = {
+static const struct serio_device_id tr_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_TOUCHRIGHT,
--
2.7.4

2017-08-17 11:40:22

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 30/30] Input: wacom_w8001: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/touchscreen/wacom_w8001.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/touchscreen/wacom_w8001.c b/drivers/input/touchscreen/wacom_w8001.c
index 85e9572..3715d1e 100644
--- a/drivers/input/touchscreen/wacom_w8001.c
+++ b/drivers/input/touchscreen/wacom_w8001.c
@@ -681,7 +681,7 @@ static int w8001_connect(struct serio *serio, struct serio_driver *drv)
return err;
}

-static struct serio_device_id w8001_serio_ids[] = {
+static const struct serio_device_id w8001_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_W8001,
--
2.7.4

2017-08-17 11:40:47

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 28/30] Input: touchwin: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/touchscreen/touchwin.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/touchscreen/touchwin.c b/drivers/input/touchscreen/touchwin.c
index ba90f44..2ba6b4c 100644
--- a/drivers/input/touchscreen/touchwin.c
+++ b/drivers/input/touchscreen/touchwin.c
@@ -159,7 +159,7 @@ static int tw_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id tw_serio_ids[] = {
+static const struct serio_device_id tw_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_TOUCHWIN,
--
2.7.4

2017-08-17 11:40:46

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 29/30] Input: tsc40: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/touchscreen/tsc40.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/touchscreen/tsc40.c b/drivers/input/touchscreen/tsc40.c
index 2968787..d4ae4ba 100644
--- a/drivers/input/touchscreen/tsc40.c
+++ b/drivers/input/touchscreen/tsc40.c
@@ -141,7 +141,7 @@ static void tsc_disconnect(struct serio *serio)
serio_set_drvdata(serio, NULL);
}

-static struct serio_device_id tsc_serio_ids[] = {
+static const struct serio_device_id tsc_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_TSC40,
--
2.7.4

2017-08-17 11:41:23

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 26/30] Input: touchit213: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/touchscreen/touchit213.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/touchscreen/touchit213.c b/drivers/input/touchscreen/touchit213.c
index c27cf8f..98a1669 100644
--- a/drivers/input/touchscreen/touchit213.c
+++ b/drivers/input/touchscreen/touchit213.c
@@ -192,7 +192,7 @@ static int touchit213_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id touchit213_serio_ids[] = {
+static const struct serio_device_id touchit213_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_TOUCHIT213,
--
2.7.4

2017-08-17 11:41:41

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 24/30] Input: mtouch: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/touchscreen/mtouch.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/touchscreen/mtouch.c b/drivers/input/touchscreen/mtouch.c
index 9b5552a..a3707fa 100644
--- a/drivers/input/touchscreen/mtouch.c
+++ b/drivers/input/touchscreen/mtouch.c
@@ -178,7 +178,7 @@ static int mtouch_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id mtouch_serio_ids[] = {
+static const struct serio_device_id mtouch_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_MICROTOUCH,
--
2.7.4

2017-08-17 11:42:01

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 23/30] Input: inexio: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/touchscreen/inexio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/touchscreen/inexio.c b/drivers/input/touchscreen/inexio.c
index adb80b6..b9bc562 100644
--- a/drivers/input/touchscreen/inexio.c
+++ b/drivers/input/touchscreen/inexio.c
@@ -165,7 +165,7 @@ static int inexio_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id inexio_serio_ids[] = {
+static const struct serio_device_id inexio_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_INEXIO,
--
2.7.4

2017-08-17 11:42:25

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 22/30] Input: hampshire: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/touchscreen/hampshire.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/touchscreen/hampshire.c b/drivers/input/touchscreen/hampshire.c
index ecb1e0e..eb052d5 100644
--- a/drivers/input/touchscreen/hampshire.c
+++ b/drivers/input/touchscreen/hampshire.c
@@ -163,7 +163,7 @@ static int hampshire_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id hampshire_serio_ids[] = {
+static const struct serio_device_id hampshire_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_HAMPSHIRE,
--
2.7.4

2017-08-17 11:42:45

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 19/30] Input: elo: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/touchscreen/elo.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/touchscreen/elo.c b/drivers/input/touchscreen/elo.c
index 8051a4b..83433e8 100644
--- a/drivers/input/touchscreen/elo.c
+++ b/drivers/input/touchscreen/elo.c
@@ -381,7 +381,7 @@ static int elo_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id elo_serio_ids[] = {
+static const struct serio_device_id elo_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_ELO,
--
2.7.4

2017-08-17 11:43:05

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 18/30] Input: dynapro: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/touchscreen/dynapro.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/touchscreen/dynapro.c b/drivers/input/touchscreen/dynapro.c
index 86237a9..5b1b66f 100644
--- a/drivers/input/touchscreen/dynapro.c
+++ b/drivers/input/touchscreen/dynapro.c
@@ -164,7 +164,7 @@ static int dynapro_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id dynapro_serio_ids[] = {
+static const struct serio_device_id dynapro_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_DYNAPRO,
--
2.7.4

2017-08-17 11:39:49

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 13/30] Input: stowaway: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/keyboard/stowaway.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/keyboard/stowaway.c b/drivers/input/keyboard/stowaway.c
index a6e0d56..8b6de9a 100644
--- a/drivers/input/keyboard/stowaway.c
+++ b/drivers/input/keyboard/stowaway.c
@@ -146,7 +146,7 @@ static void skbd_disconnect(struct serio *serio)
kfree(skbd);
}

-static struct serio_device_id skbd_serio_ids[] = {
+static const struct serio_device_id skbd_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_STOWAWAY,
--
2.7.4

2017-08-17 11:43:28

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 16/30] Input: serio: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/serio/serio_raw.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/serio/serio_raw.c b/drivers/input/serio/serio_raw.c
index 71ef5d6..516f9fe 100644
--- a/drivers/input/serio/serio_raw.c
+++ b/drivers/input/serio/serio_raw.c
@@ -410,7 +410,7 @@ static void serio_raw_disconnect(struct serio *serio)
serio_set_drvdata(serio, NULL);
}

-static struct serio_device_id serio_raw_serio_ids[] = {
+static const struct serio_device_id serio_raw_serio_ids[] = {
{
.type = SERIO_8042,
.proto = SERIO_ANY,
--
2.7.4

2017-08-17 11:39:46

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 11/30] Input: lkkbd: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/keyboard/lkkbd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/keyboard/lkkbd.c b/drivers/input/keyboard/lkkbd.c
index 9fcd9f1..471d538 100644
--- a/drivers/input/keyboard/lkkbd.c
+++ b/drivers/input/keyboard/lkkbd.c
@@ -707,7 +707,7 @@ static void lkkbd_disconnect(struct serio *serio)
kfree(lk);
}

-static struct serio_device_id lkkbd_serio_ids[] = {
+static const struct serio_device_id lkkbd_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_LKKBD,
--
2.7.4

2017-08-17 11:43:58

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 14/30] Input: sunkbd: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/keyboard/sunkbd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/keyboard/sunkbd.c b/drivers/input/keyboard/sunkbd.c
index dc6bb9d..c95707e 100644
--- a/drivers/input/keyboard/sunkbd.c
+++ b/drivers/input/keyboard/sunkbd.c
@@ -339,7 +339,7 @@ static void sunkbd_disconnect(struct serio *serio)
kfree(sunkbd);
}

-static struct serio_device_id sunkbd_serio_ids[] = {
+static const struct serio_device_id sunkbd_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_SUNKBD,
--
2.7.4

2017-08-17 11:39:42

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 09/30] Input: iatkbd: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/keyboard/atkbd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/keyboard/atkbd.c b/drivers/input/keyboard/atkbd.c
index ec876b5..7e75835 100644
--- a/drivers/input/keyboard/atkbd.c
+++ b/drivers/input/keyboard/atkbd.c
@@ -1270,7 +1270,7 @@ static int atkbd_reconnect(struct serio *serio)
return retval;
}

-static struct serio_device_id atkbd_serio_ids[] = {
+static const struct serio_device_id atkbd_serio_ids[] = {
{
.type = SERIO_8042,
.proto = SERIO_ANY,
--
2.7.4

2017-08-17 11:44:40

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 12/30] Input: newtonkbd: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/keyboard/newtonkbd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/keyboard/newtonkbd.c b/drivers/input/keyboard/newtonkbd.c
index 20f0443..fb9b8e2 100644
--- a/drivers/input/keyboard/newtonkbd.c
+++ b/drivers/input/keyboard/newtonkbd.c
@@ -142,7 +142,7 @@ static void nkbd_disconnect(struct serio *serio)
kfree(nkbd);
}

-static struct serio_device_id nkbd_serio_ids[] = {
+static const struct serio_device_id nkbd_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_NEWTON,
--
2.7.4

2017-08-17 11:45:07

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 10/30] Input: hil_kbd: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/keyboard/hil_kbd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/keyboard/hil_kbd.c b/drivers/input/keyboard/hil_kbd.c
index 5b152f2..bb29a7c 100644
--- a/drivers/input/keyboard/hil_kbd.c
+++ b/drivers/input/keyboard/hil_kbd.c
@@ -559,7 +559,7 @@ static int hil_dev_connect(struct serio *serio, struct serio_driver *drv)
return error;
}

-static struct serio_device_id hil_dev_ids[] = {
+static const struct serio_device_id hil_dev_ids[] = {
{
.type = SERIO_HIL_MLC,
.proto = SERIO_HIL,
--
2.7.4

2017-08-17 11:45:23

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 08/30] Input: zhenhua: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/joystick/zhenhua.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/joystick/zhenhua.c b/drivers/input/joystick/zhenhua.c
index 4a8258b..5c6d5de 100644
--- a/drivers/input/joystick/zhenhua.c
+++ b/drivers/input/joystick/zhenhua.c
@@ -192,7 +192,7 @@ static int zhenhua_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id zhenhua_serio_ids[] = {
+static const struct serio_device_id zhenhua_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_ZHENHUA,
--
2.7.4

2017-08-17 11:45:46

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 07/30] Input: warrior: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/joystick/warrior.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/joystick/warrior.c b/drivers/input/joystick/warrior.c
index e13a914..ef5391b 100644
--- a/drivers/input/joystick/warrior.c
+++ b/drivers/input/joystick/warrior.c
@@ -193,7 +193,7 @@ static int warrior_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id warrior_serio_ids[] = {
+static const struct serio_device_id warrior_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_WARRIOR,
--
2.7.4

2017-08-17 11:46:10

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 05/30] Input: stinger: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/joystick/stinger.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/joystick/stinger.c b/drivers/input/joystick/stinger.c
index 099c6d7..cb10e7b 100644
--- a/drivers/input/joystick/stinger.c
+++ b/drivers/input/joystick/stinger.c
@@ -184,7 +184,7 @@ static int stinger_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id stinger_serio_ids[] = {
+static const struct serio_device_id stinger_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_STINGER,
--
2.7.4

2017-08-17 11:46:34

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 03/30] Input: spaceball: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/joystick/spaceball.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/joystick/spaceball.c b/drivers/input/joystick/spaceball.c
index f4445a4..e9712a1 100644
--- a/drivers/input/joystick/spaceball.c
+++ b/drivers/input/joystick/spaceball.c
@@ -272,7 +272,7 @@ static int spaceball_connect(struct serio *serio, struct serio_driver *drv)
* The serio driver structure.
*/

-static struct serio_device_id spaceball_serio_ids[] = {
+static const struct serio_device_id spaceball_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_SPACEBALL,
--
2.7.4

2017-08-17 11:39:25

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 01/30] Input: iforce: constify serio_device_id

serio_device_id are not supposed to change at runtime. All functions
working with serio_device_id provided by <linux/serio.h> work with
const serio_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/input/joystick/iforce/iforce-serio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/input/joystick/iforce/iforce-serio.c b/drivers/input/joystick/iforce/iforce-serio.c
index 46d5041..154e827 100644
--- a/drivers/input/joystick/iforce/iforce-serio.c
+++ b/drivers/input/joystick/iforce/iforce-serio.c
@@ -164,7 +164,7 @@ static void iforce_serio_disconnect(struct serio *serio)
kfree(iforce);
}

-static struct serio_device_id iforce_serio_ids[] = {
+static const struct serio_device_id iforce_serio_ids[] = {
{
.type = SERIO_RS232,
.proto = SERIO_IFORCE,
--
2.7.4

2017-08-20 16:32:01

by Dmitry Torokhov

[permalink] [raw]
Subject: Re: [PATCH 00/30] constify input serio_device_id

On Thu, Aug 17, 2017 at 05:08:01PM +0530, Arvind Yadav wrote:
> serio_device_id are not supposed to change at runtime. All functions
> working with serio_device_id provided by <linux/serio.h> work with
> const serio_device_id. So mark the non-const structs as const.

Applied the lot, thank you.

>
> Arvind Yadav (30):
> [PATCH 01/30] Input: iforce: constify serio_device_id
> [PATCH 02/30] Input: magellan: constify serio_device_id
> [PATCH 03/30] Input: spaceball: constify serio_device_id
> [PATCH 04/30] Input: spaceorb: constify serio_device_id
> [PATCH 05/30] Input: stinger: constify serio_device_id
> [PATCH 06/30] Input: twidjoy: constify serio_device_id
> [PATCH 07/30] Input: warrior: constify serio_device_id
> [PATCH 08/30] Input: zhenhua: constify serio_device_id
> [PATCH 09/30] Input: iatkbd: constify serio_device_id
> [PATCH 10/30] Input: hil_kbd: constify serio_device_id
> [PATCH 11/30] Input: lkkbd: constify serio_device_id
> [PATCH 12/30] Input: newtonkbd: constify serio_device_id
> [PATCH 13/30] Input: stowaway: constify serio_device_id
> [PATCH 14/30] Input: sunkbd: constify serio_device_id
> [PATCH 15/30] Input: xtkbd: constify serio_device_id
> [PATCH 16/30] Input: serio: constify serio_device_id
> [PATCH 17/30] Input: wacom_serial4: constify serio_device_id
> [PATCH 18/30] Input: dynapro: constify serio_device_id
> [PATCH 19/30] Input: elo: constify serio_device_id
> [PATCH 20/30] Input: fujitsu_ts: constify serio_device_id
> [PATCH 21/30] Input: gunze: constify serio_device_id
> [PATCH 22/30] Input: hampshire: constify serio_device_id
> [PATCH 23/30] Input: inexio: constify serio_device_id
> [PATCH 24/30] Input: mtouch: constify serio_device_id
> [PATCH 25/30] Input: penmount: constify serio_device_id
> [PATCH 26/30] Input: touchit213: constify serio_device_id
> [PATCH 27/30] Input: touchright: constify serio_device_id
> [PATCH 28/30] Input: touchwin: constify serio_device_id
> [PATCH 29/30] Input: tsc40: constify serio_device_id
> [PATCH 30/30] Input: wacom_w8001: constify serio_device_id
>
> drivers/input/joystick/iforce/iforce-serio.c | 2 +-
> drivers/input/joystick/magellan.c | 2 +-
> drivers/input/joystick/spaceball.c | 2 +-
> drivers/input/joystick/spaceorb.c | 2 +-
> drivers/input/joystick/stinger.c | 2 +-
> drivers/input/joystick/twidjoy.c | 2 +-
> drivers/input/joystick/warrior.c | 2 +-
> drivers/input/joystick/zhenhua.c | 2 +-
> drivers/input/keyboard/atkbd.c | 2 +-
> drivers/input/keyboard/hil_kbd.c | 2 +-
> drivers/input/keyboard/lkkbd.c | 2 +-
> drivers/input/keyboard/newtonkbd.c | 2 +-
> drivers/input/keyboard/stowaway.c | 2 +-
> drivers/input/keyboard/sunkbd.c | 2 +-
> drivers/input/keyboard/xtkbd.c | 2 +-
> drivers/input/serio/serio_raw.c | 2 +-
> drivers/input/tablet/wacom_serial4.c | 2 +-
> drivers/input/touchscreen/dynapro.c | 2 +-
> drivers/input/touchscreen/elo.c | 2 +-
> drivers/input/touchscreen/fujitsu_ts.c | 2 +-
> drivers/input/touchscreen/gunze.c | 2 +-
> drivers/input/touchscreen/hampshire.c | 2 +-
> drivers/input/touchscreen/inexio.c | 2 +-
> drivers/input/touchscreen/mtouch.c | 2 +-
> drivers/input/touchscreen/penmount.c | 2 +-
> drivers/input/touchscreen/touchit213.c | 2 +-
> drivers/input/touchscreen/touchright.c | 2 +-
> drivers/input/touchscreen/touchwin.c | 2 +-
> drivers/input/touchscreen/tsc40.c | 2 +-
> drivers/input/touchscreen/wacom_w8001.c | 2 +-
> 30 files changed, 30 insertions(+), 30 deletions(-)
>
> --
> 2.7.4
>

--
Dmitry