diff options
author | Jingoo Han <jg1.han@samsung.com> | 2013-04-29 19:17:31 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-04-29 21:28:15 -0400 |
commit | b2dcd7be0b4cd1d226d014cbafea209f41cc70d2 (patch) | |
tree | 381270fdcca814183b055d7d1ea7476570526d45 /drivers/video/backlight | |
parent | 970354281507ec8055767c7063d26109fd80d2b5 (diff) |
drivers/video/backlight/jornada720_*.c: use dev_err()/dev_info() instead of pr_err()/pr_info()
dev_err()/dev_info() are preferred to pr_err()/pr_info().
Signed-off-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/jornada720_bl.c | 18 | ||||
-rw-r--r-- | drivers/video/backlight/jornada720_lcd.c | 21 |
2 files changed, 19 insertions, 20 deletions
diff --git a/drivers/video/backlight/jornada720_bl.c b/drivers/video/backlight/jornada720_bl.c index fef6ce4fad71..3ccb89340f22 100644 --- a/drivers/video/backlight/jornada720_bl.c +++ b/drivers/video/backlight/jornada720_bl.c | |||
@@ -9,8 +9,6 @@ | |||
9 | * | 9 | * |
10 | */ | 10 | */ |
11 | 11 | ||
12 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt | ||
13 | |||
14 | #include <linux/backlight.h> | 12 | #include <linux/backlight.h> |
15 | #include <linux/device.h> | 13 | #include <linux/device.h> |
16 | #include <linux/fb.h> | 14 | #include <linux/fb.h> |
@@ -40,11 +38,13 @@ static int jornada_bl_get_brightness(struct backlight_device *bd) | |||
40 | ret = jornada_ssp_byte(GETBRIGHTNESS); | 38 | ret = jornada_ssp_byte(GETBRIGHTNESS); |
41 | 39 | ||
42 | if (jornada_ssp_byte(GETBRIGHTNESS) != TXDUMMY) { | 40 | if (jornada_ssp_byte(GETBRIGHTNESS) != TXDUMMY) { |
43 | pr_err("get brightness timeout\n"); | 41 | dev_err(&bd->dev, "get brightness timeout\n"); |
44 | jornada_ssp_end(); | 42 | jornada_ssp_end(); |
45 | return -ETIMEDOUT; | 43 | return -ETIMEDOUT; |
46 | } else /* exchange txdummy for value */ | 44 | } else { |
45 | /* exchange txdummy for value */ | ||
47 | ret = jornada_ssp_byte(TXDUMMY); | 46 | ret = jornada_ssp_byte(TXDUMMY); |
47 | } | ||
48 | 48 | ||
49 | jornada_ssp_end(); | 49 | jornada_ssp_end(); |
50 | 50 | ||
@@ -61,7 +61,7 @@ static int jornada_bl_update_status(struct backlight_device *bd) | |||
61 | if ((bd->props.power != FB_BLANK_UNBLANK) || (bd->props.fb_blank != FB_BLANK_UNBLANK)) { | 61 | if ((bd->props.power != FB_BLANK_UNBLANK) || (bd->props.fb_blank != FB_BLANK_UNBLANK)) { |
62 | ret = jornada_ssp_byte(BRIGHTNESSOFF); | 62 | ret = jornada_ssp_byte(BRIGHTNESSOFF); |
63 | if (ret != TXDUMMY) { | 63 | if (ret != TXDUMMY) { |
64 | pr_info("brightness off timeout\n"); | 64 | dev_info(&bd->dev, "brightness off timeout\n"); |
65 | /* turn off backlight */ | 65 | /* turn off backlight */ |
66 | PPSR &= ~PPC_LDD1; | 66 | PPSR &= ~PPC_LDD1; |
67 | PPDR |= PPC_LDD1; | 67 | PPDR |= PPC_LDD1; |
@@ -72,7 +72,7 @@ static int jornada_bl_update_status(struct backlight_device *bd) | |||
72 | 72 | ||
73 | /* send command to our mcu */ | 73 | /* send command to our mcu */ |
74 | if (jornada_ssp_byte(SETBRIGHTNESS) != TXDUMMY) { | 74 | if (jornada_ssp_byte(SETBRIGHTNESS) != TXDUMMY) { |
75 | pr_info("failed to set brightness\n"); | 75 | dev_info(&bd->dev, "failed to set brightness\n"); |
76 | ret = -ETIMEDOUT; | 76 | ret = -ETIMEDOUT; |
77 | goto out; | 77 | goto out; |
78 | } | 78 | } |
@@ -86,7 +86,7 @@ static int jornada_bl_update_status(struct backlight_device *bd) | |||
86 | */ | 86 | */ |
87 | if (jornada_ssp_byte(BL_MAX_BRIGHT - bd->props.brightness) | 87 | if (jornada_ssp_byte(BL_MAX_BRIGHT - bd->props.brightness) |
88 | != TXDUMMY) { | 88 | != TXDUMMY) { |
89 | pr_err("set brightness failed\n"); | 89 | dev_err(&bd->dev, "set brightness failed\n"); |
90 | ret = -ETIMEDOUT; | 90 | ret = -ETIMEDOUT; |
91 | } | 91 | } |
92 | 92 | ||
@@ -120,7 +120,7 @@ static int jornada_bl_probe(struct platform_device *pdev) | |||
120 | 120 | ||
121 | if (IS_ERR(bd)) { | 121 | if (IS_ERR(bd)) { |
122 | ret = PTR_ERR(bd); | 122 | ret = PTR_ERR(bd); |
123 | pr_err("failed to register device, err=%x\n", ret); | 123 | dev_err(&pdev->dev, "failed to register device, err=%x\n", ret); |
124 | return ret; | 124 | return ret; |
125 | } | 125 | } |
126 | 126 | ||
@@ -134,7 +134,7 @@ static int jornada_bl_probe(struct platform_device *pdev) | |||
134 | jornada_bl_update_status(bd); | 134 | jornada_bl_update_status(bd); |
135 | 135 | ||
136 | platform_set_drvdata(pdev, bd); | 136 | platform_set_drvdata(pdev, bd); |
137 | pr_info("HP Jornada 700 series backlight driver\n"); | 137 | dev_info(&pdev->dev, "HP Jornada 700 series backlight driver\n"); |
138 | 138 | ||
139 | return 0; | 139 | return 0; |
140 | } | 140 | } |
diff --git a/drivers/video/backlight/jornada720_lcd.c b/drivers/video/backlight/jornada720_lcd.c index 635b30523fd5..b061413f1a65 100644 --- a/drivers/video/backlight/jornada720_lcd.c +++ b/drivers/video/backlight/jornada720_lcd.c | |||
@@ -9,8 +9,6 @@ | |||
9 | * | 9 | * |
10 | */ | 10 | */ |
11 | 11 | ||
12 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt | ||
13 | |||
14 | #include <linux/device.h> | 12 | #include <linux/device.h> |
15 | #include <linux/fb.h> | 13 | #include <linux/fb.h> |
16 | #include <linux/kernel.h> | 14 | #include <linux/kernel.h> |
@@ -27,7 +25,7 @@ | |||
27 | #define LCD_MAX_CONTRAST 0xff | 25 | #define LCD_MAX_CONTRAST 0xff |
28 | #define LCD_DEF_CONTRAST 0x80 | 26 | #define LCD_DEF_CONTRAST 0x80 |
29 | 27 | ||
30 | static int jornada_lcd_get_power(struct lcd_device *dev) | 28 | static int jornada_lcd_get_power(struct lcd_device *ld) |
31 | { | 29 | { |
32 | /* LDD2 in PPC = LCD POWER */ | 30 | /* LDD2 in PPC = LCD POWER */ |
33 | if (PPSR & PPC_LDD2) | 31 | if (PPSR & PPC_LDD2) |
@@ -36,17 +34,17 @@ static int jornada_lcd_get_power(struct lcd_device *dev) | |||
36 | return FB_BLANK_POWERDOWN; /* PW OFF */ | 34 | return FB_BLANK_POWERDOWN; /* PW OFF */ |
37 | } | 35 | } |
38 | 36 | ||
39 | static int jornada_lcd_get_contrast(struct lcd_device *dev) | 37 | static int jornada_lcd_get_contrast(struct lcd_device *ld) |
40 | { | 38 | { |
41 | int ret; | 39 | int ret; |
42 | 40 | ||
43 | if (jornada_lcd_get_power(dev) != FB_BLANK_UNBLANK) | 41 | if (jornada_lcd_get_power(ld) != FB_BLANK_UNBLANK) |
44 | return 0; | 42 | return 0; |
45 | 43 | ||
46 | jornada_ssp_start(); | 44 | jornada_ssp_start(); |
47 | 45 | ||
48 | if (jornada_ssp_byte(GETCONTRAST) != TXDUMMY) { | 46 | if (jornada_ssp_byte(GETCONTRAST) != TXDUMMY) { |
49 | pr_err("get contrast failed\n"); | 47 | dev_err(&ld->dev, "get contrast failed\n"); |
50 | jornada_ssp_end(); | 48 | jornada_ssp_end(); |
51 | return -ETIMEDOUT; | 49 | return -ETIMEDOUT; |
52 | } else { | 50 | } else { |
@@ -56,7 +54,7 @@ static int jornada_lcd_get_contrast(struct lcd_device *dev) | |||
56 | } | 54 | } |
57 | } | 55 | } |
58 | 56 | ||
59 | static int jornada_lcd_set_contrast(struct lcd_device *dev, int value) | 57 | static int jornada_lcd_set_contrast(struct lcd_device *ld, int value) |
60 | { | 58 | { |
61 | int ret; | 59 | int ret; |
62 | 60 | ||
@@ -67,7 +65,7 @@ static int jornada_lcd_set_contrast(struct lcd_device *dev, int value) | |||
67 | 65 | ||
68 | /* push the new value */ | 66 | /* push the new value */ |
69 | if (jornada_ssp_byte(value) != TXDUMMY) { | 67 | if (jornada_ssp_byte(value) != TXDUMMY) { |
70 | pr_err("set contrast failed\n"); | 68 | dev_err(&ld->dev, "set contrast failed\n"); |
71 | jornada_ssp_end(); | 69 | jornada_ssp_end(); |
72 | return -ETIMEDOUT; | 70 | return -ETIMEDOUT; |
73 | } | 71 | } |
@@ -78,13 +76,14 @@ static int jornada_lcd_set_contrast(struct lcd_device *dev, int value) | |||
78 | return 0; | 76 | return 0; |
79 | } | 77 | } |
80 | 78 | ||
81 | static int jornada_lcd_set_power(struct lcd_device *dev, int power) | 79 | static int jornada_lcd_set_power(struct lcd_device *ld, int power) |
82 | { | 80 | { |
83 | if (power != FB_BLANK_UNBLANK) { | 81 | if (power != FB_BLANK_UNBLANK) { |
84 | PPSR &= ~PPC_LDD2; | 82 | PPSR &= ~PPC_LDD2; |
85 | PPDR |= PPC_LDD2; | 83 | PPDR |= PPC_LDD2; |
86 | } else | 84 | } else { |
87 | PPSR |= PPC_LDD2; | 85 | PPSR |= PPC_LDD2; |
86 | } | ||
88 | 87 | ||
89 | return 0; | 88 | return 0; |
90 | } | 89 | } |
@@ -105,7 +104,7 @@ static int jornada_lcd_probe(struct platform_device *pdev) | |||
105 | 104 | ||
106 | if (IS_ERR(lcd_device)) { | 105 | if (IS_ERR(lcd_device)) { |
107 | ret = PTR_ERR(lcd_device); | 106 | ret = PTR_ERR(lcd_device); |
108 | pr_err("failed to register device\n"); | 107 | dev_err(&pdev->dev, "failed to register device\n"); |
109 | return ret; | 108 | return ret; |
110 | } | 109 | } |
111 | 110 | ||