OMAPDSS: SDI: change regulator handling
authorTomi Valkeinen <tomi.valkeinen@ti.com>
Thu, 29 Aug 2013 07:06:43 +0000 (10:06 +0300)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Thu, 29 Aug 2013 13:21:52 +0000 (16:21 +0300)
Regulator handling for DPI and SDI is currently handled in the core.c,
using the 'virtual' omapdss platform device. Nowadays we have proper
devices for both DPI and SDI, and so we can handle the regulators inside
the respective drivers.

This patch moves the regulator handling for SDI into sdi.c.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Reviewed-by: Archit Taneja <archit@ti.com>
drivers/video/omap2/dss/core.c
drivers/video/omap2/dss/dss.h
drivers/video/omap2/dss/sdi.c

index 1aeb274e30fc2728e611a8ee4033e88f23d77972..a07d626440264bbc87babe38f4876fc64d29e337 100644 (file)
@@ -44,7 +44,6 @@ static struct {
        struct platform_device *pdev;
 
        struct regulator *vdds_dsi_reg;
-       struct regulator *vdds_sdi_reg;
 
        const char *default_display_name;
 } core;
@@ -95,20 +94,6 @@ struct regulator *dss_get_vdds_dsi(void)
        return reg;
 }
 
-struct regulator *dss_get_vdds_sdi(void)
-{
-       struct regulator *reg;
-
-       if (core.vdds_sdi_reg != NULL)
-               return core.vdds_sdi_reg;
-
-       reg = devm_regulator_get(&core.pdev->dev, "vdds_sdi");
-       if (!IS_ERR(reg))
-               core.vdds_sdi_reg = reg;
-
-       return reg;
-}
-
 int dss_dsi_enable_pads(int dsi_id, unsigned lane_mask)
 {
        struct omap_dss_board_info *board_data = core.pdev->dev.platform_data;
index 50a2362ef8f882cba159af3ad0a1c6defd6a820b..7d97de46b17f01cf671ba3c9667785fd563199a7 100644 (file)
@@ -165,7 +165,6 @@ struct platform_device;
 struct platform_device *dss_get_core_pdev(void);
 struct bus_type *dss_get_bus(void);
 struct regulator *dss_get_vdds_dsi(void);
-struct regulator *dss_get_vdds_sdi(void);
 int dss_dsi_enable_pads(int dsi_id, unsigned lane_mask);
 void dss_dsi_disable_pads(int dsi_id, unsigned lane_mask);
 int dss_set_min_bus_tput(struct device *dev, unsigned long tput);
index 856af2e89760937d01d3459a1bd879ca84b6d4cd..82b64e3fbd0863f68bc61ad3bcc8b46cfce33943 100644 (file)
@@ -267,14 +267,10 @@ static int sdi_init_regulator(void)
        if (sdi.vdds_sdi_reg)
                return 0;
 
-       vdds_sdi = dss_get_vdds_sdi();
-
+       vdds_sdi = devm_regulator_get(&sdi.pdev->dev, "vdds_sdi");
        if (IS_ERR(vdds_sdi)) {
-               vdds_sdi = devm_regulator_get(&sdi.pdev->dev, "vdds_sdi");
-               if (IS_ERR(vdds_sdi)) {
-                       DSSERR("can't get VDDS_SDI regulator\n");
-                       return PTR_ERR(vdds_sdi);
-               }
+               DSSERR("can't get VDDS_SDI regulator\n");
+               return PTR_ERR(vdds_sdi);
        }
 
        sdi.vdds_sdi_reg = vdds_sdi;