aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/video/backlight
diff options
context:
space:
mode:
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>2013-04-29 19:17:54 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2013-04-29 21:28:17 -0400
commit514f920a4a37bce427ec4aab591619b158e29115 (patch)
tree720af9879f56490f407ae057712acaba16ffca58 /drivers/video/backlight
parent84eba9ed709198cda09c14a16c499a45d88a63ea (diff)
drivers/video/backlight/adp8870_bl.c: fix error return code in adp8870_led_probe()
Fix to return a negative error code from the error handling case instead of zero, as returned elsewhere in this function. Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Acked-by: Jingoo Han <jg1.han@samsung.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/video/backlight')
-rw-r--r--drivers/video/backlight/adp8870_bl.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/video/backlight/adp8870_bl.c b/drivers/video/backlight/adp8870_bl.c
index 302c8009a7dc..90049d7b5c60 100644
--- a/drivers/video/backlight/adp8870_bl.c
+++ b/drivers/video/backlight/adp8870_bl.c
@@ -274,12 +274,14 @@ static int adp8870_led_probe(struct i2c_client *client)
274 if (led_dat->id > 7 || led_dat->id < 1) { 274 if (led_dat->id > 7 || led_dat->id < 1) {
275 dev_err(&client->dev, "Invalid LED ID %d\n", 275 dev_err(&client->dev, "Invalid LED ID %d\n",
276 led_dat->id); 276 led_dat->id);
277 ret = -EINVAL;
277 goto err; 278 goto err;
278 } 279 }
279 280
280 if (pdata->bl_led_assign & (1 << (led_dat->id - 1))) { 281 if (pdata->bl_led_assign & (1 << (led_dat->id - 1))) {
281 dev_err(&client->dev, "LED %d used by Backlight\n", 282 dev_err(&client->dev, "LED %d used by Backlight\n",
282 led_dat->id); 283 led_dat->id);
284 ret = -EBUSY;
283 goto err; 285 goto err;
284 } 286 }
285 287