Dear =?iso-8859-1?Q?Andreas_Bie=DFmann?=,
In message 0CD9BF11-6356-44A8-BABA-0AA08671D9FB@googlemail.com you wrote:
+#if defined(CONFIG_OMAP) || defined(CONFIG_MX51)
- unsigned long timestamp;
- unsigned long lastinc;
+#endif +#if defined(CONFIG_S5P)
- unsigned long count_value;
- /* Internal tick units */
- unsigned long long timestamp; /* Monotonic incrementing timer */
- unsigned long lastdec; /* Last decremneter snapshot */
+#endif unsigned long relocaddr; /* Start address of U-Boot in RAM */ phys_size_t ram_size; /* RAM size */ unsigned long mon_len; /* monitor len */
can't this generalized in some way?
It can, and it should.
But I suggest we handle such a cleanup as a separate task and defer it until the next release (which means that any such patches will go to the "next" branch if they come now).
I guess if we provide something like
uint64_t timebase; uint32_t timelast;
this should cover most of the cases. We can add this without ifdef's, and only eventually a few implementations may need another variable added.
Patches welcome...
Best regards,
Wolfgang Denk