[U-Boot] [PATCH v3 0/3] ARM: AM43xx: Enable SPL_DM

[U-Boot] [PATCH v3 0/3] ARM: AM43xx: Enable SPL_DM

[Lokesh Vutla]

Enable SPL_DM on all AM4xx based boards.

This series depends on:
- DRA7 SPL_DM series[1]
- http://patchwork.ozlabs.org/patch/727106/

[1] https://www.mail-archive.com/u-boot at lists.denx.de/msg238751.html

Changes since v2:
- Adapted to use 'imply' in Kconfig

Changes since v1:
- Fixed build error with non-SPL_DM defconfigs
- Increased SYS_MALLOC_F_LEN


Lokesh Vutla (3):
  ARM: dts: am43xx: Add u-boot specific dtsi
  config: am43xx_usbhost_boot: sync with am43xx_evm
  configs: am43xx_evm: Enable SPL_DM

 arch/arm/Kconfig                          |  6 +++++
 arch/arm/dts/am437x-gp-evm-u-boot.dtsi    | 38 +++++++++++++++++++++++++++++++
 configs/am43xx_evm_defconfig              |  2 +-
 configs/am43xx_evm_usbhost_boot_defconfig |  7 +++++-
 configs/am43xx_hs_evm_defconfig           |  2 +-
 include/configs/am43xx_evm.h              |  7 ++----
 6 files changed, 54 insertions(+), 8 deletions(-)
 create mode 100644 arch/arm/dts/am437x-gp-evm-u-boot.dtsi

-- 
2.11.0

[U-Boot] [PATCH v3 1/3] ARM: dts: am43xx: Add u-boot specific dtsi

[Lokesh Vutla]

Add u-boot specific dtsi for am43xx-gp-evm so
that it will be used for SPL.

Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>
---
 arch/arm/dts/am437x-gp-evm-u-boot.dtsi | 38 ++++++++++++++++++++++++++++++++++
 1 file changed, 38 insertions(+)
 create mode 100644 arch/arm/dts/am437x-gp-evm-u-boot.dtsi

diff --git a/arch/arm/dts/am437x-gp-evm-u-boot.dtsi b/arch/arm/dts/am437x-gp-evm-u-boot.dtsi
new file mode 100644
index 0000000000..885a9a92db
--- /dev/null
+++ b/arch/arm/dts/am437x-gp-evm-u-boot.dtsi
@@ -0,0 +1,38 @@
+/*
+ * Copyright (C) 2017 Texas Instruments Incorporated - http://www.ti.com/
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ * Based on "dra7.dtsi"
+ */
+
+/{
+	ocp {
+		u-boot,dm-pre-reloc;
+	};
+};
+
+&uart0 {
+	u-boot,dm-pre-reloc;
+};
+
+&mmc1 {
+	u-boot,dm-pre-reloc;
+};
+
+&mac {
+	u-boot,dm-pre-reloc;
+};
+
+&davinci_mdio {
+	u-boot,dm-pre-reloc;
+};
+
+&cpsw_emac0 {
+	u-boot,dm-pre-reloc;
+};
+
+&phy_sel {
+	u-boot,dm-pre-reloc;
+};
-- 
2.11.0

[U-Boot] [PATCH v3 1/3] ARM: dts: am43xx: Add u-boot specific dtsi

[Tom Rini]

On Mon, Feb 20, 2017 at 11:05:31AM +0530, Lokesh Vutla wrote:

> Add u-boot specific dtsi for am43xx-gp-evm so
> that it will be used for SPL.
> 
> Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
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/20170220/cb4579f7/attachment.sig>

[U-Boot] [PATCH v3 2/3] config: am43xx_usbhost_boot: sync with am43xx_evm

[Lokesh Vutla]

am43xx_evm defconfig has been modified without making changes
in am43xx_usbhost_boot defconfig. Synce here.

Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>
---
 configs/am43xx_evm_usbhost_boot_defconfig | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/configs/am43xx_evm_usbhost_boot_defconfig b/configs/am43xx_evm_usbhost_boot_defconfig
index 5775ab16dd..e22bc7fbdb 100644
--- a/configs/am43xx_evm_usbhost_boot_defconfig
+++ b/configs/am43xx_evm_usbhost_boot_defconfig
@@ -48,13 +48,18 @@ CONFIG_DFU_MMC=y
 CONFIG_DFU_RAM=y
 CONFIG_DFU_SF=y
 CONFIG_DM_GPIO=y
+CONFIG_DM_I2C=y
 CONFIG_DM_MMC=y
 # CONFIG_DM_MMC_OPS is not set
 CONFIG_MMC_OMAP_HS=y
+CONFIG_DM_SPI_FLASH=y
 CONFIG_SPI_FLASH=y
+CONFIG_SPI_FLASH_BAR=y
 CONFIG_SPI_FLASH_MACRONIX=y
+CONFIG_DM_ETH=y
 CONFIG_DM_SERIAL=y
 CONFIG_SYS_NS16550=y
+CONFIG_DM_SPI=y
 CONFIG_TI_QSPI=y
 CONFIG_TIMER=y
 CONFIG_OMAP_TIMER=y
-- 
2.11.0

[U-Boot] [PATCH v3 2/3] config: am43xx_usbhost_boot: sync with am43xx_evm

[Tom Rini]

On Mon, Feb 20, 2017 at 11:05:32AM +0530, Lokesh Vutla wrote:

> am43xx_evm defconfig has been modified without making changes
> in am43xx_usbhost_boot defconfig. Synce here.
> 
> Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>
> ---
>  configs/am43xx_evm_usbhost_boot_defconfig | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/configs/am43xx_evm_usbhost_boot_defconfig b/configs/am43xx_evm_usbhost_boot_defconfig
> index 5775ab16dd..e22bc7fbdb 100644
> --- a/configs/am43xx_evm_usbhost_boot_defconfig
> +++ b/configs/am43xx_evm_usbhost_boot_defconfig
> @@ -48,13 +48,18 @@ CONFIG_DFU_MMC=y
>  CONFIG_DFU_RAM=y
>  CONFIG_DFU_SF=y
>  CONFIG_DM_GPIO=y
> +CONFIG_DM_I2C=y
>  CONFIG_DM_MMC=y
>  # CONFIG_DM_MMC_OPS is not set
>  CONFIG_MMC_OMAP_HS=y
> +CONFIG_DM_SPI_FLASH=y
>  CONFIG_SPI_FLASH=y
> +CONFIG_SPI_FLASH_BAR=y
>  CONFIG_SPI_FLASH_MACRONIX=y
> +CONFIG_DM_ETH=y
>  CONFIG_DM_SERIAL=y
>  CONFIG_SYS_NS16550=y
> +CONFIG_DM_SPI=y
>  CONFIG_TI_QSPI=y
>  CONFIG_TIMER=y
>  CONFIG_OMAP_TIMER=y

I like 3/3.  But instead of 2/3 here we should be moving I2C/SPI/ETH/etc
stuff over to imply (at the board level) as that still allows the user
to turn things off in their .config file, unlike a select.  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/20170220/e6262a6a/attachment.sig>

[U-Boot] [PATCH v3 3/3] configs: am43xx_evm: Enable SPL_DM

[Lokesh Vutla]

Enable SPL_DM on all AM43xx based platforms

Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>
---
 arch/arm/Kconfig                          | 6 ++++++
 configs/am43xx_evm_defconfig              | 2 +-
 configs/am43xx_evm_usbhost_boot_defconfig | 2 +-
 configs/am43xx_hs_evm_defconfig           | 2 +-
 include/configs/am43xx_evm.h              | 7 ++-----
 5 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index e84b74efb6..ceec643bc3 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -583,6 +583,12 @@ config OMAP54XX
 config AM43XX
 	bool "AM43XX SoC"
 	select ARCH_OMAP2
+	imply SPL_SYS_MALLOC_SIMPLE
+	imply SPL_SEPARATE_BSS
+	imply SPL_OF_CONTROL
+	imply SPL_DM
+	imply SPL_DM_SEQ_ALIAS
+	imply SPL_OF_TRANSLATE
 	help
 	  Support for AM43xx SOC from Texas Instruments.
 	  The AM43xx high performance SOC features a Cortex-A9
diff --git a/configs/am43xx_evm_defconfig b/configs/am43xx_evm_defconfig
index 6fb2053f33..a04ba1dddb 100644
--- a/configs/am43xx_evm_defconfig
+++ b/configs/am43xx_evm_defconfig
@@ -1,5 +1,6 @@
 CONFIG_ARM=y
 CONFIG_AM43XX=y
+CONFIG_SYS_MALLOC_F_LEN=0x2000
 CONFIG_TARGET_AM43XX_EVM=y
 CONFIG_SPL_STACK_R_ADDR=0x82000000
 CONFIG_SPL_YMODEM_SUPPORT=y
@@ -73,4 +74,3 @@ CONFIG_USB_GADGET_DOWNLOAD=y
 CONFIG_G_DNL_MANUFACTURER="Texas Instruments"
 CONFIG_G_DNL_VENDOR_NUM=0x0403
 CONFIG_G_DNL_PRODUCT_NUM=0xbd00
-CONFIG_SPL_OF_LIBFDT=y
diff --git a/configs/am43xx_evm_usbhost_boot_defconfig b/configs/am43xx_evm_usbhost_boot_defconfig
index e22bc7fbdb..68d6ee94ce 100644
--- a/configs/am43xx_evm_usbhost_boot_defconfig
+++ b/configs/am43xx_evm_usbhost_boot_defconfig
@@ -1,5 +1,6 @@
 CONFIG_ARM=y
 CONFIG_AM43XX=y
+CONFIG_SYS_MALLOC_F_LEN=0x2000
 CONFIG_TARGET_AM43XX_EVM=y
 CONFIG_ISW_ENTRY_ADDR=0x40300350
 CONFIG_SPL_STACK_R_ADDR=0x82000000
@@ -76,4 +77,3 @@ CONFIG_USB_GADGET_DOWNLOAD=y
 CONFIG_G_DNL_MANUFACTURER="Texas Instruments"
 CONFIG_G_DNL_VENDOR_NUM=0x0403
 CONFIG_G_DNL_PRODUCT_NUM=0xbd00
-CONFIG_SPL_OF_LIBFDT=y
diff --git a/configs/am43xx_hs_evm_defconfig b/configs/am43xx_hs_evm_defconfig
index 6bcbfd77ed..61e2d58fc9 100644
--- a/configs/am43xx_hs_evm_defconfig
+++ b/configs/am43xx_hs_evm_defconfig
@@ -1,5 +1,6 @@
 CONFIG_ARM=y
 CONFIG_AM43XX=y
+CONFIG_SYS_MALLOC_F_LEN=0x2000
 CONFIG_TI_SECURE_DEVICE=y
 CONFIG_TARGET_AM43XX_EVM=y
 CONFIG_ISW_ENTRY_ADDR=0x403018e0
@@ -82,4 +83,3 @@ CONFIG_USB_GADGET_DOWNLOAD=y
 CONFIG_G_DNL_MANUFACTURER="Texas Instruments"
 CONFIG_G_DNL_VENDOR_NUM=0x0403
 CONFIG_G_DNL_PRODUCT_NUM=0xbd00
-CONFIG_SPL_OF_LIBFDT=y
diff --git a/include/configs/am43xx_evm.h b/include/configs/am43xx_evm.h
index 1d622eff2f..ec99958fcd 100644
--- a/include/configs/am43xx_evm.h
+++ b/include/configs/am43xx_evm.h
@@ -18,9 +18,9 @@
 
 /* NS16550 Configuration */
 #define CONFIG_SYS_NS16550_CLK		48000000
-#if defined(CONFIG_SPL_BUILD) || !defined(CONFIG_DM_SERIAL)
+#if !defined(CONFIG_SPL_DM) || !defined(CONFIG_DM_SERIAL)
+#define CONFIG_SYS_NS16550_REG_SIZE    (-4)
 #define CONFIG_SYS_NS16550_SERIAL
-#define CONFIG_SYS_NS16550_REG_SIZE	(-4)
 #endif
 
 /* I2C Configuration */
@@ -111,9 +111,6 @@
  * DM support in SPL
  */
 #ifdef CONFIG_SPL_BUILD
-#undef CONFIG_DM_MMC
-#undef CONFIG_DM_SPI
-#undef CONFIG_DM_SPI_FLASH
 #undef CONFIG_TIMER
 #endif
 
-- 
2.11.0

[U-Boot] [PATCH v3 3/3] configs: am43xx_evm: Enable SPL_DM

[Tom Rini]

On Mon, Feb 20, 2017 at 11:05:33AM +0530, Lokesh Vutla wrote:

> Enable SPL_DM on all AM43xx based platforms
> 
> Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
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/20170220/99c92709/attachment.sig>