On 08/05/2026 00:05, 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 <[email protected]>

Reviewed-by: Casey Connolly <[email protected]>

> ---
> Cc: Casey Connolly <[email protected]>
> Cc: Neil Armstrong <[email protected]>
> Cc: Rui Miguel Silva <[email protected]>
> Cc: Sumit Garg <[email protected]>
> Cc: Tom Rini <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> ---
>  drivers/gpio/qcom_pmic_gpio.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpio/qcom_pmic_gpio.c b/drivers/gpio/qcom_pmic_gpio.c
> index 4458c55cd3d..3cabb7df88c 100644
> --- a/drivers/gpio/qcom_pmic_gpio.c
> +++ b/drivers/gpio/qcom_pmic_gpio.c
> @@ -410,7 +410,7 @@ static int 
> qcom_pmic_pinctrl_generic_pinmux_set_mux(struct udevice *dev, unsigne
>       return 0;
>  }
>  
> -struct pinctrl_ops qcom_pmic_pinctrl_ops = {
> +static const struct pinctrl_ops qcom_pmic_pinctrl_ops = {
>       .get_pins_count = qcom_pmic_pinctrl_get_pins_count,
>       .get_pin_name = qcom_pmic_pinctrl_get_pin_name,
>       .set_state = pinctrl_generic_set_state,

-- 
// Casey (she/her)

Reply via email to