On 10/22/2010 01:25 PM, Jason Liu wrote:
Fix the board version printing issue on MX51EVK. Need to read the board version via get_cpu_rev and not rely on system_rev due to the system_rev not initialized at boardchecking time.
Signed-off-by: Jason Liu r64343@freescale.com
board/freescale/mx51evk/mx51evk.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-)
Hi Jason,
diff --git a/board/freescale/mx51evk/mx51evk.c b/board/freescale/mx51evk/mx51evk.c index d6bb71c..c532603 100644 --- a/board/freescale/mx51evk/mx51evk.c +++ b/board/freescale/mx51evk/mx51evk.c @@ -438,6 +438,8 @@ int board_late_init(void)
int checkboard(void) {
u32 system_rev = get_cpu_rev();
puts("Board: MX51EVK ");
switch (system_rev & 0xff) {
Then we need to clean up other part of the code: system_rev should be not declared globally in the file and must be removed. In the same time, get_board_rev() should be changed. It seems it is in any case wrong, because it returns the same value, and this means get_cpu_rev().
As this is a cpu revision and not a board revision, it is not correct. If the board revision cannot be determined correctly at runtime, we should return a fixed value. but certainly not the cpu revision.
Best regards, Stefano Babic