All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH 1/3] am43xx: Remove ethboot / usbhost_boot variants
@ 2017-07-26 17:11 Tom Rini
  2017-07-26 17:11 ` [U-Boot] [PATCH 2/3] am335x_hs_evm: Disable CONFIG_SPL_ENV_SUPPORT Tom Rini
                   ` (2 more replies)
  0 siblings, 3 replies; 12+ messages in thread
From: Tom Rini @ 2017-07-26 17:11 UTC (permalink / raw)
  To: u-boot

The intention of having aaa_evm_xxx_defconfig variants is to allow
building for useful but non-default cases that cannot be easily done
with the standard aaa_evm_defconfig file.  In this case, we get rid of
usbhost_boot as SPL_TEXT has been in OCMC RAM for a while now, so this
can be included in the default.  We get rid of ethboot as it's fine to
have the environment in someplace that's not available and the default
will be used.  Further, now that environment location is in Kconfig it's
reasonable to expect a user of this feature to optimize their use-case
if needed.

Cc: Lokesh Vutla <lokeshvutla@ti.com>
Signed-off-by: Tom Rini <trini@konsulko.com>
---
 configs/am43xx_evm_defconfig              |  7 +++
 configs/am43xx_evm_ethboot_defconfig      | 63 --------------------------
 configs/am43xx_evm_usbhost_boot_defconfig | 75 -------------------------------
 3 files changed, 7 insertions(+), 138 deletions(-)
 delete mode 100644 configs/am43xx_evm_ethboot_defconfig
 delete mode 100644 configs/am43xx_evm_usbhost_boot_defconfig

diff --git a/configs/am43xx_evm_defconfig b/configs/am43xx_evm_defconfig
index 7a7a0b0ea0ac..ae2ae1ae516a 100644
--- a/configs/am43xx_evm_defconfig
+++ b/configs/am43xx_evm_defconfig
@@ -12,8 +12,15 @@ CONFIG_SYS_CONSOLE_INFO_QUIET=y
 CONFIG_VERSION_VARIABLE=y
 CONFIG_SPL=y
 CONFIG_SPL_STACK_R=y
+CONFIG_SPL_ETH_SUPPORT=y
 CONFIG_SPL_MTD_SUPPORT=y
+CONFIG_SPL_NET_SUPPORT=y
+CONFIG_SPL_NET_VCI_STRING="AM43xx U-Boot SPL"
 CONFIG_SPL_OS_BOOT=y
+CONFIG_SPL_USB_HOST_SUPPORT=y
+CONFIG_SPL_USB_SUPPORT=y
+CONFIG_SPL_USB_GADGET_SUPPORT=y
+CONFIG_SPL_USBETH_SUPPORT=y
 CONFIG_HUSH_PARSER=y
 # CONFIG_CMD_IMLS is not set
 # CONFIG_CMD_FLASH is not set
diff --git a/configs/am43xx_evm_ethboot_defconfig b/configs/am43xx_evm_ethboot_defconfig
deleted file mode 100644
index 9892653d27c3..000000000000
--- a/configs/am43xx_evm_ethboot_defconfig
+++ /dev/null
@@ -1,63 +0,0 @@
-CONFIG_ARM=y
-CONFIG_ARCH_OMAP2PLUS=y
-CONFIG_AM43XX=y
-CONFIG_SYS_EXTRA_OPTIONS="SERIAL1,CONS_INDEX=1,NAND"
-CONFIG_ENV_IS_IN_FAT=y
-CONFIG_SYS_CONSOLE_INFO_QUIET=y
-CONFIG_VERSION_VARIABLE=y
-CONFIG_SPL=y
-# CONFIG_SPL_SYS_MALLOC_SIMPLE is not set
-CONFIG_SPL_ETH_SUPPORT=y
-CONFIG_SPL_MTD_SUPPORT=y
-CONFIG_SPL_NET_SUPPORT=y
-CONFIG_SPL_NET_VCI_STRING="AM43xx U-Boot SPL"
-CONFIG_SPL_OS_BOOT=y
-CONFIG_HUSH_PARSER=y
-CONFIG_CMD_BOOTZ=y
-# CONFIG_CMD_IMLS is not set
-CONFIG_CMD_ASKENV=y
-CONFIG_CMD_EEPROM=y
-# CONFIG_CMD_FLASH is not set
-CONFIG_CMD_GPT=y
-CONFIG_CMD_MMC=y
-CONFIG_CMD_PART=y
-CONFIG_CMD_SF=y
-CONFIG_CMD_SPI=y
-CONFIG_CMD_I2C=y
-CONFIG_CMD_USB=y
-CONFIG_CMD_DFU=y
-CONFIG_CMD_GPIO=y
-# CONFIG_CMD_SETEXPR is not set
-CONFIG_CMD_DHCP=y
-CONFIG_CMD_MII=y
-CONFIG_CMD_PING=y
-CONFIG_CMD_EXT2=y
-CONFIG_CMD_EXT4=y
-CONFIG_CMD_EXT4_WRITE=y
-CONFIG_CMD_FAT=y
-CONFIG_CMD_FS_GENERIC=y
-CONFIG_CMD_MTDPARTS=y
-CONFIG_ISO_PARTITION=y
-CONFIG_DFU_MMC=y
-CONFIG_DFU_RAM=y
-CONFIG_DFU_SF=y
-CONFIG_MMC_OMAP_HS=y
-CONFIG_SPI_FLASH=y
-CONFIG_SPI_FLASH_MACRONIX=y
-CONFIG_SYS_NS16550=y
-CONFIG_TI_QSPI=y
-CONFIG_USB=y
-CONFIG_USB_XHCI_HCD=y
-CONFIG_USB_XHCI_DWC3=y
-CONFIG_USB_DWC3=y
-CONFIG_USB_DWC3_GADGET=y
-CONFIG_USB_DWC3_OMAP=y
-CONFIG_USB_DWC3_PHY_OMAP=y
-CONFIG_USB_STORAGE=y
-CONFIG_USB_GADGET=y
-CONFIG_USB_GADGET_DOWNLOAD=y
-CONFIG_G_DNL_MANUFACTURER="Texas Instruments"
-CONFIG_G_DNL_VENDOR_NUM=0x0403
-CONFIG_G_DNL_PRODUCT_NUM=0xbd00
-CONFIG_FAT_WRITE=y
-CONFIG_OF_LIBFDT=y
diff --git a/configs/am43xx_evm_usbhost_boot_defconfig b/configs/am43xx_evm_usbhost_boot_defconfig
deleted file mode 100644
index f4c0f8442a17..000000000000
--- a/configs/am43xx_evm_usbhost_boot_defconfig
+++ /dev/null
@@ -1,75 +0,0 @@
-CONFIG_ARM=y
-CONFIG_ARCH_OMAP2PLUS=y
-CONFIG_SYS_MALLOC_F_LEN=0x2000
-CONFIG_AM43XX=y
-CONFIG_ISW_ENTRY_ADDR=0x40300350
-CONFIG_SPL_STACK_R_ADDR=0x82000000
-CONFIG_DEFAULT_DEVICE_TREE="am437x-gp-evm"
-CONFIG_SPL_LOAD_FIT=y
-CONFIG_SYS_EXTRA_OPTIONS="SERIAL1,CONS_INDEX=1,NAND"
-CONFIG_ENV_IS_IN_FAT=y
-CONFIG_SYS_CONSOLE_INFO_QUIET=y
-CONFIG_VERSION_VARIABLE=y
-CONFIG_SPL=y
-CONFIG_SPL_STACK_R=y
-CONFIG_SPL_MTD_SUPPORT=y
-CONFIG_SPL_OS_BOOT=y
-CONFIG_SPL_USB_HOST_SUPPORT=y
-CONFIG_SPL_USB_SUPPORT=y
-CONFIG_HUSH_PARSER=y
-CONFIG_CMD_BOOTZ=y
-# CONFIG_CMD_IMLS is not set
-CONFIG_CMD_ASKENV=y
-CONFIG_CMD_EEPROM=y
-# CONFIG_CMD_FLASH is not set
-CONFIG_CMD_GPT=y
-CONFIG_CMD_MMC=y
-CONFIG_CMD_PART=y
-CONFIG_CMD_SF=y
-CONFIG_CMD_SPI=y
-CONFIG_CMD_I2C=y
-CONFIG_CMD_USB=y
-CONFIG_CMD_DFU=y
-CONFIG_CMD_GPIO=y
-# CONFIG_CMD_SETEXPR is not set
-CONFIG_CMD_DHCP=y
-CONFIG_CMD_MII=y
-CONFIG_CMD_PING=y
-CONFIG_CMD_EXT2=y
-CONFIG_CMD_EXT4=y
-CONFIG_CMD_EXT4_WRITE=y
-CONFIG_CMD_FAT=y
-CONFIG_CMD_FS_GENERIC=y
-CONFIG_CMD_MTDPARTS=y
-CONFIG_ISO_PARTITION=y
-CONFIG_OF_CONTROL=y
-CONFIG_OF_LIST="am437x-gp-evm am437x-sk-evm am43x-epos-evm am437x-idk-evm"
-CONFIG_DM=y
-# CONFIG_BLK is not set
-CONFIG_DFU_MMC=y
-CONFIG_DFU_RAM=y
-CONFIG_DFU_SF=y
-CONFIG_DM_GPIO=y
-CONFIG_DM_MMC=y
-CONFIG_MMC_OMAP_HS=y
-CONFIG_SPI_FLASH=y
-CONFIG_SPI_FLASH_MACRONIX=y
-CONFIG_DM_SERIAL=y
-CONFIG_SYS_NS16550=y
-CONFIG_TI_QSPI=y
-CONFIG_TIMER=y
-CONFIG_OMAP_TIMER=y
-CONFIG_USB=y
-CONFIG_USB_XHCI_HCD=y
-CONFIG_USB_XHCI_DWC3=y
-CONFIG_USB_DWC3=y
-CONFIG_USB_DWC3_GADGET=y
-CONFIG_USB_DWC3_OMAP=y
-CONFIG_USB_DWC3_PHY_OMAP=y
-CONFIG_USB_STORAGE=y
-CONFIG_USB_GADGET=y
-CONFIG_USB_GADGET_DOWNLOAD=y
-CONFIG_G_DNL_MANUFACTURER="Texas Instruments"
-CONFIG_G_DNL_VENDOR_NUM=0x0403
-CONFIG_G_DNL_PRODUCT_NUM=0xbd00
-CONFIG_FAT_WRITE=y
-- 
1.9.1

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

* [U-Boot] [PATCH 2/3] am335x_hs_evm: Disable CONFIG_SPL_ENV_SUPPORT
  2017-07-26 17:11 [U-Boot] [PATCH 1/3] am43xx: Remove ethboot / usbhost_boot variants Tom Rini
@ 2017-07-26 17:11 ` Tom Rini
  2017-07-27  2:46   ` Lokesh Vutla
  2017-08-01  0:41   ` [U-Boot] [U-Boot, " Tom Rini
  2017-07-26 17:11 ` [U-Boot] [PATCH 3/3] ti: Default to ENV_IS_IN_FAT if MMC_OMAP_HS Tom Rini
  2017-07-27  2:45 ` [U-Boot] [PATCH 1/3] am43xx: Remove ethboot / usbhost_boot variants Lokesh Vutla
  2 siblings, 2 replies; 12+ messages in thread
From: Tom Rini @ 2017-07-26 17:11 UTC (permalink / raw)
  To: u-boot

The main uses of CONFIG_SPL_ENV_SUPPORT are for network support and for
disabling "Falcon Mode" support at run-time.  As this build enables
neither, remove this feature.

Cc: Andrew F. Davis <afd@ti.com>
Signed-off-by: Tom Rini <trini@konsulko.com>
---
 configs/am335x_hs_evm_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/configs/am335x_hs_evm_defconfig b/configs/am335x_hs_evm_defconfig
index 2b1acb70d18f..ffdbe7585573 100644
--- a/configs/am335x_hs_evm_defconfig
+++ b/configs/am335x_hs_evm_defconfig
@@ -18,6 +18,7 @@ CONFIG_VERSION_VARIABLE=y
 CONFIG_ARCH_MISC_INIT=y
 CONFIG_SPL=y
 CONFIG_SPL_STACK_R=y
+# CONFIG_SPL_ENV_SUPPORT is not set
 # CONFIG_SPL_EXT_SUPPORT is not set
 CONFIG_SPL_MTD_SUPPORT=y
 # CONFIG_SPL_YMODEM_SUPPORT is not set
-- 
1.9.1

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

* [U-Boot] [PATCH 3/3] ti: Default to ENV_IS_IN_FAT if MMC_OMAP_HS
  2017-07-26 17:11 [U-Boot] [PATCH 1/3] am43xx: Remove ethboot / usbhost_boot variants Tom Rini
  2017-07-26 17:11 ` [U-Boot] [PATCH 2/3] am335x_hs_evm: Disable CONFIG_SPL_ENV_SUPPORT Tom Rini
@ 2017-07-26 17:11 ` Tom Rini
  2017-07-27  2:49   ` Lokesh Vutla
  2017-08-01  0:41   ` [U-Boot] [U-Boot, " Tom Rini
  2017-07-27  2:45 ` [U-Boot] [PATCH 1/3] am43xx: Remove ethboot / usbhost_boot variants Lokesh Vutla
  2 siblings, 2 replies; 12+ messages in thread
From: Tom Rini @ 2017-07-26 17:11 UTC (permalink / raw)
  To: u-boot

When we have MMC available we assume that we want to put the env as a
file on FAT.

Cc: Lokesh Vutla <lokeshvutla@ti.com>
Signed-off-by: Tom Rini <trini@konsulko.com>
---
 board/ti/common/Kconfig             | 1 +
 configs/am335x_evm_defconfig        | 1 -
 configs/am335x_evm_usbspl_defconfig | 1 -
 configs/am335x_hs_evm_defconfig     | 1 -
 configs/am43xx_evm_defconfig        | 1 -
 configs/am43xx_hs_evm_defconfig     | 1 -
 configs/k2g_evm_defconfig           | 1 -
 configs/k2g_hs_evm_defconfig        | 1 -
 8 files changed, 1 insertion(+), 7 deletions(-)

diff --git a/board/ti/common/Kconfig b/board/ti/common/Kconfig
index c21eb8c2d2e3..c81baa12becc 100644
--- a/board/ti/common/Kconfig
+++ b/board/ti/common/Kconfig
@@ -42,3 +42,4 @@ config TI_COMMON_CMD_OPTIONS
 	imply CMD_SPI
 	imply CMD_TIME
 	imply CMD_USB if USB
+	imply ENV_IS_IN_FAT if MMC_OMAP_HS
diff --git a/configs/am335x_evm_defconfig b/configs/am335x_evm_defconfig
index 4ea56178eeab..204d817c0a60 100644
--- a/configs/am335x_evm_defconfig
+++ b/configs/am335x_evm_defconfig
@@ -7,7 +7,6 @@ CONFIG_DEFAULT_DEVICE_TREE="am335x-evm"
 CONFIG_DISTRO_DEFAULTS=y
 CONFIG_SPL_LOAD_FIT=y
 CONFIG_SYS_EXTRA_OPTIONS="NAND"
-CONFIG_ENV_IS_IN_FAT=y
 CONFIG_SYS_CONSOLE_INFO_QUIET=y
 CONFIG_VERSION_VARIABLE=y
 CONFIG_ARCH_MISC_INIT=y
diff --git a/configs/am335x_evm_usbspl_defconfig b/configs/am335x_evm_usbspl_defconfig
index 51d1042c3433..84e746916cfa 100644
--- a/configs/am335x_evm_usbspl_defconfig
+++ b/configs/am335x_evm_usbspl_defconfig
@@ -6,7 +6,6 @@ CONFIG_AM33XX=y
 CONFIG_SPL_STACK_R_ADDR=0x82000000
 CONFIG_DISTRO_DEFAULTS=y
 CONFIG_SYS_EXTRA_OPTIONS="NAND"
-CONFIG_ENV_IS_IN_FAT=y
 CONFIG_SYS_CONSOLE_INFO_QUIET=y
 CONFIG_VERSION_VARIABLE=y
 CONFIG_ARCH_MISC_INIT=y
diff --git a/configs/am335x_hs_evm_defconfig b/configs/am335x_hs_evm_defconfig
index ffdbe7585573..52cbf05ab025 100644
--- a/configs/am335x_hs_evm_defconfig
+++ b/configs/am335x_hs_evm_defconfig
@@ -12,7 +12,6 @@ CONFIG_FIT_IMAGE_POST_PROCESS=y
 CONFIG_SPL_LOAD_FIT=y
 CONFIG_SPL_FIT_IMAGE_POST_PROCESS=y
 CONFIG_SYS_EXTRA_OPTIONS="NAND"
-CONFIG_ENV_IS_IN_FAT=y
 CONFIG_SYS_CONSOLE_INFO_QUIET=y
 CONFIG_VERSION_VARIABLE=y
 CONFIG_ARCH_MISC_INIT=y
diff --git a/configs/am43xx_evm_defconfig b/configs/am43xx_evm_defconfig
index ae2ae1ae516a..903bd69ad209 100644
--- a/configs/am43xx_evm_defconfig
+++ b/configs/am43xx_evm_defconfig
@@ -7,7 +7,6 @@ CONFIG_SPL_STACK_R_ADDR=0x82000000
 CONFIG_DEFAULT_DEVICE_TREE="am437x-gp-evm"
 CONFIG_SPL_LOAD_FIT=y
 CONFIG_SYS_EXTRA_OPTIONS="CONS_INDEX=1,NAND"
-CONFIG_ENV_IS_IN_FAT=y
 CONFIG_SYS_CONSOLE_INFO_QUIET=y
 CONFIG_VERSION_VARIABLE=y
 CONFIG_SPL=y
diff --git a/configs/am43xx_hs_evm_defconfig b/configs/am43xx_hs_evm_defconfig
index 5306c26b9908..b8e3fd46f272 100644
--- a/configs/am43xx_hs_evm_defconfig
+++ b/configs/am43xx_hs_evm_defconfig
@@ -15,7 +15,6 @@ CONFIG_SPL_LOAD_FIT=y
 CONFIG_SPL_FIT_IMAGE_POST_PROCESS=y
 CONFIG_OF_BOARD_SETUP=y
 CONFIG_SYS_EXTRA_OPTIONS="CONS_INDEX=1,NAND"
-CONFIG_ENV_IS_IN_FAT=y
 CONFIG_SYS_CONSOLE_INFO_QUIET=y
 CONFIG_VERSION_VARIABLE=y
 CONFIG_SPL=y
diff --git a/configs/k2g_evm_defconfig b/configs/k2g_evm_defconfig
index 332c4d18101b..1eed60576cdd 100644
--- a/configs/k2g_evm_defconfig
+++ b/configs/k2g_evm_defconfig
@@ -10,7 +10,6 @@ CONFIG_SPL_SPI_FLASH_SUPPORT=y
 CONFIG_SPL_SPI_SUPPORT=y
 CONFIG_DEFAULT_DEVICE_TREE="keystone-k2g-evm"
 CONFIG_OF_BOARD_SETUP=y
-CONFIG_ENV_IS_IN_FAT=y
 CONFIG_SYS_CONSOLE_INFO_QUIET=y
 CONFIG_DTB_RESELECT=y
 CONFIG_FIT_EMBED=y
diff --git a/configs/k2g_hs_evm_defconfig b/configs/k2g_hs_evm_defconfig
index 2e2b18c18565..51c701ed902d 100644
--- a/configs/k2g_hs_evm_defconfig
+++ b/configs/k2g_hs_evm_defconfig
@@ -5,7 +5,6 @@ CONFIG_TI_COMMON_CMD_OPTIONS=y
 CONFIG_TARGET_K2G_EVM=y
 CONFIG_DEFAULT_DEVICE_TREE="keystone-k2g-evm"
 CONFIG_OF_BOARD_SETUP=y
-CONFIG_ENV_IS_IN_FAT=y
 CONFIG_SYS_CONSOLE_INFO_QUIET=y
 CONFIG_VERSION_VARIABLE=y
 CONFIG_HUSH_PARSER=y
-- 
1.9.1

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

* [U-Boot] [PATCH 1/3] am43xx: Remove ethboot / usbhost_boot variants
  2017-07-26 17:11 [U-Boot] [PATCH 1/3] am43xx: Remove ethboot / usbhost_boot variants Tom Rini
  2017-07-26 17:11 ` [U-Boot] [PATCH 2/3] am335x_hs_evm: Disable CONFIG_SPL_ENV_SUPPORT Tom Rini
  2017-07-26 17:11 ` [U-Boot] [PATCH 3/3] ti: Default to ENV_IS_IN_FAT if MMC_OMAP_HS Tom Rini
@ 2017-07-27  2:45 ` Lokesh Vutla
  2017-07-27  6:08   ` Vignesh R
  2017-07-27 11:15   ` Tom Rini
  2 siblings, 2 replies; 12+ messages in thread
From: Lokesh Vutla @ 2017-07-27  2:45 UTC (permalink / raw)
  To: u-boot

+ Vignesh

On 7/26/2017 10:41 PM, Tom Rini wrote:
> The intention of having aaa_evm_xxx_defconfig variants is to allow
> building for useful but non-default cases that cannot be easily done
> with the standard aaa_evm_defconfig file.  In this case, we get rid of
> usbhost_boot as SPL_TEXT has been in OCMC RAM for a while now, so this

SPL_TEXT for default boot is 0x402f0400
SPL_TEXT for usb host boot is 0x40300350 as ROM uses the initial portion
for DMA when copying from USB. Initially we thought of moving SPL_TET
for default boot to 0x40300350 but this will break UART boot as it has
to be 0x402f0400 for it. So I guess we cannot remove usb host boot
defconfig.

> can be included in the default.  We get rid of ethboot as it's fine to
> have the environment in someplace that's not available and the default

With DM_ETH enabled, it enforces to have DM for usb peripheral for usb
rndis boot, which is not supported yet. Vignesh is currently working on
it. I guess once it is done, ethboot_defocnfig can be removed.

So $patch breaks both usb host and ethboot.

Thanks and regards,
Lokesh

> will be used.  Further, now that environment location is in Kconfig it's
> reasonable to expect a user of this feature to optimize their use-case
> if needed.
> 
> Cc: Lokesh Vutla <lokeshvutla@ti.com>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
>  configs/am43xx_evm_defconfig              |  7 +++
>  configs/am43xx_evm_ethboot_defconfig      | 63 --------------------------
>  configs/am43xx_evm_usbhost_boot_defconfig | 75 -------------------------------
>  3 files changed, 7 insertions(+), 138 deletions(-)
>  delete mode 100644 configs/am43xx_evm_ethboot_defconfig
>  delete mode 100644 configs/am43xx_evm_usbhost_boot_defconfig
> 
> diff --git a/configs/am43xx_evm_defconfig b/configs/am43xx_evm_defconfig
> index 7a7a0b0ea0ac..ae2ae1ae516a 100644
> --- a/configs/am43xx_evm_defconfig
> +++ b/configs/am43xx_evm_defconfig
> @@ -12,8 +12,15 @@ CONFIG_SYS_CONSOLE_INFO_QUIET=y
>  CONFIG_VERSION_VARIABLE=y
>  CONFIG_SPL=y
>  CONFIG_SPL_STACK_R=y
> +CONFIG_SPL_ETH_SUPPORT=y
>  CONFIG_SPL_MTD_SUPPORT=y
> +CONFIG_SPL_NET_SUPPORT=y
> +CONFIG_SPL_NET_VCI_STRING="AM43xx U-Boot SPL"
>  CONFIG_SPL_OS_BOOT=y
> +CONFIG_SPL_USB_HOST_SUPPORT=y
> +CONFIG_SPL_USB_SUPPORT=y
> +CONFIG_SPL_USB_GADGET_SUPPORT=y
> +CONFIG_SPL_USBETH_SUPPORT=y
>  CONFIG_HUSH_PARSER=y
>  # CONFIG_CMD_IMLS is not set
>  # CONFIG_CMD_FLASH is not set
> diff --git a/configs/am43xx_evm_ethboot_defconfig b/configs/am43xx_evm_ethboot_defconfig
> deleted file mode 100644
> index 9892653d27c3..000000000000
> --- a/configs/am43xx_evm_ethboot_defconfig
> +++ /dev/null
> @@ -1,63 +0,0 @@
> -CONFIG_ARM=y
> -CONFIG_ARCH_OMAP2PLUS=y
> -CONFIG_AM43XX=y
> -CONFIG_SYS_EXTRA_OPTIONS="SERIAL1,CONS_INDEX=1,NAND"
> -CONFIG_ENV_IS_IN_FAT=y
> -CONFIG_SYS_CONSOLE_INFO_QUIET=y
> -CONFIG_VERSION_VARIABLE=y
> -CONFIG_SPL=y
> -# CONFIG_SPL_SYS_MALLOC_SIMPLE is not set
> -CONFIG_SPL_ETH_SUPPORT=y
> -CONFIG_SPL_MTD_SUPPORT=y
> -CONFIG_SPL_NET_SUPPORT=y
> -CONFIG_SPL_NET_VCI_STRING="AM43xx U-Boot SPL"
> -CONFIG_SPL_OS_BOOT=y
> -CONFIG_HUSH_PARSER=y
> -CONFIG_CMD_BOOTZ=y
> -# CONFIG_CMD_IMLS is not set
> -CONFIG_CMD_ASKENV=y
> -CONFIG_CMD_EEPROM=y
> -# CONFIG_CMD_FLASH is not set
> -CONFIG_CMD_GPT=y
> -CONFIG_CMD_MMC=y
> -CONFIG_CMD_PART=y
> -CONFIG_CMD_SF=y
> -CONFIG_CMD_SPI=y
> -CONFIG_CMD_I2C=y
> -CONFIG_CMD_USB=y
> -CONFIG_CMD_DFU=y
> -CONFIG_CMD_GPIO=y
> -# CONFIG_CMD_SETEXPR is not set
> -CONFIG_CMD_DHCP=y
> -CONFIG_CMD_MII=y
> -CONFIG_CMD_PING=y
> -CONFIG_CMD_EXT2=y
> -CONFIG_CMD_EXT4=y
> -CONFIG_CMD_EXT4_WRITE=y
> -CONFIG_CMD_FAT=y
> -CONFIG_CMD_FS_GENERIC=y
> -CONFIG_CMD_MTDPARTS=y
> -CONFIG_ISO_PARTITION=y
> -CONFIG_DFU_MMC=y
> -CONFIG_DFU_RAM=y
> -CONFIG_DFU_SF=y
> -CONFIG_MMC_OMAP_HS=y
> -CONFIG_SPI_FLASH=y
> -CONFIG_SPI_FLASH_MACRONIX=y
> -CONFIG_SYS_NS16550=y
> -CONFIG_TI_QSPI=y
> -CONFIG_USB=y
> -CONFIG_USB_XHCI_HCD=y
> -CONFIG_USB_XHCI_DWC3=y
> -CONFIG_USB_DWC3=y
> -CONFIG_USB_DWC3_GADGET=y
> -CONFIG_USB_DWC3_OMAP=y
> -CONFIG_USB_DWC3_PHY_OMAP=y
> -CONFIG_USB_STORAGE=y
> -CONFIG_USB_GADGET=y
> -CONFIG_USB_GADGET_DOWNLOAD=y
> -CONFIG_G_DNL_MANUFACTURER="Texas Instruments"
> -CONFIG_G_DNL_VENDOR_NUM=0x0403
> -CONFIG_G_DNL_PRODUCT_NUM=0xbd00
> -CONFIG_FAT_WRITE=y
> -CONFIG_OF_LIBFDT=y
> diff --git a/configs/am43xx_evm_usbhost_boot_defconfig b/configs/am43xx_evm_usbhost_boot_defconfig
> deleted file mode 100644
> index f4c0f8442a17..000000000000
> --- a/configs/am43xx_evm_usbhost_boot_defconfig
> +++ /dev/null
> @@ -1,75 +0,0 @@
> -CONFIG_ARM=y
> -CONFIG_ARCH_OMAP2PLUS=y
> -CONFIG_SYS_MALLOC_F_LEN=0x2000
> -CONFIG_AM43XX=y
> -CONFIG_ISW_ENTRY_ADDR=0x40300350
> -CONFIG_SPL_STACK_R_ADDR=0x82000000
> -CONFIG_DEFAULT_DEVICE_TREE="am437x-gp-evm"
> -CONFIG_SPL_LOAD_FIT=y
> -CONFIG_SYS_EXTRA_OPTIONS="SERIAL1,CONS_INDEX=1,NAND"
> -CONFIG_ENV_IS_IN_FAT=y
> -CONFIG_SYS_CONSOLE_INFO_QUIET=y
> -CONFIG_VERSION_VARIABLE=y
> -CONFIG_SPL=y
> -CONFIG_SPL_STACK_R=y
> -CONFIG_SPL_MTD_SUPPORT=y
> -CONFIG_SPL_OS_BOOT=y
> -CONFIG_SPL_USB_HOST_SUPPORT=y
> -CONFIG_SPL_USB_SUPPORT=y
> -CONFIG_HUSH_PARSER=y
> -CONFIG_CMD_BOOTZ=y
> -# CONFIG_CMD_IMLS is not set
> -CONFIG_CMD_ASKENV=y
> -CONFIG_CMD_EEPROM=y
> -# CONFIG_CMD_FLASH is not set
> -CONFIG_CMD_GPT=y
> -CONFIG_CMD_MMC=y
> -CONFIG_CMD_PART=y
> -CONFIG_CMD_SF=y
> -CONFIG_CMD_SPI=y
> -CONFIG_CMD_I2C=y
> -CONFIG_CMD_USB=y
> -CONFIG_CMD_DFU=y
> -CONFIG_CMD_GPIO=y
> -# CONFIG_CMD_SETEXPR is not set
> -CONFIG_CMD_DHCP=y
> -CONFIG_CMD_MII=y
> -CONFIG_CMD_PING=y
> -CONFIG_CMD_EXT2=y
> -CONFIG_CMD_EXT4=y
> -CONFIG_CMD_EXT4_WRITE=y
> -CONFIG_CMD_FAT=y
> -CONFIG_CMD_FS_GENERIC=y
> -CONFIG_CMD_MTDPARTS=y
> -CONFIG_ISO_PARTITION=y
> -CONFIG_OF_CONTROL=y
> -CONFIG_OF_LIST="am437x-gp-evm am437x-sk-evm am43x-epos-evm am437x-idk-evm"
> -CONFIG_DM=y
> -# CONFIG_BLK is not set
> -CONFIG_DFU_MMC=y
> -CONFIG_DFU_RAM=y
> -CONFIG_DFU_SF=y
> -CONFIG_DM_GPIO=y
> -CONFIG_DM_MMC=y
> -CONFIG_MMC_OMAP_HS=y
> -CONFIG_SPI_FLASH=y
> -CONFIG_SPI_FLASH_MACRONIX=y
> -CONFIG_DM_SERIAL=y
> -CONFIG_SYS_NS16550=y
> -CONFIG_TI_QSPI=y
> -CONFIG_TIMER=y
> -CONFIG_OMAP_TIMER=y
> -CONFIG_USB=y
> -CONFIG_USB_XHCI_HCD=y
> -CONFIG_USB_XHCI_DWC3=y
> -CONFIG_USB_DWC3=y
> -CONFIG_USB_DWC3_GADGET=y
> -CONFIG_USB_DWC3_OMAP=y
> -CONFIG_USB_DWC3_PHY_OMAP=y
> -CONFIG_USB_STORAGE=y
> -CONFIG_USB_GADGET=y
> -CONFIG_USB_GADGET_DOWNLOAD=y
> -CONFIG_G_DNL_MANUFACTURER="Texas Instruments"
> -CONFIG_G_DNL_VENDOR_NUM=0x0403
> -CONFIG_G_DNL_PRODUCT_NUM=0xbd00
> -CONFIG_FAT_WRITE=y
> 

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

* [U-Boot] [PATCH 2/3] am335x_hs_evm: Disable CONFIG_SPL_ENV_SUPPORT
  2017-07-26 17:11 ` [U-Boot] [PATCH 2/3] am335x_hs_evm: Disable CONFIG_SPL_ENV_SUPPORT Tom Rini
@ 2017-07-27  2:46   ` Lokesh Vutla
  2017-07-31 14:45     ` Simon Glass
  2017-08-01  0:41   ` [U-Boot] [U-Boot, " Tom Rini
  1 sibling, 1 reply; 12+ messages in thread
From: Lokesh Vutla @ 2017-07-27  2:46 UTC (permalink / raw)
  To: u-boot



On 7/26/2017 10:41 PM, Tom Rini wrote:
> The main uses of CONFIG_SPL_ENV_SUPPORT are for network support and for
> disabling "Falcon Mode" support at run-time.  As this build enables
> neither, remove this feature.
> 
> Cc: Andrew F. Davis <afd@ti.com>
> Signed-off-by: Tom Rini <trini@konsulko.com>

Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>

Thanks and regards,
Lokesh

> ---
>  configs/am335x_hs_evm_defconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/configs/am335x_hs_evm_defconfig b/configs/am335x_hs_evm_defconfig
> index 2b1acb70d18f..ffdbe7585573 100644
> --- a/configs/am335x_hs_evm_defconfig
> +++ b/configs/am335x_hs_evm_defconfig
> @@ -18,6 +18,7 @@ CONFIG_VERSION_VARIABLE=y
>  CONFIG_ARCH_MISC_INIT=y
>  CONFIG_SPL=y
>  CONFIG_SPL_STACK_R=y
> +# CONFIG_SPL_ENV_SUPPORT is not set
>  # CONFIG_SPL_EXT_SUPPORT is not set
>  CONFIG_SPL_MTD_SUPPORT=y
>  # CONFIG_SPL_YMODEM_SUPPORT is not set
> 

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

* [U-Boot] [PATCH 3/3] ti: Default to ENV_IS_IN_FAT if MMC_OMAP_HS
  2017-07-26 17:11 ` [U-Boot] [PATCH 3/3] ti: Default to ENV_IS_IN_FAT if MMC_OMAP_HS Tom Rini
@ 2017-07-27  2:49   ` Lokesh Vutla
  2017-07-27 11:14     ` Tom Rini
  2017-08-01  0:41   ` [U-Boot] [U-Boot, " Tom Rini
  1 sibling, 1 reply; 12+ messages in thread
From: Lokesh Vutla @ 2017-07-27  2:49 UTC (permalink / raw)
  To: u-boot



On 7/26/2017 10:41 PM, Tom Rini wrote:
> When we have MMC available we assume that we want to put the env as a
> file on FAT.

On DRA7 and AM57 we use eMMC for saving env. Is it fine to imply FAT and
change it in defconfig to use raw partitions?

Thanks and regards,
Lokesh

> 
> Cc: Lokesh Vutla <lokeshvutla@ti.com>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
>  board/ti/common/Kconfig             | 1 +
>  configs/am335x_evm_defconfig        | 1 -
>  configs/am335x_evm_usbspl_defconfig | 1 -
>  configs/am335x_hs_evm_defconfig     | 1 -
>  configs/am43xx_evm_defconfig        | 1 -
>  configs/am43xx_hs_evm_defconfig     | 1 -
>  configs/k2g_evm_defconfig           | 1 -
>  configs/k2g_hs_evm_defconfig        | 1 -
>  8 files changed, 1 insertion(+), 7 deletions(-)
> 
> diff --git a/board/ti/common/Kconfig b/board/ti/common/Kconfig
> index c21eb8c2d2e3..c81baa12becc 100644
> --- a/board/ti/common/Kconfig
> +++ b/board/ti/common/Kconfig
> @@ -42,3 +42,4 @@ config TI_COMMON_CMD_OPTIONS
>  	imply CMD_SPI
>  	imply CMD_TIME
>  	imply CMD_USB if USB
> +	imply ENV_IS_IN_FAT if MMC_OMAP_HS
> diff --git a/configs/am335x_evm_defconfig b/configs/am335x_evm_defconfig
> index 4ea56178eeab..204d817c0a60 100644
> --- a/configs/am335x_evm_defconfig
> +++ b/configs/am335x_evm_defconfig
> @@ -7,7 +7,6 @@ CONFIG_DEFAULT_DEVICE_TREE="am335x-evm"
>  CONFIG_DISTRO_DEFAULTS=y
>  CONFIG_SPL_LOAD_FIT=y
>  CONFIG_SYS_EXTRA_OPTIONS="NAND"
> -CONFIG_ENV_IS_IN_FAT=y
>  CONFIG_SYS_CONSOLE_INFO_QUIET=y
>  CONFIG_VERSION_VARIABLE=y
>  CONFIG_ARCH_MISC_INIT=y
> diff --git a/configs/am335x_evm_usbspl_defconfig b/configs/am335x_evm_usbspl_defconfig
> index 51d1042c3433..84e746916cfa 100644
> --- a/configs/am335x_evm_usbspl_defconfig
> +++ b/configs/am335x_evm_usbspl_defconfig
> @@ -6,7 +6,6 @@ CONFIG_AM33XX=y
>  CONFIG_SPL_STACK_R_ADDR=0x82000000
>  CONFIG_DISTRO_DEFAULTS=y
>  CONFIG_SYS_EXTRA_OPTIONS="NAND"
> -CONFIG_ENV_IS_IN_FAT=y
>  CONFIG_SYS_CONSOLE_INFO_QUIET=y
>  CONFIG_VERSION_VARIABLE=y
>  CONFIG_ARCH_MISC_INIT=y
> diff --git a/configs/am335x_hs_evm_defconfig b/configs/am335x_hs_evm_defconfig
> index ffdbe7585573..52cbf05ab025 100644
> --- a/configs/am335x_hs_evm_defconfig
> +++ b/configs/am335x_hs_evm_defconfig
> @@ -12,7 +12,6 @@ CONFIG_FIT_IMAGE_POST_PROCESS=y
>  CONFIG_SPL_LOAD_FIT=y
>  CONFIG_SPL_FIT_IMAGE_POST_PROCESS=y
>  CONFIG_SYS_EXTRA_OPTIONS="NAND"
> -CONFIG_ENV_IS_IN_FAT=y
>  CONFIG_SYS_CONSOLE_INFO_QUIET=y
>  CONFIG_VERSION_VARIABLE=y
>  CONFIG_ARCH_MISC_INIT=y
> diff --git a/configs/am43xx_evm_defconfig b/configs/am43xx_evm_defconfig
> index ae2ae1ae516a..903bd69ad209 100644
> --- a/configs/am43xx_evm_defconfig
> +++ b/configs/am43xx_evm_defconfig
> @@ -7,7 +7,6 @@ CONFIG_SPL_STACK_R_ADDR=0x82000000
>  CONFIG_DEFAULT_DEVICE_TREE="am437x-gp-evm"
>  CONFIG_SPL_LOAD_FIT=y
>  CONFIG_SYS_EXTRA_OPTIONS="CONS_INDEX=1,NAND"
> -CONFIG_ENV_IS_IN_FAT=y
>  CONFIG_SYS_CONSOLE_INFO_QUIET=y
>  CONFIG_VERSION_VARIABLE=y
>  CONFIG_SPL=y
> diff --git a/configs/am43xx_hs_evm_defconfig b/configs/am43xx_hs_evm_defconfig
> index 5306c26b9908..b8e3fd46f272 100644
> --- a/configs/am43xx_hs_evm_defconfig
> +++ b/configs/am43xx_hs_evm_defconfig
> @@ -15,7 +15,6 @@ CONFIG_SPL_LOAD_FIT=y
>  CONFIG_SPL_FIT_IMAGE_POST_PROCESS=y
>  CONFIG_OF_BOARD_SETUP=y
>  CONFIG_SYS_EXTRA_OPTIONS="CONS_INDEX=1,NAND"
> -CONFIG_ENV_IS_IN_FAT=y
>  CONFIG_SYS_CONSOLE_INFO_QUIET=y
>  CONFIG_VERSION_VARIABLE=y
>  CONFIG_SPL=y
> diff --git a/configs/k2g_evm_defconfig b/configs/k2g_evm_defconfig
> index 332c4d18101b..1eed60576cdd 100644
> --- a/configs/k2g_evm_defconfig
> +++ b/configs/k2g_evm_defconfig
> @@ -10,7 +10,6 @@ CONFIG_SPL_SPI_FLASH_SUPPORT=y
>  CONFIG_SPL_SPI_SUPPORT=y
>  CONFIG_DEFAULT_DEVICE_TREE="keystone-k2g-evm"
>  CONFIG_OF_BOARD_SETUP=y
> -CONFIG_ENV_IS_IN_FAT=y
>  CONFIG_SYS_CONSOLE_INFO_QUIET=y
>  CONFIG_DTB_RESELECT=y
>  CONFIG_FIT_EMBED=y
> diff --git a/configs/k2g_hs_evm_defconfig b/configs/k2g_hs_evm_defconfig
> index 2e2b18c18565..51c701ed902d 100644
> --- a/configs/k2g_hs_evm_defconfig
> +++ b/configs/k2g_hs_evm_defconfig
> @@ -5,7 +5,6 @@ CONFIG_TI_COMMON_CMD_OPTIONS=y
>  CONFIG_TARGET_K2G_EVM=y
>  CONFIG_DEFAULT_DEVICE_TREE="keystone-k2g-evm"
>  CONFIG_OF_BOARD_SETUP=y
> -CONFIG_ENV_IS_IN_FAT=y
>  CONFIG_SYS_CONSOLE_INFO_QUIET=y
>  CONFIG_VERSION_VARIABLE=y
>  CONFIG_HUSH_PARSER=y
> 

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

* [U-Boot] [PATCH 1/3] am43xx: Remove ethboot / usbhost_boot variants
  2017-07-27  2:45 ` [U-Boot] [PATCH 1/3] am43xx: Remove ethboot / usbhost_boot variants Lokesh Vutla
@ 2017-07-27  6:08   ` Vignesh R
  2017-07-27 11:15   ` Tom Rini
  1 sibling, 0 replies; 12+ messages in thread
From: Vignesh R @ 2017-07-27  6:08 UTC (permalink / raw)
  To: u-boot



On Thursday 27 July 2017 08:15 AM, Lokesh Vutla wrote:
> + Vignesh
> 
> On 7/26/2017 10:41 PM, Tom Rini wrote:
>> The intention of having aaa_evm_xxx_defconfig variants is to allow
>> building for useful but non-default cases that cannot be easily done
>> with the standard aaa_evm_defconfig file.  In this case, we get rid of
>> usbhost_boot as SPL_TEXT has been in OCMC RAM for a while now, so this
> 
> SPL_TEXT for default boot is 0x402f0400
> SPL_TEXT for usb host boot is 0x40300350 as ROM uses the initial portion
> for DMA when copying from USB. Initially we thought of moving SPL_TET
> for default boot to 0x40300350 but this will break UART boot as it has
> to be 0x402f0400 for it. So I guess we cannot remove usb host boot
> defconfig.
> 
>> can be included in the default.  We get rid of ethboot as it's fine to
>> have the environment in someplace that's not available and the default
> 
> With DM_ETH enabled, it enforces to have DM for usb peripheral for usb
> rndis boot, which is not supported yet. Vignesh is currently working on
> it. I guess once it is done, ethboot_defocnfig can be removed.
> 
> So $patch breaks both usb host and ethboot.
> 

Has ethboot been tested with this patch because previously
ethboot_defconfig did not imply DM_ETH?


> Thanks and regards,
> Lokesh
> 
>> will be used.  Further, now that environment location is in Kconfig it's
>> reasonable to expect a user of this feature to optimize their use-case
>> if needed.
>>
>> Cc: Lokesh Vutla <lokeshvutla@ti.com>
>> Signed-off-by: Tom Rini <trini@konsulko.com>
>> ---
>>  configs/am43xx_evm_defconfig              |  7 +++
>>  configs/am43xx_evm_ethboot_defconfig      | 63 --------------------------
>>  configs/am43xx_evm_usbhost_boot_defconfig | 75 -------------------------------
>>  3 files changed, 7 insertions(+), 138 deletions(-)
>>  delete mode 100644 configs/am43xx_evm_ethboot_defconfig
>>  delete mode 100644 configs/am43xx_evm_usbhost_boot_defconfig
>>
>> diff --git a/configs/am43xx_evm_defconfig b/configs/am43xx_evm_defconfig
>> index 7a7a0b0ea0ac..ae2ae1ae516a 100644
>> --- a/configs/am43xx_evm_defconfig
>> +++ b/configs/am43xx_evm_defconfig
>> @@ -12,8 +12,15 @@ CONFIG_SYS_CONSOLE_INFO_QUIET=y
>>  CONFIG_VERSION_VARIABLE=y
>>  CONFIG_SPL=y
>>  CONFIG_SPL_STACK_R=y
>> +CONFIG_SPL_ETH_SUPPORT=y
>>  CONFIG_SPL_MTD_SUPPORT=y
>> +CONFIG_SPL_NET_SUPPORT=y
>> +CONFIG_SPL_NET_VCI_STRING="AM43xx U-Boot SPL"
>>  CONFIG_SPL_OS_BOOT=y
>> +CONFIG_SPL_USB_HOST_SUPPORT=y
>> +CONFIG_SPL_USB_SUPPORT=y
>> +CONFIG_SPL_USB_GADGET_SUPPORT=y
>> +CONFIG_SPL_USBETH_SUPPORT=y
>>  CONFIG_HUSH_PARSER=y
>>  # CONFIG_CMD_IMLS is not set
>>  # CONFIG_CMD_FLASH is not set
>> diff --git a/configs/am43xx_evm_ethboot_defconfig b/configs/am43xx_evm_ethboot_defconfig
>> deleted file mode 100644
>> index 9892653d27c3..000000000000
>> --- a/configs/am43xx_evm_ethboot_defconfig
>> +++ /dev/null
>> @@ -1,63 +0,0 @@
>> -CONFIG_ARM=y
>> -CONFIG_ARCH_OMAP2PLUS=y
>> -CONFIG_AM43XX=y
>> -CONFIG_SYS_EXTRA_OPTIONS="SERIAL1,CONS_INDEX=1,NAND"
>> -CONFIG_ENV_IS_IN_FAT=y
>> -CONFIG_SYS_CONSOLE_INFO_QUIET=y
>> -CONFIG_VERSION_VARIABLE=y
>> -CONFIG_SPL=y
>> -# CONFIG_SPL_SYS_MALLOC_SIMPLE is not set
>> -CONFIG_SPL_ETH_SUPPORT=y
>> -CONFIG_SPL_MTD_SUPPORT=y
>> -CONFIG_SPL_NET_SUPPORT=y
>> -CONFIG_SPL_NET_VCI_STRING="AM43xx U-Boot SPL"
>> -CONFIG_SPL_OS_BOOT=y
>> -CONFIG_HUSH_PARSER=y
>> -CONFIG_CMD_BOOTZ=y
>> -# CONFIG_CMD_IMLS is not set
>> -CONFIG_CMD_ASKENV=y
>> -CONFIG_CMD_EEPROM=y
>> -# CONFIG_CMD_FLASH is not set
>> -CONFIG_CMD_GPT=y
>> -CONFIG_CMD_MMC=y
>> -CONFIG_CMD_PART=y
>> -CONFIG_CMD_SF=y
>> -CONFIG_CMD_SPI=y
>> -CONFIG_CMD_I2C=y
>> -CONFIG_CMD_USB=y
>> -CONFIG_CMD_DFU=y
>> -CONFIG_CMD_GPIO=y
>> -# CONFIG_CMD_SETEXPR is not set
>> -CONFIG_CMD_DHCP=y
>> -CONFIG_CMD_MII=y
>> -CONFIG_CMD_PING=y
>> -CONFIG_CMD_EXT2=y
>> -CONFIG_CMD_EXT4=y
>> -CONFIG_CMD_EXT4_WRITE=y
>> -CONFIG_CMD_FAT=y
>> -CONFIG_CMD_FS_GENERIC=y
>> -CONFIG_CMD_MTDPARTS=y
>> -CONFIG_ISO_PARTITION=y
>> -CONFIG_DFU_MMC=y
>> -CONFIG_DFU_RAM=y
>> -CONFIG_DFU_SF=y
>> -CONFIG_MMC_OMAP_HS=y
>> -CONFIG_SPI_FLASH=y
>> -CONFIG_SPI_FLASH_MACRONIX=y
>> -CONFIG_SYS_NS16550=y
>> -CONFIG_TI_QSPI=y
>> -CONFIG_USB=y
>> -CONFIG_USB_XHCI_HCD=y
>> -CONFIG_USB_XHCI_DWC3=y
>> -CONFIG_USB_DWC3=y
>> -CONFIG_USB_DWC3_GADGET=y
>> -CONFIG_USB_DWC3_OMAP=y
>> -CONFIG_USB_DWC3_PHY_OMAP=y
>> -CONFIG_USB_STORAGE=y
>> -CONFIG_USB_GADGET=y
>> -CONFIG_USB_GADGET_DOWNLOAD=y
>> -CONFIG_G_DNL_MANUFACTURER="Texas Instruments"
>> -CONFIG_G_DNL_VENDOR_NUM=0x0403
>> -CONFIG_G_DNL_PRODUCT_NUM=0xbd00
>> -CONFIG_FAT_WRITE=y
>> -CONFIG_OF_LIBFDT=y
>> diff --git a/configs/am43xx_evm_usbhost_boot_defconfig b/configs/am43xx_evm_usbhost_boot_defconfig
>> deleted file mode 100644
>> index f4c0f8442a17..000000000000
>> --- a/configs/am43xx_evm_usbhost_boot_defconfig
>> +++ /dev/null
>> @@ -1,75 +0,0 @@
>> -CONFIG_ARM=y
>> -CONFIG_ARCH_OMAP2PLUS=y
>> -CONFIG_SYS_MALLOC_F_LEN=0x2000
>> -CONFIG_AM43XX=y
>> -CONFIG_ISW_ENTRY_ADDR=0x40300350
>> -CONFIG_SPL_STACK_R_ADDR=0x82000000
>> -CONFIG_DEFAULT_DEVICE_TREE="am437x-gp-evm"
>> -CONFIG_SPL_LOAD_FIT=y
>> -CONFIG_SYS_EXTRA_OPTIONS="SERIAL1,CONS_INDEX=1,NAND"
>> -CONFIG_ENV_IS_IN_FAT=y
>> -CONFIG_SYS_CONSOLE_INFO_QUIET=y
>> -CONFIG_VERSION_VARIABLE=y
>> -CONFIG_SPL=y
>> -CONFIG_SPL_STACK_R=y
>> -CONFIG_SPL_MTD_SUPPORT=y
>> -CONFIG_SPL_OS_BOOT=y
>> -CONFIG_SPL_USB_HOST_SUPPORT=y
>> -CONFIG_SPL_USB_SUPPORT=y
>> -CONFIG_HUSH_PARSER=y
>> -CONFIG_CMD_BOOTZ=y
>> -# CONFIG_CMD_IMLS is not set
>> -CONFIG_CMD_ASKENV=y
>> -CONFIG_CMD_EEPROM=y
>> -# CONFIG_CMD_FLASH is not set
>> -CONFIG_CMD_GPT=y
>> -CONFIG_CMD_MMC=y
>> -CONFIG_CMD_PART=y
>> -CONFIG_CMD_SF=y
>> -CONFIG_CMD_SPI=y
>> -CONFIG_CMD_I2C=y
>> -CONFIG_CMD_USB=y
>> -CONFIG_CMD_DFU=y
>> -CONFIG_CMD_GPIO=y
>> -# CONFIG_CMD_SETEXPR is not set
>> -CONFIG_CMD_DHCP=y
>> -CONFIG_CMD_MII=y
>> -CONFIG_CMD_PING=y
>> -CONFIG_CMD_EXT2=y
>> -CONFIG_CMD_EXT4=y
>> -CONFIG_CMD_EXT4_WRITE=y
>> -CONFIG_CMD_FAT=y
>> -CONFIG_CMD_FS_GENERIC=y
>> -CONFIG_CMD_MTDPARTS=y
>> -CONFIG_ISO_PARTITION=y
>> -CONFIG_OF_CONTROL=y
>> -CONFIG_OF_LIST="am437x-gp-evm am437x-sk-evm am43x-epos-evm am437x-idk-evm"
>> -CONFIG_DM=y
>> -# CONFIG_BLK is not set
>> -CONFIG_DFU_MMC=y
>> -CONFIG_DFU_RAM=y
>> -CONFIG_DFU_SF=y
>> -CONFIG_DM_GPIO=y
>> -CONFIG_DM_MMC=y
>> -CONFIG_MMC_OMAP_HS=y
>> -CONFIG_SPI_FLASH=y
>> -CONFIG_SPI_FLASH_MACRONIX=y
>> -CONFIG_DM_SERIAL=y
>> -CONFIG_SYS_NS16550=y
>> -CONFIG_TI_QSPI=y
>> -CONFIG_TIMER=y
>> -CONFIG_OMAP_TIMER=y
>> -CONFIG_USB=y
>> -CONFIG_USB_XHCI_HCD=y
>> -CONFIG_USB_XHCI_DWC3=y
>> -CONFIG_USB_DWC3=y
>> -CONFIG_USB_DWC3_GADGET=y
>> -CONFIG_USB_DWC3_OMAP=y
>> -CONFIG_USB_DWC3_PHY_OMAP=y
>> -CONFIG_USB_STORAGE=y
>> -CONFIG_USB_GADGET=y
>> -CONFIG_USB_GADGET_DOWNLOAD=y
>> -CONFIG_G_DNL_MANUFACTURER="Texas Instruments"
>> -CONFIG_G_DNL_VENDOR_NUM=0x0403
>> -CONFIG_G_DNL_PRODUCT_NUM=0xbd00
>> -CONFIG_FAT_WRITE=y
>>

-- 
Regards
Vignesh

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

* [U-Boot] [PATCH 3/3] ti: Default to ENV_IS_IN_FAT if MMC_OMAP_HS
  2017-07-27  2:49   ` Lokesh Vutla
@ 2017-07-27 11:14     ` Tom Rini
  0 siblings, 0 replies; 12+ messages in thread
From: Tom Rini @ 2017-07-27 11:14 UTC (permalink / raw)
  To: u-boot

On Thu, Jul 27, 2017 at 08:19:22AM +0530, Lokesh Vutla wrote:

> On 7/26/2017 10:41 PM, Tom Rini wrote:
> > When we have MMC available we assume that we want to put the env as a
> > file on FAT.
> 
> On DRA7 and AM57 we use eMMC for saving env. Is it fine to imply FAT and
> change it in defconfig to use raw partitions?

Yes, I'd just like to do it as a follow-up to my series that moves the
implementation side of FAT env support to Kconfig, so that we don't need
to add a few more entires into the header file.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170727/98c95816/attachment.sig>

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

* [U-Boot] [PATCH 1/3] am43xx: Remove ethboot / usbhost_boot variants
  2017-07-27  2:45 ` [U-Boot] [PATCH 1/3] am43xx: Remove ethboot / usbhost_boot variants Lokesh Vutla
  2017-07-27  6:08   ` Vignesh R
@ 2017-07-27 11:15   ` Tom Rini
  1 sibling, 0 replies; 12+ messages in thread
From: Tom Rini @ 2017-07-27 11:15 UTC (permalink / raw)
  To: u-boot

On Thu, Jul 27, 2017 at 08:15:25AM +0530, Lokesh Vutla wrote:
> + Vignesh
> 
> On 7/26/2017 10:41 PM, Tom Rini wrote:
> > The intention of having aaa_evm_xxx_defconfig variants is to allow
> > building for useful but non-default cases that cannot be easily done
> > with the standard aaa_evm_defconfig file.  In this case, we get rid of
> > usbhost_boot as SPL_TEXT has been in OCMC RAM for a while now, so this
> 
> SPL_TEXT for default boot is 0x402f0400
> SPL_TEXT for usb host boot is 0x40300350 as ROM uses the initial portion
> for DMA when copying from USB. Initially we thought of moving SPL_TET
> for default boot to 0x40300350 but this will break UART boot as it has
> to be 0x402f0400 for it. So I guess we cannot remove usb host boot
> defconfig.
> 
> > can be included in the default.  We get rid of ethboot as it's fine to
> > have the environment in someplace that's not available and the default
> 
> With DM_ETH enabled, it enforces to have DM for usb peripheral for usb
> rndis boot, which is not supported yet. Vignesh is currently working on
> it. I guess once it is done, ethboot_defocnfig can be removed.
> 
> So $patch breaks both usb host and ethboot.

Aw, ok, thanks!

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

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

* [U-Boot] [PATCH 2/3] am335x_hs_evm: Disable CONFIG_SPL_ENV_SUPPORT
  2017-07-27  2:46   ` Lokesh Vutla
@ 2017-07-31 14:45     ` Simon Glass
  0 siblings, 0 replies; 12+ messages in thread
From: Simon Glass @ 2017-07-31 14:45 UTC (permalink / raw)
  To: u-boot

On 26 July 2017 at 20:46, Lokesh Vutla <lokeshvutla@ti.com> wrote:
>
>
> On 7/26/2017 10:41 PM, Tom Rini wrote:
>> The main uses of CONFIG_SPL_ENV_SUPPORT are for network support and for
>> disabling "Falcon Mode" support at run-time.  As this build enables
>> neither, remove this feature.
>>
>> Cc: Andrew F. Davis <afd@ti.com>
>> Signed-off-by: Tom Rini <trini@konsulko.com>
>
> Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>
>
> Thanks and regards,
> Lokesh
>
>> ---
>>  configs/am335x_hs_evm_defconfig | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/configs/am335x_hs_evm_defconfig b/configs/am335x_hs_evm_defconfig
>> index 2b1acb70d18f..ffdbe7585573 100644
>> --- a/configs/am335x_hs_evm_defconfig
>> +++ b/configs/am335x_hs_evm_defconfig
>> @@ -18,6 +18,7 @@ CONFIG_VERSION_VARIABLE=y
>>  CONFIG_ARCH_MISC_INIT=y
>>  CONFIG_SPL=y
>>  CONFIG_SPL_STACK_R=y
>> +# CONFIG_SPL_ENV_SUPPORT is not set
>>  # CONFIG_SPL_EXT_SUPPORT is not set
>>  CONFIG_SPL_MTD_SUPPORT=y
>>  # CONFIG_SPL_YMODEM_SUPPORT is not set

Reviewed-by: Simon Glass <sjg@chromium.org>

>>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot

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

* [U-Boot] [U-Boot, 2/3] am335x_hs_evm: Disable CONFIG_SPL_ENV_SUPPORT
  2017-07-26 17:11 ` [U-Boot] [PATCH 2/3] am335x_hs_evm: Disable CONFIG_SPL_ENV_SUPPORT Tom Rini
  2017-07-27  2:46   ` Lokesh Vutla
@ 2017-08-01  0:41   ` Tom Rini
  1 sibling, 0 replies; 12+ messages in thread
From: Tom Rini @ 2017-08-01  0:41 UTC (permalink / raw)
  To: u-boot

On Wed, Jul 26, 2017 at 01:11:36PM -0400, Tom Rini wrote:

> The main uses of CONFIG_SPL_ENV_SUPPORT are for network support and for
> disabling "Falcon Mode" support at run-time.  As this build enables
> neither, remove this feature.
> 
> Cc: Andrew F. Davis <afd@ti.com>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>

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: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170731/d23d1519/attachment.sig>

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

* [U-Boot] [U-Boot, 3/3] ti: Default to ENV_IS_IN_FAT if MMC_OMAP_HS
  2017-07-26 17:11 ` [U-Boot] [PATCH 3/3] ti: Default to ENV_IS_IN_FAT if MMC_OMAP_HS Tom Rini
  2017-07-27  2:49   ` Lokesh Vutla
@ 2017-08-01  0:41   ` Tom Rini
  1 sibling, 0 replies; 12+ messages in thread
From: Tom Rini @ 2017-08-01  0:41 UTC (permalink / raw)
  To: u-boot

On Wed, Jul 26, 2017 at 01:11:37PM -0400, Tom Rini wrote:

> When we have MMC available we assume that we want to put the env as a
> file on FAT.
> 
> Cc: Lokesh Vutla <lokeshvutla@ti.com>
> Signed-off-by: Tom Rini <trini@konsulko.com>

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: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170731/6a805974/attachment.sig>

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

end of thread, other threads:[~2017-08-01  0:41 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-26 17:11 [U-Boot] [PATCH 1/3] am43xx: Remove ethboot / usbhost_boot variants Tom Rini
2017-07-26 17:11 ` [U-Boot] [PATCH 2/3] am335x_hs_evm: Disable CONFIG_SPL_ENV_SUPPORT Tom Rini
2017-07-27  2:46   ` Lokesh Vutla
2017-07-31 14:45     ` Simon Glass
2017-08-01  0:41   ` [U-Boot] [U-Boot, " Tom Rini
2017-07-26 17:11 ` [U-Boot] [PATCH 3/3] ti: Default to ENV_IS_IN_FAT if MMC_OMAP_HS Tom Rini
2017-07-27  2:49   ` Lokesh Vutla
2017-07-27 11:14     ` Tom Rini
2017-08-01  0:41   ` [U-Boot] [U-Boot, " Tom Rini
2017-07-27  2:45 ` [U-Boot] [PATCH 1/3] am43xx: Remove ethboot / usbhost_boot variants Lokesh Vutla
2017-07-27  6:08   ` Vignesh R
2017-07-27 11:15   ` 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.