[PATCH 1/3] pinctrl: pinctrl_stm32: Update pinmux_mode definition

Patrick DELAUNAY patrick.delaunay at foss.st.com
Fri May 6 10:37:10 CEST 2022


Hi,

On 4/22/22 09:38, Patrice Chotard wrote:
> pinmux_mode[] is linked to gpio_function[] defined in gpio-uclass.c
> So reuse the same gpio_func_t enum value
>
> Signed-off-by: Patrice Chotard <patrice.chotard at foss.st.com>
> ---
>
>   drivers/pinctrl/pinctrl_stm32.c | 13 ++++++-------
>   1 file changed, 6 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/pinctrl/pinctrl_stm32.c b/drivers/pinctrl/pinctrl_stm32.c
> index 5729799b12..5d8e156d62 100644
> --- a/drivers/pinctrl/pinctrl_stm32.c
> +++ b/drivers/pinctrl/pinctrl_stm32.c
> @@ -42,13 +42,12 @@ struct stm32_gpio_bank {
>   #ifndef CONFIG_SPL_BUILD
>   
>   static char pin_name[PINNAME_SIZE];
> -#define PINMUX_MODE_COUNT		5
> -static const char * const pinmux_mode[PINMUX_MODE_COUNT] = {
> -	"gpio input",
> -	"gpio output",
> -	"analog",
> -	"unknown",
> -	"alt function",
> +static const char * const pinmux_mode[GPIOF_COUNT] = {
> +	[GPIOF_INPUT] = "gpio input",
> +	[GPIOF_OUTPUT] = "gpio output",
> +	[GPIOF_UNUSED] = "analog",
> +	[GPIOF_UNKNOWN] = "unknown",
> +	[GPIOF_FUNC] = "alt function",
>   };
>   
>   static const char * const pinmux_bias[] = {


Reviewed-by: Patrick Delaunay <patrick.delaunay at foss.st.com>

Thanks
Patrick




More information about the U-Boot mailing list