[PATCHv3 02/10] gpio: mxc_gpio: add support to read status of output gpios

Fabio Estevam festevam at gmail.com
Sun Aug 23 16:44:35 CEST 2020


Hi Sebastian,

On Fri, Aug 21, 2020 at 1:16 PM Sebastian Reichel
<sebastian.reichel at collabora.com> wrote:
>
> This is supported by the hardware when the pinmux is configured

I assume you are referring to the SION bit.

> correctly. Usually it is not, so this adds explicit code for this.
> This fixes all GPIO regulators being shown as disabled.

This has come up in the past and the consensus was to set the SION bit
when reading the output of a GPIO is needed.

Please see commit:

commit 5dafa4543c399d329c7b01df1afa98437861cac0
Author: Benoît Thébaudeau <benoit.thebaudeau at advansee.com>
Date:   Mon Aug 20 10:55:41 2012 +0000

    mxc: Make gpio_get_value() use PSR

    gpio_get_value() should use PSR like Linux, not DR, because DR
does not always
    reflect the pin state, while PSR does. This is especially useful to detect a
    short circuit on a GPIO pin configured as output, or to read the
level of a pin
    controlled by a non-GPIO IOMUX function.

    Signed-off-by: Benoît Thébaudeau <benoit.thebaudeau at advansee.com>
    Cc: Stefano Babic <sbabic at denx.de>
    Acked-by: Stefano Babic <sbabic at denx.de>


More information about the U-Boot mailing list