Skip to contentSkip to conversation

mpc83xx: set gpio level before direction

PATCH New Peter Korsgaard 2 participants 3 messages Aug 29, 2008

lore
Share & tools
lore
#0 lore raw

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
index 67c9e57..4514dbb 100644
@@ -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
#1 lore raw

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.

---

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

#2 lore raw

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