diff mbox

[U-Boot,3/3] davinci: omapl138_lcdk: add MMC/SD SPL boot support

Message ID 20161116163335.4560-3-fparent@baylibre.com
State Superseded
Delegated to: Tom Rini
Headers show

Commit Message

Fabien Parent Nov. 16, 2016, 4:33 p.m. UTC
Define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR in order to be able
to boot from MMC/SD.

The SPL is stored at sector 0x75, while u-boot will follow at
sector 0xb5.

Signed-off-by: Fabien Parent <fparent@baylibre.com>
---
 include/configs/omapl138_lcdk.h | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Tom Rini Nov. 17, 2016, 10:23 p.m. UTC | #1
On Wed, Nov 16, 2016 at 05:33:35PM +0100, Fabien Parent wrote:
> Define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR in order to be able
> to boot from MMC/SD.
> 
> The SPL is stored at sector 0x75, while u-boot will follow at
> sector 0xb5.
> 
> Signed-off-by: Fabien Parent <fparent@baylibre.com>
> ---
>  include/configs/omapl138_lcdk.h | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/include/configs/omapl138_lcdk.h b/include/configs/omapl138_lcdk.h
> index 91ddbb1..0964176 100644
> --- a/include/configs/omapl138_lcdk.h
> +++ b/include/configs/omapl138_lcdk.h
> @@ -272,6 +272,14 @@
>  #define CONFIG_SPL_PAD_TO	32768
>  #endif
>  
> +/* Load U-Boot Image From MMC */
> +#ifdef CONFIG_SPL_MMC_LOAD
> +#define SPL_SECTOR				0x75
> +#define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR	(SPL_SECTOR + \
> +							CONFIG_SPL_PAD_TO / 512)
> +#undef CONFIG_SPL_SPI_LOAD
> +#endif
> +
>  /* AIS Image */
>  #define CONFIG_AIS_CONFIG_FILE	"board/davinci/da8xxevm/omapl138_lcdk_ais.cfg"

We need to do this on top of Sam's series, and we need to do this in
Kconfig and not adding more stuff to the config header.
Sam Protsenko Nov. 22, 2016, 4:40 p.m. UTC | #2
On Fri, Nov 18, 2016 at 12:23 AM, Tom Rini <trini@konsulko.com> wrote:
> On Wed, Nov 16, 2016 at 05:33:35PM +0100, Fabien Parent wrote:
>> Define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR in order to be able
>> to boot from MMC/SD.
>>
>> The SPL is stored at sector 0x75, while u-boot will follow at
>> sector 0xb5.
>>
>> Signed-off-by: Fabien Parent <fparent@baylibre.com>
>> ---
>>  include/configs/omapl138_lcdk.h | 8 ++++++++
>>  1 file changed, 8 insertions(+)
>>
>> diff --git a/include/configs/omapl138_lcdk.h b/include/configs/omapl138_lcdk.h
>> index 91ddbb1..0964176 100644
>> --- a/include/configs/omapl138_lcdk.h
>> +++ b/include/configs/omapl138_lcdk.h
>> @@ -272,6 +272,14 @@
>>  #define CONFIG_SPL_PAD_TO    32768
>>  #endif
>>
>> +/* Load U-Boot Image From MMC */
>> +#ifdef CONFIG_SPL_MMC_LOAD
>> +#define SPL_SECTOR                           0x75
>> +#define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR      (SPL_SECTOR + \
>> +                                                     CONFIG_SPL_PAD_TO / 512)
>> +#undef CONFIG_SPL_SPI_LOAD
>> +#endif
>> +
>>  /* AIS Image */
>>  #define CONFIG_AIS_CONFIG_FILE       "board/davinci/da8xxevm/omapl138_lcdk_ais.cfg"
>
> We need to do this on top of Sam's series, and we need to do this in
> Kconfig and not adding more stuff to the config header.
>

Just FYI: mentioned patch is [1]. For the Kconfig option itself see diif at [2].

[1] http://git.denx.de/?p=u-boot.git;a=commit;h=38fed8abe7d2e7ba90deb352d0e7aed4364c5236
[2] http://git.denx.de/?p=u-boot.git;a=blobdiff;f=common/spl/Kconfig;h=df9e0ce68f9ea2348af8aec3ebe5aea25456062e;hp=bb99f1fcff4b4ba3ae909be3d3134323741f6c84;hb=38fed8abe7d2e7ba90deb352d0e7aed4364c5236;hpb=c2cbd164ea5b5f564fcf03447c7bf9ec4a9f5699

> --
> Tom
diff mbox

Patch

diff --git a/include/configs/omapl138_lcdk.h b/include/configs/omapl138_lcdk.h
index 91ddbb1..0964176 100644
--- a/include/configs/omapl138_lcdk.h
+++ b/include/configs/omapl138_lcdk.h
@@ -272,6 +272,14 @@ 
 #define CONFIG_SPL_PAD_TO	32768
 #endif
 
+/* Load U-Boot Image From MMC */
+#ifdef CONFIG_SPL_MMC_LOAD
+#define SPL_SECTOR				0x75
+#define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR	(SPL_SECTOR + \
+							CONFIG_SPL_PAD_TO / 512)
+#undef CONFIG_SPL_SPI_LOAD
+#endif
+
 /* AIS Image */
 #define CONFIG_AIS_CONFIG_FILE	"board/davinci/da8xxevm/omapl138_lcdk_ais.cfg"