[U-Boot] [PATCH v3] arm926ejs: timer: Replace bss variable by gdr

Albert ARIBAUD albert.aribaud at free.fr
Sat Jan 22 08:46:20 CET 2011


Le 22/01/2011 06:39, Alexander Holler a écrit :
> Hello,
>
> Am 21.01.2011 09:56, schrieb Heiko Schocher:
>
>> -static ulong timestamp;
>> -static ulong lastdec;
>> +DECLARE_GLOBAL_DATA_PTR;
>> +
>> +#define timestamp gd->tbl
>> +#define lastdec gd->lastinc
>
> I'm the only one who doesn't like such defines? They might be handy for
> quick fixes, but in regard to style and readablity I don't like them.
> When looking at teh code where they will used, you won't see the actual
> place where they are stored. And in more complex expression they might
> become dangerous to use because they hide the operator "->".

I accept the patch because it un-breaks support for ARM cpus, and I 
prefer a working fix to a perfect fix in this specific, transitional, 
situation.

However a general rework of ARM timer code is in order so that all SoCs 
and CPUs share the same set of gd variables with the same names and the 
same logic; and when we get that, this code shall move along.

About this rework, as the saying goes... "Patches Welcome ©". :)

> Regards,
>
> Alexander

Amicalement,
-- 
Albert.


More information about the U-Boot mailing list