[coreboot] Patch set updated for coreboot: 775bfbd exynos5250: clean-ups for clock_init

David Hendricks (dhendrix@chromium.org) gerrit at coreboot.org
Thu Jan 10 05:11:58 CET 2013


David Hendricks (dhendrix at chromium.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2127

-gerrit

commit 775bfbdd6c8a6958b733e272e9ede3cdad9e4fc5
Author: David Hendricks <dhendrix at chromium.org>
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 <dhendrix at chromium.org>
---
 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;
 }
 



More information about the coreboot mailing list