[PATCH 0/2] Fix checkpatch warnings

[PATCH 0/2] Fix checkpatch warnings

[Roberta Dobrescu]

This patchset fixes warnings detected using checkpatch.pl,
such as: incomplete config symbol description, prohibited
space before ','.

Roberta Dobrescu (2):
  iio: light: Annotate Kconfig entry with module name information
  iio: light: lm3533-als: Remove space before ','

 drivers/iio/light/Kconfig      | 3 +++
 drivers/iio/light/lm3533-als.c | 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)

-- 
1.9.1

[PATCH 1/2] iio: light: Annotate Kconfig entry with module name information

[Roberta Dobrescu]

This patch fixes the following checkpatch.pl warning:
WARNING: please write a paragraph that describes the config symbol fully

Signed-off-by: Roberta Dobrescu <roberta.dobrescu@gmail.com>
---
 drivers/iio/light/Kconfig | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/iio/light/Kconfig b/drivers/iio/light/Kconfig
index 5bea821..fce4d688 100644
--- a/drivers/iio/light/Kconfig
+++ b/drivers/iio/light/Kconfig
@@ -95,6 +95,9 @@ config HID_SENSOR_ALS
 	  Say yes here to build support for the HID SENSOR
 	  Ambient light sensor.
 
+	  To compile this driver as a module, choose M here: the
+	  module will be called hid-sensor-als.
+
 config HID_SENSOR_PROX
 	depends on HID_SENSOR_HUB
 	select IIO_BUFFER
-- 
1.9.1

Re: [PATCH 1/2] iio: light: Annotate Kconfig entry with module name information

[Jonathan Cameron]

On 30/12/14 18:48, Roberta Dobrescu wrote:
> This patch fixes the following checkpatch.pl warning:
> WARNING: please write a paragraph that describes the config symbol fully
> 
> Signed-off-by: Roberta Dobrescu <roberta.dobrescu@gmail.com>
Applied.
> ---
>  drivers/iio/light/Kconfig | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/iio/light/Kconfig b/drivers/iio/light/Kconfig
> index 5bea821..fce4d688 100644
> --- a/drivers/iio/light/Kconfig
> +++ b/drivers/iio/light/Kconfig
> @@ -95,6 +95,9 @@ config HID_SENSOR_ALS
>  	  Say yes here to build support for the HID SENSOR
>  	  Ambient light sensor.
>  
> +	  To compile this driver as a module, choose M here: the
> +	  module will be called hid-sensor-als.
> +
>  config HID_SENSOR_PROX
>  	depends on HID_SENSOR_HUB
>  	select IIO_BUFFER
> 

[PATCH 2/2] iio: light: lm3533-als: Remove space before ','

[Roberta Dobrescu]

This patch fixes the following checkpatch.pl error:
ERROR: space prohibited before that ',' (ctx:WxE)

Signed-off-by: Roberta Dobrescu <roberta.dobrescu@gmail.com>
---
 drivers/iio/light/lm3533-als.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/iio/light/lm3533-als.c b/drivers/iio/light/lm3533-als.c
index ae3c71b..076bc46 100644
--- a/drivers/iio/light/lm3533-als.c
+++ b/drivers/iio/light/lm3533-als.c
@@ -657,7 +657,7 @@ static ALS_HYSTERESIS_ATTR_RO(3);
 #define ILLUMINANCE_ATTR_RO(_name) \
 	DEVICE_ATTR(in_illuminance0_##_name, S_IRUGO, show_##_name, NULL)
 #define ILLUMINANCE_ATTR_RW(_name) \
-	DEVICE_ATTR(in_illuminance0_##_name, S_IRUGO | S_IWUSR , \
+	DEVICE_ATTR(in_illuminance0_##_name, S_IRUGO | S_IWUSR, \
 						show_##_name, store_##_name)
 /*
  * ALS Zone threshold-event enable
-- 
1.9.1

Re: [PATCH 2/2] iio: light: lm3533-als: Remove space before ','

[Jonathan Cameron]

On 30/12/14 18:48, Roberta Dobrescu wrote:
> This patch fixes the following checkpatch.pl error:
> ERROR: space prohibited before that ',' (ctx:WxE)
> 
> Signed-off-by: Roberta Dobrescu <roberta.dobrescu@gmail.com>
Applied.
> ---
>  drivers/iio/light/lm3533-als.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/iio/light/lm3533-als.c b/drivers/iio/light/lm3533-als.c
> index ae3c71b..076bc46 100644
> --- a/drivers/iio/light/lm3533-als.c
> +++ b/drivers/iio/light/lm3533-als.c
> @@ -657,7 +657,7 @@ static ALS_HYSTERESIS_ATTR_RO(3);
>  #define ILLUMINANCE_ATTR_RO(_name) \
>  	DEVICE_ATTR(in_illuminance0_##_name, S_IRUGO, show_##_name, NULL)
>  #define ILLUMINANCE_ATTR_RW(_name) \
> -	DEVICE_ATTR(in_illuminance0_##_name, S_IRUGO | S_IWUSR , \
> +	DEVICE_ATTR(in_illuminance0_##_name, S_IRUGO | S_IWUSR, \
>  						show_##_name, store_##_name)
>  /*
>   * ALS Zone threshold-event enable
>