[PATCH 16/36] mtd: onenand: omap: Remove regulator management code

2014-06-11 Thread Roger Quadros
None of the OMAP platforms are suppying the "regulator_can_sleep"
parameter via platform data. Regulator management is generic
enough to be done in onenand_base driver if required.

Mark the "regulator_can_sleep" platform data parameter as deprecated.

Signed-off-by: Roger Quadros 
---
 drivers/mtd/onenand/omap2.c | 42 +
 include/linux/platform_data/mtd-onenand-omap2.h |  3 +-
 2 files changed, 3 insertions(+), 42 deletions(-)

diff --git a/drivers/mtd/onenand/omap2.c b/drivers/mtd/onenand/omap2.c
index d945473..a5f5ad8 100644
--- a/drivers/mtd/onenand/omap2.c
+++ b/drivers/mtd/onenand/omap2.c
@@ -34,7 +34,6 @@
 #include 
 #include 
 #include 
-#include 
 
 #include 
 #include 
@@ -59,7 +58,6 @@ struct omap2_onenand {
int dma_channel;
int freq;
int (*setup)(void __iomem *base, int *freq_ptr);
-   struct regulator *regulator;
u8 flags;
 };
 
@@ -583,30 +581,6 @@ static void omap2_onenand_shutdown(struct platform_device 
*pdev)
memset((__force void *)c->onenand.base, 0, ONENAND_BUFRAM_SIZE);
 }
 
-static int omap2_onenand_enable(struct mtd_info *mtd)
-{
-   int ret;
-   struct omap2_onenand *c = container_of(mtd, struct omap2_onenand, mtd);
-
-   ret = regulator_enable(c->regulator);
-   if (ret != 0)
-   dev_err(>pdev->dev, "can't enable regulator\n");
-
-   return ret;
-}
-
-static int omap2_onenand_disable(struct mtd_info *mtd)
-{
-   int ret;
-   struct omap2_onenand *c = container_of(mtd, struct omap2_onenand, mtd);
-
-   ret = regulator_disable(c->regulator);
-   if (ret != 0)
-   dev_err(>pdev->dev, "can't disable regulator\n");
-
-   return ret;
-}
-
 static int omap2_onenand_probe(struct platform_device *pdev)
 {
struct omap_onenand_platform_data *pdata;
@@ -728,22 +702,11 @@ static int omap2_onenand_probe(struct platform_device 
*pdev)
}
}
 
-   if (pdata->regulator_can_sleep) {
-   c->regulator = regulator_get(>dev, "vonenand");
-   if (IS_ERR(c->regulator)) {
-   dev_err(>dev,  "Failed to get regulator\n");
-   r = PTR_ERR(c->regulator);
-   goto err_release_dma;
-   }
-   c->onenand.enable = omap2_onenand_enable;
-   c->onenand.disable = omap2_onenand_disable;
-   }
-
if (pdata->skip_initial_unlocking)
this->options |= ONENAND_SKIP_INITIAL_UNLOCKING;
 
if ((r = onenand_scan(>mtd, 1)) < 0)
-   goto err_release_regulator;
+   goto err_release_dma;
 
ppdata.of_node = pdata->of_node;
r = mtd_device_parse_register(>mtd, NULL, ,
@@ -758,8 +721,6 @@ static int omap2_onenand_probe(struct platform_device *pdev)
 
 err_release_onenand:
onenand_release(>mtd);
-err_release_regulator:
-   regulator_put(c->regulator);
 err_release_dma:
if (c->dma_channel != -1)
omap_free_dma(c->dma_channel);
@@ -783,7 +744,6 @@ static int omap2_onenand_remove(struct platform_device 
*pdev)
struct omap2_onenand *c = dev_get_drvdata(>dev);
 
onenand_release(>mtd);
-   regulator_put(c->regulator);
if (c->dma_channel != -1)
omap_free_dma(c->dma_channel);
omap2_onenand_shutdown(pdev);
diff --git a/include/linux/platform_data/mtd-onenand-omap2.h 
b/include/linux/platform_data/mtd-onenand-omap2.h
index 56ff0e6..445b41e 100644
--- a/include/linux/platform_data/mtd-onenand-omap2.h
+++ b/include/linux/platform_data/mtd-onenand-omap2.h
@@ -25,10 +25,11 @@ struct omap_onenand_platform_data {
int (*onenand_setup)(void __iomem *, int *freq_ptr);
int dma_channel;
u8  flags;
-   u8  regulator_can_sleep;
u8  skip_initial_unlocking;
 
/* for passing the partitions */
struct device_node  *of_node;
+
+   u8  regulator_can_sleep;/* deprecated */
 };
 #endif
-- 
1.8.3.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 16/36] mtd: onenand: omap: Remove regulator management code

2014-06-11 Thread Roger Quadros
None of the OMAP platforms are suppying the regulator_can_sleep
parameter via platform data. Regulator management is generic
enough to be done in onenand_base driver if required.

Mark the regulator_can_sleep platform data parameter as deprecated.

Signed-off-by: Roger Quadros rog...@ti.com
---
 drivers/mtd/onenand/omap2.c | 42 +
 include/linux/platform_data/mtd-onenand-omap2.h |  3 +-
 2 files changed, 3 insertions(+), 42 deletions(-)

diff --git a/drivers/mtd/onenand/omap2.c b/drivers/mtd/onenand/omap2.c
index d945473..a5f5ad8 100644
--- a/drivers/mtd/onenand/omap2.c
+++ b/drivers/mtd/onenand/omap2.c
@@ -34,7 +34,6 @@
 #include linux/dma-mapping.h
 #include linux/io.h
 #include linux/slab.h
-#include linux/regulator/consumer.h
 
 #include asm/mach/flash.h
 #include linux/platform_data/mtd-onenand-omap2.h
@@ -59,7 +58,6 @@ struct omap2_onenand {
int dma_channel;
int freq;
int (*setup)(void __iomem *base, int *freq_ptr);
-   struct regulator *regulator;
u8 flags;
 };
 
@@ -583,30 +581,6 @@ static void omap2_onenand_shutdown(struct platform_device 
*pdev)
memset((__force void *)c-onenand.base, 0, ONENAND_BUFRAM_SIZE);
 }
 
-static int omap2_onenand_enable(struct mtd_info *mtd)
-{
-   int ret;
-   struct omap2_onenand *c = container_of(mtd, struct omap2_onenand, mtd);
-
-   ret = regulator_enable(c-regulator);
-   if (ret != 0)
-   dev_err(c-pdev-dev, can't enable regulator\n);
-
-   return ret;
-}
-
-static int omap2_onenand_disable(struct mtd_info *mtd)
-{
-   int ret;
-   struct omap2_onenand *c = container_of(mtd, struct omap2_onenand, mtd);
-
-   ret = regulator_disable(c-regulator);
-   if (ret != 0)
-   dev_err(c-pdev-dev, can't disable regulator\n);
-
-   return ret;
-}
-
 static int omap2_onenand_probe(struct platform_device *pdev)
 {
struct omap_onenand_platform_data *pdata;
@@ -728,22 +702,11 @@ static int omap2_onenand_probe(struct platform_device 
*pdev)
}
}
 
-   if (pdata-regulator_can_sleep) {
-   c-regulator = regulator_get(pdev-dev, vonenand);
-   if (IS_ERR(c-regulator)) {
-   dev_err(pdev-dev,  Failed to get regulator\n);
-   r = PTR_ERR(c-regulator);
-   goto err_release_dma;
-   }
-   c-onenand.enable = omap2_onenand_enable;
-   c-onenand.disable = omap2_onenand_disable;
-   }
-
if (pdata-skip_initial_unlocking)
this-options |= ONENAND_SKIP_INITIAL_UNLOCKING;
 
if ((r = onenand_scan(c-mtd, 1))  0)
-   goto err_release_regulator;
+   goto err_release_dma;
 
ppdata.of_node = pdata-of_node;
r = mtd_device_parse_register(c-mtd, NULL, ppdata,
@@ -758,8 +721,6 @@ static int omap2_onenand_probe(struct platform_device *pdev)
 
 err_release_onenand:
onenand_release(c-mtd);
-err_release_regulator:
-   regulator_put(c-regulator);
 err_release_dma:
if (c-dma_channel != -1)
omap_free_dma(c-dma_channel);
@@ -783,7 +744,6 @@ static int omap2_onenand_remove(struct platform_device 
*pdev)
struct omap2_onenand *c = dev_get_drvdata(pdev-dev);
 
onenand_release(c-mtd);
-   regulator_put(c-regulator);
if (c-dma_channel != -1)
omap_free_dma(c-dma_channel);
omap2_onenand_shutdown(pdev);
diff --git a/include/linux/platform_data/mtd-onenand-omap2.h 
b/include/linux/platform_data/mtd-onenand-omap2.h
index 56ff0e6..445b41e 100644
--- a/include/linux/platform_data/mtd-onenand-omap2.h
+++ b/include/linux/platform_data/mtd-onenand-omap2.h
@@ -25,10 +25,11 @@ struct omap_onenand_platform_data {
int (*onenand_setup)(void __iomem *, int *freq_ptr);
int dma_channel;
u8  flags;
-   u8  regulator_can_sleep;
u8  skip_initial_unlocking;
 
/* for passing the partitions */
struct device_node  *of_node;
+
+   u8  regulator_can_sleep;/* deprecated */
 };
 #endif
-- 
1.8.3.2

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/