backlight: fix broken regulator API usage in l4f00242t03
authorMark Brown <broonie@opensource.wolfsonmicro.com>
Tue, 1 Nov 2011 00:11:48 +0000 (17:11 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 1 Nov 2011 00:30:54 +0000 (17:30 -0700)
The regulator support in the l4f00242t03 is very non-idiomatic.  Rather
than requesting the regulators based on the device name and the supply
names used by the device the driver requires boards to pass system
specific supply names around through platform data.  The driver also
conditionally requests the regulators based on this platform data, adding
unneeded conditional code to the driver.

Fix this by removing the platform data and converting to the standard
idiom, also updating all in tree users of the driver.  As no datasheet
appears to be available for the LCD I'm guessing the names for the
supplies based on the existing users and I've no ability to do anything
more than compile test.

The use of regulator_set_voltage() in the driver is also problematic,
since fixed voltages are required the expectation would be that the
voltages would be fixed in the constraints set by the machines rather than
manually configured by the driver, but is less problematic.

Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Tested-by: Fabio Estevam <fabio.estevam@freescale.com>
Cc: Richard Purdie <rpurdie@rpsys.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/arm/mach-imx/mach-mx27_3ds.c
arch/arm/mach-imx/mach-mx31_3ds.c
drivers/video/backlight/l4f00242t03.c
include/linux/spi/l4f00242t03.h

index 2eafbac2c76356dad02db7bb13a08ec64f1b5726..377230497dcc5007166e89238de1024aabbc2b2f 100644 (file)
@@ -241,7 +241,7 @@ static struct regulator_init_data gpo_init = {
 };
 
 static struct regulator_consumer_supply vmmc1_consumers[] = {
-       REGULATOR_SUPPLY("lcd_2v8", NULL),
+       REGULATOR_SUPPLY("vcore", "spi0.0"),
 };
 
 static struct regulator_init_data vmmc1_init = {
@@ -257,7 +257,7 @@ static struct regulator_init_data vmmc1_init = {
 };
 
 static struct regulator_consumer_supply vgen_consumers[] = {
-       REGULATOR_SUPPLY("vdd_lcdio", NULL),
+       REGULATOR_SUPPLY("vdd", "spi0.0"),
 };
 
 static struct regulator_init_data vgen_init = {
@@ -348,8 +348,6 @@ static const struct imx_fb_platform_data mx27_3ds_fb_data __initconst = {
 static struct l4f00242t03_pdata mx27_3ds_lcd_pdata = {
        .reset_gpio             = LCD_RESET,
        .data_enable_gpio       = LCD_ENABLE,
-       .core_supply            = "lcd_2v8",
-       .io_supply              = "vdd_lcdio",
 };
 
 static struct spi_board_info mx27_3ds_spi_devs[] __initdata = {
index 589066fb3316ae794dc84ff98e67e37c1a716232..6484db525bd7934be102246e322dbb4af84a4baa 100644 (file)
@@ -285,8 +285,6 @@ static struct mx3fb_platform_data mx3fb_pdata __initdata = {
 static struct l4f00242t03_pdata mx31_3ds_l4f00242t03_pdata = {
        .reset_gpio             = IOMUX_TO_GPIO(MX31_PIN_LCS1),
        .data_enable_gpio       = IOMUX_TO_GPIO(MX31_PIN_SER_RS),
-       .core_supply            = "lcd_2v8",
-       .io_supply              = "vdd_lcdio",
 };
 
 /*
@@ -411,7 +409,7 @@ static struct regulator_init_data vmmc2_init = {
 };
 
 static struct regulator_consumer_supply vmmc1_consumers[] = {
-       REGULATOR_SUPPLY("lcd_2v8", NULL),
+       REGULATOR_SUPPLY("vcore", "spi0.0"),
        REGULATOR_SUPPLY("cmos_2v8", "soc-camera-pdrv.0"),
 };
 
@@ -428,7 +426,7 @@ static struct regulator_init_data vmmc1_init = {
 };
 
 static struct regulator_consumer_supply vgen_consumers[] = {
-       REGULATOR_SUPPLY("vdd_lcdio", NULL),
+       REGULATOR_SUPPLY("vdd", "spi0.0"),
 };
 
 static struct regulator_init_data vgen_init = {
index 98ad3e5f7c853cead7388f09e725e96c80a5e8ef..77ec70bcb3034069e1a8df15617340c57a49010e 100644 (file)
@@ -52,15 +52,11 @@ static void l4f00242t03_lcd_init(struct spi_device *spi)
 
        dev_dbg(&spi->dev, "initializing LCD\n");
 
-       if (priv->io_reg) {
-               regulator_set_voltage(priv->io_reg, 1800000, 1800000);
-               regulator_enable(priv->io_reg);
-       }
+       regulator_set_voltage(priv->io_reg, 1800000, 1800000);
+       regulator_enable(priv->io_reg);
 
-       if (priv->core_reg) {
-               regulator_set_voltage(priv->core_reg, 2800000, 2800000);
-               regulator_enable(priv->core_reg);
-       }
+       regulator_set_voltage(priv->core_reg, 2800000, 2800000);
+       regulator_enable(priv->core_reg);
 
        l4f00242t03_reset(pdata->reset_gpio);
 
@@ -78,11 +74,8 @@ static void l4f00242t03_lcd_powerdown(struct spi_device *spi)
 
        gpio_set_value(pdata->data_enable_gpio, 0);
 
-       if (priv->io_reg)
-               regulator_disable(priv->io_reg);
-
-       if (priv->core_reg)
-               regulator_disable(priv->core_reg);
+       regulator_disable(priv->io_reg);
+       regulator_disable(priv->core_reg);
 }
 
 static int l4f00242t03_lcd_power_get(struct lcd_device *ld)
@@ -201,24 +194,18 @@ static int __devinit l4f00242t03_probe(struct spi_device *spi)
        if (ret)
                goto err3;
 
-       if (pdata->io_supply) {
-               priv->io_reg = regulator_get(NULL, pdata->io_supply);
-
-               if (IS_ERR(priv->io_reg)) {
-                       pr_err("%s: Unable to get the IO regulator\n",
-                                                               __func__);
-                       goto err3;
-               }
+       priv->io_reg = regulator_get(&spi->dev, "vdd");
+       if (IS_ERR(priv->io_reg)) {
+               dev_err(&spi->dev, "%s: Unable to get the IO regulator\n",
+                      __func__);
+               goto err3;
        }
 
-       if (pdata->core_supply) {
-               priv->core_reg = regulator_get(NULL, pdata->core_supply);
-
-               if (IS_ERR(priv->core_reg)) {
-                       pr_err("%s: Unable to get the core regulator\n",
-                                                               __func__);
-                       goto err4;
-               }
+       priv->core_reg = regulator_get(&spi->dev, "vcore");
+       if (IS_ERR(priv->core_reg)) {
+               dev_err(&spi->dev, "%s: Unable to get the core regulator\n",
+                      __func__);
+               goto err4;
        }
 
        priv->ld = lcd_device_register("l4f00242t03",
@@ -238,11 +225,9 @@ static int __devinit l4f00242t03_probe(struct spi_device *spi)
        return 0;
 
 err5:
-       if (priv->core_reg)
-               regulator_put(priv->core_reg);
+       regulator_put(priv->core_reg);
 err4:
-       if (priv->io_reg)
-               regulator_put(priv->io_reg);
+       regulator_put(priv->io_reg);
 err3:
        gpio_free(pdata->data_enable_gpio);
 err2:
@@ -266,10 +251,8 @@ static int __devexit l4f00242t03_remove(struct spi_device *spi)
        gpio_free(pdata->data_enable_gpio);
        gpio_free(pdata->reset_gpio);
 
-       if (priv->io_reg)
-               regulator_put(priv->io_reg);
-       if (priv->core_reg)
-               regulator_put(priv->core_reg);
+       regulator_put(priv->io_reg);
+       regulator_put(priv->core_reg);
 
        kfree(priv);
 
index aee1dbda4edc2b4a8a53134c0fdc24575d1e65aa..bc8677c8eba92c2ed22e23a1089d19486062a512 100644 (file)
@@ -24,8 +24,6 @@
 struct l4f00242t03_pdata {
        unsigned int    reset_gpio;
        unsigned int    data_enable_gpio;
-       const char      *io_supply;     /* will be set to 1.8 V */
-       const char      *core_supply;   /* will be set to 2.8 V */
 };
 
 #endif /* _INCLUDE_LINUX_SPI_L4F00242T03_H_ */