mbox series

[0/3] gpio: update gpio_get_status()

Message ID 20220802090910.277343-1-patrice.chotard@foss.st.com
Headers show
Series gpio: update gpio_get_status() | expand

Message

Patrice CHOTARD Aug. 2, 2022, 9:09 a.m. UTC
Currently, if pin's function is GPIOF_FUNC, only "func" if displayed
without any other information. It would be interesting, if information is
available, to indicate which pinmuxing's name is used.


Patrice Chotard (3):
  gpio: Allow to print pin's label even for pin with GPIOF_FUNC function
  gpio: Fix pin's status display for pin with GPIOF_UNUSED function
  pinctrl: pinctrl_stm32: Populate uc_priv->name[] with pinmux node's
    name

 drivers/gpio/gpio-uclass.c      | 18 ++++++++++++------
 drivers/pinctrl/pinctrl_stm32.c |  8 ++++++--
 2 files changed, 18 insertions(+), 8 deletions(-)

Comments

Heinrich Schuchardt Aug. 2, 2022, 2:56 p.m. UTC | #1
On 8/2/22 11:09, Patrice Chotard wrote:
>
> Currently, if pin's function is GPIOF_FUNC, only "func" if displayed
> without any other information. It would be interesting, if information is
> available, to indicate which pinmuxing's name is used.

Maybe you can add on top of my gpio man-page patch a description of the
output fields of gpio status.

[PATCH 1/1] doc: man-page for gpio command
https://lists.denx.de/pipermail/u-boot/2022-August/490666.html

Best regards

Heinrich

>
>
> Patrice Chotard (3):
>    gpio: Allow to print pin's label even for pin with GPIOF_FUNC function
>    gpio: Fix pin's status display for pin with GPIOF_UNUSED function
>    pinctrl: pinctrl_stm32: Populate uc_priv->name[] with pinmux node's
>      name
>
>   drivers/gpio/gpio-uclass.c      | 18 ++++++++++++------
>   drivers/pinctrl/pinctrl_stm32.c |  8 ++++++--
>   2 files changed, 18 insertions(+), 8 deletions(-)
>
Patrice CHOTARD Aug. 3, 2022, 1:11 p.m. UTC | #2
Hi Heinrich

On 8/2/22 16:56, Heinrich Schuchardt wrote:
> On 8/2/22 11:09, Patrice Chotard wrote:
>>
>> Currently, if pin's function is GPIOF_FUNC, only "func" if displayed
>> without any other information. It would be interesting, if information is
>> available, to indicate which pinmuxing's name is used.
> 
> Maybe you can add on top of my gpio man-page patch a description of the
> output fields of gpio status.
> 
> [PATCH 1/1] doc: man-page for gpio command
> https://lists.denx.de/pipermail/u-boot/2022-August/490666.html

Yes for sure, thanks for the link

Patrice

> 
> Best regards
> 
> Heinrich
> 
>>
>>
>> Patrice Chotard (3):
>>    gpio: Allow to print pin's label even for pin with GPIOF_FUNC function
>>    gpio: Fix pin's status display for pin with GPIOF_UNUSED function
>>    pinctrl: pinctrl_stm32: Populate uc_priv->name[] with pinmux node's
>>      name
>>
>>   drivers/gpio/gpio-uclass.c      | 18 ++++++++++++------
>>   drivers/pinctrl/pinctrl_stm32.c |  8 ++++++--
>>   2 files changed, 18 insertions(+), 8 deletions(-)
>>
>