ARM: ti: DFU environment consolidation + AM57x DFU

PATCH New Sekhar Nori 2 participants 7 messages Nov 25, 2016 – Dec 3, 2016

lore

Share & tools

lore

Sekhar Nori

author

2016-11-25 08:55 UTC #0 lore raw

v3 cover ARM: ti: DFU environment consolidation + AM57x DFU

This patch series consolidates DFU environment variables into newly created include/environment/ti/dfu.h

Then goes on to add DFU support for AM57x GP EVM.

Build tested using the following buildman command:

$ tools/buildman/buildman -j 16 -b dfu -c 2 -o test am33xx omap

v2 -> v3
Rebase against latest master, drop 1/3 in the process.

v1 submission here:
https://www.mail-archive.com/u-boot at lists.denx.de/msg222838.html

Sekhar Nori (2):
ARM: ti: consolidate dfu environment variables
ARM: am57xx_evm: enable DFU support

 configs/am57xx_evm_defconfig    |  3 ++
 configs/am57xx_hs_evm_defconfig |  3 ++
 include/configs/am335x_evm.h    | 33 +-----------------
 include/configs/am43xx_evm.h    | 34 ++-----------------
 include/configs/am57xx_evm.h    |  8 +++++
 include/configs/dra7xx_evm.h    | 51 ++--------------------------
 include/configs/omap5_uevm.h    | 37 ++------------------
 include/environment/ti/dfu.h    | 75 +++++++++++++++++++++++++++++++++++++++++
 8 files changed, 96 insertions(+), 148 deletions(-)
 create mode 100644 include/environment/ti/dfu.h

Signature · 1 line

2.9.0

Sekhar Nori

author

2016-11-25 08:55 UTC #1 lore raw

PATCH v3 1/2 ARM: ti: consolidate dfu environment variables

Introduce include/environment/ti/dfu.h that
consolidates environment variable definitions
for various TI boards that support DFU today.

Tested on AM335x EVM, AM437x SK EVM and DRA74x
EVM by using DFU to write to SD card.

Signed-off-by Sekhar Nori <[email protected]>

---
 include/configs/am335x_evm.h | 33 +------------------
 include/configs/am43xx_evm.h | 34 ++------------------
 include/configs/dra7xx_evm.h | 51 ++----------------------------
 include/configs/omap5_uevm.h | 37 ++--------------------
 include/environment/ti/dfu.h | 75 ++++++++++++++++++++++++++++++++++++++++++++
 5 files changed, 82 insertions(+), 148 deletions(-)
 create mode 100644 include/environment/ti/dfu.h

include/configs/am335x_evm.h +1 ~~-32~~

index 8fa8e39f71e4..27d26af6f1c8 100644
@@ -17,6 +17,7 @@
 #define __CONFIG_AM335X_EVM_H
 
 #include <configs/ti_am335x_common.h>
+#include <environment/ti/dfu.h>
 
 #ifndef CONFIG_SPL_BUILD
 # define CONFIG_TIMESTAMP

@@ -330,38 +331,6 @@
 
 /* USB Device Firmware Update support */
 #ifndef CONFIG_SPL_BUILD
-#define DFU_ALT_INFO_MMC \
-	"dfu_alt_info_mmc=" \
-	"boot part 0 1;" \
-	"rootfs part 0 2;" \
-	"MLO fat 0 1;" \
-	"MLO.raw raw 0x100 0x100;" \
-	"u-boot.img.raw raw 0x300 0x400;" \
-	"spl-os-args.raw raw 0x80 0x80;" \
-	"spl-os-image.raw raw 0x900 0x2000;" \
-	"spl-os-args fat 0 1;" \
-	"spl-os-image fat 0 1;" \
-	"u-boot.img fat 0 1;" \
-	"uEnv.txt fat 0 1\0"
-#ifdef CONFIG_NAND
-#define DFU_ALT_INFO_NAND \
-	"dfu_alt_info_nand=" \
-	"SPL part 0 1;" \
-	"SPL.backup1 part 0 2;" \
-	"SPL.backup2 part 0 3;" \
-	"SPL.backup3 part 0 4;" \
-	"u-boot part 0 5;" \
-	"u-boot-spl-os part 0 6;" \
-	"kernel part 0 8;" \
-	"rootfs part 0 9\0"
-#else
-#define DFU_ALT_INFO_NAND ""
-#endif
-#define DFU_ALT_INFO_RAM \
-	"dfu_alt_info_ram=" \
-	"kernel ram 0x80200000 0xD80000;" \
-	"fdt ram 0x80F80000 0x80000;" \
-	"ramdisk ram 0x81000000 0x4000000\0"
 #define DFUARGS \
 	"dfu_alt_info_emmc=rawemmc raw 0 3751936\0" \
 	DFU_ALT_INFO_MMC \

include/configs/am43xx_evm.h +2 ~~-32~~

index 0a6c06a4bba2..6416d55c4250 100644
@@ -14,6 +14,7 @@
 #define CONFIG_MAX_RAM_BANK_SIZE	(1024 << 21)	/* 2GB */
 #define CONFIG_SYS_TIMERBASE		0x48040000	/* Use Timer2 */
 
+#include <environment/ti/dfu.h>
 #include <asm/arch/omap.h>
 
 /* NS16550 Configuration */

@@ -122,43 +123,12 @@
 
 #ifndef CONFIG_SPL_BUILD
 /* USB Device Firmware Update support */
-
-#define DFU_ALT_INFO_MMC \
-	"dfu_alt_info_mmc=" \
-	"boot part 0 1;" \
-	"rootfs part 0 2;" \
-	"MLO fat 0 1;" \
-	"spl-os-args fat 0 1;" \
-	"spl-os-image fat 0 1;" \
-	"u-boot.img fat 0 1;" \
-	"uEnv.txt fat 0 1\0"
-
-#define DFU_ALT_INFO_EMMC \
-	"dfu_alt_info_emmc=" \
-	"MLO raw 0x100 0x100 mmcpart 0;" \
-	"u-boot.img raw 0x300 0x1000 mmcpart 0\0"
-
-#define DFU_ALT_INFO_RAM \
-	"dfu_alt_info_ram=" \
-	"kernel ram 0x80200000 0x4000000;" \
-	"fdt ram 0x80f80000 0x80000;" \
-	"ramdisk ram 0x81000000 0x4000000\0"
-
-#define DFU_ALT_INFO_QSPI \
-	"dfu_alt_info_qspi=" \
-	"u-boot.bin raw 0x0 0x080000;" \
-	"u-boot.backup raw 0x080000 0x080000;" \
-	"u-boot-spl-os raw 0x100000 0x010000;" \
-	"u-boot-env raw 0x110000 0x010000;" \
-	"u-boot-env.backup raw 0x120000 0x010000;" \
-	"kernel raw 0x130000 0x800000\0"
-
 #define DFUARGS \
 	"dfu_bufsiz=0x10000\0" \
 	DFU_ALT_INFO_MMC \
 	DFU_ALT_INFO_EMMC \
 	DFU_ALT_INFO_RAM \
-	DFU_ALT_INFO_QSPI
+	DFU_ALT_INFO_QSPI_XIP
 #else
 #define DFUARGS
 #endif

include/configs/dra7xx_evm.h +2 ~~-49~~

69 lines not on this page · raw message

include/configs/omap5_uevm.h +2 ~~-35~~

index 22614fb9c3a0..736d8043aae3 100644
@@ -12,6 +12,8 @@
 #ifndef __CONFIG_OMAP5_EVM_H
 #define __CONFIG_OMAP5_EVM_H
 
+#include <environment/ti/dfu.h>
+
 #ifndef CONFIG_SPL_BUILD
 /* Define the default GPT table for eMMC */
 #define PARTS_DEFAULT \

@@ -19,41 +21,6 @@
 	"name=rootfs,start=2MiB,size=-,uuid=${uuid_gpt_rootfs}"
 #endif
 
-#define DFU_ALT_INFO_MMC \
-	"dfu_alt_info_mmc=" \
-	"boot part 0 1;" \
-	"rootfs part 0 2;" \
-	"MLO fat 0 1;" \
-	"MLO.raw raw 0x100 0x100;" \
-	"u-boot.img.raw raw 0x300 0x400;" \
-	"spl-os-args.raw raw 0x80 0x80;" \
-	"spl-os-image.raw raw 0x900 0x2000;" \
-	"spl-os-args fat 0 1;" \
-	"spl-os-image fat 0 1;" \
-	"u-boot.img fat 0 1;" \
-	"uEnv.txt fat 0 1\0"
-
-#define DFU_ALT_INFO_EMMC \
-	"dfu_alt_info_emmc=" \
-	"rawemmc raw 0 3751936;" \
-	"boot part 1 1;" \
-	"rootfs part 1 2;" \
-	"MLO fat 1 1;" \
-	"MLO.raw raw 0x100 0x100;" \
-	"u-boot.img.raw raw 0x300 0x400;" \
-	"spl-os-args.raw raw 0x80 0x80;" \
-	"spl-os-image.raw raw 0x900 0x2000;" \
-	"spl-os-args fat 1 1;" \
-	"spl-os-image fat 1 1;" \
-	"u-boot.img fat 1 1;" \
-	"uEnv.txt fat 1 1\0"
-
-#define DFU_ALT_INFO_RAM \
-	"dfu_alt_info_ram=" \
-	"kernel ram 0x80200000 0x4000000;" \
-	"fdt ram 0x80f80000 0x80000;" \
-	"ramdisk ram 0x81000000 0x4000000\0"
-
 #define DFUARGS \
 	"dfu_bufsiz=0x10000\0" \
 	DFU_ALT_INFO_MMC \

include/environment/ti/dfu.h new file +75 -0

75 lines not on this page · raw message

Signature · 1 line

2.9.0

Tom Rini

2016-11-26 16:16 UTC #2 lore raw

PATCH v3 1/2 ARM: ti: consolidate dfu environment variables

Quoted 8 lines · Sekhar Nori

Introduce include/environment/ti/dfu.h that
consolidates environment variable definitions
for various TI boards that support DFU today.
Tested on AM335x EVM, AM437x SK EVM and DRA74x
EVM by using DFU to write to SD card.
Signed-off-by Sekhar Nori <[email protected]>

Reviewed-by Tom Rini <[email protected]>

Signature · 8 lines

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/20161126/77cf3bbe/attachment.sig>

Tom Rini

2016-12-03 23:47 UTC #3 lore raw

PATCH v3 1/2 ARM: ti: consolidate dfu environment variables

In reply to Sekhar Nori

Quoted 9 lines · Sekhar Nori

Introduce include/environment/ti/dfu.h that consolidates environment variable definitions for various TI boards that support DFU today.
Tested on AM335x EVM, AM437x SK EVM and DRA74x EVM by using DFU to write to SD card.
Signed-off-by Sekhar Nori <[email protected]>
Reviewed-by Tom Rini <[email protected]>

Applied to u-boot/master, thanks!

Signature · 8 lines

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/20161203/a3ecdcdd/attachment.sig>

Sekhar Nori

author

2016-11-25 08:55 UTC #4 lore raw

PATCH v3 2/2 ARM: am57xx_evm: enable DFU support

In reply to Sekhar Nori

AM57xx GP EVM has USB2 port of the SoC exposed as
USB client port.

It is useful to be able to use this port for USB
DFU downloads.

Enable USB DFU support. Tested on AM57x GP EVM Rev A3 using DFU to download to connected SD card.

configs for HS version of the AM57x EVM are
included in the patch but not really tested.

Signed-off-by Sekhar Nori <[email protected]>

---
 configs/am57xx_evm_defconfig    | 3 +++
 configs/am57xx_hs_evm_defconfig | 3 +++
 include/configs/am57xx_evm.h    | 8 ++++++++
 3 files changed, 14 insertions(+)

configs/am57xx_evm_defconfig +3 -0

index 12d9a1425f88..a7f0fb60301a 100644
@@ -35,6 +35,9 @@ CONFIG_CMD_SF=y
 CONFIG_CMD_SPI=y
 CONFIG_CMD_I2C=y
 CONFIG_CMD_USB=y
+CONFIG_CMD_DFU=y
+CONFIG_DFU_RAM=y
+CONFIG_DFU_MMC=y
 CONFIG_CMD_GPIO=y
 # CONFIG_CMD_SETEXPR is not set
 CONFIG_CMD_DHCP=y

configs/am57xx_hs_evm_defconfig +3 -0

index c508b8e4dabc..2b1d6958b64f 100644
@@ -36,6 +36,9 @@ CONFIG_CMD_SF=y
 CONFIG_CMD_SPI=y
 CONFIG_CMD_I2C=y
 CONFIG_CMD_USB=y
+CONFIG_CMD_DFU=y
+CONFIG_DFU_RAM=y
+CONFIG_DFU_MMC=y
 CONFIG_CMD_GPIO=y
 # CONFIG_CMD_SETEXPR is not set
 CONFIG_CMD_DHCP=y

include/configs/am57xx_evm.h +8 -0

index d009900c3976..c457fd4b5406 100644
@@ -12,6 +12,8 @@
 #ifndef __CONFIG_AM57XX_EVM_H
 #define __CONFIG_AM57XX_EVM_H
 
+#include <environment/ti/dfu.h>
+
 #define CONFIG_AM57XX
 
 #ifdef CONFIG_SPL_BUILD

@@ -59,6 +61,12 @@
 	"name=ipu2,size=1M,uuid=${uuid_gpt_ipu2};" \
 	"name=userdata,size=-,uuid=${uuid_gpt_userdata}"
 
+#define DFUARGS \
+	"dfu_bufsiz=0x10000\0" \
+	DFU_ALT_INFO_MMC \
+	DFU_ALT_INFO_EMMC \
+	DFU_ALT_INFO_RAM \
+
 #include <configs/ti_omap5_common.h>
 
 /* Enhance our eMMC support / experience. */

Signature · 1 line

2.9.0

Tom Rini

2016-11-26 16:17 UTC #5 lore raw

PATCH v3 2/2 ARM: am57xx_evm: enable DFU support

Quoted 6 lines · Sekhar Nori

AM57xx GP EVM has USB2 port of the SoC exposed as
USB client port.
It is useful to be able to use this port for USB
DFU downloads.

Quoted 7 lines

Enable USB DFU support. Tested on AM57x GP EVM Rev A3 using DFU to download to connected SD card.
configs for HS version of the AM57x EVM are
included in the patch but not really tested.
Signed-off-by Sekhar Nori <[email protected]>

Reviewed-by Tom Rini <[email protected]>

Signature · 8 lines

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/20161126/43292283/attachment.sig>

Tom Rini

2016-12-03 23:47 UTC #6 lore raw

PATCH v3 2/2 ARM: am57xx_evm: enable DFU support

In reply to Sekhar Nori

Quoted 6 lines · Sekhar Nori

AM57xx GP EVM has USB2 port of the SoC exposed as USB client port.
It is useful to be able to use this port for USB DFU downloads.

Quoted 8 lines

Enable USB DFU support. Tested on AM57x GP EVM Rev A3 using DFU to download to connected SD card.
configs for HS version of the AM57x EVM are included in the patch but not really tested.
Signed-off-by Sekhar Nori <[email protected]>
Reviewed-by Tom Rini <[email protected]>

Applied to u-boot/master, thanks!

Signature · 8 lines

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/20161203/fa11deec/attachment.sig>