lkml.org 
[lkml]   [2012]   [Jun]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH] regulator: fixed: support deferred probe for DT GPIOs
Date
From: Stephen Warren <swarren@nvidia.com>

of_get_named_gpio() needs the driver hosting the GPIO that the DT
property references to have been probed. Detect this specific failure,
and defer the probe of the whole regulator until this API can complete.

Note that of_get_named_gpio() currently returns -ENODEV in this case,
but a patch has been sent to make it return -EPROBE_DEFER. Hence, this
patch checks both so that it works with/without the other patch.

Signed-off-by: Stephen Warren <swarren@nvidia.com>
---
drivers/regulator/fixed.c | 19 +++++++++++++++----
1 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/drivers/regulator/fixed.c b/drivers/regulator/fixed.c
index 8bda365..1acd60a 100644
--- a/drivers/regulator/fixed.c
+++ b/drivers/regulator/fixed.c
@@ -51,13 +51,15 @@ struct fixed_voltage_data {
* alloc fails.
*/
static struct fixed_voltage_config *
-of_get_fixed_voltage_config(struct device *dev)
+of_get_fixed_voltage_config(struct device *dev, bool *defer_probe)
{
struct fixed_voltage_config *config;
struct device_node *np = dev->of_node;
const __be32 *delay;
struct regulator_init_data *init_data;

+ *defer_probe = false;
+
config = devm_kzalloc(dev, sizeof(struct fixed_voltage_config),
GFP_KERNEL);
if (!config)
@@ -83,6 +85,11 @@ of_get_fixed_voltage_config(struct device *dev)
config->enabled_at_boot = true;

config->gpio = of_get_named_gpio(np, "gpio", 0);
+ if ((config->gpio == -ENODEV) || (config->gpio == -EPROBE_DEFER)) {
+ *defer_probe = true;
+ return NULL;
+ }
+
delay = of_get_property(np, "startup-delay-us", NULL);
if (delay)
config->startup_delay = be32_to_cpu(*delay);
@@ -167,15 +174,19 @@ static struct regulator_ops fixed_voltage_ops = {

static int __devinit reg_fixed_voltage_probe(struct platform_device *pdev)
{
+ bool defer_probe;
struct fixed_voltage_config *config;
struct fixed_voltage_data *drvdata;
struct regulator_config cfg = { };
int ret;

- if (pdev->dev.of_node)
- config = of_get_fixed_voltage_config(&pdev->dev);
- else
+ if (pdev->dev.of_node) {
+ config = of_get_fixed_voltage_config(&pdev->dev, &defer_probe);
+ if (defer_probe)
+ return -EPROBE_DEFER;
+ } else {
config = pdev->dev.platform_data;
+ }

if (!config)
return -ENOMEM;
--
1.7.0.4


\
 
 \ /
  Last update: 2012-06-29 01:01    [W:0.038 / U:0.776 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site