diff mbox

[U-Boot,u-boot,v4,03/10] ti_armv7_keystone2: Define scratch space in SRAM

Message ID 1489410273-10159-4-git-send-email-rogerq@ti.com
State Accepted
Delegated to: Tom Rini
Headers show

Commit Message

Roger Quadros March 13, 2017, 1:04 p.m. UTC
From: Franklin S Cooper Jr <fcooper@ti.com>

Scratch space can be used for features such as board detection. Define
an area within SRAM that can be used for this purpose.

[rogerq@ti.com] Rename EEPROM macro

Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com>
Signed-off-by: Roger Quadros <rogerq@ti.com>
---
 include/configs/ti_armv7_keystone2.h | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Tom Rini March 17, 2017, 2:08 p.m. UTC | #1
On Mon, Mar 13, 2017 at 03:04:26PM +0200, Roger Quadros wrote:

> From: Franklin S Cooper Jr <fcooper@ti.com>
> 
> Scratch space can be used for features such as board detection. Define
> an area within SRAM that can be used for this purpose.
> 
> [rogerq@ti.com] Rename EEPROM macro
> 
> Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com>
> Signed-off-by: Roger Quadros <rogerq@ti.com>
> ---
>  include/configs/ti_armv7_keystone2.h | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/include/configs/ti_armv7_keystone2.h b/include/configs/ti_armv7_keystone2.h
> index 5d4ef58..f76e0a5 100644
> --- a/include/configs/ti_armv7_keystone2.h
> +++ b/include/configs/ti_armv7_keystone2.h
> @@ -55,6 +55,13 @@
>  #define CONFIG_SPL_SPI_LOAD
>  #define CONFIG_SYS_SPI_U_BOOT_OFFS	CONFIG_SPL_PAD_TO
>  
> +/* SRAM scratch space entries  */
> +#define SRAM_SCRATCH_SPACE_ADDR	CONFIG_SPL_STACK + 0x8
> +
> +#define TI_SRAM_SCRATCH_BOARD_EEPROM_START	(SRAM_SCRATCH_SPACE_ADDR)
> +#define TI_SRAM_SCRATCH_BOARD_EEPROM_END	(SRAM_SCRATCH_SPACE_ADDR + 0x200)
> +#define KEYSTONE_SRAM_SCRATCH_SPACE_END		(TI_SRAM_SCRATCH_BOARD_EEPROM_END)
> +
>  /* UART Configuration */
>  #define CONFIG_SYS_NS16550_MEM32
>  #if defined(CONFIG_SPL_BUILD) || !defined(CONFIG_DM_SERIAL)

I think I said this in Franklin's posting of this patch as well, we need
to move this to some header elsewhere rather than include/configs/,
Thanks!
Tom Rini March 21, 2017, 6:08 p.m. UTC | #2
On Mon, Mar 13, 2017 at 03:04:26PM +0200, Roger Quadros wrote:

> From: Franklin S Cooper Jr <fcooper@ti.com>
> 
> Scratch space can be used for features such as board detection. Define
> an area within SRAM that can be used for this purpose.
> 
> [rogerq@ti.com] Rename EEPROM macro
> 
> Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com>
> Signed-off-by: Roger Quadros <rogerq@ti.com>

Applied to u-boot/master, thanks!
diff mbox

Patch

diff --git a/include/configs/ti_armv7_keystone2.h b/include/configs/ti_armv7_keystone2.h
index 5d4ef58..f76e0a5 100644
--- a/include/configs/ti_armv7_keystone2.h
+++ b/include/configs/ti_armv7_keystone2.h
@@ -55,6 +55,13 @@ 
 #define CONFIG_SPL_SPI_LOAD
 #define CONFIG_SYS_SPI_U_BOOT_OFFS	CONFIG_SPL_PAD_TO
 
+/* SRAM scratch space entries  */
+#define SRAM_SCRATCH_SPACE_ADDR	CONFIG_SPL_STACK + 0x8
+
+#define TI_SRAM_SCRATCH_BOARD_EEPROM_START	(SRAM_SCRATCH_SPACE_ADDR)
+#define TI_SRAM_SCRATCH_BOARD_EEPROM_END	(SRAM_SCRATCH_SPACE_ADDR + 0x200)
+#define KEYSTONE_SRAM_SCRATCH_SPACE_END		(TI_SRAM_SCRATCH_BOARD_EEPROM_END)
+
 /* UART Configuration */
 #define CONFIG_SYS_NS16550_MEM32
 #if defined(CONFIG_SPL_BUILD) || !defined(CONFIG_DM_SERIAL)