pinctrl: sunxi: always look for apb block
authorAndre Przywara <andre.przywara@arm.com>
Sat, 3 Mar 2018 12:25:54 +0000 (12:25 +0000)
committerLinus Walleij <linus.walleij@linaro.org>
Fri, 23 Mar 2018 02:43:48 +0000 (03:43 +0100)
The Allwinner pinctrl device tree binding suggests that a clock named
"apb" would drive the pin controller IP. However (for legacy reasons) we
rely on this clock actually being the first clock defined.
Since named clocks can be in any order, let's explicitly check for a
clock called "apb" if there is more than one clock referenced.

Kudo to Maxime for suggesting this much more elegant approach.

Signed-off-by: Andre Przywara <andre.przywara@arm.com>
Acked-by: Maxime Ripard <maxime.ripard@bootlin.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/sunxi/pinctrl-sunxi.c

index 341312d665120a29472ce174474f870c913e5458..bc3d59f2173f2ad78faa28904e81a91c041d9a94 100644 (file)
@@ -1363,7 +1363,8 @@ int sunxi_pinctrl_init_with_variant(struct platform_device *pdev,
                        goto gpiochip_error;
        }
 
-       clk = devm_clk_get(&pdev->dev, NULL);
+       ret = of_count_phandle_with_args(node, "clocks", "#clock-cells");
+       clk = devm_clk_get(&pdev->dev, ret == 1 ? NULL : "apb");
        if (IS_ERR(clk)) {
                ret = PTR_ERR(clk);
                goto gpiochip_error;