Message ID | 1354634875-5182-5-git-send-email-benjamin.tissoires@gmail.com (mailing list archive) |
---|---|
State | New, archived |
Delegated to: | Jiri Kosina |
Headers | show |
On Tue, 4 Dec 2012 16:27:45 +0100, Benjamin Tissoires wrote: > We should not initialize to 0 static declarations. > > Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com> > --- > drivers/hid/i2c-hid/i2c-hid.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/hid/i2c-hid/i2c-hid.c b/drivers/hid/i2c-hid/i2c-hid.c > index ad771a7..9ee6555 100644 > --- a/drivers/hid/i2c-hid/i2c-hid.c > +++ b/drivers/hid/i2c-hid/i2c-hid.c > @@ -46,7 +46,7 @@ > #define I2C_HID_PWR_SLEEP 0x01 > > /* debug option */ > -static bool debug = false; > +static bool debug; > module_param(debug, bool, 0444); > MODULE_PARM_DESC(debug, "print a lot of debug information"); > Reviewed-by: Jean Delvare <khali@linux-fr.org>
On Tue, 4 Dec 2012, Jean Delvare wrote: > > We should not initialize to 0 static declarations. > > > > Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com> > > --- > > drivers/hid/i2c-hid/i2c-hid.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/hid/i2c-hid/i2c-hid.c b/drivers/hid/i2c-hid/i2c-hid.c > > index ad771a7..9ee6555 100644 > > --- a/drivers/hid/i2c-hid/i2c-hid.c > > +++ b/drivers/hid/i2c-hid/i2c-hid.c > > @@ -46,7 +46,7 @@ > > #define I2C_HID_PWR_SLEEP 0x01 > > > > /* debug option */ > > -static bool debug = false; > > +static bool debug; > > module_param(debug, bool, 0444); > > MODULE_PARM_DESC(debug, "print a lot of debug information"); > > > > Reviewed-by: Jean Delvare <khali@linux-fr.org> Applied, thanks.
diff --git a/drivers/hid/i2c-hid/i2c-hid.c b/drivers/hid/i2c-hid/i2c-hid.c index ad771a7..9ee6555 100644 --- a/drivers/hid/i2c-hid/i2c-hid.c +++ b/drivers/hid/i2c-hid/i2c-hid.c @@ -46,7 +46,7 @@ #define I2C_HID_PWR_SLEEP 0x01 /* debug option */ -static bool debug = false; +static bool debug; module_param(debug, bool, 0444); MODULE_PARM_DESC(debug, "print a lot of debug information");
We should not initialize to 0 static declarations. Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com> --- drivers/hid/i2c-hid/i2c-hid.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)