David Hendricks ([email protected]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2127
-gerrit commit 085bfb17120cfb001cf949d3fba593908bd9e4ac Author: David Hendricks <[email protected]> Date: Wed Jan 9 18:06:41 2013 -0800 exynos5250: clean-ups for clock_init This does some clean-up for the exynos5250 clock_init.c: - No global data. - Remove some unused #includes - Hard-code the memory type for Elpida DRAM. This will need to be fixed eventually (or the system will be unstable), but is good enough for early bring-up and until we finish other re-factoring. Change-Id: Icd2cf8ba35058cbd1131666db311dfb77ef1a160 Signed-off-by: David Hendricks <[email protected]> --- src/cpu/samsung/exynos5250/clock_init.c | 27 ++++++++------------------- 1 file changed, 8 insertions(+), 19 deletions(-) diff --git a/src/cpu/samsung/exynos5250/clock_init.c b/src/cpu/samsung/exynos5250/clock_init.c index fd2a90f..ea91bb4 100644 --- a/src/cpu/samsung/exynos5250/clock_init.c +++ b/src/cpu/samsung/exynos5250/clock_init.c @@ -22,18 +22,6 @@ * MA 02111-1307 USA */ -#if 0 -#include <common.h> -#include <config.h> -//#include <fdtdec.h> -#include <arch/io.h> -#include <asm/arch/board.h> -#include <asm/arch/clk.h> -#include <asm/arch/clock.h> -#include <cpu/samsung/exynos5250/cpu.h> -#include <asm/arch/gpio.h> -#include <asm/arch-exynos/spl.h> -#endif #include <delay.h> #include <stdlib.h> #include <types.h> @@ -51,8 +39,6 @@ #include "clock_init.h" #include "setup.h" -//DECLARE_GLOBAL_DATA_PTR; - struct arm_clk_ratios arm_clk_ratios[] = { { .arm_freq_mhz = 600, @@ -585,10 +571,16 @@ struct mem_timings mem_timings[] = { */ #define BOARD_REV_ELPIDA_MEMORY 3 #define BOARD_REV_SAMSUNG_MEMORY 4 + +static inline int board_get_revision(void) +{ + /* FIXME: yuck! */ + return BOARD_REV_ELPIDA_MEMORY; +} + static int autodetect_memory(void) { -// int board_rev = board_get_revision(); - int board_rev = BOARD_REV_ELPIDA_MEMORY; + int board_rev = board_get_revision(); if (board_rev == -1) return -1; @@ -749,9 +741,6 @@ struct mem_timings *clock_get_mem_timings(void) return mem; } } - /* TODO: Call panic() here */ - while (1) - ; return NULL; } -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot

