Hello Peng,

Am 15.06.2016 um 08:15 schrieb Peng Fan:
Add CONFIG_{SD|NOR|NAND|ONENAND|SPI|QSPI|SATA}_BOOT kconfig entries.

SoCs supports loading U-Boot from different medias to DRAM, such as
i.MX6/7 supports loading U-Boot to DRAM from sd/emmc/nand/qspi/spi/sata
and etc. For i.MX, imximage will generate different IVT headers according
to boot medias.

Signed-off-by: Peng Fan <peng....@nxp.com>
Cc: Simon Glass <s...@chromium.org>
Cc: Heiko Schocher <h...@denx.de>
Cc: Joe Hershberger <joe.hershber...@ni.com>
Cc: Bin Meng <bmeng...@gmail.com>
Cc: Christophe Ricard <christophe-h.ric...@st.com>
Cc: Nikita Kiryanov <nik...@compulab.co.il>
Cc: Francois Retief <fgret...@spaceteq.co.za>
Cc: Tom Rini <tr...@konsulko.com>
---
  common/Kconfig | 46 ++++++++++++++++++++++++++++++++++++++++++++++
  1 file changed, 46 insertions(+)

Hmm... are there no places where the introduced config options
already used in config files ?

diff --git a/common/Kconfig b/common/Kconfig
index 4d17b10..a41b670 100644
--- a/common/Kconfig
+++ b/common/Kconfig
@@ -97,6 +97,52 @@ config BOOTSTAGE_STASH_SIZE

  endmenu

+menu "Boot media"
+
+config SD_BOOT
+       bool "Boot from SD"
+       default n
+       help
+         U-Boot image is stored in SD card.
+
+config NOR_BOOT
+       bool "Boot from NOR"
+       default n
+       help
+         U-Boot image is stored in NOR flash.

For example this is already defined in

./board/ti/am335x/Kconfig

So I miss at least the lines, which remove this part in your patch,
else I think, this would lead in a Kconfig error, if we have the
same symbol in two places ... or?

+
+config NAND_BOOT
+       bool "Boot from NAND"
+       default n
+       help
+         U-Boot image is stored in NAND flash.
+
+config ONENAND_BOOT
+       bool "Boot from ONENAND"
+       default n
+       help
+         U-Boot image is stored in ONENAND flash.
+
+config SPI_BOOT
+       bool "Boot from SPI"
+       default n
+       help
+         U-Boot image is stored in SPI flash.
+
+config QSPI_BOOT
+       bool "Boot from QSPI"
+       default n
+       help
+         U-Boot image is stored in QSPI flash.
+
+config SATA_BOOT
+       bool "Boot from SATA"
+       default n
+       help
+         U-Boot image is stored in SATA.
+
+endmenu
+
  config BOOTDELAY
        int "delay in seconds before automatically booting"
        default 0


may you want to use my brand new tbot testcase [1] for testing, if your
patch not breaks boards?

bye,
Heiko
[1] http://lists.denx.de/pipermail/u-boot/2016-June/258119.html
--
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to