All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] ARM: da850evm: Fix TI boot scripts for MMC
@ 2019-04-17 18:24 Adam Ford
  2019-04-27 14:47 ` [U-Boot] " Tom Rini
  0 siblings, 1 reply; 2+ messages in thread
From: Adam Ford @ 2019-04-17 18:24 UTC (permalink / raw)
  To: u-boot

The da850evm include environment/ti/mmc.h and places
DEFAULT_MMC_TI_ARGS, defined int that file, in
CONFIG_EXTRA_ENV_SETTINGS.  This has been broken for some time
since neither CMD_PART nor CMD_UUID are available, so manually
changing the environmental variables was always required before
booting from MMC. With this patch, these scripts should work
again, and by default they point to mmc 0, partition 2, and
the dtb file exists in a /boot directory on partition 2.

Signed-off-by: Adam Ford <aford173@gmail.com>

diff --git a/configs/da850evm_defconfig b/configs/da850evm_defconfig
index 745ae2b4b1..ee39b0b1bc 100644
--- a/configs/da850evm_defconfig
+++ b/configs/da850evm_defconfig
@@ -9,9 +9,9 @@ CONFIG_SPL_LIBGENERIC_SUPPORT=y
 CONFIG_SYS_MALLOC_F_LEN=0x800
 CONFIG_SPL_SERIAL_SUPPORT=y
 CONFIG_SPL=y
+CONFIG_NR_DRAM_BANKS=1
 CONFIG_SPL_SPI_FLASH_SUPPORT=y
 CONFIG_SPL_SPI_SUPPORT=y
-CONFIG_NR_DRAM_BANKS=1
 CONFIG_SYS_EXTRA_OPTIONS="MAC_ADDR_IN_SPIFLASH"
 CONFIG_BOOTDELAY=3
 CONFIG_DEFAULT_FDT_FILE="da850-evm.dtb"
@@ -30,11 +30,10 @@ CONFIG_CRC32_VERIFY=y
 CONFIG_CMD_DM=y
 # CONFIG_CMD_FLASH is not set
 # CONFIG_CMD_GPT is not set
-# CONFIG_CMD_PART is not set
 # CONFIG_CMD_SETEXPR is not set
 # CONFIG_CMD_TIME is not set
 # CONFIG_CMD_EXT4 is not set
-# CONFIG_CMD_FS_GENERIC is not set
+CONFIG_CMD_FS_UUID=y
 CONFIG_CMD_MTDPARTS=y
 CONFIG_MTDIDS_DEFAULT="nor0=spi0.0"
 CONFIG_MTDPARTS_DEFAULT="mtdparts=spi0.0:512k(u-boot.ais),64k(u-boot-env),7552k(kernel-spare),64k(MAC-Address)"
-- 
2.17.1

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

* [U-Boot] ARM: da850evm: Fix TI boot scripts for MMC
  2019-04-17 18:24 [U-Boot] [PATCH] ARM: da850evm: Fix TI boot scripts for MMC Adam Ford
@ 2019-04-27 14:47 ` Tom Rini
  0 siblings, 0 replies; 2+ messages in thread
From: Tom Rini @ 2019-04-27 14:47 UTC (permalink / raw)
  To: u-boot

On Wed, Apr 17, 2019 at 01:24:33PM -0500, Adam Ford wrote:

> The da850evm include environment/ti/mmc.h and places
> DEFAULT_MMC_TI_ARGS, defined int that file, in
> CONFIG_EXTRA_ENV_SETTINGS.  This has been broken for some time
> since neither CMD_PART nor CMD_UUID are available, so manually
> changing the environmental variables was always required before
> booting from MMC. With this patch, these scripts should work
> again, and by default they point to mmc 0, partition 2, and
> the dtb file exists in a /boot directory on partition 2.
> 
> Signed-off-by: Adam Ford <aford173@gmail.com>
> 
> diff --git a/configs/da850evm_defconfig b/configs/da850evm_defconfig
> index 745ae2b4b1..ee39b0b1bc 100644

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190427/75f56a38/attachment.sig>

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

end of thread, other threads:[~2019-04-27 14:47 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-17 18:24 [U-Boot] [PATCH] ARM: da850evm: Fix TI boot scripts for MMC Adam Ford
2019-04-27 14:47 ` [U-Boot] " Tom Rini

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.