[U-Boot] [PATCH] mx6: Add IOMUX_CONFIG_SION flag to all GPIO pins

Stefano Babic sbabic at denx.de
Tue Oct 1 10:33:54 CEST 2013


Hi Otavio,

On 30/09/2013 00:15, Otavio Salvador wrote:
> The IOMUX_CONFIG_SION allows for reading PAD value from PSR register.
> 
> The following quote from the datasheet:
> 
> ,----
> | ...
> | 28.4.2.2 GPIO Write Mode
> | The programming sequence for driving output signals should be as follows:
> | 1. Configure IOMUX to select GPIO mode (Via IOMUXC), also enable SION if need
> | to read loopback pad value through PSR
> | 2. Configure GPIO direction register to output (GPIO_GDIR[GDIR] set to 1b).
> | 3. Write value to data register (GPIO_DR).
> | ...
> `----
> 
> This fixes the gpio_get_value to properly work when a GPIO is set for
> output and has no conflicts.
> 
> Thanks for Benoît Thébaudeau <benoit.thebaudeau at advansee.com>, Fabio
> Estevam <fabio.estevam at freescale.com> and Eric Bénard
> <eric at eukrea.com> for helping to properly trace this down.
> 
> Signed-off-by: Otavio Salvador <otavio at ossystems.com.br>
> ---

I come later in the discussion, but I see you all have found the
solution ;-).

I suggest you add in the commit message that this patch is only for
i.MX6 (if you do not plan to extend it to the other i.MXes...), so that
we can track that the same must be done also for the other SOCs.

Thanks,
Stefano


-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================


More information about the U-Boot mailing list