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

[Wolfgang Denk]

In message <B27D27F93BC429468DBC3B0DA043AA4401FF270D at ILPTEX02.ecitele.com> you wrote:
> > 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.
> 
> Specifying parallel I/O ports has whole registers is really
> uncomfortable: If for example one would like to write a value to a
> parallel I/O port, then he'd need to read the data register first, then
> mask off all irrelevant bits, and then write the shifted value to this
> register. The pario command does this job in a much more comfortable
> way.

Nothing prevents you to write your code such that  "name"  is  not  a
register  name  but  a  parallel  port  name,  and that the code that
handles it does exactly what you describe above transparently for the
user.

> Again - we can keep this command in our boards' common code, but I think
> it would be a pity, as I think this functionality can be useful for many
> other CPU types as well.

Agreed.

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
The most exciting phrase to hear in science, the one that heralds new
discoveries, is not "Eureka!" (I found it!) but "That's funny ..."
                                                      -- Isaac Asimov