regulator: gpio-regulator: fix forgotten gpios-states reading
authorHeiko Stuebner <heiko.stuebner@bqreaders.com>
Thu, 13 Feb 2014 15:34:32 +0000 (16:34 +0100)
committerMark Brown <broonie@linaro.org>
Fri, 14 Feb 2014 14:58:57 +0000 (14:58 +0000)
Commit 934624d6e9f0 ("regulator: gpio-regulator: do not open-code counting
and access of dt array elements") forgot to convert the recently added
gpios-states property using the same pattern.

Convert this instance to use the of-helpers too, resolving the build error.

Signed-off-by: Heiko Stuebner <heiko.stuebner@bqreaders.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
drivers/regulator/gpio-regulator.c

index 9fd55611016c7c50052f124421577b7c25295fa4..989b23b377c0d2bb511227749e871ce72f46398b 100644 (file)
@@ -171,13 +171,14 @@ of_get_gpio_regulator_config(struct device *dev, struct device_node *np)
        if (!config->gpios)
                return ERR_PTR(-ENOMEM);
 
-       prop = of_find_property(np, "gpios-states", NULL);
-       if (prop) {
-               proplen = prop->length / sizeof(int);
-               if (proplen != config->nr_gpios) {
-                       dev_warn(dev, "gpios <-> gpios-states mismatch\n");
-                       prop = NULL;
-               }
+       proplen = of_property_count_u32_elems(np, "gpios-states");
+       /* optional property */
+       if (proplen < 0)
+               proplen = 0;
+
+       if (proplen > 0 && proplen != config->nr_gpios) {
+               dev_warn(dev, "gpios <-> gpios-states mismatch\n");
+               proplen = 0;
        }
 
        for (i = 0; i < config->nr_gpios; i++) {
@@ -185,8 +186,11 @@ of_get_gpio_regulator_config(struct device *dev, struct device_node *np)
                if (gpio < 0)
                        break;
                config->gpios[i].gpio = gpio;
-               if (prop && be32_to_cpup((int *)prop->value + i))
-                       config->gpios[i].flags = GPIOF_OUT_INIT_HIGH;
+               if (proplen > 0) {
+                       of_property_read_u32_index(np, "gpios-states", i, &ret);
+                       if (ret)
+                               config->gpios[i].flags = GPIOF_OUT_INIT_HIGH;
+               }
        }
 
        /* Fetch states. */
This page took 0.038349 seconds and 5 git commands to generate.