[U-Boot] [PATCH 0/4] ppc4xx: Overhaul for cmd reginfo
Stefan Roese
sr at denx.de
Sun Oct 4 13:31:06 CEST 2009
Hi Niklaus,
On Friday 02 October 2009 20:12:07 Niklaus Giger wrote:
> The command "reginfo" got an overhaul for the ppc4xx. It dumps all the
> relevant HW configuration registers (address, symbolic name, content).
> This allows to easily detect errors in *.h files and changes in the HW
> configuration.
Thanks.
> It is split in the following parts:
> - Cleanup some HW register names:
> Here you find all the changes in the include directory for new register
> names and adapting other ones to the names used by AMCC in their manuals,
> e.g. For 440EPx/GRPPC440EPx/GRX, Revision 1.15 ق September 22, 2008 For
> PPC405GP Embedded Processor, Revision 1.02 ق March 22, 2006 - Apply new
> HW register names
> Modify all existing *.c files to use the new register names
> - Rework cmd reginfo
> Here the real work done to improve the reginfo command.
> - respect 80-chars per line in ppc*.h files
> After running checkstyle.pl on the three previous patches I noted that in
> the *.h files there were a lot of long lines. This patch solves this
> problem.
A general comment to all your 4 patches. Please put the description here also
into the patches itself. This patch (0/4) won't be applied, so this
information is lost.
> I tested the changes on my PPC405GPr board HCU4 and PPC440EPx board HCU5.
>
> Only ran MAKEALL 4xx as I have no other cross-compilers installed.
>
> The DMA-registers are not dumped for the PPC440.
>
> I know that I could spend not only many hours but weeks to cleanup all the
> PPC4xx register naming conventions.
Right, this is time consuming. Thanks for your effort so far. Really
appretiated.
Cheers,
Stefan
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de
More information about the U-Boot
mailing list