media: usb: make i2c_client const
authorBhumika Goyal <bhumirks@gmail.com>
Mon, 28 Aug 2017 07:21:35 +0000 (03:21 -0400)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Sat, 23 Sep 2017 12:27:42 +0000 (08:27 -0400)
Make these const as they are only used in a copy operation.
Done using Coccinelle.

@match disable optional_qualifier@
identifier s;
@@
static struct i2c_client s = {...};

@ref@
position p;
identifier match.s;
@@
s@p

@good1@
position ref.p;
identifier match.s,f,c;
expression e;
@@
(
e = s@p
|
e = s@p.f
|
c(...,s@p.f,...)
|
c(...,s@p,...)
)

@bad depends on  !good1@
position ref.p;
identifier match.s;
@@
s@p

@depends on forall !bad disable optional_qualifier@
identifier match.s;
@@
static
+ const
struct i2c_client s;

Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
Signed-off-by: Hans Verkuil <hansverk@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/usb/au0828/au0828-i2c.c
drivers/media/usb/em28xx/em28xx-i2c.c
drivers/media/usb/stk1160/stk1160-i2c.c

index ef7d1b8..1b8ec5d 100644 (file)
@@ -342,7 +342,7 @@ static const struct i2c_adapter au0828_i2c_adap_template = {
        .algo              = &au0828_i2c_algo_template,
 };
 
-static struct i2c_client au0828_i2c_client_template = {
+static const struct i2c_client au0828_i2c_client_template = {
        .name   = "au0828 internal",
 };
 
index 66c5012..9bf49d6 100644 (file)
@@ -882,7 +882,7 @@ static const struct i2c_adapter em28xx_adap_template = {
        .algo = &em28xx_algo,
 };
 
-static struct i2c_client em28xx_client_template = {
+static const struct i2c_client em28xx_client_template = {
        .name = "em28xx internal",
 };
 
index 2c70173..62a12d5 100644 (file)
@@ -246,7 +246,7 @@ static const struct i2c_adapter adap_template = {
        .algo = &algo,
 };
 
-static struct i2c_client client_template = {
+static const struct i2c_client client_template = {
        .name = "stk1160 internal",
 };