[PATCH V2 00/18] backlight: fix checkpatch warnings

[PATCH V2 00/18] backlight: fix checkpatch warnings

[Jingoo Han]

This patchset fixes checkpatch warnings as follows.
There is no functional change.

   WARNING: Missing a blank line after declarations
   WARNING: else is not generally useful after a break or return
   WARNING: void function return statements are not generally useful

Changes for V2
- Added Lee Jones's Acked-by for 1~18th patches, except for 17th patch.
- Fixed 17th patch, per Lee Jones's feedback.
   
Jingoo Han (18)
      backlight: adp5520: add blank line after declarations
      backlight: adp8860: add blank line after declarations
      backlight: adp8870: add blank line after declarations
      backlight: ams369fg06: remove 'else' after a return
      backlight: corgi_lcd: add blank line after declarations
      backlight: cr_bllcd: add blank line after declarations
      backlight: ili922x: remove 'else' after a return
      backlight: ld9040: remove 'else' after a return
      backlight: lm3639: remove unnecessary return statements
      backlight: lms501kf03: remove 'else' after a return
      backlight: lp855x: add blank line after declarations
      backlight: pcf50633: add blank line after declarations
      backlight: s6e63m0: remove 'else' after a return
      backlight: tdo24m: add blank line after declarations
      backlight: wm831x_bl: add blank line after declarations
      backlight: jornada720: remove 'else' after a return
      backlight: jornada720: remove 'else' after a return
      backlight: omap1: add blank line after declarations
   
---
 drivers/video/backlight/adp5520_bl.c         |  1 +
 drivers/video/backlight/adp8860_bl.c         |  3 +++
 drivers/video/backlight/adp8870_bl.c         |  4 ++++
 drivers/video/backlight/ams369fg06.c         |  6 +++---
 drivers/video/backlight/corgi_lcd.c          |  1 +
 drivers/video/backlight/cr_bllcd.c           |  1 +
 drivers/video/backlight/ili922x.c            | 11 ++++++-----
 drivers/video/backlight/jornada720_bl.c      |  6 +++---
 drivers/video/backlight/jornada720_lcd.c     |  6 +-----
 drivers/video/backlight/ld9040.c             |  6 +++---
 drivers/video/backlight/lm3639_bl.c          |  2 --
 drivers/video/backlight/lms501kf03.c         | 12 ++++++------
 drivers/video/backlight/lp855x_bl.c          |  2 ++
 drivers/video/backlight/omap1_bl.c           |  1 +
 drivers/video/backlight/pcf50633-backlight.c |  1 +
 drivers/video/backlight/s6e63m0.c            | 12 ++++++------
 drivers/video/backlight/tdo24m.c             |  2 ++
 drivers/video/backlight/wm831x_bl.c          |  1 +
 18 files changed, 45 insertions(+), 33 deletions(-)

[PATCH V2 01/18] backlight: adp5520: add blank line after declarations

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: Missing a blank line after declarations

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/adp5520_bl.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/video/backlight/adp5520_bl.c b/drivers/video/backlight/adp5520_bl.c
index f37097a261a2..8280245d7c12 100644
--- a/drivers/video/backlight/adp5520_bl.c
+++ b/drivers/video/backlight/adp5520_bl.c
@@ -67,6 +67,7 @@ static int adp5520_bl_set(struct backlight_device *bl, int brightness)
 static int adp5520_bl_update_status(struct backlight_device *bl)
 {
 	int brightness = bl->props.brightness;
+
 	if (bl->props.power != FB_BLANK_UNBLANK)
 		brightness = 0;
 
-- 
2.0.0

[PATCH V2 02/18] backlight: adp8860: add blank line after declarations

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: Missing a blank line after declarations

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/adp8860_bl.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/video/backlight/adp8860_bl.c b/drivers/video/backlight/adp8860_bl.c
index be8d83deca7d..71147f4461b8 100644
--- a/drivers/video/backlight/adp8860_bl.c
+++ b/drivers/video/backlight/adp8860_bl.c
@@ -181,6 +181,7 @@ static int adp8860_clr_bits(struct i2c_client *client, int reg, uint8_t bit_mask
 static void adp8860_led_work(struct work_struct *work)
 {
 	struct adp8860_led *led = container_of(work, struct adp8860_led, work);
+
 	adp8860_write(led->client, ADP8860_ISC1 - led->id + 1,
 			 led->new_brightness >> 1);
 }
@@ -362,6 +363,7 @@ static int adp8860_bl_set(struct backlight_device *bl, int brightness)
 static int adp8860_bl_update_status(struct backlight_device *bl)
 {
 	int brightness = bl->props.brightness;
+
 	if (bl->props.power != FB_BLANK_UNBLANK)
 		brightness = 0;
 
@@ -499,6 +501,7 @@ static ssize_t adp8860_bl_l1_daylight_max_store(struct device *dev,
 {
 	struct adp8860_bl *data = dev_get_drvdata(dev);
 	int ret = kstrtoul(buf, 10, &data->cached_daylight_max);
+
 	if (ret)
 		return ret;
 
-- 
2.0.0

[PATCH V2 03/18] backlight: adp8870: add blank line after declarations

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: Missing a blank line after declarations

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/adp8870_bl.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/video/backlight/adp8870_bl.c b/drivers/video/backlight/adp8870_bl.c
index 251af4d38d86..037e43083343 100644
--- a/drivers/video/backlight/adp8870_bl.c
+++ b/drivers/video/backlight/adp8870_bl.c
@@ -144,6 +144,7 @@ static int adp8870_read(struct i2c_client *client, int reg, uint8_t *val)
 static int adp8870_write(struct i2c_client *client, u8 reg, u8 val)
 {
 	int ret = i2c_smbus_write_byte_data(client, reg, val);
+
 	if (ret)
 		dev_err(&client->dev, "failed to write\n");
 
@@ -195,6 +196,7 @@ static int adp8870_clr_bits(struct i2c_client *client, int reg, uint8_t bit_mask
 static void adp8870_led_work(struct work_struct *work)
 {
 	struct adp8870_led *led = container_of(work, struct adp8870_led, work);
+
 	adp8870_write(led->client, ADP8870_ISC1 + led->id - 1,
 			 led->new_brightness >> 1);
 }
@@ -399,6 +401,7 @@ static int adp8870_bl_set(struct backlight_device *bl, int brightness)
 static int adp8870_bl_update_status(struct backlight_device *bl)
 {
 	int brightness = bl->props.brightness;
+
 	if (bl->props.power != FB_BLANK_UNBLANK)
 		brightness = 0;
 
@@ -649,6 +652,7 @@ static ssize_t adp8870_bl_l1_daylight_max_store(struct device *dev,
 {
 	struct adp8870_bl *data = dev_get_drvdata(dev);
 	int ret = kstrtoul(buf, 10, &data->cached_daylight_max);
+
 	if (ret)
 		return ret;
 
-- 
2.0.0

[PATCH V2 04/18] backlight: ams369fg06: remove 'else' after a return

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: else is not generally useful after a break or return

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/ams369fg06.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/video/backlight/ams369fg06.c b/drivers/video/backlight/ams369fg06.c
index 4726c8be626f..5f897f99cc9b 100644
--- a/drivers/video/backlight/ams369fg06.c
+++ b/drivers/video/backlight/ams369fg06.c
@@ -325,11 +325,11 @@ static int ams369fg06_power_on(struct ams369fg06 *lcd)
 	if (!pd->reset) {
 		dev_err(lcd->dev, "reset is NULL.\n");
 		return -EINVAL;
-	} else {
-		pd->reset(lcd->ld);
-		msleep(pd->reset_delay);
 	}
 
+	pd->reset(lcd->ld);
+	msleep(pd->reset_delay);
+
 	ret = ams369fg06_ldi_init(lcd);
 	if (ret) {
 		dev_err(lcd->dev, "failed to initialize ldi.\n");
-- 
2.0.0

[PATCH V2 05/18] backlight: corgi_lcd: add blank line after declarations

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: Missing a blank line after declarations

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/corgi_lcd.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/video/backlight/corgi_lcd.c b/drivers/video/backlight/corgi_lcd.c
index 51d18d637e2b..d7c37a8ccd1f 100644
--- a/drivers/video/backlight/corgi_lcd.c
+++ b/drivers/video/backlight/corgi_lcd.c
@@ -143,6 +143,7 @@ static void lcdtg_i2c_send_byte(struct corgi_lcd *lcd,
 				uint8_t base, uint8_t data)
 {
 	int i;
+
 	for (i = 0; i < 8; i++) {
 		if (data & 0x80)
 			lcdtg_i2c_send_bit(lcd, base | POWER0_COM_DOUT);
-- 
2.0.0

[PATCH V2 06/18] backlight: cr_bllcd: add blank line after declarations

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: Missing a blank line after declarations

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/cr_bllcd.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/video/backlight/cr_bllcd.c b/drivers/video/backlight/cr_bllcd.c
index f3fed9ef745f..3e3880fc8c8e 100644
--- a/drivers/video/backlight/cr_bllcd.c
+++ b/drivers/video/backlight/cr_bllcd.c
@@ -235,6 +235,7 @@ static int cr_backlight_probe(struct platform_device *pdev)
 static int cr_backlight_remove(struct platform_device *pdev)
 {
 	struct cr_panel *crp = platform_get_drvdata(pdev);
+
 	crp->cr_backlight_device->props.power = FB_BLANK_POWERDOWN;
 	crp->cr_backlight_device->props.brightness = 0;
 	crp->cr_backlight_device->props.max_brightness = 0;
-- 
2.0.0

[PATCH V2 07/18] backlight: ili922x: remove 'else' after a return

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: else is not generally useful after a break or return

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/ili922x.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/drivers/video/backlight/ili922x.c b/drivers/video/backlight/ili922x.c
index ea67fe199e34..e7f0890cc211 100644
--- a/drivers/video/backlight/ili922x.c
+++ b/drivers/video/backlight/ili922x.c
@@ -495,17 +495,18 @@ static int ili922x_probe(struct spi_device *spi)
 			"no LCD found: Chip ID 0x%x, ret %d\n",
 			reg, ret);
 		return -ENODEV;
-	} else {
-		dev_info(&spi->dev, "ILI%x found, SPI freq %d, mode %d\n",
-			 reg, spi->max_speed_hz, spi->mode);
 	}
 
+	dev_info(&spi->dev, "ILI%x found, SPI freq %d, mode %d\n",
+		 reg, spi->max_speed_hz, spi->mode);
+
 	ret = ili922x_read_status(spi, &reg);
 	if (ret) {
 		dev_err(&spi->dev, "reading RS failed...\n");
 		return ret;
-	} else
-		dev_dbg(&spi->dev, "status: 0x%x\n", reg);
+	}
+
+	dev_dbg(&spi->dev, "status: 0x%x\n", reg);
 
 	ili922x_display_init(spi);
 
-- 
2.0.0

[PATCH V2 08/18] backlight: ld9040: remove 'else' after a return

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: else is not generally useful after a break or return

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/ld9040.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/video/backlight/ld9040.c b/drivers/video/backlight/ld9040.c
index ccb44e8e4927..f71eaf10c4eb 100644
--- a/drivers/video/backlight/ld9040.c
+++ b/drivers/video/backlight/ld9040.c
@@ -566,11 +566,11 @@ static int ld9040_power_on(struct ld9040 *lcd)
 	if (!pd->reset) {
 		dev_err(lcd->dev, "reset is NULL.\n");
 		return -EINVAL;
-	} else {
-		pd->reset(lcd->ld);
-		msleep(pd->reset_delay);
 	}
 
+	pd->reset(lcd->ld);
+	msleep(pd->reset_delay);
+
 	ret = ld9040_ldi_init(lcd);
 	if (ret) {
 		dev_err(lcd->dev, "failed to initialize ldi.\n");
-- 
2.0.0

[PATCH V2 09/18] backlight: lm3639: remove unnecessary return statements

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: void function return statements are not generally useful

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/lm3639_bl.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/video/backlight/lm3639_bl.c b/drivers/video/backlight/lm3639_bl.c
index 5f36808d214f..cd50df5807ea 100644
--- a/drivers/video/backlight/lm3639_bl.c
+++ b/drivers/video/backlight/lm3639_bl.c
@@ -254,7 +254,6 @@ static void lm3639_torch_brightness_set(struct led_classdev *cdev,
 	return;
 out:
 	dev_err(pchip->dev, "i2c failed to access register\n");
-	return;
 }
 
 /* flash */
@@ -293,7 +292,6 @@ static void lm3639_flash_brightness_set(struct led_classdev *cdev,
 	return;
 out:
 	dev_err(pchip->dev, "i2c failed to access register\n");
-	return;
 }
 
 static const struct regmap_config lm3639_regmap = {
-- 
2.0.0

[PATCH V2 10/18] backlight: lms501kf03: remove 'else' after a return

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: else is not generally useful after a break or return

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/lms501kf03.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/video/backlight/lms501kf03.c b/drivers/video/backlight/lms501kf03.c
index 77258b7b04be..7e3810308c3e 100644
--- a/drivers/video/backlight/lms501kf03.c
+++ b/drivers/video/backlight/lms501kf03.c
@@ -232,19 +232,19 @@ static int lms501kf03_power_on(struct lms501kf03 *lcd)
 	if (!pd->power_on) {
 		dev_err(lcd->dev, "power_on is NULL.\n");
 		return -EINVAL;
-	} else {
-		pd->power_on(lcd->ld, 1);
-		msleep(pd->power_on_delay);
 	}
 
+	pd->power_on(lcd->ld, 1);
+	msleep(pd->power_on_delay);
+
 	if (!pd->reset) {
 		dev_err(lcd->dev, "reset is NULL.\n");
 		return -EINVAL;
-	} else {
-		pd->reset(lcd->ld);
-		msleep(pd->reset_delay);
 	}
 
+	pd->reset(lcd->ld);
+	msleep(pd->reset_delay);
+
 	ret = lms501kf03_ldi_init(lcd);
 	if (ret) {
 		dev_err(lcd->dev, "failed to initialize ldi.\n");
-- 
2.0.0

[PATCH V2 11/18] backlight: lp855x: add blank line after declarations

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: Missing a blank line after declarations

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/lp855x_bl.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/video/backlight/lp855x_bl.c b/drivers/video/backlight/lp855x_bl.c
index dcdd5443efcf..25fb8e3d75b1 100644
--- a/drivers/video/backlight/lp855x_bl.c
+++ b/drivers/video/backlight/lp855x_bl.c
@@ -268,6 +268,7 @@ static int lp855x_bl_update_status(struct backlight_device *bl)
 
 	} else if (lp->mode == REGISTER_BASED) {
 		u8 val = bl->props.brightness;
+
 		lp855x_write_byte(lp, lp->cfg->reg_brightness, val);
 	}
 
@@ -308,6 +309,7 @@ static ssize_t lp855x_get_chip_id(struct device *dev,
 				struct device_attribute *attr, char *buf)
 {
 	struct lp855x *lp = dev_get_drvdata(dev);
+
 	return scnprintf(buf, PAGE_SIZE, "%s\n", lp->chipname);
 }
 
-- 
2.0.0

[PATCH V2 12/18] backlight: pcf50633: add blank line after declarations

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: Missing a blank line after declarations

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/pcf50633-backlight.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/video/backlight/pcf50633-backlight.c b/drivers/video/backlight/pcf50633-backlight.c
index b95d3b0aaffe..85bd573b6d15 100644
--- a/drivers/video/backlight/pcf50633-backlight.c
+++ b/drivers/video/backlight/pcf50633-backlight.c
@@ -90,6 +90,7 @@ static int pcf50633_bl_update_status(struct backlight_device *bl)
 static int pcf50633_bl_get_brightness(struct backlight_device *bl)
 {
 	struct pcf50633_bl *pcf_bl = bl_get_data(bl);
+
 	return pcf_bl->brightness;
 }
 
-- 
2.0.0

[PATCH V2 13/18] backlight: s6e63m0: remove 'else' after a return

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: else is not generally useful after a break or return

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/s6e63m0.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/video/backlight/s6e63m0.c b/drivers/video/backlight/s6e63m0.c
index f3a65c8940ed..28bfa127fee4 100644
--- a/drivers/video/backlight/s6e63m0.c
+++ b/drivers/video/backlight/s6e63m0.c
@@ -507,19 +507,19 @@ static int s6e63m0_power_on(struct s6e63m0 *lcd)
 	if (!pd->power_on) {
 		dev_err(lcd->dev, "power_on is NULL.\n");
 		return -EINVAL;
-	} else {
-		pd->power_on(lcd->ld, 1);
-		msleep(pd->power_on_delay);
 	}
 
+	pd->power_on(lcd->ld, 1);
+	msleep(pd->power_on_delay);
+
 	if (!pd->reset) {
 		dev_err(lcd->dev, "reset is NULL.\n");
 		return -EINVAL;
-	} else {
-		pd->reset(lcd->ld);
-		msleep(pd->reset_delay);
 	}
 
+	pd->reset(lcd->ld);
+	msleep(pd->reset_delay);
+
 	ret = s6e63m0_ldi_init(lcd);
 	if (ret) {
 		dev_err(lcd->dev, "failed to initialize ldi.\n");
-- 
2.0.0

[PATCH V2 14/18] backlight: tdo24m: add blank line after declarations

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: Missing a blank line after declarations

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/tdo24m.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/video/backlight/tdo24m.c b/drivers/video/backlight/tdo24m.c
index 908016fc5829..30afce33ef2a 100644
--- a/drivers/video/backlight/tdo24m.c
+++ b/drivers/video/backlight/tdo24m.c
@@ -300,12 +300,14 @@ static int tdo24m_power(struct tdo24m *lcd, int power)
 static int tdo24m_set_power(struct lcd_device *ld, int power)
 {
 	struct tdo24m *lcd = lcd_get_data(ld);
+
 	return tdo24m_power(lcd, power);
 }
 
 static int tdo24m_get_power(struct lcd_device *ld)
 {
 	struct tdo24m *lcd = lcd_get_data(ld);
+
 	return lcd->power;
 }
 
-- 
2.0.0

[PATCH V2 15/18] backlight: wm831x_bl: add blank line after declarations

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: Missing a blank line after declarations

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/wm831x_bl.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/video/backlight/wm831x_bl.c b/drivers/video/backlight/wm831x_bl.c
index 8b9455e93069..07ca98e2a5d4 100644
--- a/drivers/video/backlight/wm831x_bl.c
+++ b/drivers/video/backlight/wm831x_bl.c
@@ -111,6 +111,7 @@ static int wm831x_backlight_update_status(struct backlight_device *bl)
 static int wm831x_backlight_get_brightness(struct backlight_device *bl)
 {
 	struct wm831x_backlight_data *data = bl_get_data(bl);
+
 	return data->current_brightness;
 }
 
-- 
2.0.0

[PATCH V2 16/18] backlight: jornada720: remove 'else' after a return

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: else is not generally useful after a break or return

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/jornada720_bl.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/video/backlight/jornada720_bl.c b/drivers/video/backlight/jornada720_bl.c
index 6ce96b4a8796..7e6ff5346892 100644
--- a/drivers/video/backlight/jornada720_bl.c
+++ b/drivers/video/backlight/jornada720_bl.c
@@ -41,11 +41,11 @@ static int jornada_bl_get_brightness(struct backlight_device *bd)
 		dev_err(&bd->dev, "get brightness timeout\n");
 		jornada_ssp_end();
 		return -ETIMEDOUT;
-	} else {
-		/* exchange txdummy for value */
-		ret = jornada_ssp_byte(TXDUMMY);
 	}
 
+	/* exchange txdummy for value */
+	ret = jornada_ssp_byte(TXDUMMY);
+
 	jornada_ssp_end();
 
 	return BL_MAX_BRIGHT - ret;
-- 
2.0.0

[PATCH V2 17/18] backlight: jornada720: remove 'else' after a return

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: else is not generally useful after a break or return

Suggested-by: Lee Jones <lee.jones@linaro.org>
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/video/backlight/jornada720_lcd.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/drivers/video/backlight/jornada720_lcd.c b/drivers/video/backlight/jornada720_lcd.c
index 228bc319de19..dfa0fa0d5c78 100644
--- a/drivers/video/backlight/jornada720_lcd.c
+++ b/drivers/video/backlight/jornada720_lcd.c
@@ -27,11 +27,7 @@
 
 static int jornada_lcd_get_power(struct lcd_device *ld)
 {
-	/* LDD2 in PPC = LCD POWER */
-	if (PPSR & PPC_LDD2)
-		return FB_BLANK_UNBLANK;	/* PW ON */
-	else
-		return FB_BLANK_POWERDOWN;	/* PW OFF */
+	return PPSR & PPC_LDD2 ? FB_BLANK_UNBLANK : FB_BLANK_POWERDOWN;
 }
 
 static int jornada_lcd_get_contrast(struct lcd_device *ld)
-- 
2.0.0

[PATCH V2 18/18] backlight: omap1: add blank line after declarations

[Jingoo Han]

Fixed the following checkpatch warning.

  WARNING: Missing a blank line after declarations

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/video/backlight/omap1_bl.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/video/backlight/omap1_bl.c b/drivers/video/backlight/omap1_bl.c
index a0dcd88ac74f..546d94df21d5 100644
--- a/drivers/video/backlight/omap1_bl.c
+++ b/drivers/video/backlight/omap1_bl.c
@@ -120,6 +120,7 @@ static int omapbl_update_status(struct backlight_device *dev)
 static int omapbl_get_intensity(struct backlight_device *dev)
 {
 	struct omap_backlight *bl = bl_get_data(dev);
+
 	return bl->current_intensity;
 }
 
-- 
2.0.0

Re: [PATCH V2 00/18] backlight: fix checkpatch warnings

[Lee Jones]

On Wed, 27 Aug 2014, Jingoo Han wrote:
> This patchset fixes checkpatch warnings as follows.
> There is no functional change.
> 
>    WARNING: Missing a blank line after declarations
>    WARNING: else is not generally useful after a break or return
>    WARNING: void function return statements are not generally useful
> 
> Changes for V2
> - Added Lee Jones's Acked-by for 1~18th patches, except for 17th patch.
> - Fixed 17th patch, per Lee Jones's feedback.
>    
> Jingoo Han (18)
>       backlight: adp5520: add blank line after declarations
>       backlight: adp8860: add blank line after declarations
>       backlight: adp8870: add blank line after declarations
>       backlight: ams369fg06: remove 'else' after a return
>       backlight: corgi_lcd: add blank line after declarations
>       backlight: cr_bllcd: add blank line after declarations
>       backlight: ili922x: remove 'else' after a return
>       backlight: ld9040: remove 'else' after a return
>       backlight: lm3639: remove unnecessary return statements
>       backlight: lms501kf03: remove 'else' after a return
>       backlight: lp855x: add blank line after declarations
>       backlight: pcf50633: add blank line after declarations
>       backlight: s6e63m0: remove 'else' after a return
>       backlight: tdo24m: add blank line after declarations
>       backlight: wm831x_bl: add blank line after declarations
>       backlight: jornada720: remove 'else' after a return
>       backlight: jornada720: remove 'else' after a return
>       backlight: omap1: add blank line after declarations

I'll wait to see if Bryan has anything to add.  If not, I'll apply
them in a couple of days.

> ---
>  drivers/video/backlight/adp5520_bl.c         |  1 +
>  drivers/video/backlight/adp8860_bl.c         |  3 +++
>  drivers/video/backlight/adp8870_bl.c         |  4 ++++
>  drivers/video/backlight/ams369fg06.c         |  6 +++---
>  drivers/video/backlight/corgi_lcd.c          |  1 +
>  drivers/video/backlight/cr_bllcd.c           |  1 +
>  drivers/video/backlight/ili922x.c            | 11 ++++++-----
>  drivers/video/backlight/jornada720_bl.c      |  6 +++---
>  drivers/video/backlight/jornada720_lcd.c     |  6 +-----
>  drivers/video/backlight/ld9040.c             |  6 +++---
>  drivers/video/backlight/lm3639_bl.c          |  2 --
>  drivers/video/backlight/lms501kf03.c         | 12 ++++++------
>  drivers/video/backlight/lp855x_bl.c          |  2 ++
>  drivers/video/backlight/omap1_bl.c           |  1 +
>  drivers/video/backlight/pcf50633-backlight.c |  1 +
>  drivers/video/backlight/s6e63m0.c            | 12 ++++++------
>  drivers/video/backlight/tdo24m.c             |  2 ++
>  drivers/video/backlight/wm831x_bl.c          |  1 +
>  18 files changed, 45 insertions(+), 33 deletions(-)
> 
> 

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH V2 00/18] backlight: fix checkpatch warnings

[Lee Jones]

On Wed, 27 Aug 2014, Lee Jones wrote:

> On Wed, 27 Aug 2014, Jingoo Han wrote:
> > This patchset fixes checkpatch warnings as follows.
> > There is no functional change.
> > 
> >    WARNING: Missing a blank line after declarations
> >    WARNING: else is not generally useful after a break or return
> >    WARNING: void function return statements are not generally useful
> > 
> > Changes for V2
> > - Added Lee Jones's Acked-by for 1~18th patches, except for 17th patch.
> > - Fixed 17th patch, per Lee Jones's feedback.
> >    
> > Jingoo Han (18)
> >       backlight: adp5520: add blank line after declarations
> >       backlight: adp8860: add blank line after declarations
> >       backlight: adp8870: add blank line after declarations
> >       backlight: ams369fg06: remove 'else' after a return
> >       backlight: corgi_lcd: add blank line after declarations
> >       backlight: cr_bllcd: add blank line after declarations
> >       backlight: ili922x: remove 'else' after a return
> >       backlight: ld9040: remove 'else' after a return
> >       backlight: lm3639: remove unnecessary return statements
> >       backlight: lms501kf03: remove 'else' after a return
> >       backlight: lp855x: add blank line after declarations
> >       backlight: pcf50633: add blank line after declarations
> >       backlight: s6e63m0: remove 'else' after a return
> >       backlight: tdo24m: add blank line after declarations
> >       backlight: wm831x_bl: add blank line after declarations
> >       backlight: jornada720: remove 'else' after a return
> >       backlight: jornada720: remove 'else' after a return
> >       backlight: omap1: add blank line after declarations
> 
> I'll wait to see if Bryan has anything to add.  If not, I'll apply
> them in a couple of days.

All patches applied and pushed.

> > ---
> >  drivers/video/backlight/adp5520_bl.c         |  1 +
> >  drivers/video/backlight/adp8860_bl.c         |  3 +++
> >  drivers/video/backlight/adp8870_bl.c         |  4 ++++
> >  drivers/video/backlight/ams369fg06.c         |  6 +++---
> >  drivers/video/backlight/corgi_lcd.c          |  1 +
> >  drivers/video/backlight/cr_bllcd.c           |  1 +
> >  drivers/video/backlight/ili922x.c            | 11 ++++++-----
> >  drivers/video/backlight/jornada720_bl.c      |  6 +++---
> >  drivers/video/backlight/jornada720_lcd.c     |  6 +-----
> >  drivers/video/backlight/ld9040.c             |  6 +++---
> >  drivers/video/backlight/lm3639_bl.c          |  2 --
> >  drivers/video/backlight/lms501kf03.c         | 12 ++++++------
> >  drivers/video/backlight/lp855x_bl.c          |  2 ++
> >  drivers/video/backlight/omap1_bl.c           |  1 +
> >  drivers/video/backlight/pcf50633-backlight.c |  1 +
> >  drivers/video/backlight/s6e63m0.c            | 12 ++++++------
> >  drivers/video/backlight/tdo24m.c             |  2 ++
> >  drivers/video/backlight/wm831x_bl.c          |  1 +
> >  18 files changed, 45 insertions(+), 33 deletions(-)
> > 
> > 
> 

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog