[PATCH] pinctrl: armada-38x: Staticize and constify driver ops

Stefan Roese stefan.roese at mailbox.org
Fri May 8 15:56:14 CEST 2026


On 5/8/26 14:21, Marek Vasut wrote:
> Set the ops structure as static const. The structure is not accessible
> from outside of this driver and is not going to be modified at runtime.
> 
> Signed-off-by: Marek Vasut <marek.vasut+renesas at mailbox.org>
> ---
> Cc: Stefan Roese <stefan.roese at mailbox.org>
> Cc: Tom Rini <trini at konsulko.com>
> Cc: u-boot at lists.denx.de

Reviewed-by: Stefan Roese <stefan.roese at mailbox.org>

Thanks,
Stefan

> ---
>   drivers/pinctrl/mvebu/pinctrl-armada-38x.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-38x.c b/drivers/pinctrl/mvebu/pinctrl-armada-38x.c
> index 78184d2860a..c18afe958dc 100644
> --- a/drivers/pinctrl/mvebu/pinctrl-armada-38x.c
> +++ b/drivers/pinctrl/mvebu/pinctrl-armada-38x.c
> @@ -550,7 +550,7 @@ static int armada_38x_pinctrl_probe(struct udevice *dev)
>   	return 0;
>   }
>   
> -struct pinctrl_ops armada_37xx_pinctrl_ops = {
> +static const struct pinctrl_ops armada_37xx_pinctrl_ops = {
>   	.get_pins_count = armada_38x_pinctrl_get_pins_count,
>   	.get_pin_name = armada_38x_pinctrl_get_pin_name,
>   	.get_functions_count = armada_38x_pinctrl_get_functions_count,



More information about the U-Boot mailing list