Hi Mike,
On Saturday 05 November 2005 18:43, Mike Wellington wrote:
In include/asm-ppc/processor.h the definitions for the Time Base Upper and Lower registers are incorrect for the PPC440GX.
#define SPRN_TBRL 0x10D /* Time Base Read Lower Register */ #define SPRN_TBRU 0x10C /* Time Base Read Upper Register */ #define SPRN_TBWL 0x11D /* Time Base Write Lower Register */ #define SPRN_TBWU 0x11C /* Time Base Write Upper Register */
i fixed this in my own code by adding the following ifdef:
#ifdef CONFIG_440GX #define SPRN_TBRL 0x10C /* Time Base Read Lower Register */ #define SPRN_TBRU 0x10D /* Time Base Read Upper Register */ #define SPRN_TBWL 0x11C /* Time Base Write Lower Register */ #define SPRN_TBWU 0x11D /* Time Base Write Upper Register */ #else #define SPRN_TBRL 0x10D /* Time Base Read Lower Register */ #define SPRN_TBRU 0x10C /* Time Base Read Upper Register */ #define SPRN_TBWL 0x11D /* Time Base Write Lower Register */ #define SPRN_TBWU 0x11C /* Time Base Write Upper Register */ #endif
After a quick look in some other ppc manuals (AMCC and Freescale) this seems to be not only a 440GX problem, but a generic ppc problem. I will prepare a patch for this.
I didn't find any code referencing these registers. Do you have an command/application using these timebase regsisters, which could be interesting for the community? If so, please share it it with us.
And please submit a proper patch next time (CHANGELOG, etc...)
Best regards, Stefan