[U-Boot] [PATCH V2] ppc4xx: use SPRN_TCR macro instead of tcr
Wolfgang Denk
wd at denx.de
Tue Jul 21 12:06:13 CEST 2009
Dear Matthias Fuchs,
In message <200907211154.16125.matthias.fuchs at esd.eu> you wrote:
> This patch migrates 4xx code from using the lowercase tcr macro
> to the SPRN_TCR macro. Both macros have been defined in the past.
> So one is to much and uppercase macros are much more common.
>
> The lowercase macro conflicts with the upcoming
> introduction of the ppc4xx_gpio struct and it's tcr field.
>
> Doing more such migration and getting rid of many defines from
> include/ppc405.h and include/ppc440.h should be put on
> our TODO list.
>
> patch v2: better use SPRN_TCR instead of TCR
Hm... why are you just fixing this single register? It's not the only
one...
> diff --git a/include/ppc405.h b/include/ppc405.h
> index 917afec..50fc7be 100644
> --- a/include/ppc405.h
> +++ b/include/ppc405.h
> @@ -61,7 +61,6 @@
> #define dear 0x3d5 /* data exeption address register */
> #define evpr 0x3d6 /* exeption vector prefix register */
> #define tsr 0x3d8 /* timer status register */
> - #define tcr 0x3da /* timer control register */
> #define pit 0x3db /* programmable interval timer */
> #define sgr 0x3b9 /* storage guarded reg */
> #define dcwr 0x3ba /* data cache write-thru reg*/
> diff --git a/include/ppc440.h b/include/ppc440.h
> index 01f6eaf..df78eea 100644
> --- a/include/ppc440.h
> +++ b/include/ppc440.h
> @@ -90,7 +90,6 @@
> #define dvc1 0x13e /* data value compare 1 */
> #define dvc2 0x13f /* data value compare 2 */
> #define tsr 0x150 /* timer status register */
> -#define tcr 0x154 /* timer control register */
> #define ivor0 0x190 /* interrupt vector offset register 0 */
> #define ivor1 0x191 /* interrupt vector offset register 1 */
> #define ivor2 0x192 /* interrupt vector offset register 2 */
_ALL_ these #defines for Special Purpose Registers should go away.
They are redundant.
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
"At least they're __________EXPERIENCED incompetents"
More information about the U-Boot
mailing list