]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
pinctrl: renesas: Move fixed assignments to 'pinctrl_desc' definition
authorKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Wed, 11 Jun 2025 06:13:46 +0000 (08:13 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 18 Jun 2025 11:26:37 +0000 (13:26 +0200)
Assign 'struct pinctrl_desc' .pins and .npins members in definition to
make clear that number of pins is fixed and have less code in the probe.

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/20250611-pinctrl-const-desc-v2-14-b11c1d650384@linaro.org
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/renesas/pinctrl-rzn1.c

index d442d4f9981c908389407275522b936dbce294d3..fb874867dbfb9877b012988543ecbfd1bfb11270 100644 (file)
@@ -680,6 +680,8 @@ static struct pinctrl_desc rzn1_pinctrl_desc = {
        .pmxops = &rzn1_pmx_ops,
        .confops = &rzn1_pinconf_ops,
        .owner = THIS_MODULE,
+       .pins = rzn1_pins,
+       .npins = ARRAY_SIZE(rzn1_pins),
 };
 
 static int rzn1_pinctrl_parse_groups(struct device_node *np,
@@ -878,8 +880,6 @@ static int rzn1_pinctrl_probe(struct platform_device *pdev)
 
        ipctl->dev = &pdev->dev;
        rzn1_pinctrl_desc.name = dev_name(&pdev->dev);
-       rzn1_pinctrl_desc.pins = rzn1_pins;
-       rzn1_pinctrl_desc.npins = ARRAY_SIZE(rzn1_pins);
 
        ret = rzn1_pinctrl_probe_dt(pdev, ipctl);
        if (ret) {