[U-Boot] [PATCH V2] ppc4xx: use SPRN_TCR macro instead of tcr

Matthias Fuchs matthias.fuchs at esd.eu
Tue Jul 21 12:28:20 CEST 2009


On Tuesday 21 July 2009 12:06, Wolfgang Denk wrote:
> 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...
You are right. As you might think of, I want to get my PMC405DE board
support patch accepted in this closing merge window. 
I also noticed that there are other macros that are obsolete and tha't 
why I netioned this in my commit message. So I am aware of this.
But that's another story.

I will see if I find some time later today.

Matthias

> 
> > 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
> 



More information about the U-Boot mailing list