[U-Boot-Users] [PATCH 3/3] QE IO - Add pario command

[Wolfgang Denk]

In message <B27D27F93BC429468DBC3B0DA043AA4401FF2660 at ILPTEX02.ecitele.com> you wrote:
> 
> > I don't think is a good idea. The code as presented is  actually  not
> > suited  for such an extension. And I definitly don't want to plan for
> > another #ifdef maze.
> 
> > Are you aware of the "reginfo" command? Few people seem to know it.
> 
> > Maybe you should add such code there.
> 
> As far as I managed to understand, reginfo just shows a dump of CPU
> based register values, while pario allows you to set and get the value
> of a GPIO port - two different things. We found this very useful for
> debugging our board, and I think it can be useful for many other
> processor families as well. 

I'm aware of this. Changing register contents seems a useful extension
to me, too. That's why I wrote "add such code". I think something like

Print all (or a predefined set of) registers:
	=> reg

Print a specific register:
	=> reg name

Set a specific register to "value":
	=> reg name value

would be generally useful, not only for parallel I/O registers.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"If it ain't broke, don't fix it."                       - Bert Lantz