Commit 18fab399 authored by Geert Uytterhoeven's avatar Geert Uytterhoeven

pinctrl: sh-pfc: Stop calling gpiochip_add_pin_range() on DT platforms

On platforms where the PFC/GPIO controller is instantiated from DT, the
mapping between GPIOs and pins is set up using the "gpio-ranges"
property in DT.

Hence stop setting up the mapping from C code on DT platforms.
This code is still used for SH or ARM-legacy platforms.
Signed-off-by: default avatarGeert Uytterhoeven <geert+renesas@glider.be>
Acked-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Acked-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
parent 6ff33f39
...@@ -379,22 +379,29 @@ int sh_pfc_register_gpiochip(struct sh_pfc *pfc) ...@@ -379,22 +379,29 @@ int sh_pfc_register_gpiochip(struct sh_pfc *pfc)
pfc->gpio = chip; pfc->gpio = chip;
/* Register the GPIO to pin mappings. As pins with GPIO ports must come if (IS_ENABLED(CONFIG_OF) && pfc->dev->of_node)
* first in the ranges, skip the pins without GPIO ports by stopping at return 0;
* the first range that contains such a pin.
*/
for (i = 0; i < pfc->nr_ranges; ++i) {
const struct sh_pfc_pin_range *range = &pfc->ranges[i];
if (range->start >= pfc->nr_gpio_pins)
break;
ret = gpiochip_add_pin_range(&chip->gpio_chip, if (IS_ENABLED(CONFIG_SUPERH) ||
dev_name(pfc->dev), IS_ENABLED(CONFIG_ARCH_SHMOBILE_LEGACY)) {
range->start, range->start, /*
range->end - range->start + 1); * Register the GPIO to pin mappings. As pins with GPIO ports
if (ret < 0) * must come first in the ranges, skip the pins without GPIO
return ret; * ports by stopping at the first range that contains such a
* pin.
*/
for (i = 0; i < pfc->nr_ranges; ++i) {
const struct sh_pfc_pin_range *range = &pfc->ranges[i];
if (range->start >= pfc->nr_gpio_pins)
break;
ret = gpiochip_add_pin_range(&chip->gpio_chip,
dev_name(pfc->dev), range->start, range->start,
range->end - range->start + 1);
if (ret < 0)
return ret;
}
} }
/* Register the function GPIOs chip. */ /* Register the function GPIOs chip. */
......
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