Hello Albert,
Albert ARIBAUD wrote:
Le 20/01/2011 21:43, Albert ARIBAUD a écrit :
Le 10/12/2010 10:33, Heiko Schocher a écrit :
Reuse the gd->tbl value for timestamp and add gd->lastinc for lastinc bss values in the arm926ejs timers implementation.
The usage of bss values in drivers before initialisation of bss is forbidden. In that special case some data in .rel.dyn gets corrupted.
This patch is similiar to the patch Dirk Behme posted for the armv7/omap-common/timer.c and added suggestions from Reinhard Meyer.
Tested on the arm926ejs mx27 based magnesium board Tested on the arm926ejs kirkwood based suen3 board
Signed-off-by: Heiko Schocherhs@denx.de cc: Albert ARIBAUDalbert.aribaud@free.fr cc: Prafulla Wadaskarprafulla@marvell.com cc: Stefano Babicsbabic@denx.de cc: Reinhard Meyeru-boot@emk-elektronik.de
Signed-off-by: Heiko Schocherhs@denx.de
arch/arm/cpu/arm926ejs/davinci/timer.c | 6 ++++-- arch/arm/cpu/arm926ejs/kirkwood/timer.c | 6 ++++-- arch/arm/cpu/arm926ejs/mb86r0x/timer.c | 6 ++++-- arch/arm/cpu/arm926ejs/mx25/timer.c | 6 ++++-- arch/arm/cpu/arm926ejs/mx27/timer.c | 6 ++++-- arch/arm/cpu/arm926ejs/omap/timer.c | 6 ++++-- arch/arm/cpu/arm926ejs/orion5x/timer.c | 6 ++++-- arch/arm/cpu/arm926ejs/spear/timer.c | 6 ++++-- arch/arm/cpu/arm926ejs/versatile/timer.c | 6 ++++-- 9 files changed, 36 insertions(+), 18 deletions(-)
[...]
#define TIMER_ENABLE (1<< 7) #define TIMER_MODE_MSK (1<< 6)
Applied to u-boot-arm, with a trivial merge for davinci.
Correction -- can't apply it *and* retain correct authorship in commit. Heiko, can you post a rebased version?
Ok, I rebase this patch against current mainline, and send a new version.
bye, Heiko