mpc83xx: set gpio level before direction
PATCH 1/1 mpc83xx: set gpio level before direction
Set gpio level register before direction register to inhibit glitches on high level output pins.
Dir and data gets cleared at powerup, so high level output lines see a short low pulse between setting the direction and level registers.
Issue was seen on a new board with the nReset line of the NOR flash connected to a GPIO. Setting the direction register puts the NOR flash in reset so the next instruction to set the level cannot get executed.
--- cpu/mpc83xx/cpu_init.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)
cpu/mpc83xx/cpu_init.c +2 -2
@@ -283,12 +283,12 @@ void cpu_init_f (volatile immap_t * im) im->sysconf.lblaw[7 ].ar =CFG_LBLAWAR7_PRELIM ;#endif #ifdef CFG_GPIO1_PRELIM - im->gpio[0 ].dir =CFG_GPIO1_DIR ; im->gpio[0 ].dat =CFG_GPIO1_DAT ; + im->gpio[0 ].dir =CFG_GPIO1_DIR ;#endif #ifdef CFG_GPIO2_PRELIM - im->gpio[1 ].dir =CFG_GPIO2_DIR ; im->gpio[1 ].dat =CFG_GPIO2_DAT ; + im->gpio[1 ].dir =CFG_GPIO2_DIR ;#endif }
Signature · 1 line
1.5.6.3
PATCH 1/1 mpc83xx: set gpio level before direction
Quoted 6 lines · On Fri, 29 Aug 2008 20:37:25 +0200 Peter Korsgaard <[email protected]> wrote:
Set gpio level register before direction register to inhibit glitches on high level output pins.
Dir and data gets cleared at powerup, so high level output lines see a short low pulse between setting the direction and level registers.
Quoted 6 lines
Issue was seen on a new board with the nReset line of the NOR flash connected to a GPIO. Setting the direction register puts the NOR flash in reset so the next instruction to set the level cannot get executed.
Signed-off-by Peter Korsgaard <[email protected]>---
Hi Peter,
this is a duplicate of:
http://www.nabble.com/-U-Boot---PATCH--MPC83XX%3A-Fix-GPIO-configuration-to19119329.html#a19119329
so I'll merge your better description and s-o-b to that one if you don't mind.
Kim
PATCH 1/1 mpc83xx: set gpio level before direction
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
"Kim" == Kim Phillips <[email protected]> writes:
Kim> this is a duplicate of:
Ahh sorry, I must have missed that one.
Kim> so I'll merge your better description and s-o-b to that one if you Kim> don't mind.
Yeah, that's fine.
Signature · 1 line
Bye, Peter Korsgaard