[PATCH v2] qe_lib: Set gpio data before changing the direction to output
avorontsov at ru.mvista.com
Wed Aug 19 08:56:07 EST 2009
On Tue, Aug 18, 2009 at 05:33:00PM -0500, Timur Tabi wrote:
> Anton Vorontsov wrote:
> > On Tue, Aug 18, 2009 at 05:20:44PM -0400, Michael Barkowski wrote:
> >> This avoids having a short glitch if the desired initial value is not
> >> the same as what was previously in the data register.
> >> Signed-off-by: Michael Barkowski <michaelbarkowski at ruggedcom.com>
> > Acked-by: Anton Vorontsov <avorontsov at ru.mvista.com>
> I don't have the time to test this patch, so I abstain from acking. :-)
> If Anton likes it, that's good enough for me.
You made me doubt for a moment. :-) Thanks for the suspiciousness.
What happens if a pin was previously configured as input? Does our
write to the data register survive? For MPC8xxx GPIO controllers
it does. And randomly taken QE spec says:
A write to CPDAT is latched, and if the corresponding CPDIR
bits have configured the port pin as an output, the latched
value is driven onto the respective pin. However, if the
corresponding CPDIR bits have configured the port pin as an
input, the latched value is prevented from reaching the pin.
I guess we're safe, but Michael, could you actually test it
(if not already)?
email: cbouatmailru at gmail.com
More information about the Linuxppc-dev