From mboxrd@z Thu Jan  1 00:00:00 1970
From: Daniel Schwierzeck <daniel.schwierzeck@googlemail.com>
Date: Sat, 16 Jul 2011 15:36:28 +0200
Subject: [U-Boot] [PATCH v3 09/12] omap: add basic SPL support
In-Reply-To: <1310820802-24984-10-git-send-email-aneesh@ti.com>
References: <1298893591-17636-1-git-send-email-aneesh@ti.com>
	<1310820802-24984-10-git-send-email-aneesh@ti.com>
Message-ID: <4E2193DC.8040207@googlemail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Aneesh,

On 07/16/2011 02:53 PM, Aneesh V wrote:

> diff --git a/include/configs/omap4_panda.h b/include/configs/omap4_panda.h
> index a8dd861..3244fc1 100644
> --- a/include/configs/omap4_panda.h
> +++ b/include/configs/omap4_panda.h
> @@ -243,4 +243,30 @@
>   #define CONFIG_SYS_DEFAULT_LPDDR2_TIMINGS
>   #endif
>
> +/* Defines for SPL */
> +#define CONFIG_SPL
> +#define CONFIG_SYS_SPL_TEXT_BASE	0x40304350

Does this work? The SPL framework patches currently use

ifneq ($(CONFIG_SPL_TEXT_BASE),)
CPPFLAGS += -DCONFIG_SPL_TEXT_BASE=$(CONFIG_SPL_TEXT_BASE)
endif

But CONFIG_SYS_SPL_TEXT_BASE seems to be the right name for this option.
Shall I fix it in the SPL framework patches?

> +#define CONFIG_SYS_SPL_MAX_SIZE		0x8000	/* 32 K */
> +#define CONFIG_SYS_SPL_STACK		LOW_LEVEL_SRAM_STACK
> +
> +#define CONFIG_SYS_SPL_BSS_START_ADDR		0x80000000
> +#define CONFIG_SYS_SPL_BSS_MAX_SIZE		0x80000		/* 512 KB */
> +
> +#define CONFIG_SPL_LIBCOMMON_SUPPORT
> +#define CONFIG_SPL_LIBDISK_SUPPORT
> +#define CONFIG_SPL_I2C_SUPPORT
> +#define CONFIG_SPL_MMC_SUPPORT
> +#define CONFIG_SPL_FAT_SUPPORT
> +#define CONFIG_SPL_LIBGENERIC_SUPPORT
> +#define CONFIG_SPL_SERIAL_SUPPORT
> +#define CONFIG_SYS_SPL_LDSCRIPT "arch/arm/cpu/armv7/omap-common/u-boot-spl.lds"
> +
> +/*
> + * 1MB into the SDRAM to allow for SPL's bss at the beginning of SDRAM
> + * 64 bytes before this address should be set aside for u-boot.img's
> + * header. That is 0x800FFFC0--0x80100000 should not be used for any
> + * other needs.
> + */
> +#define CONFIG_SYS_TEXT_BASE		0x80100000
> +
>   #endif /* __CONFIG_H */
> diff --git a/include/configs/omap4_sdp4430.h b/include/configs/omap4_sdp4430.h
> index 534f89a..5c486c3 100644
> --- a/include/configs/omap4_sdp4430.h
> +++ b/include/configs/omap4_sdp4430.h
> @@ -249,4 +249,30 @@
>   #define CONFIG_SYS_DEFAULT_LPDDR2_TIMINGS
>   #endif
>
> +/* Defines for SPL */
> +#define CONFIG_SPL
> +#define CONFIG_SYS_SPL_TEXT_BASE	0x40304350
> +#define CONFIG_SYS_SPL_MAX_SIZE		0x8000	/* 32 K */
> +#define CONFIG_SYS_SPL_STACK		LOW_LEVEL_SRAM_STACK
> +
> +#define CONFIG_SYS_SPL_BSS_START_ADDR		0x80000000
> +#define CONFIG_SYS_SPL_BSS_MAX_SIZE		0x80000		/* 512 KB */
> +
> +#define CONFIG_SPL_LIBCOMMON_SUPPORT
> +#define CONFIG_SPL_LIBDISK_SUPPORT
> +#define CONFIG_SPL_I2C_SUPPORT
> +#define CONFIG_SPL_MMC_SUPPORT
> +#define CONFIG_SPL_FAT_SUPPORT
> +#define CONFIG_SPL_LIBGENERIC_SUPPORT
> +#define CONFIG_SPL_SERIAL_SUPPORT
> +#define CONFIG_SYS_SPL_LDSCRIPT "arch/arm/cpu/armv7/omap-common/u-boot-spl.lds"
> +
> +/*
> + * 1MB into the SDRAM to allow for SPL's bss at the beginning of SDRAM
> + * 64 bytes before this address should be set aside for u-boot.img's
> + * header. That is 0x800FFFC0--0x80100000 should not be used for any
> + * other needs.
> + */
> +#define CONFIG_SYS_TEXT_BASE		0x80100000
> +
>   #endif /* __CONFIG_H */