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

Patrice CHOTARD patrice.chotard at foss.st.com
Tue May 10 10:06:13 CEST 2022



On 5/6/22 10:37, Patrick DELAUNAY wrote:
> 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
> 
> 
Applied to u-boot-stm32

Thanks
Patrice


More information about the U-Boot mailing list