Thread (8 messages) flat view 8 messages, 4 authors, 2011-12-02

backlight/ld9040.c: regulator control in the lcd driver

From: Linus Walleij <hidden>
Date: 2011-12-02 08:49:16
Also in: linux-fbdev

On Fri, Dec 2, 2011 at 8:45 AM, leedonghwa [off-list ref] wrote:

[From your mail headers]
X-Mailer: Microsoft Office Outlook 12.0
No please. That mailer does not work, all your whitespace is screwed up.
Consult: Documentation/email-clients.txt
This patch supports regulator power control in the driver.
Always CC the regulator maintainers on patches like this please.
+???????? lcd->reg_vdd3 = regulator_get(lcd->dev, "vdd");
+???????? if (IS_ERR(lcd->reg_vdd3)) {
+?????????????????? dev_info(lcd->dev, "no %s regulator found\n", "vdd");
+?????????????????? lcd->reg_vdd3 = NULL;
+???????? }
+
+???????? lcd->reg_vci = regulator_get(lcd->dev, "vci");
+???????? if (IS_ERR(lcd->reg_vci)) {
+?????????????????? dev_info(lcd->dev, "no %s regulator found\n", "vci");
+?????????????????? lcd->reg_vci = NULL;
+???????? }
As explained in earlier discussion with Mark regarding the SMSC911x
driver regulator, treat these as errors and do not fail
"gracefully" like this.

Reference:
http://marc.info/?l=linux-netdev&m=131914562120725&w=2

Yours,
Linus Walleij
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help