All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] Exynos5250: Remove lowlevelinit
@ 2014-02-14  8:22 Rajeshwari Shinde
  2014-02-14  9:41 ` Albert ARIBAUD
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Rajeshwari Shinde @ 2014-02-14  8:22 UTC (permalink / raw)
  To: u-boot

From: Rajeshwari S Shinde <rajeshwari.s@samsung.com>

Since we use CONFIG_SKIP_LOWLEVEL_INIT for Exynos baords, we dont need to a 
lowlevel_init.S file.

Signed-off-by: Rajeshwari S Shinde <rajeshwari.s@samsung.com>
---
 board/samsung/smdk5250/lowlevel_init.S | 82 ----------------------------------
 1 file changed, 82 deletions(-)
 delete mode 100644 board/samsung/smdk5250/lowlevel_init.S

diff --git a/board/samsung/smdk5250/lowlevel_init.S b/board/samsung/smdk5250/lowlevel_init.S
deleted file mode 100644
index 9003e2d..0000000
--- a/board/samsung/smdk5250/lowlevel_init.S
+++ /dev/null
@@ -1,82 +0,0 @@
-/*
- * Lowlevel setup for SMDK5250 board based on S5PC520
- *
- * Copyright (C) 2012 Samsung Electronics
- *
- * SPDX-License-Identifier:	GPL-2.0+
- */
-
-#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	arch_cpu_init
-	bl	tzpc_init
-	ldmia	r13!, {ip,pc}
-
-wakeup_reset:
-	bl	system_clock_init
-	bl	mem_ctrl_init
-	bl	arch_cpu_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
-- 
1.7.12.4

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] Exynos5250: Remove lowlevelinit
  2014-02-14  8:22 [U-Boot] [PATCH] Exynos5250: Remove lowlevelinit Rajeshwari Shinde
@ 2014-02-14  9:41 ` Albert ARIBAUD
  2014-02-14 13:03   ` Rajeshwari Birje
  2014-02-14 16:23 ` Simon Glass
  2014-02-19  9:47 ` Albert ARIBAUD
  2 siblings, 1 reply; 5+ messages in thread
From: Albert ARIBAUD @ 2014-02-14  9:41 UTC (permalink / raw)
  To: u-boot

Hi Rajeshwari,

On Fri, 14 Feb 2014 13:52:48 +0530, Rajeshwari Shinde
<rajeshwari.s@samsung.com> wrote:

> From: Rajeshwari S Shinde <rajeshwari.s@samsung.com>
> 
> Since we use CONFIG_SKIP_LOWLEVEL_INIT for Exynos baords, we dont need to a 
> lowlevel_init.S file.
> 
> Signed-off-by: Rajeshwari S Shinde <rajeshwari.s@samsung.com>
> ---

Builds fine on my side, and as it just removes dead code, I've assigned
this patch to myself and will include it in my next PR. Thanks!

Amicalement,
-- 
Albert.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] Exynos5250: Remove lowlevelinit
  2014-02-14  9:41 ` Albert ARIBAUD
@ 2014-02-14 13:03   ` Rajeshwari Birje
  0 siblings, 0 replies; 5+ messages in thread
From: Rajeshwari Birje @ 2014-02-14 13:03 UTC (permalink / raw)
  To: u-boot

Hi Albert,

On Fri, Feb 14, 2014 at 3:11 PM, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Hi Rajeshwari,
>
> On Fri, 14 Feb 2014 13:52:48 +0530, Rajeshwari Shinde
> <rajeshwari.s@samsung.com> wrote:
>
>> From: Rajeshwari S Shinde <rajeshwari.s@samsung.com>
>>
>> Since we use CONFIG_SKIP_LOWLEVEL_INIT for Exynos baords, we dont need to a
>> lowlevel_init.S file.
>>
>> Signed-off-by: Rajeshwari S Shinde <rajeshwari.s@samsung.com>
>> ---
>
> Builds fine on my side, and as it just removes dead code, I've assigned
> this patch to myself and will include it in my next PR. Thanks!

Sure Thanx

-- 
Regards,
Rajeshwari Shinde
>
> Amicalement,
> --
> Albert.
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] Exynos5250: Remove lowlevelinit
  2014-02-14  8:22 [U-Boot] [PATCH] Exynos5250: Remove lowlevelinit Rajeshwari Shinde
  2014-02-14  9:41 ` Albert ARIBAUD
@ 2014-02-14 16:23 ` Simon Glass
  2014-02-19  9:47 ` Albert ARIBAUD
  2 siblings, 0 replies; 5+ messages in thread
From: Simon Glass @ 2014-02-14 16:23 UTC (permalink / raw)
  To: u-boot

On 14 February 2014 01:22, Rajeshwari Shinde <rajeshwari.s@samsung.com> wrote:
> From: Rajeshwari S Shinde <rajeshwari.s@samsung.com>
>
> Since we use CONFIG_SKIP_LOWLEVEL_INIT for Exynos baords, we dont need to a
> lowlevel_init.S file.
>
> Signed-off-by: Rajeshwari S Shinde <rajeshwari.s@samsung.com>

Acked-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] Exynos5250: Remove lowlevelinit
  2014-02-14  8:22 [U-Boot] [PATCH] Exynos5250: Remove lowlevelinit Rajeshwari Shinde
  2014-02-14  9:41 ` Albert ARIBAUD
  2014-02-14 16:23 ` Simon Glass
@ 2014-02-19  9:47 ` Albert ARIBAUD
  2 siblings, 0 replies; 5+ messages in thread
From: Albert ARIBAUD @ 2014-02-19  9:47 UTC (permalink / raw)
  To: u-boot

Hi Rajeshwari,

On Fri, 14 Feb 2014 13:52:48 +0530, Rajeshwari Shinde
<rajeshwari.s@samsung.com> wrote:

> From: Rajeshwari S Shinde <rajeshwari.s@samsung.com>
> 
> Since we use CONFIG_SKIP_LOWLEVEL_INIT for Exynos baords, we dont need to a 
> lowlevel_init.S file.
> 
> Signed-off-by: Rajeshwari S Shinde <rajeshwari.s@samsung.com>
> ---
>  board/samsung/smdk5250/lowlevel_init.S | 82 ----------------------------------
>  1 file changed, 82 deletions(-)
>  delete mode 100644 board/samsung/smdk5250/lowlevel_init.S
> 
> diff --git a/board/samsung/smdk5250/lowlevel_init.S b/board/samsung/smdk5250/lowlevel_init.S
> deleted file mode 100644
> index 9003e2d..0000000
> --- a/board/samsung/smdk5250/lowlevel_init.S
> +++ /dev/null
> @@ -1,82 +0,0 @@
> -/*
> - * Lowlevel setup for SMDK5250 board based on S5PC520
> - *
> - * Copyright (C) 2012 Samsung Electronics
> - *
> - * SPDX-License-Identifier:	GPL-2.0+
> - */
> -
> -#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	arch_cpu_init
> -	bl	tzpc_init
> -	ldmia	r13!, {ip,pc}
> -
> -wakeup_reset:
> -	bl	system_clock_init
> -	bl	mem_ctrl_init
> -	bl	arch_cpu_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

Applied to u-boot-arm/master, thanks!

Amicalement,
-- 
Albert.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2014-02-19  9:47 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-02-14  8:22 [U-Boot] [PATCH] Exynos5250: Remove lowlevelinit Rajeshwari Shinde
2014-02-14  9:41 ` Albert ARIBAUD
2014-02-14 13:03   ` Rajeshwari Birje
2014-02-14 16:23 ` Simon Glass
2014-02-19  9:47 ` Albert ARIBAUD

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.