[v4,03/10] ti_armv7_keystone2: Define scratch space in SRAM

Message ID 1489410273-10159-4-git-send-email-rogerq@ti.com
State New
Headers show
Series
  • am57xx-idk LCD and am571x-idk 6 port ethernet pinmux
Related show

Commit Message

Roger Quadros March 13, 2017, 1:04 p.m.
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. | #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
Tom Rini March 21, 2017, 6:08 p.m. | #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!

-- 
Tom

Patch hide | download patch | download mbox

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)