pinctrl: ocelot: fix confops resource index
authorMichael Walle <michael@walle.cc>
Wed, 16 Feb 2022 08:20:20 +0000 (09:20 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Tue, 15 Mar 2022 00:05:25 +0000 (01:05 +0100)
Prior to commit ad96111e658a ("pinctrl: ocelot: combine get resource and
ioremap into single call") the resource index was 1, now it is 0. But 0
is the base region for the pinctrl block. Fix it.
I noticed this because there was an error that the memory region was
ioremapped twice.

Fixes: ad96111e658a ("pinctrl: ocelot: combine get resource and ioremap into single call")
Signed-off-by: Michael Walle <michael@walle.cc>
Reviewed-by: Colin Foster <colin.foster@in-advantage.com>
Link: https://lore.kernel.org/r/20220216082020.981797-1-michael@walle.cc
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinctrl-ocelot.c

index 685c79e..a859fbc 100644 (file)
@@ -1892,7 +1892,7 @@ static struct regmap *ocelot_pinctrl_create_pincfg(struct platform_device *pdev)
                .max_register = 32,
        };
 
-       base = devm_platform_ioremap_resource(pdev, 0);
+       base = devm_platform_ioremap_resource(pdev, 1);
        if (IS_ERR(base)) {
                dev_dbg(&pdev->dev, "Failed to ioremap config registers (no extended pinconf)\n");
                return NULL;