SMDK5250: Convert lowlevel_init.S to .c and set PMIC values

This patch adds functions to intialsise power registers during spl boot.

Signed-off-by Rajeshwari Shinde <[email protected]>

---
Changes in V2:
- Moved from second patch to first patch
- Renamed ps_hold_setup to power_ps_hold_setup
- Added explanation before each power function definition

 arch/arm/cpu/armv7/exynos/power.c        |   27 +++++++++++++++++++++++++++
 arch/arm/include/asm/arch-exynos/power.h |   10 ++++++++++
 2 files changed, 37 insertions(+), 0 deletions(-)

arch/arm/cpu/armv7/exynos/power.c +27 -0

index d4bce6d..f0c107c 100644
@@ -95,3 +95,30 @@ void set_dp_phy_ctrl(unsigned int enable)
        if (cpu_is_exynos5())
                exynos5_dp_phy_control(enable);
 }
+
+uint32_t power_read_reset_status(void)
+{
+       struct exynos5_power *power =
+               (struct exynos5_power *)samsung_get_base_power();
+
+       return power->inform1;
+}
+
+void power_ps_hold_setup(void)
+{
+       struct exynos5_power *power =
+               (struct exynos5_power *)samsung_get_base_power();
+
+       /* Set PS-Hold high */
+       setbits_le32(&power->ps_hold_control,
+                       EXYNOS_PS_HOLD_CONTROL_DATA_HIGH);
+}
+
+void power_exit_wakeup(void)
+{
+       struct exynos5_power *power =
+               (struct exynos5_power *)samsung_get_base_power();
+       typedef void (*resume_func)(void);
+
+       ((resume_func)power->inform0)();
+}

arch/arm/include/asm/arch-exynos/power.h +0 -0

index d2fdb59..f6d0278 100644

arch/arm/include/asm/arch-exynos/power.h +4 -1

+void power_ps_hold_setup(void);
+
+/* Read the resume function and call it */
+void power_exit_wakeup(void);
 #endif
--

1.7.4.4

This patch converts lowlevel_init.S to lowlevel_init_c.c for SMDK5250.
Lowlevel.S as of now added only for SMDK5250 and same can be extended to other SOC in future.

Signed-off-by Rajeshwari Shinde <[email protected]>

---
Changes in V2:
- Renamed lowlevel_init.S to lowlevel.S and moved to arch/arm/cpu/armv7/exynos/
- Moved power mode defines to power.h
- Added early serial support.
- Renamed mem_reset to reset.

 arch/arm/cpu/armv7/exynos/Makefile       |    6 ++
 arch/arm/cpu/armv7/exynos/lowlevel.S     |   35 ++++++++
 arch/arm/include/asm/arch-exynos/power.h |    8 ++
 board/samsung/smdk5250/Makefile          |    2 +-
 board/samsung/smdk5250/dmc_common.c      |    4 +-
 board/samsung/smdk5250/dmc_init_ddr3.c   |    6 +-
 board/samsung/smdk5250/lowlevel_init.S   |   96 --------------------
 board/samsung/smdk5250/lowlevel_init.c   |   81 +++++++++++++++++

 board/samsung/smdk5250/setup.h           |   19 ++++-
 board/samsung/smdk5250/spl_boot.c        |  140 +++++++++++++++++++++++++++--
 spl/Makefile                             |    4 +
 11 files changed, 288 insertions(+), 113 deletions(-)
 create mode 100644 arch/arm/cpu/armv7/exynos/lowlevel.S
 delete mode 100644 board/samsung/smdk5250/lowlevel_init.S
 create mode 100644 board/samsung/smdk5250/lowlevel_init.c

arch/arm/cpu/armv7/exynos/Makefile +0 -0

index 9119961..2aa2722 100644

arch/arm/cpu/armv7/exynos/Makefile +3 -0

+endif
+endif
+
 COBJS  += clock.o power.o soc.o system.o pinmux.o
 
 SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)

arch/arm/cpu/armv7/exynos/lowlevel.S new file +35 -0

new file mode 100644
index 0000000..7307959
@@ -0,0 +1,35 @@
+/*
+ * Lowlevel setup for SMDK5250 board based on S5PC520
+ *
+ * Copyright (C) 2012 Samsung Electronics
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <config.h>
+#include <asm/arch/cpu.h>
+
+       .globl lowlevel_init
+lowlevel_init:
+       /*
+        * Set the stack pointer, although it will be overwritten by the caller
+        * It seems we will not boot if this function is empty.
+        */
+       ldr     sp, =CONFIG_IRAM_STACK
+       mov     pc, lr

arch/arm/include/asm/arch-exynos/power.h +8 -0

index f6d0278..d6fd29e 100644
@@ -874,4 +874,12 @@ void power_ps_hold_setup(void);
 
 /* Read the resume function and call it */
 void power_exit_wakeup(void);
+
+
+/* Power Down Modes
+ * User defined values in inform1 register
+ */
+#define EXYNOS_CHECK_SLEEP     0x00000BAD
+#define EXYNOS_CHECK_DIDLE     0xBAD00000
+#define EXYNOS_CHECK_LPA       0xABAD0000
 #endif

board/samsung/smdk5250/Makefile +0 -1

index 47c6a5a..7eaef09 100644
@@ -24,7 +24,6 @@ include $(TOPDIR)/config.mk
 
 LIB    = $(obj)lib$(BOARD).o
 
-SOBJS  := lowlevel_init.o
 
 COBJS  := clock_init.o
 COBJS  += dmc_common.o dmc_init_ddr3.o

board/samsung/smdk5250/Makefile +0 -0

 endif
 
 SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)

board/samsung/smdk5250/dmc_common.c +2 -2

index 109602a..f637bf9 100644
@@ -175,7 +175,7 @@ void dmc_config_memory(struct mem_timings *mem, struct exynos5_dmc *dmc)
        writel(DMC_MEMBASECONFIG1_VAL, &dmc->membaseconfig1);
 }
 
-void mem_ctrl_init()
+void mem_ctrl_init(int reset)
 {
        struct spl_machine_param *param = spl_get_machine_params();
        struct mem_timings *mem;

@@ -185,7 +185,7 @@ void mem_ctrl_init()
 
        /* If there are any other memory variant, add their init call below */
        if (param->mem_type == DDR_MODE_DDR3) {
-               ret = ddr3_mem_ctrl_init(mem, param->mem_iv_size);
+               ret = ddr3_mem_ctrl_init(mem, param->mem_iv_size, reset);
                if (ret) {
                        /* will hang if failed to init memory control */
                        while (1)

board/samsung/smdk5250/dmc_init_ddr3.c +4 -2

index e050790..a5a70df 100644
@@ -40,7 +40,8 @@ static void reset_phy_ctrl(void)
        writel(DDR3PHY_CTRL_PHY_RESET, &clk->lpddr3phy_ctrl);
 }
 
-int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size)
+int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size,
+                       int reset)
 {
        unsigned int val;
        struct exynos5_phy_control *phy0_ctrl, *phy1_ctrl;

@@ -51,7 +52,8 @@ int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size)
        phy1_ctrl = (struct exynos5_phy_control *)EXYNOS5_DMC_PHY1_BASE;
        dmc = (struct exynos5_dmc *)EXYNOS5_DMC_CTRL_BASE;
 
-       reset_phy_ctrl();
+       if (reset)
+               reset_phy_ctrl();
 
        /* Set Impedance Output Driver */
        val = (mem->impedance << CA_CK_DRVR_DS_OFFSET) |

board/samsung/smdk5250/lowlevel_init.S deleted +0 -96

deleted file mode 100644
index bc6cb6f..0000000
@@ -1,96 +0,0 @@
-/*
- * Lowlevel setup for SMDK5250 board based on S5PC520
- *
- * Copyright (C) 2012 Samsung Electronics
- *
- * See file CREDITS for list of people who contributed to this
- * project.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
- * MA 02111-1307 USA
- */
-
-#include <config.h>
-#include <version.h>
-#include <asm/arch/cpu.h>
-
-_TEXT_BASE:
-       .word   CONFIG_SYS_TEXT_BASE
-
-       .globl lowlevel_init
-lowlevel_init:
-
-       /* use iRAM stack in bl2 */
-       ldr     sp, =CONFIG_IRAM_STACK
-       stmdb   r13!, {ip,lr}
-
-       /* check reset status */
-       ldr     r0, =(EXYNOS5_POWER_BASE + INFORM1_OFFSET)
-       ldr     r1, [r0]
-
-       /* AFTR wakeup reset */
-       ldr     r2, =S5P_CHECK_DIDLE
-       cmp     r1, r2
-       beq     exit_wakeup
-
-       /* LPA wakeup reset */
-       ldr     r2, =S5P_CHECK_LPA
-       cmp     r1, r2
-       beq     exit_wakeup
-
-       /* Sleep wakeup reset */
-       ldr     r2, =S5P_CHECK_SLEEP
-       cmp     r1, r2
-       beq     wakeup_reset
-
-       /*
-        * If U-boot is already running in RAM, no need to relocate U-Boot.
-        * Memory controller must be configured before relocating U-Boot
-        * in ram.
-        */
-       ldr     r0, =0x0ffffff          /* r0 <- Mask Bits*/
-       bic     r1, pc, r0              /* pc <- current addr of code */
-                                       /* r1 <- unmasked bits of pc */
-       ldr     r2, _TEXT_BASE          /* r2 <- original base addr in ram */
-       bic     r2, r2, r0              /* r2 <- unmasked bits of r2*/
-       cmp     r1, r2                  /* compare r1, r2 */
-       beq     1f                      /* r0 == r1 then skip sdram init */
-
-       /* init system clock */
-       bl      system_clock_init
-
-       /* Memory initialize */
-       bl      mem_ctrl_init
-
-1:
-       bl      tzpc_init
-       ldmia   r13!, {ip,pc}
-
-wakeup_reset:
-       bl      system_clock_init
-       bl      mem_ctrl_init
-       bl      tzpc_init
-
-exit_wakeup:
-       /* Load return address and jump to kernel */
-       ldr     r0, =(EXYNOS5_POWER_BASE + INFORM0_OFFSET)
-
-       /* r1 = physical address of exynos5_cpu_resume function*/
-       ldr     r1, [r0]
-
-       /* Jump to kernel */
-       mov     pc, r1
-       nop
-       nop

board/samsung/smdk5250/lowlevel_init.c new file +67 -0

new file mode 100644
index 0000000..22bdd2b
@@ -0,0 +1,81 @@
+/*
+ * Lowlevel setup for SMDK5250 board based on S5PC520
+ *
+ * Copyright (C) 2012 Samsung Electronics
+ * Copyright (c) 2012 The Chromium OS Authors.
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <config.h>
+#include <asm/arch/cpu.h>
+#include <asm/arch/dmc.h>
+#include <asm/arch/power.h>
+#include <asm/arch/tzpc.h>
+#include <asm/arch/periph.h>
+#include <asm/arch/pinmux.h>
+#include "setup.h"
+
+/* These are the things we can do during low-level init */
+enum {
+       DO_WAKEUP       = 1 << 0,
+       DO_CLOCKS       = 1 << 1,
+       DO_MEM_RESET    = 1 << 2,
+       DO_UART         = 1 << 3,
+};
+
+int do_lowlevel_init(void)
+{
+       uint32_t reset_status;
+       int actions = 0;
+
+       arch_cpu_init();
+
+       reset_status = power_read_reset_status();
+
+       switch (reset_status) {
+       case EXYNOS_CHECK_SLEEP:
+               actions = DO_CLOCKS | DO_WAKEUP;
+               break;
+       case EXYNOS_CHECK_DIDLE:
+       case EXYNOS_CHECK_LPA:
+               actions = DO_WAKEUP;
+               break;
+       default:
+               /* This is a normal boot (not a wake from sleep) */
+               actions = DO_CLOCKS | DO_MEM_RESET | DO_UART;
+       }
+
+       if (actions & DO_CLOCKS)
+               system_clock_init();

board/samsung/smdk5250/lowlevel_init.c new file +4 -0

+       }
+
+       return actions & DO_WAKEUP;
+}

board/samsung/smdk5250/setup.h +17 -2

index a159601..f1d9f79 100644
@@ -537,9 +537,11 @@ enum {
  *                     which the DMC uses to decide how to split a memory
  *                     chunk into smaller chunks to support concurrent
  *                     accesses; may vary across boards.
+ * @param reset                Reset DDR PHY during initialization.
  * @return 0 if ok, SETUP_ERR_... if there is a problem
  */
-int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size);
+int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size,
+                       int reset);
 
 /*
  * Configure ZQ I/O interface

@@ -587,8 +589,21 @@ void dmc_config_memory(struct mem_timings *mem, struct exynos5_dmc *dmc);
  */
 void update_reset_dll(struct exynos5_dmc *, enum ddr_mode);
 
+/*
+ * Memory initialization
+ *
+ * @param reset     Reset PHY during initialization.
+ */
+void mem_ctrl_init(int reset);
+
 void sdelay(unsigned long);
-void mem_ctrl_init(void);
 void system_clock_init(void);
 void tzpc_init(void);
+
+/**
+ * Init subsystems according to the reset status
+ *
+ * @return 0 for a normal boot, non-zero for a resume
+ */
+int do_lowlevel_init(void);
 #endif

board/samsung/smdk5250/spl_boot.c +6 -3

index d8f3c1e..a1c8d3d 100644
@@ -20,18 +20,16 @@
  * MA 02111-1307 USA
  */
 
-#include<common.h>
-#include<config.h>
+#include <common.h>
+#include <config.h>
+#include <asm/arch/spl.h>
+#include <asm/arch/cpu.h>
+#include <asm/arch/power.h>
+#include "setup.h"
 
-enum boot_mode {

board/samsung/smdk5250/spl_boot.c +91 -0

 /*
 * Copy U-boot from mmc to RAM:

@@ -62,15 +60,49 @@ void copy_uboot_to_ram(void)
        }
 }
 
+void memzero(void *s, size_t n)
+{
+       char *ptr = s;
+       size_t i;
+
+       for (i = 0; i < n; i++)
+               *ptr++ = '\0';
+}
+
+/**
+ * Set up the U-Boot global_data pointer
+ *
+ * This sets the address of the global data, and sets up basic values.
+ *
+ * @param gdp   Value to give to gd
+ */
+static void setup_global_data(gd_t *gdp)
+{
+       gd = gdp;
+       memzero((void *)gd, sizeof(gd_t));
+       gd->flags |= GD_FLG_RELOC;
+       gd->baudrate = CONFIG_BAUDRATE;
+       gd->have_console = 1;
+}
+
 void board_init_f(unsigned long bootflag)
 {
+       __attribute__((aligned(8))) gd_t local_gd;
        __attribute__((noreturn)) void (*uboot)(void);
+
+       setup_global_data(&local_gd);
+
+       if (do_lowlevel_init())
+               power_exit_wakeup();
+
        copy_uboot_to_ram();
 
        /* Jump to U-Boot image */
        uboot = (void *)CONFIG_SYS_TEXT_BASE;
        (*uboot)();
+
        /* Never returns Here */
+       panic("%s: u-boot jump failed", __func__);
 }
 
 /* Place Holders */

@@ -83,3 +115,91 @@ void board_init_r(gd_t *id, ulong dest_addr)
 }
 
 void save_boot_params(u32 r0, u32 r1, u32 r2, u32 r3) {}
+
+/*
+ * The following functions are required when linking console library to SPL.
+ *
+ * Enabling UART in SPL u-boot requires console library. But some
+ * functions we needed in the console library depends on a bunch
+ * of library in libgeneric, like lib/ctype.o, lib/div64.o, lib/string.o,
+ * and lib/vsprintf.o. Adding them makes the SPL u-boot too large and not
+ * fit into the expected size.
+ *
+ * So we mock these functions in SPL, i.e. vsprintf(), panic(), etc.,
+ * in order to cut its dependency.
+ */
+static int _vscnprintf(char *buf, size_t size, const char *fmt, va_list args)
+{
+       char *str = buf, *s;
+       char *end = str + size - 1;
+       ulong u;
+
+       if (size == 0)
+               return -1;
+
+       /*
+        * We won't implement all full functions of vsprintf().
+        * We only implement %s and %u, and ignore others and directly use
+        * the original format string as its result.
+        */
+
+       while (*fmt && (str < end)) {
+               if (*fmt != '%') {
+                       *str++ = *fmt++;
+                       continue;
+               }
+               fmt++;
+               switch (*fmt) {
+               case '%':
+                       *str++ = *fmt++;
+                       break;
+               case 's':
+                       fmt++;
+                       s = va_arg(args, char *);
+                       while (*s && (str < end))
+                               *str++ = *s++;
+                       break;
+               case 'u':
+                       fmt++;
+                       u = va_arg(args, ulong);
+                       s = simple_itoa(u);
+                               while (*s && (str < end))
+                                       *str++ = *s++;
+                               break;
+               default:
+                       /* Print the original string for unsupported formats */
+                       *str++ = '%';
+                       if  (str < end)
+                               *str++ = *fmt++;
+                       }

board/samsung/smdk5250/spl_boot.c +21 -0

+
+char *simple_itoa(ulong i)
+{
+       /* 21 digits plus null terminator, good for 64-bit or smaller ints */
+       static char local[22] __attribute__((section(".data")));
+       char *p = &local[21];
+
+       *p-- = '\0';
+       do {
+               *p-- = '0' + i % 10;
+               i /= 10;
+       } while (i > 0);
+       return p + 1;
+}
+
+void hang(void)
+{
+       puts("### ERROR ### Please RESET the board ###\n");
+       for (;;)
+               ;
+}

spl/Makefile +4 -1

index 6dbb105..3aab466 100644
@@ -86,6 +86,10 @@ ifneq ($(CONFIG_AM33XX)$(CONFIG_OMAP34XX)$(CONFIG_OMAP44XX)$(CONFIG_OMAP54XX),)
 LIBS-y += $(CPUDIR)/omap-common/libomap-common.o
 endif
 
+ifneq ($(CONFIG_EXYNOS4)$(CONFIG_EXYNOS5),)
+LIBS-y += $(CPUDIR)/s5p-common/libs5p-common.o
+endif
+
 ifeq ($(SOC),tegra20)
 LIBS-y += arch/$(ARCH)/cpu/$(SOC)-common/lib$(SOC)-common.o
 LIBS-y += arch/$(ARCH)/cpu/tegra-common/libcputegra-common.o
--

1.7.4.4

Hi Rajeshwari,

Quoted 4 lines · Rajeshwari Shinde

This patch converts lowlevel_init.S to lowlevel_init_c.c for SMDK5250.
Lowlevel.S as of now added only for SMDK5250 and same can be extended to other SOC in future.

Should perhaps also mention new feature (controllable memory reset for resume?)

Quoted 10 lines

Signed-off-by Rajeshwari Shinde <[email protected]>

---
Changes in V2:
- Renamed lowlevel_init.S to lowlevel.S and moved to arch/arm/cpu/armv7/exynos/
- Moved power mode defines to power.h
- Added early serial support.
- Renamed mem_reset to reset.

Quoted 8 lines

 arch/arm/cpu/armv7/exynos/Makefile       |    6 ++
 arch/arm/cpu/armv7/exynos/lowlevel.S     |   35 ++++++++
 arch/arm/include/asm/arch-exynos/power.h |    8 ++
 board/samsung/smdk5250/Makefile          |    2 +-
 board/samsung/smdk5250/dmc_common.c      |    4 +-
 board/samsung/smdk5250/dmc_init_ddr3.c   |    6 +-
 board/samsung/smdk5250/lowlevel_init.S   |   96 --------------------
 board/samsung/smdk5250/lowlevel_init.c   |   81 +++++++++++++++++

Any change we could move all of this to arch/arm/cpu/armv7/exynos...?

It really doesn't relate to this board alone, but to the chip.

Quoted 13 lines

 board/samsung/smdk5250/setup.h           |   19 ++++-
 board/samsung/smdk5250/spl_boot.c        |  140 +++++++++++++++++++++++++++--
 spl/Makefile                             |    4 +
 11 files changed, 288 insertions(+), 113 deletions(-)
 create mode 100644 arch/arm/cpu/armv7/exynos/lowlevel.S
 delete mode 100644 board/samsung/smdk5250/lowlevel_init.S
 create mode 100644 board/samsung/smdk5250/lowlevel_init.c

arch/arm/cpu/armv7/exynos/Makefile +0 -0

index 9119961..2aa2722 100644

arch/arm/cpu/armv7/exynos/Makefile include $(TOPDIR)/config.mk · line 27

Quoted 6 lines

arch/arm/cpu/armv7/exynos/Makefile +3 -0

@@ -22,6 +22,12 @@ include $(TOPDIR)/config.mk
 
 LIB    = $(obj)lib$(SOC).o
 
+ifdef CONFIG_SMDK5250
+ifdef CONFIG_SPL
+COBJS  += lowlevel.o

Could do:

COBJS-$(CONFIG_SPL) += lowlevel.o

and remove the inner ifdef

Quoted 78 lines

arch/arm/cpu/armv7/exynos/Makefile +3 -0

+endif
+endif
+
 COBJS  += clock.o power.o soc.o system.o pinmux.o
 
 SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)

arch/arm/cpu/armv7/exynos/lowlevel.S new file +35 -0

new file mode 100644
index 0000000..7307959
@@ -0,0 +1,35 @@
+/*
+ * Lowlevel setup for SMDK5250 board based on S5PC520
+ *
+ * Copyright (C) 2012 Samsung Electronics
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <config.h>
+#include <asm/arch/cpu.h>
+
+       .globl lowlevel_init
+lowlevel_init:
+       /*
+        * Set the stack pointer, although it will be overwritten by the caller
+        * It seems we will not boot if this function is empty.
+        */
+       ldr     sp, =CONFIG_IRAM_STACK
+       mov     pc, lr

arch/arm/include/asm/arch-exynos/power.h +8 -0

index f6d0278..d6fd29e 100644
@@ -874,4 +874,12 @@ void power_ps_hold_setup(void);
 
 /* Read the resume function and call it */
 void power_exit_wakeup(void);
+
+
+/* Power Down Modes
+ * User defined values in inform1 register
+ */
+#define EXYNOS_CHECK_SLEEP     0x00000BAD
+#define EXYNOS_CHECK_DIDLE     0xBAD00000
+#define EXYNOS_CHECK_LPA       0xABAD0000
 #endif

board/samsung/smdk5250/Makefile +0 -1

index 47c6a5a..7eaef09 100644
@@ -24,7 +24,6 @@ include $(TOPDIR)/config.mk
 
 LIB    = $(obj)lib$(BOARD).o
 
-SOBJS  := lowlevel_init.o
 
 COBJS  := clock_init.o
 COBJS  += dmc_common.o dmc_init_ddr3.o

board/samsung/smdk5250/Makefile endif · line 39

Quoted 4 lines

board/samsung/smdk5250/Makefile +1 -0

@@ -37,6 +36,7 @@ endif
 
 ifdef CONFIG_SPL_BUILD
 COBJS  += spl_boot.o
+COBJS  += lowlevel_init.o

Can you use this form instead of ifdef here?

COBJS-$(CONFIG_SPL) += lowlevel_init.o

Quoted 228 lines

board/samsung/smdk5250/Makefile +0 -0

 endif
 
 SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)

board/samsung/smdk5250/dmc_common.c +2 -2

index 109602a..f637bf9 100644
@@ -175,7 +175,7 @@ void dmc_config_memory(struct mem_timings *mem, struct exynos5_dmc *dmc)
        writel(DMC_MEMBASECONFIG1_VAL, &dmc->membaseconfig1);
 }
 
-void mem_ctrl_init()
+void mem_ctrl_init(int reset)
 {
        struct spl_machine_param *param = spl_get_machine_params();
        struct mem_timings *mem;

@@ -185,7 +185,7 @@ void mem_ctrl_init()
 
        /* If there are any other memory variant, add their init call below */
        if (param->mem_type == DDR_MODE_DDR3) {
-               ret = ddr3_mem_ctrl_init(mem, param->mem_iv_size);
+               ret = ddr3_mem_ctrl_init(mem, param->mem_iv_size, reset);
                if (ret) {
                        /* will hang if failed to init memory control */
                        while (1)

board/samsung/smdk5250/dmc_init_ddr3.c +4 -2

index e050790..a5a70df 100644
@@ -40,7 +40,8 @@ static void reset_phy_ctrl(void)
        writel(DDR3PHY_CTRL_PHY_RESET, &clk->lpddr3phy_ctrl);
 }
 
-int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size)
+int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size,
+                       int reset)
 {
        unsigned int val;
        struct exynos5_phy_control *phy0_ctrl, *phy1_ctrl;

@@ -51,7 +52,8 @@ int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size)
        phy1_ctrl = (struct exynos5_phy_control *)EXYNOS5_DMC_PHY1_BASE;
        dmc = (struct exynos5_dmc *)EXYNOS5_DMC_CTRL_BASE;
 
-       reset_phy_ctrl();
+       if (reset)
+               reset_phy_ctrl();
 
        /* Set Impedance Output Driver */
        val = (mem->impedance << CA_CK_DRVR_DS_OFFSET) |

board/samsung/smdk5250/lowlevel_init.S deleted +0 -96

deleted file mode 100644
index bc6cb6f..0000000
@@ -1,96 +0,0 @@
-/*
- * Lowlevel setup for SMDK5250 board based on S5PC520
- *
- * Copyright (C) 2012 Samsung Electronics
- *
- * See file CREDITS for list of people who contributed to this
- * project.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
- * MA 02111-1307 USA
- */
-
-#include <config.h>
-#include <version.h>
-#include <asm/arch/cpu.h>
-
-_TEXT_BASE:
-       .word   CONFIG_SYS_TEXT_BASE
-
-       .globl lowlevel_init
-lowlevel_init:
-
-       /* use iRAM stack in bl2 */
-       ldr     sp, =CONFIG_IRAM_STACK
-       stmdb   r13!, {ip,lr}
-
-       /* check reset status */
-       ldr     r0, =(EXYNOS5_POWER_BASE + INFORM1_OFFSET)
-       ldr     r1, [r0]
-
-       /* AFTR wakeup reset */
-       ldr     r2, =S5P_CHECK_DIDLE
-       cmp     r1, r2
-       beq     exit_wakeup
-
-       /* LPA wakeup reset */
-       ldr     r2, =S5P_CHECK_LPA
-       cmp     r1, r2
-       beq     exit_wakeup
-
-       /* Sleep wakeup reset */
-       ldr     r2, =S5P_CHECK_SLEEP
-       cmp     r1, r2
-       beq     wakeup_reset
-
-       /*
-        * If U-boot is already running in RAM, no need to relocate U-Boot.
-        * Memory controller must be configured before relocating U-Boot
-        * in ram.
-        */
-       ldr     r0, =0x0ffffff          /* r0 <- Mask Bits*/
-       bic     r1, pc, r0              /* pc <- current addr of code */
-                                       /* r1 <- unmasked bits of pc */
-       ldr     r2, _TEXT_BASE          /* r2 <- original base addr in ram */
-       bic     r2, r2, r0              /* r2 <- unmasked bits of r2*/
-       cmp     r1, r2                  /* compare r1, r2 */
-       beq     1f                      /* r0 == r1 then skip sdram init */
-
-       /* init system clock */
-       bl      system_clock_init
-
-       /* Memory initialize */
-       bl      mem_ctrl_init
-
-1:
-       bl      tzpc_init
-       ldmia   r13!, {ip,pc}
-
-wakeup_reset:
-       bl      system_clock_init
-       bl      mem_ctrl_init
-       bl      tzpc_init
-
-exit_wakeup:
-       /* Load return address and jump to kernel */
-       ldr     r0, =(EXYNOS5_POWER_BASE + INFORM0_OFFSET)
-
-       /* r1 = physical address of exynos5_cpu_resume function*/
-       ldr     r1, [r0]
-
-       /* Jump to kernel */
-       mov     pc, r1
-       nop
-       nop

board/samsung/smdk5250/lowlevel_init.c new file +71 -0

new file mode 100644
index 0000000..22bdd2b
@@ -0,0 +1,81 @@
+/*
+ * Lowlevel setup for SMDK5250 board based on S5PC520
+ *
+ * Copyright (C) 2012 Samsung Electronics
+ * Copyright (c) 2012 The Chromium OS Authors.
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <config.h>
+#include <asm/arch/cpu.h>
+#include <asm/arch/dmc.h>
+#include <asm/arch/power.h>
+#include <asm/arch/tzpc.h>
+#include <asm/arch/periph.h>
+#include <asm/arch/pinmux.h>
+#include "setup.h"
+
+/* These are the things we can do during low-level init */
+enum {
+       DO_WAKEUP       = 1 << 0,
+       DO_CLOCKS       = 1 << 1,
+       DO_MEM_RESET    = 1 << 2,
+       DO_UART         = 1 << 3,
+};
+
+int do_lowlevel_init(void)
+{
+       uint32_t reset_status;
+       int actions = 0;
+
+       arch_cpu_init();
+
+       reset_status = power_read_reset_status();
+
+       switch (reset_status) {
+       case EXYNOS_CHECK_SLEEP:
+               actions = DO_CLOCKS | DO_WAKEUP;
+               break;
+       case EXYNOS_CHECK_DIDLE:
+       case EXYNOS_CHECK_LPA:
+               actions = DO_WAKEUP;
+               break;
+       default:
+               /* This is a normal boot (not a wake from sleep) */
+               actions = DO_CLOCKS | DO_MEM_RESET | DO_UART;
+       }
+
+       if (actions & DO_CLOCKS)
+               system_clock_init();
+
+       if (actions & DO_UART) {
+               exynos_pinmux_config(PERIPH_ID_UART3, PINMUX_FLAG_NONE);
+               serial_init();

board/samsung/smdk5250/lowlevel_init.c line 77

Quoted 6 lines

board/samsung/smdk5250/lowlevel_init.c new file +6 -0

+               timer_init();
+       }
+
+       if (actions & DO_CLOCKS) {
+               mem_ctrl_init(actions & DO_MEM_RESET);
+               tzpc_init();

I think serial SPL support is coming later.

Quoted 63 lines

board/samsung/smdk5250/lowlevel_init.c new file +4 -0

+       }
+
+       return actions & DO_WAKEUP;
+}

board/samsung/smdk5250/setup.h +17 -2

index a159601..f1d9f79 100644
@@ -537,9 +537,11 @@ enum {
  *                     which the DMC uses to decide how to split a memory
  *                     chunk into smaller chunks to support concurrent
  *                     accesses; may vary across boards.
+ * @param reset                Reset DDR PHY during initialization.
  * @return 0 if ok, SETUP_ERR_... if there is a problem
  */
-int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size);
+int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size,
+                       int reset);
 
 /*
  * Configure ZQ I/O interface

@@ -587,8 +589,21 @@ void dmc_config_memory(struct mem_timings *mem, struct exynos5_dmc *dmc);
  */
 void update_reset_dll(struct exynos5_dmc *, enum ddr_mode);
 
+/*
+ * Memory initialization
+ *
+ * @param reset     Reset PHY during initialization.
+ */
+void mem_ctrl_init(int reset);
+
 void sdelay(unsigned long);
-void mem_ctrl_init(void);
 void system_clock_init(void);
 void tzpc_init(void);
+
+/**
+ * Init subsystems according to the reset status
+ *
+ * @return 0 for a normal boot, non-zero for a resume
+ */
+int do_lowlevel_init(void);
 #endif

board/samsung/smdk5250/spl_boot.c +6 -3

index d8f3c1e..a1c8d3d 100644
@@ -20,18 +20,16 @@
  * MA 02111-1307 USA
  */
 
-#include<common.h>
-#include<config.h>
+#include <common.h>
+#include <config.h>
+#include <asm/arch/spl.h>
+#include <asm/arch/cpu.h>
+#include <asm/arch/power.h>
+#include "setup.h"
 
-enum boot_mode {

board/samsung/smdk5250/spl_boot.c line 32

Quoted 10 lines

board/samsung/smdk5250/spl_boot.c +2 -7

-       BOOT_MODE_MMC = 4,
-       BOOT_MODE_SERIAL = 20,
-       /* Boot based on Operating Mode pin settings */
-       BOOT_MODE_OM = 32,
-       BOOT_MODE_USB,  /* Boot using USB download */
-};
+DECLARE_GLOBAL_DATA_PTR;
 
-       typedef u32 (*spi_copy_func_t)(u32 offset, u32 nblock, u32 dst);
+typedef u32 (*spi_copy_func_t)(u32 offset, u32 nblock, u32 dst);

What is happening with thsi file? I think there is another patch which changes things here.

Quoted 118 lines

board/samsung/smdk5250/spl_boot.c +95 -0

 /*
 * Copy U-boot from mmc to RAM:

@@ -62,15 +60,49 @@ void copy_uboot_to_ram(void)
        }
 }
 
+void memzero(void *s, size_t n)
+{
+       char *ptr = s;
+       size_t i;
+
+       for (i = 0; i < n; i++)
+               *ptr++ = '\0';
+}
+
+/**
+ * Set up the U-Boot global_data pointer
+ *
+ * This sets the address of the global data, and sets up basic values.
+ *
+ * @param gdp   Value to give to gd
+ */
+static void setup_global_data(gd_t *gdp)
+{
+       gd = gdp;
+       memzero((void *)gd, sizeof(gd_t));
+       gd->flags |= GD_FLG_RELOC;
+       gd->baudrate = CONFIG_BAUDRATE;
+       gd->have_console = 1;
+}
+
 void board_init_f(unsigned long bootflag)
 {
+       __attribute__((aligned(8))) gd_t local_gd;
        __attribute__((noreturn)) void (*uboot)(void);
+
+       setup_global_data(&local_gd);
+
+       if (do_lowlevel_init())
+               power_exit_wakeup();
+
        copy_uboot_to_ram();
 
        /* Jump to U-Boot image */
        uboot = (void *)CONFIG_SYS_TEXT_BASE;
        (*uboot)();
+
        /* Never returns Here */
+       panic("%s: u-boot jump failed", __func__);
 }
 
 /* Place Holders */

@@ -83,3 +115,91 @@ void board_init_r(gd_t *id, ulong dest_addr)
 }
 
 void save_boot_params(u32 r0, u32 r1, u32 r2, u32 r3) {}
+
+/*
+ * The following functions are required when linking console library to SPL.
+ *
+ * Enabling UART in SPL u-boot requires console library. But some
+ * functions we needed in the console library depends on a bunch
+ * of library in libgeneric, like lib/ctype.o, lib/div64.o, lib/string.o,
+ * and lib/vsprintf.o. Adding them makes the SPL u-boot too large and not
+ * fit into the expected size.
+ *
+ * So we mock these functions in SPL, i.e. vsprintf(), panic(), etc.,
+ * in order to cut its dependency.
+ */
+static int _vscnprintf(char *buf, size_t size, const char *fmt, va_list args)
+{
+       char *str = buf, *s;
+       char *end = str + size - 1;
+       ulong u;
+
+       if (size == 0)
+               return -1;
+
+       /*
+        * We won't implement all full functions of vsprintf().
+        * We only implement %s and %u, and ignore others and directly use
+        * the original format string as its result.
+        */
+
+       while (*fmt && (str < end)) {
+               if (*fmt != '%') {
+                       *str++ = *fmt++;
+                       continue;
+               }
+               fmt++;
+               switch (*fmt) {
+               case '%':
+                       *str++ = *fmt++;
+                       break;
+               case 's':
+                       fmt++;
+                       s = va_arg(args, char *);
+                       while (*s && (str < end))
+                               *str++ = *s++;
+                       break;
+               case 'u':
+                       fmt++;
+                       u = va_arg(args, ulong);
+                       s = simple_itoa(u);
+                               while (*s && (str < end))
+                                       *str++ = *s++;
+                               break;
+               default:
+                       /* Print the original string for unsupported formats */
+                       *str++ = '%';
+                       if  (str < end)
+                               *str++ = *fmt++;
+                       }
+               }
+       *str = '\0';
+       return str - buf;
+}

board/samsung/smdk5250/spl_boot.c void board_init_r(gd_t *id, ulong dest_addr) · line 184

Quoted 6 lines

board/samsung/smdk5250/spl_boot.c +6 -0

+
+/* Implement vsprintf in case someone doesn't have CONFIG_SYS_VSNPRINTF */
+int vsprintf(char *buf, const char *fmt, va_list args)
+{
+       return _vscnprintf(buf, CONFIG_SYS_PBSIZE, fmt, args);
+}

This is ready for use by SPL serial support, right?

Quoted 38 lines

board/samsung/smdk5250/spl_boot.c +21 -0

+
+char *simple_itoa(ulong i)
+{
+       /* 21 digits plus null terminator, good for 64-bit or smaller ints */
+       static char local[22] __attribute__((section(".data")));
+       char *p = &local[21];
+
+       *p-- = '\0';
+       do {
+               *p-- = '0' + i % 10;
+               i /= 10;
+       } while (i > 0);
+       return p + 1;
+}
+
+void hang(void)
+{
+       puts("### ERROR ### Please RESET the board ###\n");
+       for (;;)
+               ;
+}

spl/Makefile +4 -1

index 6dbb105..3aab466 100644
@@ -86,6 +86,10 @@ ifneq ($(CONFIG_AM33XX)$(CONFIG_OMAP34XX)$(CONFIG_OMAP44XX)$(CONFIG_OMAP54XX),)
 LIBS-y += $(CPUDIR)/omap-common/libomap-common.o
 endif
 
+ifneq ($(CONFIG_EXYNOS4)$(CONFIG_EXYNOS5),)
+LIBS-y += $(CPUDIR)/s5p-common/libs5p-common.o
+endif
+
 ifeq ($(SOC),tegra20)
 LIBS-y += arch/$(ARCH)/cpu/$(SOC)-common/lib$(SOC)-common.o
 LIBS-y += arch/$(ARCH)/cpu/tegra-common/libcputegra-common.o
--

Quoted 2 lines

1.7.4.4

Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Hi Simon,

Thank you for comments.

Quoted 10 lines · Simon Glass

Hi Rajeshwari,

Quoted 4 lines · Rajeshwari Shinde

This patch converts lowlevel_init.S to lowlevel_init_c.c for SMDK5250.
Lowlevel.S as of now added only for SMDK5250 and same can be extended to other SOC in future.

Should perhaps also mention new feature (controllable memory reset for resume?)

-OK

Quoted 19 lines

Quoted 18 lines

Signed-off-by Rajeshwari Shinde <[email protected]>

---
Changes in V2:
- Renamed lowlevel_init.S to lowlevel.S and moved to arch/arm/cpu/armv7/exynos/
- Moved power mode defines to power.h
- Added early serial support.
- Renamed mem_reset to reset.

 arch/arm/cpu/armv7/exynos/Makefile       |    6 ++
 arch/arm/cpu/armv7/exynos/lowlevel.S     |   35 ++++++++
 arch/arm/include/asm/arch-exynos/power.h |    8 ++
 board/samsung/smdk5250/Makefile          |    2 +-
 board/samsung/smdk5250/dmc_common.c      |    4 +-
 board/samsung/smdk5250/dmc_init_ddr3.c   |    6 +-
 board/samsung/smdk5250/lowlevel_init.S   |   96 --------------------
 board/samsung/smdk5250/lowlevel_init.c   |   81 +++++++++++++++++

Quoted 1 line

Any change we could move all of this to arch/arm/cpu/armv7/exynos...?

We do not have a separate directory for exynos5 and exynos4 and if we add all these files in arch/arm/cpu/armv7/exynos it would break the compilation for EXYNOS4. Also Later versions of exynos5 have different memory and timing variants.

Quoted 25 lines

It really doesn't relate to this board alone, but to the chip.

Quoted 13 lines

 board/samsung/smdk5250/setup.h           |   19 ++++-
 board/samsung/smdk5250/spl_boot.c        |  140 +++++++++++++++++++++++++++--
 spl/Makefile                             |    4 +
 11 files changed, 288 insertions(+), 113 deletions(-)
 create mode 100644 arch/arm/cpu/armv7/exynos/lowlevel.S
 delete mode 100644 board/samsung/smdk5250/lowlevel_init.S
 create mode 100644 board/samsung/smdk5250/lowlevel_init.c

arch/arm/cpu/armv7/exynos/Makefile +0 -0

index 9119961..2aa2722 100644

arch/arm/cpu/armv7/exynos/Makefile include $(TOPDIR)/config.mk · line 27

Quoted 6 lines

arch/arm/cpu/armv7/exynos/Makefile +3 -0

@@ -22,6 +22,12 @@ include $(TOPDIR)/config.mk
 
 LIB    = $(obj)lib$(SOC).o
 
+ifdef CONFIG_SMDK5250
+ifdef CONFIG_SPL
+COBJS  += lowlevel.o

Could do:

Quoted 3 lines

COBJS-$(CONFIG_SPL) += lowlevel.o

and remove the inner ifdef

- OK

Quoted 316 lines

Quoted 78 lines

arch/arm/cpu/armv7/exynos/Makefile +3 -0

+endif
+endif
+
 COBJS  += clock.o power.o soc.o system.o pinmux.o
 
 SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)

arch/arm/cpu/armv7/exynos/lowlevel.S new file +35 -0

new file mode 100644
index 0000000..7307959
@@ -0,0 +1,35 @@
+/*
+ * Lowlevel setup for SMDK5250 board based on S5PC520
+ *
+ * Copyright (C) 2012 Samsung Electronics
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <config.h>
+#include <asm/arch/cpu.h>
+
+       .globl lowlevel_init
+lowlevel_init:
+       /*
+        * Set the stack pointer, although it will be overwritten by the caller
+        * It seems we will not boot if this function is empty.
+        */
+       ldr     sp, =CONFIG_IRAM_STACK
+       mov     pc, lr

arch/arm/include/asm/arch-exynos/power.h +8 -0

index f6d0278..d6fd29e 100644
@@ -874,4 +874,12 @@ void power_ps_hold_setup(void);
 
 /* Read the resume function and call it */
 void power_exit_wakeup(void);
+
+
+/* Power Down Modes
+ * User defined values in inform1 register
+ */
+#define EXYNOS_CHECK_SLEEP     0x00000BAD
+#define EXYNOS_CHECK_DIDLE     0xBAD00000
+#define EXYNOS_CHECK_LPA       0xABAD0000
 #endif

board/samsung/smdk5250/Makefile +0 -1

index 47c6a5a..7eaef09 100644
@@ -24,7 +24,6 @@ include $(TOPDIR)/config.mk
 
 LIB    = $(obj)lib$(BOARD).o
 
-SOBJS  := lowlevel_init.o
 
 COBJS  := clock_init.o
 COBJS  += dmc_common.o dmc_init_ddr3.o

board/samsung/smdk5250/Makefile endif · line 39

Quoted 4 lines

board/samsung/smdk5250/Makefile +1 -0

@@ -37,6 +36,7 @@ endif
 
 ifdef CONFIG_SPL_BUILD
 COBJS  += spl_boot.o
+COBJS  += lowlevel_init.o

Can you use this form instead of ifdef here?

COBJS-$(CONFIG_SPL) += lowlevel_init.o

Quoted 228 lines

board/samsung/smdk5250/Makefile +0 -0

 endif
 
 SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)

board/samsung/smdk5250/dmc_common.c +2 -2

index 109602a..f637bf9 100644
@@ -175,7 +175,7 @@ void dmc_config_memory(struct mem_timings *mem, struct exynos5_dmc *dmc)
        writel(DMC_MEMBASECONFIG1_VAL, &dmc->membaseconfig1);
 }
 
-void mem_ctrl_init()
+void mem_ctrl_init(int reset)
 {
        struct spl_machine_param *param = spl_get_machine_params();
        struct mem_timings *mem;

@@ -185,7 +185,7 @@ void mem_ctrl_init()
 
        /* If there are any other memory variant, add their init call below */
        if (param->mem_type == DDR_MODE_DDR3) {
-               ret = ddr3_mem_ctrl_init(mem, param->mem_iv_size);
+               ret = ddr3_mem_ctrl_init(mem, param->mem_iv_size, reset);
                if (ret) {
                        /* will hang if failed to init memory control */
                        while (1)

board/samsung/smdk5250/dmc_init_ddr3.c +4 -2

index e050790..a5a70df 100644
@@ -40,7 +40,8 @@ static void reset_phy_ctrl(void)
        writel(DDR3PHY_CTRL_PHY_RESET, &clk->lpddr3phy_ctrl);
 }
 
-int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size)
+int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size,
+                       int reset)
 {
        unsigned int val;
        struct exynos5_phy_control *phy0_ctrl, *phy1_ctrl;

@@ -51,7 +52,8 @@ int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size)
        phy1_ctrl = (struct exynos5_phy_control *)EXYNOS5_DMC_PHY1_BASE;
        dmc = (struct exynos5_dmc *)EXYNOS5_DMC_CTRL_BASE;
 
-       reset_phy_ctrl();
+       if (reset)
+               reset_phy_ctrl();
 
        /* Set Impedance Output Driver */
        val = (mem->impedance << CA_CK_DRVR_DS_OFFSET) |

board/samsung/smdk5250/lowlevel_init.S deleted +0 -96

deleted file mode 100644
index bc6cb6f..0000000
@@ -1,96 +0,0 @@
-/*
- * Lowlevel setup for SMDK5250 board based on S5PC520
- *
- * Copyright (C) 2012 Samsung Electronics
- *
- * See file CREDITS for list of people who contributed to this
- * project.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
- * MA 02111-1307 USA
- */
-
-#include <config.h>
-#include <version.h>
-#include <asm/arch/cpu.h>
-
-_TEXT_BASE:
-       .word   CONFIG_SYS_TEXT_BASE
-
-       .globl lowlevel_init
-lowlevel_init:
-
-       /* use iRAM stack in bl2 */
-       ldr     sp, =CONFIG_IRAM_STACK
-       stmdb   r13!, {ip,lr}
-
-       /* check reset status */
-       ldr     r0, =(EXYNOS5_POWER_BASE + INFORM1_OFFSET)
-       ldr     r1, [r0]
-
-       /* AFTR wakeup reset */
-       ldr     r2, =S5P_CHECK_DIDLE
-       cmp     r1, r2
-       beq     exit_wakeup
-
-       /* LPA wakeup reset */
-       ldr     r2, =S5P_CHECK_LPA
-       cmp     r1, r2
-       beq     exit_wakeup
-
-       /* Sleep wakeup reset */
-       ldr     r2, =S5P_CHECK_SLEEP
-       cmp     r1, r2
-       beq     wakeup_reset
-
-       /*
-        * If U-boot is already running in RAM, no need to relocate U-Boot.
-        * Memory controller must be configured before relocating U-Boot
-        * in ram.
-        */
-       ldr     r0, =0x0ffffff          /* r0 <- Mask Bits*/
-       bic     r1, pc, r0              /* pc <- current addr of code */
-                                       /* r1 <- unmasked bits of pc */
-       ldr     r2, _TEXT_BASE          /* r2 <- original base addr in ram */
-       bic     r2, r2, r0              /* r2 <- unmasked bits of r2*/
-       cmp     r1, r2                  /* compare r1, r2 */
-       beq     1f                      /* r0 == r1 then skip sdram init */
-
-       /* init system clock */
-       bl      system_clock_init
-
-       /* Memory initialize */
-       bl      mem_ctrl_init
-
-1:
-       bl      tzpc_init
-       ldmia   r13!, {ip,pc}
-
-wakeup_reset:
-       bl      system_clock_init
-       bl      mem_ctrl_init
-       bl      tzpc_init
-
-exit_wakeup:
-       /* Load return address and jump to kernel */
-       ldr     r0, =(EXYNOS5_POWER_BASE + INFORM0_OFFSET)
-
-       /* r1 = physical address of exynos5_cpu_resume function*/
-       ldr     r1, [r0]
-
-       /* Jump to kernel */
-       mov     pc, r1
-       nop
-       nop

board/samsung/smdk5250/lowlevel_init.c new file +71 -0

new file mode 100644
index 0000000..22bdd2b
@@ -0,0 +1,81 @@
+/*
+ * Lowlevel setup for SMDK5250 board based on S5PC520
+ *
+ * Copyright (C) 2012 Samsung Electronics
+ * Copyright (c) 2012 The Chromium OS Authors.
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <config.h>
+#include <asm/arch/cpu.h>
+#include <asm/arch/dmc.h>
+#include <asm/arch/power.h>
+#include <asm/arch/tzpc.h>
+#include <asm/arch/periph.h>
+#include <asm/arch/pinmux.h>
+#include "setup.h"
+
+/* These are the things we can do during low-level init */
+enum {
+       DO_WAKEUP       = 1 << 0,
+       DO_CLOCKS       = 1 << 1,
+       DO_MEM_RESET    = 1 << 2,
+       DO_UART         = 1 << 3,
+};
+
+int do_lowlevel_init(void)
+{
+       uint32_t reset_status;
+       int actions = 0;
+
+       arch_cpu_init();
+
+       reset_status = power_read_reset_status();
+
+       switch (reset_status) {
+       case EXYNOS_CHECK_SLEEP:
+               actions = DO_CLOCKS | DO_WAKEUP;
+               break;
+       case EXYNOS_CHECK_DIDLE:
+       case EXYNOS_CHECK_LPA:
+               actions = DO_WAKEUP;
+               break;
+       default:
+               /* This is a normal boot (not a wake from sleep) */
+               actions = DO_CLOCKS | DO_MEM_RESET | DO_UART;
+       }
+
+       if (actions & DO_CLOCKS)
+               system_clock_init();
+
+       if (actions & DO_UART) {
+               exynos_pinmux_config(PERIPH_ID_UART3, PINMUX_FLAG_NONE);
+               serial_init();

Quoted 8 lines

board/samsung/smdk5250/lowlevel_init.c line 77

Quoted 6 lines

board/samsung/smdk5250/lowlevel_init.c new file +6 -0

+               timer_init();
+       }
+
+       if (actions & DO_CLOCKS) {
+               mem_ctrl_init(actions & DO_MEM_RESET);
+               tzpc_init();

I think serial SPL support is coming later.

So you want to initialise serial SPL support after mem_ctrl_init and tzpc_init?

Quoted 195 lines

Quoted 63 lines

board/samsung/smdk5250/lowlevel_init.c new file +4 -0

+       }
+
+       return actions & DO_WAKEUP;
+}

board/samsung/smdk5250/setup.h +17 -2

index a159601..f1d9f79 100644
@@ -537,9 +537,11 @@ enum {
  *                     which the DMC uses to decide how to split a memory
  *                     chunk into smaller chunks to support concurrent
  *                     accesses; may vary across boards.
+ * @param reset                Reset DDR PHY during initialization.
  * @return 0 if ok, SETUP_ERR_... if there is a problem
  */
-int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size);
+int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size,
+                       int reset);
 
 /*
  * Configure ZQ I/O interface

@@ -587,8 +589,21 @@ void dmc_config_memory(struct mem_timings *mem, struct exynos5_dmc *dmc);
  */
 void update_reset_dll(struct exynos5_dmc *, enum ddr_mode);
 
+/*
+ * Memory initialization
+ *
+ * @param reset     Reset PHY during initialization.
+ */
+void mem_ctrl_init(int reset);
+
 void sdelay(unsigned long);
-void mem_ctrl_init(void);
 void system_clock_init(void);
 void tzpc_init(void);
+
+/**
+ * Init subsystems according to the reset status
+ *
+ * @return 0 for a normal boot, non-zero for a resume
+ */
+int do_lowlevel_init(void);
 #endif

board/samsung/smdk5250/spl_boot.c +6 -3

index d8f3c1e..a1c8d3d 100644
@@ -20,18 +20,16 @@
  * MA 02111-1307 USA
  */
 
-#include<common.h>
-#include<config.h>
+#include <common.h>
+#include <config.h>
+#include <asm/arch/spl.h>
+#include <asm/arch/cpu.h>
+#include <asm/arch/power.h>
+#include "setup.h"
 
-enum boot_mode {

board/samsung/smdk5250/spl_boot.c line 32

Quoted 10 lines

board/samsung/smdk5250/spl_boot.c +2 -7

-       BOOT_MODE_MMC = 4,
-       BOOT_MODE_SERIAL = 20,
-       /* Boot based on Operating Mode pin settings */
-       BOOT_MODE_OM = 32,
-       BOOT_MODE_USB,  /* Boot using USB download */
-};
+DECLARE_GLOBAL_DATA_PTR;
 
-       typedef u32 (*spi_copy_func_t)(u32 offset, u32 nblock, u32 dst);
+typedef u32 (*spi_copy_func_t)(u32 offset, u32 nblock, u32 dst);

What is happening with thsi file? I think there is another patch which changes things here.

Quoted 118 lines

board/samsung/smdk5250/spl_boot.c +95 -0

 /*
 * Copy U-boot from mmc to RAM:

@@ -62,15 +60,49 @@ void copy_uboot_to_ram(void)
        }
 }
 
+void memzero(void *s, size_t n)
+{
+       char *ptr = s;
+       size_t i;
+
+       for (i = 0; i < n; i++)
+               *ptr++ = '\0';
+}
+
+/**
+ * Set up the U-Boot global_data pointer
+ *
+ * This sets the address of the global data, and sets up basic values.
+ *
+ * @param gdp   Value to give to gd
+ */
+static void setup_global_data(gd_t *gdp)
+{
+       gd = gdp;
+       memzero((void *)gd, sizeof(gd_t));
+       gd->flags |= GD_FLG_RELOC;
+       gd->baudrate = CONFIG_BAUDRATE;
+       gd->have_console = 1;
+}
+
 void board_init_f(unsigned long bootflag)
 {
+       __attribute__((aligned(8))) gd_t local_gd;
        __attribute__((noreturn)) void (*uboot)(void);
+
+       setup_global_data(&local_gd);
+
+       if (do_lowlevel_init())
+               power_exit_wakeup();
+
        copy_uboot_to_ram();
 
        /* Jump to U-Boot image */
        uboot = (void *)CONFIG_SYS_TEXT_BASE;
        (*uboot)();
+
        /* Never returns Here */
+       panic("%s: u-boot jump failed", __func__);
 }
 
 /* Place Holders */

@@ -83,3 +115,91 @@ void board_init_r(gd_t *id, ulong dest_addr)
 }
 
 void save_boot_params(u32 r0, u32 r1, u32 r2, u32 r3) {}
+
+/*
+ * The following functions are required when linking console library to SPL.
+ *
+ * Enabling UART in SPL u-boot requires console library. But some
+ * functions we needed in the console library depends on a bunch
+ * of library in libgeneric, like lib/ctype.o, lib/div64.o, lib/string.o,
+ * and lib/vsprintf.o. Adding them makes the SPL u-boot too large and not
+ * fit into the expected size.
+ *
+ * So we mock these functions in SPL, i.e. vsprintf(), panic(), etc.,
+ * in order to cut its dependency.
+ */
+static int _vscnprintf(char *buf, size_t size, const char *fmt, va_list args)
+{
+       char *str = buf, *s;
+       char *end = str + size - 1;
+       ulong u;
+
+       if (size == 0)
+               return -1;
+
+       /*
+        * We won't implement all full functions of vsprintf().
+        * We only implement %s and %u, and ignore others and directly use
+        * the original format string as its result.
+        */
+
+       while (*fmt && (str < end)) {
+               if (*fmt != '%') {
+                       *str++ = *fmt++;
+                       continue;
+               }
+               fmt++;
+               switch (*fmt) {
+               case '%':
+                       *str++ = *fmt++;
+                       break;
+               case 's':
+                       fmt++;
+                       s = va_arg(args, char *);
+                       while (*s && (str < end))
+                               *str++ = *s++;
+                       break;
+               case 'u':
+                       fmt++;
+                       u = va_arg(args, ulong);
+                       s = simple_itoa(u);
+                               while (*s && (str < end))
+                                       *str++ = *s++;
+                               break;
+               default:
+                       /* Print the original string for unsupported formats */
+                       *str++ = '%';
+                       if  (str < end)
+                               *str++ = *fmt++;
+                       }
+               }
+       *str = '\0';
+       return str - buf;
+}

Quoted 8 lines

board/samsung/smdk5250/spl_boot.c void board_init_r(gd_t *id, ulong dest_addr) · line 184

Quoted 6 lines

board/samsung/smdk5250/spl_boot.c +6 -0

+
+/* Implement vsprintf in case someone doesn't have CONFIG_SYS_VSNPRINTF */
+int vsprintf(char *buf, const char *fmt, va_list args)
+{
+       return _vscnprintf(buf, CONFIG_SYS_PBSIZE, fmt, args);
+}

This is ready for use by SPL serial support, right?

Yes it is.

Quoted 39 lines

Quoted 39 lines

board/samsung/smdk5250/spl_boot.c +21 -0

+
+char *simple_itoa(ulong i)
+{
+       /* 21 digits plus null terminator, good for 64-bit or smaller ints */
+       static char local[22] __attribute__((section(".data")));
+       char *p = &local[21];
+
+       *p-- = '\0';
+       do {
+               *p-- = '0' + i % 10;
+               i /= 10;
+       } while (i > 0);
+       return p + 1;
+}
+
+void hang(void)
+{
+       puts("### ERROR ### Please RESET the board ###\n");
+       for (;;)
+               ;
+}

spl/Makefile +4 -1

index 6dbb105..3aab466 100644
@@ -86,6 +86,10 @@ ifneq ($(CONFIG_AM33XX)$(CONFIG_OMAP34XX)$(CONFIG_OMAP44XX)$(CONFIG_OMAP54XX),)
 LIBS-y += $(CPUDIR)/omap-common/libomap-common.o
 endif
 
+ifneq ($(CONFIG_EXYNOS4)$(CONFIG_EXYNOS5),)
+LIBS-y += $(CPUDIR)/s5p-common/libs5p-common.o
+endif
+
 ifeq ($(SOC),tegra20)
 LIBS-y += arch/$(ARCH)/cpu/$(SOC)-common/lib$(SOC)-common.o
 LIBS-y += arch/$(ARCH)/cpu/tegra-common/libcputegra-common.o
--

1.7.4.4

Quoted 8 lines

Quoted 1 line

Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

--
Regards,
Rajeshwari Shinde

OK, but is this something you intend to fix, perhaps in a future series? It seems like you need separate directories sooner rather than later.I don't think this is board-specific code, but chip-specific.

Hi Rajeshwari,

Quoted 35 lines · Rajeshwari Birje

Hi Simon,

Thank you for comments.

Quoted 10 lines · Simon Glass

Hi Rajeshwari,

Quoted 4 lines · Rajeshwari Shinde

This patch converts lowlevel_init.S to lowlevel_init_c.c for SMDK5250.
Lowlevel.S as of now added only for SMDK5250 and same can be extended to other SOC in future.

Should perhaps also mention new feature (controllable memory reset for resume?)

-OK

Quoted 19 lines

Quoted 18 lines

Signed-off-by Rajeshwari Shinde <[email protected]>

---
Changes in V2:
- Renamed lowlevel_init.S to lowlevel.S and moved to arch/arm/cpu/armv7/exynos/
- Moved power mode defines to power.h
- Added early serial support.
- Renamed mem_reset to reset.

 arch/arm/cpu/armv7/exynos/Makefile       |    6 ++
 arch/arm/cpu/armv7/exynos/lowlevel.S     |   35 ++++++++
 arch/arm/include/asm/arch-exynos/power.h |    8 ++
 board/samsung/smdk5250/Makefile          |    2 +-
 board/samsung/smdk5250/dmc_common.c      |    4 +-
 board/samsung/smdk5250/dmc_init_ddr3.c   |    6 +-
 board/samsung/smdk5250/lowlevel_init.S   |   96 --------------------
 board/samsung/smdk5250/lowlevel_init.c   |   81 +++++++++++++++++

Quoted 5 lines

Quoted 1 line

Any change we could move all of this to arch/arm/cpu/armv7/exynos...?

We do not have a separate directory for exynos5 and exynos4 and if we add all these files in arch/arm/cpu/armv7/exynos it would break the compilation for EXYNOS4. Also Later versions of exynos5 have different memory and timing variants.

OK, but is this something you intend to fix, perhaps in a future series? It seems like you need separate directories sooner rather than later.I don't think this is board-specific code, but chip-specific.

Quoted 353 lines

Quoted 25 lines

It really doesn't relate to this board alone, but to the chip.

Quoted 13 lines

 board/samsung/smdk5250/setup.h           |   19 ++++-
 board/samsung/smdk5250/spl_boot.c        |  140 +++++++++++++++++++++++++++--
 spl/Makefile                             |    4 +
 11 files changed, 288 insertions(+), 113 deletions(-)
 create mode 100644 arch/arm/cpu/armv7/exynos/lowlevel.S
 delete mode 100644 board/samsung/smdk5250/lowlevel_init.S
 create mode 100644 board/samsung/smdk5250/lowlevel_init.c

arch/arm/cpu/armv7/exynos/Makefile +0 -0

index 9119961..2aa2722 100644

arch/arm/cpu/armv7/exynos/Makefile include $(TOPDIR)/config.mk · line 27

Quoted 6 lines

arch/arm/cpu/armv7/exynos/Makefile +3 -0

@@ -22,6 +22,12 @@ include $(TOPDIR)/config.mk
 
 LIB    = $(obj)lib$(SOC).o
 
+ifdef CONFIG_SMDK5250
+ifdef CONFIG_SPL
+COBJS  += lowlevel.o

Could do:

Quoted 3 lines

COBJS-$(CONFIG_SPL) += lowlevel.o

and remove the inner ifdef

- OK

Quoted 316 lines

Quoted 78 lines

arch/arm/cpu/armv7/exynos/Makefile +3 -0

+endif
+endif
+
 COBJS  += clock.o power.o soc.o system.o pinmux.o
 
 SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)

arch/arm/cpu/armv7/exynos/lowlevel.S new file +35 -0

new file mode 100644
index 0000000..7307959
@@ -0,0 +1,35 @@
+/*
+ * Lowlevel setup for SMDK5250 board based on S5PC520
+ *
+ * Copyright (C) 2012 Samsung Electronics
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <config.h>
+#include <asm/arch/cpu.h>
+
+       .globl lowlevel_init
+lowlevel_init:
+       /*
+        * Set the stack pointer, although it will be overwritten by the caller
+        * It seems we will not boot if this function is empty.
+        */
+       ldr     sp, =CONFIG_IRAM_STACK
+       mov     pc, lr

arch/arm/include/asm/arch-exynos/power.h +8 -0

index f6d0278..d6fd29e 100644
@@ -874,4 +874,12 @@ void power_ps_hold_setup(void);
 
 /* Read the resume function and call it */
 void power_exit_wakeup(void);
+
+
+/* Power Down Modes
+ * User defined values in inform1 register
+ */
+#define EXYNOS_CHECK_SLEEP     0x00000BAD
+#define EXYNOS_CHECK_DIDLE     0xBAD00000
+#define EXYNOS_CHECK_LPA       0xABAD0000
 #endif

board/samsung/smdk5250/Makefile +0 -1

index 47c6a5a..7eaef09 100644
@@ -24,7 +24,6 @@ include $(TOPDIR)/config.mk
 
 LIB    = $(obj)lib$(BOARD).o
 
-SOBJS  := lowlevel_init.o
 
 COBJS  := clock_init.o
 COBJS  += dmc_common.o dmc_init_ddr3.o

board/samsung/smdk5250/Makefile endif · line 39

Quoted 4 lines

board/samsung/smdk5250/Makefile +1 -0

@@ -37,6 +36,7 @@ endif
 
 ifdef CONFIG_SPL_BUILD
 COBJS  += spl_boot.o
+COBJS  += lowlevel_init.o

Can you use this form instead of ifdef here?

COBJS-$(CONFIG_SPL) += lowlevel_init.o

Quoted 228 lines

board/samsung/smdk5250/Makefile +0 -0

 endif
 
 SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)

board/samsung/smdk5250/dmc_common.c +2 -2

index 109602a..f637bf9 100644
@@ -175,7 +175,7 @@ void dmc_config_memory(struct mem_timings *mem, struct exynos5_dmc *dmc)
        writel(DMC_MEMBASECONFIG1_VAL, &dmc->membaseconfig1);
 }
 
-void mem_ctrl_init()
+void mem_ctrl_init(int reset)
 {
        struct spl_machine_param *param = spl_get_machine_params();
        struct mem_timings *mem;

@@ -185,7 +185,7 @@ void mem_ctrl_init()
 
        /* If there are any other memory variant, add their init call below */
        if (param->mem_type == DDR_MODE_DDR3) {
-               ret = ddr3_mem_ctrl_init(mem, param->mem_iv_size);
+               ret = ddr3_mem_ctrl_init(mem, param->mem_iv_size, reset);
                if (ret) {
                        /* will hang if failed to init memory control */
                        while (1)

board/samsung/smdk5250/dmc_init_ddr3.c +4 -2

index e050790..a5a70df 100644
@@ -40,7 +40,8 @@ static void reset_phy_ctrl(void)
        writel(DDR3PHY_CTRL_PHY_RESET, &clk->lpddr3phy_ctrl);
 }
 
-int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size)
+int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size,
+                       int reset)
 {
        unsigned int val;
        struct exynos5_phy_control *phy0_ctrl, *phy1_ctrl;

@@ -51,7 +52,8 @@ int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size)
        phy1_ctrl = (struct exynos5_phy_control *)EXYNOS5_DMC_PHY1_BASE;
        dmc = (struct exynos5_dmc *)EXYNOS5_DMC_CTRL_BASE;
 
-       reset_phy_ctrl();
+       if (reset)
+               reset_phy_ctrl();
 
        /* Set Impedance Output Driver */
        val = (mem->impedance << CA_CK_DRVR_DS_OFFSET) |

board/samsung/smdk5250/lowlevel_init.S deleted +0 -96

deleted file mode 100644
index bc6cb6f..0000000
@@ -1,96 +0,0 @@
-/*
- * Lowlevel setup for SMDK5250 board based on S5PC520
- *
- * Copyright (C) 2012 Samsung Electronics
- *
- * See file CREDITS for list of people who contributed to this
- * project.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
- * MA 02111-1307 USA
- */
-
-#include <config.h>
-#include <version.h>
-#include <asm/arch/cpu.h>
-
-_TEXT_BASE:
-       .word   CONFIG_SYS_TEXT_BASE
-
-       .globl lowlevel_init
-lowlevel_init:
-
-       /* use iRAM stack in bl2 */
-       ldr     sp, =CONFIG_IRAM_STACK
-       stmdb   r13!, {ip,lr}
-
-       /* check reset status */
-       ldr     r0, =(EXYNOS5_POWER_BASE + INFORM1_OFFSET)
-       ldr     r1, [r0]
-
-       /* AFTR wakeup reset */
-       ldr     r2, =S5P_CHECK_DIDLE
-       cmp     r1, r2
-       beq     exit_wakeup
-
-       /* LPA wakeup reset */
-       ldr     r2, =S5P_CHECK_LPA
-       cmp     r1, r2
-       beq     exit_wakeup
-
-       /* Sleep wakeup reset */
-       ldr     r2, =S5P_CHECK_SLEEP
-       cmp     r1, r2
-       beq     wakeup_reset
-
-       /*
-        * If U-boot is already running in RAM, no need to relocate U-Boot.
-        * Memory controller must be configured before relocating U-Boot
-        * in ram.
-        */
-       ldr     r0, =0x0ffffff          /* r0 <- Mask Bits*/
-       bic     r1, pc, r0              /* pc <- current addr of code */
-                                       /* r1 <- unmasked bits of pc */
-       ldr     r2, _TEXT_BASE          /* r2 <- original base addr in ram */
-       bic     r2, r2, r0              /* r2 <- unmasked bits of r2*/
-       cmp     r1, r2                  /* compare r1, r2 */
-       beq     1f                      /* r0 == r1 then skip sdram init */
-
-       /* init system clock */
-       bl      system_clock_init
-
-       /* Memory initialize */
-       bl      mem_ctrl_init
-
-1:
-       bl      tzpc_init
-       ldmia   r13!, {ip,pc}
-
-wakeup_reset:
-       bl      system_clock_init
-       bl      mem_ctrl_init
-       bl      tzpc_init
-
-exit_wakeup:
-       /* Load return address and jump to kernel */
-       ldr     r0, =(EXYNOS5_POWER_BASE + INFORM0_OFFSET)
-
-       /* r1 = physical address of exynos5_cpu_resume function*/
-       ldr     r1, [r0]
-
-       /* Jump to kernel */
-       mov     pc, r1
-       nop
-       nop

board/samsung/smdk5250/lowlevel_init.c new file +71 -0

new file mode 100644
index 0000000..22bdd2b
@@ -0,0 +1,81 @@
+/*
+ * Lowlevel setup for SMDK5250 board based on S5PC520
+ *
+ * Copyright (C) 2012 Samsung Electronics
+ * Copyright (c) 2012 The Chromium OS Authors.
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <config.h>
+#include <asm/arch/cpu.h>
+#include <asm/arch/dmc.h>
+#include <asm/arch/power.h>
+#include <asm/arch/tzpc.h>
+#include <asm/arch/periph.h>
+#include <asm/arch/pinmux.h>
+#include "setup.h"
+
+/* These are the things we can do during low-level init */
+enum {
+       DO_WAKEUP       = 1 << 0,
+       DO_CLOCKS       = 1 << 1,
+       DO_MEM_RESET    = 1 << 2,
+       DO_UART         = 1 << 3,
+};
+
+int do_lowlevel_init(void)
+{
+       uint32_t reset_status;
+       int actions = 0;
+
+       arch_cpu_init();
+
+       reset_status = power_read_reset_status();
+
+       switch (reset_status) {
+       case EXYNOS_CHECK_SLEEP:
+               actions = DO_CLOCKS | DO_WAKEUP;
+               break;
+       case EXYNOS_CHECK_DIDLE:
+       case EXYNOS_CHECK_LPA:
+               actions = DO_WAKEUP;
+               break;
+       default:
+               /* This is a normal boot (not a wake from sleep) */
+               actions = DO_CLOCKS | DO_MEM_RESET | DO_UART;
+       }
+
+       if (actions & DO_CLOCKS)
+               system_clock_init();
+
+       if (actions & DO_UART) {
+               exynos_pinmux_config(PERIPH_ID_UART3, PINMUX_FLAG_NONE);
+               serial_init();

Quoted 8 lines

board/samsung/smdk5250/lowlevel_init.c line 77

Quoted 6 lines

board/samsung/smdk5250/lowlevel_init.c new file +6 -0

+               timer_init();
+       }
+
+       if (actions & DO_CLOCKS) {
+               mem_ctrl_init(actions & DO_MEM_RESET);
+               tzpc_init();

I think serial SPL support is coming later.

Quoted 1 line

So you want to initialise serial SPL support after mem_ctrl_init and tzpc_init?

No, I meant earlier, but please ignore my comment, i was confused. At present we can't move it earlier because the clocks need to be started up, and that happens in a big monolithic operation.

What you have here is fine.

Quoted 257 lines

Quoted 195 lines

Quoted 63 lines

board/samsung/smdk5250/lowlevel_init.c new file +4 -0

+       }
+
+       return actions & DO_WAKEUP;
+}

board/samsung/smdk5250/setup.h +17 -2

index a159601..f1d9f79 100644
@@ -537,9 +537,11 @@ enum {
  *                     which the DMC uses to decide how to split a memory
  *                     chunk into smaller chunks to support concurrent
  *                     accesses; may vary across boards.
+ * @param reset                Reset DDR PHY during initialization.
  * @return 0 if ok, SETUP_ERR_... if there is a problem
  */
-int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size);
+int ddr3_mem_ctrl_init(struct mem_timings *mem, unsigned long mem_iv_size,
+                       int reset);
 
 /*
  * Configure ZQ I/O interface

@@ -587,8 +589,21 @@ void dmc_config_memory(struct mem_timings *mem, struct exynos5_dmc *dmc);
  */
 void update_reset_dll(struct exynos5_dmc *, enum ddr_mode);
 
+/*
+ * Memory initialization
+ *
+ * @param reset     Reset PHY during initialization.
+ */
+void mem_ctrl_init(int reset);
+
 void sdelay(unsigned long);
-void mem_ctrl_init(void);
 void system_clock_init(void);
 void tzpc_init(void);
+
+/**
+ * Init subsystems according to the reset status
+ *
+ * @return 0 for a normal boot, non-zero for a resume
+ */
+int do_lowlevel_init(void);
 #endif

board/samsung/smdk5250/spl_boot.c +6 -3

index d8f3c1e..a1c8d3d 100644
@@ -20,18 +20,16 @@
  * MA 02111-1307 USA
  */
 
-#include<common.h>
-#include<config.h>
+#include <common.h>
+#include <config.h>
+#include <asm/arch/spl.h>
+#include <asm/arch/cpu.h>
+#include <asm/arch/power.h>
+#include "setup.h"
 
-enum boot_mode {

board/samsung/smdk5250/spl_boot.c line 32

Quoted 10 lines

board/samsung/smdk5250/spl_boot.c +2 -7

-       BOOT_MODE_MMC = 4,
-       BOOT_MODE_SERIAL = 20,
-       /* Boot based on Operating Mode pin settings */
-       BOOT_MODE_OM = 32,
-       BOOT_MODE_USB,  /* Boot using USB download */
-};
+DECLARE_GLOBAL_DATA_PTR;
 
-       typedef u32 (*spi_copy_func_t)(u32 offset, u32 nblock, u32 dst);
+typedef u32 (*spi_copy_func_t)(u32 offset, u32 nblock, u32 dst);

What is happening with thsi file? I think there is another patch which changes things here.

Quoted 118 lines

board/samsung/smdk5250/spl_boot.c +95 -0

 /*
 * Copy U-boot from mmc to RAM:

@@ -62,15 +60,49 @@ void copy_uboot_to_ram(void)
        }
 }
 
+void memzero(void *s, size_t n)
+{
+       char *ptr = s;
+       size_t i;
+
+       for (i = 0; i < n; i++)
+               *ptr++ = '\0';
+}
+
+/**
+ * Set up the U-Boot global_data pointer
+ *
+ * This sets the address of the global data, and sets up basic values.
+ *
+ * @param gdp   Value to give to gd
+ */
+static void setup_global_data(gd_t *gdp)
+{
+       gd = gdp;
+       memzero((void *)gd, sizeof(gd_t));
+       gd->flags |= GD_FLG_RELOC;
+       gd->baudrate = CONFIG_BAUDRATE;
+       gd->have_console = 1;
+}
+
 void board_init_f(unsigned long bootflag)
 {
+       __attribute__((aligned(8))) gd_t local_gd;
        __attribute__((noreturn)) void (*uboot)(void);
+
+       setup_global_data(&local_gd);
+
+       if (do_lowlevel_init())
+               power_exit_wakeup();
+
        copy_uboot_to_ram();
 
        /* Jump to U-Boot image */
        uboot = (void *)CONFIG_SYS_TEXT_BASE;
        (*uboot)();
+
        /* Never returns Here */
+       panic("%s: u-boot jump failed", __func__);
 }
 
 /* Place Holders */

@@ -83,3 +115,91 @@ void board_init_r(gd_t *id, ulong dest_addr)
 }
 
 void save_boot_params(u32 r0, u32 r1, u32 r2, u32 r3) {}
+
+/*
+ * The following functions are required when linking console library to SPL.
+ *
+ * Enabling UART in SPL u-boot requires console library. But some
+ * functions we needed in the console library depends on a bunch
+ * of library in libgeneric, like lib/ctype.o, lib/div64.o, lib/string.o,
+ * and lib/vsprintf.o. Adding them makes the SPL u-boot too large and not
+ * fit into the expected size.
+ *
+ * So we mock these functions in SPL, i.e. vsprintf(), panic(), etc.,
+ * in order to cut its dependency.
+ */
+static int _vscnprintf(char *buf, size_t size, const char *fmt, va_list args)
+{
+       char *str = buf, *s;
+       char *end = str + size - 1;
+       ulong u;
+
+       if (size == 0)
+               return -1;
+
+       /*
+        * We won't implement all full functions of vsprintf().
+        * We only implement %s and %u, and ignore others and directly use
+        * the original format string as its result.
+        */
+
+       while (*fmt && (str < end)) {
+               if (*fmt != '%') {
+                       *str++ = *fmt++;
+                       continue;
+               }
+               fmt++;
+               switch (*fmt) {
+               case '%':
+                       *str++ = *fmt++;
+                       break;
+               case 's':
+                       fmt++;
+                       s = va_arg(args, char *);
+                       while (*s && (str < end))
+                               *str++ = *s++;
+                       break;
+               case 'u':
+                       fmt++;
+                       u = va_arg(args, ulong);
+                       s = simple_itoa(u);
+                               while (*s && (str < end))
+                                       *str++ = *s++;
+                               break;
+               default:
+                       /* Print the original string for unsupported formats */
+                       *str++ = '%';
+                       if  (str < end)
+                               *str++ = *fmt++;
+                       }
+               }
+       *str = '\0';
+       return str - buf;
+}

Quoted 8 lines

board/samsung/smdk5250/spl_boot.c void board_init_r(gd_t *id, ulong dest_addr) · line 184

Quoted 6 lines

board/samsung/smdk5250/spl_boot.c +6 -0

+
+/* Implement vsprintf in case someone doesn't have CONFIG_SYS_VSNPRINTF */
+int vsprintf(char *buf, const char *fmt, va_list args)
+{
+       return _vscnprintf(buf, CONFIG_SYS_PBSIZE, fmt, args);
+}

This is ready for use by SPL serial support, right?

Yes it is.

Quoted 39 lines

Quoted 39 lines

board/samsung/smdk5250/spl_boot.c +21 -0

+
+char *simple_itoa(ulong i)
+{
+       /* 21 digits plus null terminator, good for 64-bit or smaller ints */
+       static char local[22] __attribute__((section(".data")));
+       char *p = &local[21];
+
+       *p-- = '\0';
+       do {
+               *p-- = '0' + i % 10;
+               i /= 10;
+       } while (i > 0);
+       return p + 1;
+}
+
+void hang(void)
+{
+       puts("### ERROR ### Please RESET the board ###\n");
+       for (;;)
+               ;
+}

spl/Makefile +4 -1

index 6dbb105..3aab466 100644
@@ -86,6 +86,10 @@ ifneq ($(CONFIG_AM33XX)$(CONFIG_OMAP34XX)$(CONFIG_OMAP44XX)$(CONFIG_OMAP54XX),)
 LIBS-y += $(CPUDIR)/omap-common/libomap-common.o
 endif
 
+ifneq ($(CONFIG_EXYNOS4)$(CONFIG_EXYNOS5),)
+LIBS-y += $(CPUDIR)/s5p-common/libs5p-common.o
+endif
+
 ifeq ($(SOC),tegra20)
 LIBS-y += arch/$(ARCH)/cpu/$(SOC)-common/lib$(SOC)-common.o
 LIBS-y += arch/$(ARCH)/cpu/tegra-common/libcputegra-common.o
--

1.7.4.4

Quoted 8 lines

Quoted 1 line

Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

--
Regards,
Rajeshwari Shinde

Hi Minkyu Kang,

Quoted 3 lines

OK, but is this something you intend to fix, perhaps in a future series? It seems like you need separate directories sooner rather than later.I don't think this is board-specific code, but chip-specific.

Please do let me know your opinion on the above comment.

Regards,
Rajeshwari Shinde.

Quoted 49 lines · Simon Glass

Hi Rajeshwari,

Quoted 35 lines · Rajeshwari Birje

Hi Simon,

Thank you for comments.

Quoted 10 lines · Simon Glass

Hi Rajeshwari,

Quoted 4 lines · Rajeshwari Shinde

This patch converts lowlevel_init.S to lowlevel_init_c.c for SMDK5250.
Lowlevel.S as of now added only for SMDK5250 and same can be extended to other SOC in future.

Should perhaps also mention new feature (controllable memory reset for resume?)

-OK

Quoted 19 lines

Quoted 18 lines

Signed-off-by Rajeshwari Shinde <[email protected]>

---
Changes in V2:
- Renamed lowlevel_init.S to lowlevel.S and moved to arch/arm/cpu/armv7/exynos/
- Moved power mode defines to power.h
- Added early serial support.
- Renamed mem_reset to reset.

 arch/arm/cpu/armv7/exynos/Makefile       |    6 ++
 arch/arm/cpu/armv7/exynos/lowlevel.S     |   35 ++++++++
 arch/arm/include/asm/arch-exynos/power.h |    8 ++
 board/samsung/smdk5250/Makefile          |    2 +-
 board/samsung/smdk5250/dmc_common.c      |    4 +-
 board/samsung/smdk5250/dmc_init_ddr3.c   |    6 +-
 board/samsung/smdk5250/lowlevel_init.S   |   96 --------------------
 board/samsung/smdk5250/lowlevel_init.c   |   81 +++++++++++++++++

Quoted 5 lines

Quoted 1 line

Any change we could move all of this to arch/arm/cpu/armv7/exynos...?

We do not have a separate directory for exynos5 and exynos4 and if we add all these files in arch/arm/cpu/armv7/exynos it would break the compilation for EXYNOS4. Also Later versions of exynos5 have different memory and timing variants.

OK, but is this something you intend to fix, perhaps in a future series? It seems like you need separate directories sooner rather than later.I don't think this is board-specific code, but chip-specific.

This patch converts lowlevel_init.S to lowlevel_init_c.c for SMDK5250.
Lowlevel.S as of now added only for SMDK5250 and same can be extended to other SOC in future.

Signed-off-by Rajeshwari Shinde <[email protected]>

---
Changes in V2:
- Renamed lowlevel_init.S to lowlevel.S and moved to arch/arm/cpu/armv7/exynos/
- Moved power mode defines to power.h
- Added early serial support.
- Renamed mem_reset to reset.

 arch/arm/cpu/armv7/exynos/Makefile       |    6 ++
 arch/arm/cpu/armv7/exynos/lowlevel.S     |   35 ++++++++
 arch/arm/include/asm/arch-exynos/power.h |    8 ++
 board/samsung/smdk5250/Makefile          |    2 +-
 board/samsung/smdk5250/dmc_common.c      |    4 +-
 board/samsung/smdk5250/dmc_init_ddr3.c   |    6 +-
 board/samsung/smdk5250/lowlevel_init.S   |   96 --------------------
 board/samsung/smdk5250/lowlevel_init.c   |   81 +++++++++++++++++
 board/samsung/smdk5250/setup.h           |   19 ++++-
 board/samsung/smdk5250/spl_boot.c        |  140 +++++++++++++++++++++++++++--
 spl/Makefile                             |    4 +
 11 files changed, 288 insertions(+), 113 deletions(-)
 create mode 100644 arch/arm/cpu/armv7/exynos/lowlevel.S
 delete mode 100644 board/samsung/smdk5250/lowlevel_init.S
 create mode 100644 board/samsung/smdk5250/lowlevel_init.c

arch/arm/cpu/armv7/exynos/Makefile +0 -0

index 9119961..2aa2722 100644

arch/arm/cpu/armv7/exynos/Makefile +3 -0

+endif
+endif
+
 COBJS	+= clock.o power.o soc.o system.o pinmux.o
 
 SRCS	:= $(SOBJS:.o=.S) $(COBJS:.o=.c)

arch/arm/cpu/armv7/exynos/lowlevel.S new file +0 -0

new file mode 100644
index 0000000..7307959

arch/arm/cpu/armv7/exynos/lowlevel.S new file +33 -0

+ *
+ * Copyright (C) 2012 Samsung Electronics
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <config.h>
+#include <asm/arch/cpu.h>
+
+	.globl lowlevel_init
+lowlevel_init:
+	/*
+	 * Set the stack pointer, although it will be overwritten by the caller
+	 * It seems we will not boot if this function is empty.
+	 */
+	ldr	sp, =CONFIG_IRAM_STACK
+	mov	pc, lr

board/samsung/smdk5250/lowlevel_init.c new file +81 -0

new file mode 100644
index 0000000..22bdd2b
@@ -0,0 +1,81 @@
+/*
+ * Lowlevel setup for SMDK5250 board based on S5PC520
+ *
+ * Copyright (C) 2012 Samsung Electronics
+ * Copyright (c) 2012 The Chromium OS Authors.
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <config.h>
+#include <asm/arch/cpu.h>
+#include <asm/arch/dmc.h>
+#include <asm/arch/power.h>
+#include <asm/arch/tzpc.h>
+#include <asm/arch/periph.h>
+#include <asm/arch/pinmux.h>
+#include "setup.h"
+
+/* These are the things we can do during low-level init */
+enum {
+	DO_WAKEUP	= 1 << 0,
+	DO_CLOCKS	= 1 << 1,
+	DO_MEM_RESET	= 1 << 2,
+	DO_UART		= 1 << 3,
+};
+
+int do_lowlevel_init(void)
+{
+	uint32_t reset_status;
+	int actions = 0;
+
+	arch_cpu_init();
+
+	reset_status = power_read_reset_status();
+
+	switch (reset_status) {
+	case EXYNOS_CHECK_SLEEP:
+		actions = DO_CLOCKS | DO_WAKEUP;
+		break;
+	case EXYNOS_CHECK_DIDLE:
+	case EXYNOS_CHECK_LPA:
+		actions = DO_WAKEUP;
+		break;
+	default:
+		/* This is a normal boot (not a wake from sleep) */
+		actions = DO_CLOCKS | DO_MEM_RESET | DO_UART;
+	}
+
+	if (actions & DO_CLOCKS)
+		system_clock_init();
+
+	if (actions & DO_UART) {
+		exynos_pinmux_config(PERIPH_ID_UART3, PINMUX_FLAG_NONE);
+		serial_init();
+		timer_init();
+	}
+
+	if (actions & DO_CLOCKS) {
+		mem_ctrl_init(actions & DO_MEM_RESET);
+		tzpc_init();
+	}
+
+	return actions & DO_WAKEUP;
+}

board/samsung/smdk5250/spl_boot.c +90 -10

index d8f3c1e..a1c8d3d 100644
@@ -20,18 +20,16 @@
  * MA 02111-1307 USA
  */
 
-#include<common.h>
-#include<config.h>
+#include <common.h>
+#include <config.h>
+#include <asm/arch/spl.h>
+#include <asm/arch/cpu.h>
+#include <asm/arch/power.h>
+#include "setup.h"
 
-enum boot_mode {
-	BOOT_MODE_MMC = 4,
-	BOOT_MODE_SERIAL = 20,
-	/* Boot based on Operating Mode pin settings */
-	BOOT_MODE_OM = 32,
-	BOOT_MODE_USB,	/* Boot using USB download */
-};
+DECLARE_GLOBAL_DATA_PTR;
 
-	typedef u32 (*spi_copy_func_t)(u32 offset, u32 nblock, u32 dst);
+typedef u32 (*spi_copy_func_t)(u32 offset, u32 nblock, u32 dst);
 
 /*
 * Copy U-boot from mmc to RAM:

@@ -62,15 +60,49 @@ void copy_uboot_to_ram(void)
 	}
 }
 
+void memzero(void *s, size_t n)
+{
+	char *ptr = s;
+	size_t i;
+
+	for (i = 0; i < n; i++)
+		*ptr++ = '\0';
+}
+
+/**
+ * Set up the U-Boot global_data pointer
+ *
+ * This sets the address of the global data, and sets up basic values.
+ *
+ * @param gdp   Value to give to gd
+ */
+static void setup_global_data(gd_t *gdp)
+{
+	gd = gdp;
+	memzero((void *)gd, sizeof(gd_t));
+	gd->flags |= GD_FLG_RELOC;
+	gd->baudrate = CONFIG_BAUDRATE;
+	gd->have_console = 1;
+}
+
 void board_init_f(unsigned long bootflag)
 {
+	__attribute__((aligned(8))) gd_t local_gd;
 	__attribute__((noreturn)) void (*uboot)(void);
+
+	setup_global_data(&local_gd);
+
+	if (do_lowlevel_init())
+		power_exit_wakeup();
+
 	copy_uboot_to_ram();
 
 	/* Jump to U-Boot image */
 	uboot = (void *)CONFIG_SYS_TEXT_BASE;
 	(*uboot)();
+
 	/* Never returns Here */
+	panic("%s: u-boot jump failed", __func__);
 }
 
 /* Place Holders */

@@ -83,3 +115,91 @@ void board_init_r(gd_t *id, ulong dest_addr)
 }
 
 void save_boot_params(u32 r0, u32 r1, u32 r2, u32 r3) {}
+
+/*
+ * The following functions are required when linking console library to SPL.
+ *
+ * Enabling UART in SPL u-boot requires console library. But some
+ * functions we needed in the console library depends on a bunch
+ * of library in libgeneric, like lib/ctype.o, lib/div64.o, lib/string.o,
+ * and lib/vsprintf.o. Adding them makes the SPL u-boot too large and not
+ * fit into the expected size.
+ *
+ * So we mock these functions in SPL, i.e. vsprintf(), panic(), etc.,
+ * in order to cut its dependency.
+ */
+static int _vscnprintf(char *buf, size_t size, const char *fmt, va_list args)
+{
+	char *str = buf, *s;
+	char *end = str + size - 1;
+	ulong u;
+
+	if (size == 0)
+		return -1;
+
+	/*
+	 * We won't implement all full functions of vsprintf().
+	 * We only implement %s and %u, and ignore others and directly use
+	 * the original format string as its result.
+	 */
+
+	while (*fmt && (str < end)) {
+		if (*fmt != '%') {
+			*str++ = *fmt++;
+			continue;
+		}
+		fmt++;
+		switch (*fmt) {
+		case '%':
+			*str++ = *fmt++;
+			break;
+		case 's':
+			fmt++;
+			s = va_arg(args, char *);
+			while (*s && (str < end))
+				*str++ = *s++;
+			break;
+		case 'u':
+			fmt++;
+			u = va_arg(args, ulong);
+			s = simple_itoa(u);

board/samsung/smdk5250/spl_boot.c +20 -0

+	*str = '\0';
+	return str - buf;
+}
+
+/* Implement vsprintf in case someone doesn't have CONFIG_SYS_VSNPRINTF */
+int vsprintf(char *buf, const char *fmt, va_list args)
+{
+	return _vscnprintf(buf, CONFIG_SYS_PBSIZE, fmt, args);
+}
+
+char *simple_itoa(ulong i)
+{
+	/* 21 digits plus null terminator, good for 64-bit or smaller ints */
+	static char local[22] __attribute__((section(".data")));
+	char *p = &local[21];
+
+	*p-- = '\0';
+	do {
+		*p-- = '0' + i % 10;
+		i /= 10;

Dear Rajeshwari,

Quoted 35 lines · Rajeshwari Shinde

This patch converts lowlevel_init.S to lowlevel_init_c.c for SMDK5250.
Lowlevel.S as of now added only for SMDK5250 and same can be extended to other SOC in future.

Signed-off-by Rajeshwari Shinde <[email protected]>

---
Changes in V2:
- Renamed lowlevel_init.S to lowlevel.S and moved to arch/arm/cpu/armv7/exynos/
- Moved power mode defines to power.h
- Added early serial support.
- Renamed mem_reset to reset.

 arch/arm/cpu/armv7/exynos/Makefile       |    6 ++
 arch/arm/cpu/armv7/exynos/lowlevel.S     |   35 ++++++++
 arch/arm/include/asm/arch-exynos/power.h |    8 ++
 board/samsung/smdk5250/Makefile          |    2 +-
 board/samsung/smdk5250/dmc_common.c      |    4 +-
 board/samsung/smdk5250/dmc_init_ddr3.c   |    6 +-
 board/samsung/smdk5250/lowlevel_init.S   |   96 --------------------
 board/samsung/smdk5250/lowlevel_init.c   |   81 +++++++++++++++++
 board/samsung/smdk5250/setup.h           |   19 ++++-
 board/samsung/smdk5250/spl_boot.c        |  140 +++++++++++++++++++++++++++--
 spl/Makefile                             |    4 +
 11 files changed, 288 insertions(+), 113 deletions(-)
 create mode 100644 arch/arm/cpu/armv7/exynos/lowlevel.S
 delete mode 100644 board/samsung/smdk5250/lowlevel_init.S
 create mode 100644 board/samsung/smdk5250/lowlevel_init.c

arch/arm/cpu/armv7/exynos/Makefile +0 -0

index 9119961..2aa2722 100644
@@ -22,6 +22,12 @@ include $(TOPDIR)/config.mk
 

arch/arm/cpu/armv7/exynos/Makefile include $(TOPDIR)/config.mk · line 25

Quoted 3 lines

arch/arm/cpu/armv7/exynos/Makefile +1 -0

 LIB  = $(obj)lib$(SOC).o
 
+ifdef CONFIG_SMDK5250

Is it SMDK5250 specific?

arch/arm/cpu/armv7/exynos/Makefile include $(TOPDIR)/config.mk · line 27

Quoted 3 lines

arch/arm/cpu/armv7/exynos/Makefile +2 -0

+ifdef CONFIG_SPL
+COBJS  += lowlevel.o

It's a SOBJS.

Quoted 6 lines

arch/arm/cpu/armv7/exynos/Makefile +3 -0

+endif
+endif
+
 COBJS        += clock.o power.o soc.o system.o pinmux.o
 

arch/arm/cpu/armv7/exynos/lowlevel.S line 2

Quoted 9 lines

arch/arm/cpu/armv7/exynos/Makefile +0 -0

 SRCS := $(SOBJS:.o=.S) $(COBJS:.o=.c)

arch/arm/cpu/armv7/exynos/lowlevel.S new file +2 -0

new file mode 100644
index 0000000..7307959
@@ -0,0 +1,35 @@
+/*
+ * Lowlevel setup for SMDK5250 board based on S5PC520

please fix this comment.
Maybe this file is not a board specific.

Quoted 258 lines

arch/arm/cpu/armv7/exynos/lowlevel.S new file +33 -0

+ *
+ * Copyright (C) 2012 Samsung Electronics
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <config.h>
+#include <asm/arch/cpu.h>
+
+     .globl lowlevel_init
+lowlevel_init:
+     /*
+      * Set the stack pointer, although it will be overwritten by the caller
+      * It seems we will not boot if this function is empty.
+      */
+     ldr     sp, =CONFIG_IRAM_STACK
+     mov     pc, lr

board/samsung/smdk5250/lowlevel_init.c new file +81 -0

new file mode 100644
index 0000000..22bdd2b
@@ -0,0 +1,81 @@
+/*
+ * Lowlevel setup for SMDK5250 board based on S5PC520
+ *
+ * Copyright (C) 2012 Samsung Electronics
+ * Copyright (c) 2012 The Chromium OS Authors.
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <config.h>
+#include <asm/arch/cpu.h>
+#include <asm/arch/dmc.h>
+#include <asm/arch/power.h>
+#include <asm/arch/tzpc.h>
+#include <asm/arch/periph.h>
+#include <asm/arch/pinmux.h>
+#include "setup.h"
+
+/* These are the things we can do during low-level init */
+enum {
+     DO_WAKEUP       = 1 << 0,
+     DO_CLOCKS       = 1 << 1,
+     DO_MEM_RESET    = 1 << 2,
+     DO_UART         = 1 << 3,
+};
+
+int do_lowlevel_init(void)
+{
+     uint32_t reset_status;
+     int actions = 0;
+
+     arch_cpu_init();
+
+     reset_status = power_read_reset_status();
+
+     switch (reset_status) {
+     case EXYNOS_CHECK_SLEEP:
+             actions = DO_CLOCKS | DO_WAKEUP;
+             break;
+     case EXYNOS_CHECK_DIDLE:
+     case EXYNOS_CHECK_LPA:
+             actions = DO_WAKEUP;
+             break;
+     default:
+             /* This is a normal boot (not a wake from sleep) */
+             actions = DO_CLOCKS | DO_MEM_RESET | DO_UART;
+     }
+
+     if (actions & DO_CLOCKS)
+             system_clock_init();
+
+     if (actions & DO_UART) {
+             exynos_pinmux_config(PERIPH_ID_UART3, PINMUX_FLAG_NONE);
+             serial_init();
+             timer_init();
+     }
+
+     if (actions & DO_CLOCKS) {
+             mem_ctrl_init(actions & DO_MEM_RESET);
+             tzpc_init();
+     }
+
+     return actions & DO_WAKEUP;
+}

board/samsung/smdk5250/spl_boot.c +94 -10

index d8f3c1e..a1c8d3d 100644
@@ -20,18 +20,16 @@
  * MA 02111-1307 USA
  */
 
-#include<common.h>
-#include<config.h>
+#include <common.h>
+#include <config.h>
+#include <asm/arch/spl.h>
+#include <asm/arch/cpu.h>
+#include <asm/arch/power.h>
+#include "setup.h"
 
-enum boot_mode {
-     BOOT_MODE_MMC = 4,
-     BOOT_MODE_SERIAL = 20,
-     /* Boot based on Operating Mode pin settings */
-     BOOT_MODE_OM = 32,
-     BOOT_MODE_USB,  /* Boot using USB download */
-};
+DECLARE_GLOBAL_DATA_PTR;
 
-     typedef u32 (*spi_copy_func_t)(u32 offset, u32 nblock, u32 dst);
+typedef u32 (*spi_copy_func_t)(u32 offset, u32 nblock, u32 dst);
 
 /*
 * Copy U-boot from mmc to RAM:

@@ -62,15 +60,49 @@ void copy_uboot_to_ram(void)
      }
 }
 
+void memzero(void *s, size_t n)
+{
+     char *ptr = s;
+     size_t i;
+
+     for (i = 0; i < n; i++)
+             *ptr++ = '\0';
+}
+
+/**
+ * Set up the U-Boot global_data pointer
+ *
+ * This sets the address of the global data, and sets up basic values.
+ *
+ * @param gdp   Value to give to gd
+ */
+static void setup_global_data(gd_t *gdp)
+{
+     gd = gdp;
+     memzero((void *)gd, sizeof(gd_t));
+     gd->flags |= GD_FLG_RELOC;
+     gd->baudrate = CONFIG_BAUDRATE;
+     gd->have_console = 1;
+}
+
 void board_init_f(unsigned long bootflag)
 {
+     __attribute__((aligned(8))) gd_t local_gd;
      __attribute__((noreturn)) void (*uboot)(void);
+
+     setup_global_data(&local_gd);
+
+     if (do_lowlevel_init())
+             power_exit_wakeup();
+
      copy_uboot_to_ram();
 
      /* Jump to U-Boot image */
      uboot = (void *)CONFIG_SYS_TEXT_BASE;
      (*uboot)();
+
      /* Never returns Here */
+     panic("%s: u-boot jump failed", __func__);
 }
 
 /* Place Holders */

@@ -83,3 +115,91 @@ void board_init_r(gd_t *id, ulong dest_addr)
 }
 
 void save_boot_params(u32 r0, u32 r1, u32 r2, u32 r3) {}
+
+/*
+ * The following functions are required when linking console library to SPL.
+ *
+ * Enabling UART in SPL u-boot requires console library. But some
+ * functions we needed in the console library depends on a bunch
+ * of library in libgeneric, like lib/ctype.o, lib/div64.o, lib/string.o,
+ * and lib/vsprintf.o. Adding them makes the SPL u-boot too large and not
+ * fit into the expected size.
+ *
+ * So we mock these functions in SPL, i.e. vsprintf(), panic(), etc.,
+ * in order to cut its dependency.
+ */
+static int _vscnprintf(char *buf, size_t size, const char *fmt, va_list args)
+{
+     char *str = buf, *s;
+     char *end = str + size - 1;
+     ulong u;
+
+     if (size == 0)
+             return -1;
+
+     /*
+      * We won't implement all full functions of vsprintf().
+      * We only implement %s and %u, and ignore others and directly use
+      * the original format string as its result.
+      */
+
+     while (*fmt && (str < end)) {
+             if (*fmt != '%') {
+                     *str++ = *fmt++;
+                     continue;
+             }
+             fmt++;
+             switch (*fmt) {
+             case '%':
+                     *str++ = *fmt++;
+                     break;
+             case 's':
+                     fmt++;
+                     s = va_arg(args, char *);
+                     while (*s && (str < end))
+                             *str++ = *s++;
+                     break;
+             case 'u':
+                     fmt++;
+                     u = va_arg(args, ulong);
+                     s = simple_itoa(u);
+                             while (*s && (str < end))
+                                     *str++ = *s++;
+                             break;
+             default:

board/samsung/smdk5250/spl_boot.c void board_init_r(gd_t *id, ulong dest_addr) · line 175

Quoted 6 lines

board/samsung/smdk5250/spl_boot.c +6 -0

+                     /* Print the original string for unsupported formats */
+                     *str++ = '%';
+                     if  (str < end)
+                             *str++ = *fmt++;
+                     }
+             }

indentation error.

Quoted 21 lines

board/samsung/smdk5250/spl_boot.c +20 -0

+     *str = '\0';
+     return str - buf;
+}
+
+/* Implement vsprintf in case someone doesn't have CONFIG_SYS_VSNPRINTF */
+int vsprintf(char *buf, const char *fmt, va_list args)
+{
+     return _vscnprintf(buf, CONFIG_SYS_PBSIZE, fmt, args);
+}
+
+char *simple_itoa(ulong i)
+{
+     /* 21 digits plus null terminator, good for 64-bit or smaller ints */
+     static char local[22] __attribute__((section(".data")));
+     char *p = &local[21];
+
+     *p-- = '\0';
+     do {
+             *p-- = '0' + i % 10;
+             i /= 10;

board/samsung/smdk5250/spl_boot.c void board_init_r(gd_t *id, ulong dest_addr) · line 205

Quoted 10 lines

board/samsung/smdk5250/spl_boot.c +10 -0

+     } while (i > 0);
+     return p + 1;
+}
+
+void hang(void)
+{
+     puts("### ERROR ### Please RESET the board ###\n");
+     for (;;)
+             ;
+}

Thanks,
Minkyu Kang.

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Hi Minkyu Kang,

Thank you for comments.

Quoted 38 lines · Minkyu Kang

Dear Rajeshwari,

Quoted 35 lines · Rajeshwari Shinde

This patch converts lowlevel_init.S to lowlevel_init_c.c for SMDK5250.
Lowlevel.S as of now added only for SMDK5250 and same can be extended to other SOC in future.

Signed-off-by Rajeshwari Shinde <[email protected]>

---
Changes in V2:
- Renamed lowlevel_init.S to lowlevel.S and moved to arch/arm/cpu/armv7/exynos/
- Moved power mode defines to power.h
- Added early serial support.
- Renamed mem_reset to reset.

 arch/arm/cpu/armv7/exynos/Makefile       |    6 ++
 arch/arm/cpu/armv7/exynos/lowlevel.S     |   35 ++++++++
 arch/arm/include/asm/arch-exynos/power.h |    8 ++
 board/samsung/smdk5250/Makefile          |    2 +-
 board/samsung/smdk5250/dmc_common.c      |    4 +-
 board/samsung/smdk5250/dmc_init_ddr3.c   |    6 +-
 board/samsung/smdk5250/lowlevel_init.S   |   96 --------------------
 board/samsung/smdk5250/lowlevel_init.c   |   81 +++++++++++++++++
 board/samsung/smdk5250/setup.h           |   19 ++++-
 board/samsung/smdk5250/spl_boot.c        |  140 +++++++++++++++++++++++++++--
 spl/Makefile                             |    4 +
 11 files changed, 288 insertions(+), 113 deletions(-)
 create mode 100644 arch/arm/cpu/armv7/exynos/lowlevel.S
 delete mode 100644 board/samsung/smdk5250/lowlevel_init.S
 create mode 100644 board/samsung/smdk5250/lowlevel_init.c

arch/arm/cpu/armv7/exynos/Makefile +0 -0

index 9119961..2aa2722 100644
@@ -22,6 +22,12 @@ include $(TOPDIR)/config.mk
 

Quoted 5 lines

arch/arm/cpu/armv7/exynos/Makefile include $(TOPDIR)/config.mk · line 25

Quoted 3 lines

arch/arm/cpu/armv7/exynos/Makefile +1 -0

 LIB  = $(obj)lib$(SOC).o
 
+ifdef CONFIG_SMDK5250

Is it SMDK5250 specific?

- Yes, As mentioned in the commit message it is currently been done only for SMDK5250 and can be also
extended to other SOC in Future.

Quoted 1 line