Commit ed32213f authored by Krzysztof Kozlowski's avatar Krzysztof Kozlowski Committed by Linus Walleij
Browse files

pinctrl: max77620: Move fixed assignments to 'pinctrl_desc' definition 



Assign 'struct pinctrl_desc' .pins, .npins and other members in
definition to make clear that number of pins is fixed and have less code
in the probe.

Signed-off-by: default avatarKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/20250611-pinctrl-const-desc-v2-12-b11c1d650384@linaro.org


Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
parent 1bd634e5

drivers/pinctrl/pinctrl-max77620.c

+4 −5
Original line number Diff line number Diff line
@@ -543,6 +543,10 @@ static struct pinctrl_desc max77620_pinctrl_desc = { 
	.pctlops = &max77620_pinctrl_ops,

	.pmxops = &max77620_pinmux_ops,

	.confops = &max77620_pinconf_ops,

	.pins = max77620_pins_desc,

	.npins = ARRAY_SIZE(max77620_pins_desc),

	.num_custom_params = ARRAY_SIZE(max77620_cfg_params),

	.custom_params = max77620_cfg_params,

};



static int max77620_pinctrl_probe(struct platform_device *pdev)
@@ -569,11 +573,6 @@ static int max77620_pinctrl_probe(struct platform_device *pdev) 
	platform_set_drvdata(pdev, mpci);



	max77620_pinctrl_desc.name = dev_name(&pdev->dev);

	max77620_pinctrl_desc.pins = max77620_pins_desc;

	max77620_pinctrl_desc.npins = ARRAY_SIZE(max77620_pins_desc);

	max77620_pinctrl_desc.num_custom_params =

				ARRAY_SIZE(max77620_cfg_params);

	max77620_pinctrl_desc.custom_params = max77620_cfg_params;



	for (i = 0; i < MAX77620_PIN_NUM; ++i) {

		mpci->fps_config[i].active_fps_src = -1;