diff options
author | Sachin Kamat <sachin.kamat@linaro.org> | 2012-11-25 08:30:44 +0100 |
---|---|---|
committer | Bryan Wu <cooloney@gmail.com> | 2012-11-27 23:49:30 +0100 |
commit | 09f5fe756329fd7fb6ccb94563960d6a22eaaab2 (patch) | |
tree | 79ec40078784e74d99f6f77a121969572c6b9811 /drivers/leds | |
parent | leds: ledtrig-backlight: Fix checkpatch error (diff) | |
download | linux-09f5fe756329fd7fb6ccb94563960d6a22eaaab2.tar.xz linux-09f5fe756329fd7fb6ccb94563960d6a22eaaab2.zip |
leds: led-triggers: Fix checkpatch warnings
Fixes the following type of warnings:
WARNING: Prefer netdev_warn(netdev, ... then dev_warn(dev, ...
then pr_warn(... to printk(KERN_WARNING ...
WARNING: quoted string split across lines
While at it also fix a coding style issue w.r.t braces.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Bryan Wu <cooloney@gmail.com>
Diffstat (limited to 'drivers/leds')
-rw-r--r-- | drivers/leds/led-triggers.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/drivers/leds/led-triggers.c b/drivers/leds/led-triggers.c index a4fa4bf02d53..3c972b2f9893 100644 --- a/drivers/leds/led-triggers.c +++ b/drivers/leds/led-triggers.c @@ -313,13 +313,13 @@ void led_trigger_register_simple(const char *name, struct led_trigger **tp) if (err < 0) { kfree(trig); trig = NULL; - printk(KERN_WARNING "LED trigger %s failed to register" - " (%d)\n", name, err); + pr_warn("LED trigger %s failed to register (%d)\n", + name, err); } - } else - printk(KERN_WARNING "LED trigger %s failed to register" - " (no memory)\n", name); - + } else { + pr_warn("LED trigger %s failed to register (no memory)\n", + name); + } *tp = trig; } EXPORT_SYMBOL_GPL(led_trigger_register_simple); |