Commit 3f9547e1 authored by Masahiro Yamada's avatar Masahiro Yamada Committed by Linus Walleij

gpio: of: optimize "gpios" property parsing of of_parse_own_gpio()

Call of_property_read_u32_array() only once rather than iterating
of_property_read_u32_index().
Signed-off-by: default avatarMasahiro Yamada <yamada.masahiro@socionext.com>
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
parent 7e7c059c
...@@ -140,7 +140,7 @@ static struct gpio_desc *of_parse_own_gpio(struct device_node *np, ...@@ -140,7 +140,7 @@ static struct gpio_desc *of_parse_own_gpio(struct device_node *np,
.flags = &xlate_flags, .flags = &xlate_flags,
}; };
u32 tmp; u32 tmp;
int i, ret; int ret;
chip_np = np->parent; chip_np = np->parent;
if (!chip_np) if (!chip_np)
...@@ -159,12 +159,10 @@ static struct gpio_desc *of_parse_own_gpio(struct device_node *np, ...@@ -159,12 +159,10 @@ static struct gpio_desc *of_parse_own_gpio(struct device_node *np,
gg_data.gpiospec.args_count = tmp; gg_data.gpiospec.args_count = tmp;
gg_data.gpiospec.np = chip_np; gg_data.gpiospec.np = chip_np;
for (i = 0; i < tmp; i++) { ret = of_property_read_u32_array(np, "gpios", gg_data.gpiospec.args,
ret = of_property_read_u32_index(np, "gpios", i, tmp);
&gg_data.gpiospec.args[i]); if (ret)
if (ret) return ERR_PTR(ret);
return ERR_PTR(ret);
}
gpiochip_find(&gg_data, of_gpiochip_find_and_xlate); gpiochip_find(&gg_data, of_gpiochip_find_and_xlate);
if (!gg_data.out_gpio) { if (!gg_data.out_gpio) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment