Skip to contentSkip to conversation

Adds support for Exynos5422 odroid xu3 board

PATCH Superseded Hyungwon Hwang 6 participants 22 messages Dec 2, 2014 – Dec 9, 2014

Subsystems (2)

v10 of 11 Superseded Latest is v11 →

lore
Share & tools
lore
#0 lore raw

v10 cover Adds support for Exynos5422 odroid xu3 board

This is v10 of the patchset adding support Odroud XU3 board.

link to the previous version:
v2: https://www.mail-archive.com/u-boot at lists.denx.de/msg152275.html
v3: https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html
v4: https://patchwork.ozlabs.org/patch/407411/
v5: https://patchwork.ozlabs.org/patch/407941/
v6: https://patchwork.ozlabs.org/patch/408647/
v7: https://patchwork.ozlabs.org/patch/410204/
v8: https://patchwork.ozlabs.org/patch/410709/
v9: https://patchwork.ozlabs.org/patch/415514/

The patchset by Akshay Saraswat is already merged into u-boot-samsung. Please ignore the instruction about the additional patchset in v8.

How to test this patch:
1. git clone http://git.denx.de/u-boot-samsung.git
2. Apply this patchset
3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- odroid-xu3_config
4. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- -j8
5. Now you can use u-boot-dtb.bin for your downloading

WARNING: (update in V10)
It was impossible to make the u-boot image smaller than 335872 bytes, and also I did not want to waste my time to make it fit for that restriction, because the discussion about this issue is in progress in malining list. I used BL2 by Suriyan Ramasami <[email protected]> which the restriction is removed for testing. If you need the BL2 for testing, please contact me or Suriyan.

Note: If you use micro SD card for your test you have to apply the below patch additionally. This patch is needed, because micro sd card is recognized as MMC1 instead of MMC0. Additional work is needed to make it work regardless of device id.

Thanks for Sjoerd Simons. Not only did he comment my patchset, but he also made real patch for me. I squashed some part of that patchset, and included the remainings into my patchset.

include/configs/exynos5-common.h +1 -1
index ba591e7..437eaae 100644
@@ -109,7 +109,7 @@
 
 #define CONFIG_SYS_MONITOR_BASE        0x00000000
 
-#define CONFIG_SYS_MMC_ENV_DEV         0
+#define CONFIG_SYS_MMC_ENV_DEV         1
 
 #define CONFIG_SECURE_BL1_ONLY
 

Changes for v2:
- Add a patch to add new common setup header file for Odroid X2/U3 and Odroid XU3

Changes for v3:
- Remove the patch which adds new common setup header file from v2
- Remove the wrong patch to fix GPIO information of Exynos 5800
- Remove unnecessary node from DT file
- Remove unnecessary features from config file
- Fix some trivial typos in comments

Changes for v4:
- Add MMC FIFO buffer's configuration to DT file
- Make CONFIG_OF_CONTROL be set by the target information
- Add basic document to doc/README.odroid-xu3
- Add CONFIG_CMD_EXT4 to config file
- Add environment size and offset to config file
- Add extra default environment to make bootable without modification
- Remove unnecessary features from config file

Changes for v5:
- Add the specific build instruction
- Update the information of patchset by Akshay Saraswat
- Convert /include/ to #include in DT file

Changes for v6:
- Separate out the documentation to new commit
- Remove unnecessary header file inclusuib from the board-specific setup file
- Make the function board_clock_init be declared, only when CONFIG_BOARD_EARLY_INIT_F is defined

Changes for v7:
- Fix several errata in the documentation
- Remove OF_CONTROL dependency from !SPL_BUILD

Changes for v8:
- Remove unnecessary properties in DT mmc node

Changes for v9:
- Remove the first patch which change GPIO entries order. It became useless, after DT support is added
- Remove useless variables in the default environment
- Add the new contents to the documentation of Odroid X2/U2, instead of making new document for Odorid XU3
- Remove the detailed information in the header file, and leave the reference to the documentation

Changes for v10:
- Move the non common configs in Exynos5420 config file to each board file
- Remove the config unsets which are added to make the result image small This is needless now, because the image is not small enough even though these unsets are added.
- Remove redundant DT node and properties
- Remove the odroid-xu3 board file and make odroid-xu3 a variant of smdk5420
- Fix some errata in documentation
- Include the patch by Sjoerd Simons as 4th patch

Hyungwon Hwang (3):
config: exynos5420: move non common configs to specific board files
Odroid-XU3: Add support for Odroid-XU3
Odroid-XU3: Add documentation for Odroid-XU3

Sjoerd Simons (1):
Odroid-XU3: Add entry for DTS EHCI GPIO

 arch/arm/cpu/armv7/exynos/Kconfig     |  4 +++
 arch/arm/dts/Makefile                 |  3 +-
 arch/arm/dts/exynos5422-odroidxu3.dts | 49 +++++++++++++++++++++++++++++++
 board/samsung/smdk5420/Kconfig        | 13 +++++++++
 configs/odroid-xu3_defconfig          |  4 +++
 doc/README.odroid                     | 46 ++++++++++++++++++-----------
 include/configs/exynos5420-common.h   | 18 ------------
 include/configs/odroid_xu3.h          | 54 +++++++++++++++++++++++++++++++++++
 include/configs/peach-pi.h            |  6 ++++
 include/configs/peach-pit.h           |  6 ++++
 include/configs/smdk5420.h            | 11 +++++++
 11 files changed, 179 insertions(+), 35 deletions(-)
 create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts
 create mode 100644 configs/odroid-xu3_defconfig
 create mode 100644 include/configs/odroid_xu3.h
Signature · 1 line
1.9.1
#1 lore raw

PATCH v10 1/4 config: exynos5420: move non common configs to specific board files

The media for boot and environment is a board-specific feature, not a processor-specific. This is same to console port number and some other addresses. This patch moves the that kinds of configs to each board-specific files from the common config file for Exynos5420.

Cc Minkyu Kang <[email protected]>
Cc Lukasz Majewski <[email protected]>
Cc Sjoerd Simons <[email protected]>
Cc Javier Martinez Canillas <[email protected]>
Cc Simon Glass <[email protected]>

---
Changes for v10:
- Newly added

 include/configs/exynos5420-common.h | 18 ------------------
 include/configs/peach-pi.h          |  6 ++++++
 include/configs/peach-pit.h         |  6 ++++++
 include/configs/smdk5420.h          | 11 +++++++++++
 4 files changed, 23 insertions(+), 18 deletions(-)
include/configs/exynos5420-common.h +0 -18
index ef6e155..7322f33 100644
@@ -13,13 +13,6 @@
 /* A variant of Exynos5420 (Exynos5 Family) */
 #define CONFIG_EXYNOS5800
 
-#define CONFIG_ENV_IS_IN_SPI_FLASH
-#define CONFIG_SPI_FLASH
-#define CONFIG_ENV_SPI_BASE	0x12D30000
-#define FLASH_SIZE		(0x4 << 20)
-#define CONFIG_ENV_OFFSET	(FLASH_SIZE - CONFIG_BL2_SIZE)
-#define CONFIG_SPI_BOOTING
-
 #include <configs/exynos5-common.h>
 
 #define CONFIG_ARCH_EARLY_INIT_R
@@ -29,8 +22,6 @@
 
 #define CONFIG_VAR_SIZE_SPL
 
-#define CONFIG_SYS_SDRAM_BASE		0x20000000
-#define CONFIG_SYS_TEXT_BASE		0x23E00000
 #ifdef CONFIG_VAR_SIZE_SPL
 #define CONFIG_SPL_TEXT_BASE		0x02024410
 #else
@@ -49,13 +40,4 @@
 
 #define CONFIG_SYS_USB_XHCI_MAX_ROOT_PORTS	2
 
-/*
- * Put the initial stack pointer 1KB below this to allow room for the
- * SPL marker. This value is arbitrary, but gd_t is placed starting here.
- */
-#define CONFIG_SYS_INIT_SP_ADDR	(CONFIG_IRAM_TOP - 0x800)
-
-/* Miscellaneous configurable options */
-#define CONFIG_DEFAULT_CONSOLE		"console=ttySAC1,115200n8\0"
-
 #endif	/* __CONFIG_EXYNOS5420_H */
include/configs/peach-pi.h +6 -0
index 8a82402..a1c980d 100644
@@ -14,14 +14,20 @@
 #define CONFIG_ENV_SPI_BASE	0x12D30000
 #define FLASH_SIZE		(0x4 << 20)
 #define CONFIG_ENV_OFFSET	(FLASH_SIZE - CONFIG_BL2_SIZE)
+#define CONFIG_SPI_BOOTING
 
 #include <configs/exynos5420-common.h>
 #include <configs/exynos5-dt-common.h>
 
 #define CONFIG_BOARD_COMMON
 
+#define CONFIG_SYS_SDRAM_BASE	0x20000000
+#define CONFIG_SYS_TEXT_BASE	0x23E00000
+#define CONFIG_SYS_INIT_SP_ADDR	(CONFIG_IRAM_TOP - 0x800)
+
 /* select serial console configuration */
 #define CONFIG_SERIAL3		/* use SERIAL 3 */
+#define CONFIG_DEFAULT_CONSOLE	"console=ttySAC1,115200n8\0"
 
 #define CONFIG_SYS_PROMPT	"Peach-Pi # "
 #define CONFIG_IDENT_STRING	" for Peach-Pi"
include/configs/peach-pit.h +6 -0
index ad5db57..6516a72 100644
@@ -14,14 +14,20 @@
 #define CONFIG_ENV_SPI_BASE	0x12D30000
 #define FLASH_SIZE		(0x4 << 20)
 #define CONFIG_ENV_OFFSET	(FLASH_SIZE - CONFIG_BL2_SIZE)
+#define CONFIG_SPI_BOOTING
 
 #include <configs/exynos5420-common.h>
 #include <configs/exynos5-dt-common.h>
 
 #define CONFIG_BOARD_COMMON
 
+#define CONFIG_SYS_SDRAM_BASE	0x20000000
+#define CONFIG_SYS_TEXT_BASE	0x23E00000
+#define CONFIG_SYS_INIT_SP_ADDR	(CONFIG_IRAM_TOP - 0x800)
+
 /* select serial console configuration */
 #define CONFIG_SERIAL3		/* use SERIAL 3 */
+#define CONFIG_DEFAULT_CONSOLE	"console=ttySAC1,115200n8\0"
 
 #define CONFIG_SYS_PROMPT	"Peach-Pit # "
 #define CONFIG_IDENT_STRING	" for Peach-Pit"
include/configs/smdk5420.h +11 -0
index 5c9a3c0..9742427 100644
@@ -11,13 +11,24 @@
 
 #include <configs/exynos5420-common.h>
 
+#define CONFIG_ENV_IS_IN_SPI_FLASH
+#define CONFIG_SPI_FLASH
+#define CONFIG_ENV_SPI_BASE	0x12D30000
+#define FLASH_SIZE		(0x4 << 20)
+#define CONFIG_ENV_OFFSET	(FLASH_SIZE - CONFIG_BL2_SIZE)
+#define CONFIG_SPI_BOOTING
+
 #define CONFIG_BOARD_COMMON
 
 #define CONFIG_SMDK5420			/* which is in a SMDK5420 */
 
+#define CONFIG_SYS_SDRAM_BASE	0x20000000
+#define CONFIG_SYS_TEXT_BASE	0x23E00000
+#define CONFIG_SYS_INIT_SP_ADDR	(CONFIG_IRAM_TOP - 0x800)
 
 /* select serial console configuration */
 #define CONFIG_SERIAL3		/* use SERIAL 3 */
+#define CONFIG_DEFAULT_CONSOLE	"console=ttySAC1,115200n8\0"
 
 #define CONFIG_SYS_PROMPT	"SMDK5420 # "
 #define CONFIG_IDENT_STRING	" for SMDK5420"
Signature · 1 line
1.9.1
#2 lore raw

PATCH v10 1/4 config: exynos5420: move non common configs to specific board files

Quoted 11 lines · Hyungwon Hwang

The media for boot and environment is a board-specific feature, not a processor-specific. This is same to console port number and some other addresses. This patch moves the that kinds of configs to each board-specific files from the common config file for Exynos5420.

Cc Minkyu Kang <[email protected]>
Cc Lukasz Majewski <[email protected]>
Cc Sjoerd Simons <[email protected]>
Cc Javier Martinez Canillas <[email protected]>
Cc Simon Glass <[email protected]>
Quoted 3 lines

---
Changes for v10:
- Newly added

#3 lore raw

PATCH v10 1/4 config: exynos5420: move non common configs to specific board files

In reply to Hyungwon Hwang
Quoted 28 lines · Hyungwon Hwang

The media for boot and environment is a board-specific feature, not a processor-specific. This is same to console port number and some other addresses. This patch moves the that kinds of configs to each board-specific files from the common config file for Exynos5420.

Cc Minkyu Kang <[email protected]>
Cc Lukasz Majewski <[email protected]>
Cc Sjoerd Simons <[email protected]>
Cc Javier Martinez Canillas <[email protected]>
Cc Simon Glass <[email protected]>

---
Changes for v10:
- Newly added

 include/configs/exynos5420-common.h | 18 ------------------
 include/configs/peach-pi.h          |  6 ++++++
 include/configs/peach-pit.h         |  6 ++++++
 include/configs/smdk5420.h          | 11 +++++++++++
 4 files changed, 23 insertions(+), 18 deletions(-)
include/configs/smdk5420.h +0 -0
index 5c9a3c0..9742427 100644

include/configs/smdk5420.h line 19

Quoted 9 lines
include/configs/smdk5420.h +6 -0
@@ -11,13 +11,24 @@
 
 #include <configs/exynos5420-common.h>
 
+#define CONFIG_ENV_IS_IN_SPI_FLASH
+#define CONFIG_SPI_FLASH
+#define CONFIG_ENV_SPI_BASE	0x12D30000
+#define FLASH_SIZE		(0x4 << 20)
+#define CONFIG_ENV_OFFSET	(FLASH_SIZE - CONFIG_BL2_SIZE)
+#define CONFIG_SPI_BOOTING

These need to be defined before including configs/exynos5420-common.h. With this patch smdk5420_defconfig doesn't build any longer

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 6170 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141205/e1977a3e/attachment.bin>

#4 lore raw

PATCH v10 2/4 Odroid-XU3: Add support for Odroid-XU3

In reply to Hyungwon Hwang

This patch adds support for Odroid-XU3.

Cc Minkyu Kang <[email protected]>
Cc Lukasz Majewski <[email protected]>
Cc Sjoerd Simons <[email protected]>
Cc Javier Martinez Canillas <[email protected]>
Cc Simon Glass <[email protected]>

---
Changes for v3:
- Remove unnecessary node from DT file
- Remove unnecessary features from config file
- Remove unnecessary macros from board-specific header file
- Fix some trivial typos in comments

Changes for v4:
- Add MMC FIFO buffer's configuration to DT file
- Make CONFIG_OF_CONTROL be set by the target information
- Add basic document to doc/README.odroid-xu3
- Add CONFIG_CMD_EXT4 to config file
- Add environment size and offset to config file
- Add extra default environment to make bootable without modification
- Remove unnecessary features from config file

Changes for v5:
- Convert /include/ to #include in DT file

Changes for v6:
- Separate out the documentation to new commit
- Remove unnecessary header file inclusions from the board-specific setup file
- Make the function board_clock_init be declared, only when CONFIG_BOARD_EARLY_INIT_F is defined

Changes for v7:
- Remove OF_CONTROL dependency from !SPL_BUILD

Changes for v8:
- Remove unnecessary properties in DT mmc node

Changes for v9:
- Remove useless variables in the default environment
- Replace the detailed information to the reference to the documentation

Changes for v10:
- Remove the config unsets which are added to make the result image small This is needless now, because the image is not small enough even though these unsets are added.
- Remove redundant DT node and properties
- Remove the odroid-xu3 board file and make odroid-xu3 a variant of smdk5420

 arch/arm/cpu/armv7/exynos/Kconfig     |  4 +++
 arch/arm/dts/Makefile                 |  3 +-
 arch/arm/dts/exynos5422-odroidxu3.dts | 45 +++++++++++++++++++++++++++++
 board/samsung/smdk5420/Kconfig        | 13 +++++++++
 configs/odroid-xu3_defconfig          |  4 +++
 include/configs/odroid_xu3.h          | 54 +++++++++++++++++++++++++++++++++++
 6 files changed, 122 insertions(+), 1 deletion(-)
 create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts
 create mode 100644 configs/odroid-xu3_defconfig
 create mode 100644 include/configs/odroid_xu3.h
arch/arm/cpu/armv7/exynos/Kconfig +4 -0
index f3eadb4..7fcb5d2 100644
@@ -24,6 +24,10 @@ config TARGET_TRATS2
 config TARGET_ODROID
 	bool "Exynos4412 Odroid board"
 
+config TARGET_ODROID_XU3
+	bool "Exynos5422 Odroid board"
+	select OF_CONTROL
+
 config TARGET_ARNDALE
 	bool "Exynos5250 Arndale board"
 	select CPU_V7_HAS_NONSEC
arch/arm/dts/Makefile +2 -1
index e5846ea..a811b1b 100644
@@ -13,7 +13,8 @@ dtb-$(CONFIG_EXYNOS5) += exynos5250-arndale.dtb \
 	exynos5250-smdk5250.dtb \
 	exynos5420-smdk5420.dtb \
 	exynos5420-peach-pit.dtb \
-	exynos5800-peach-pi.dtb
+	exynos5800-peach-pi.dtb \
+	exynos5422-odroidxu3.dtb
 dtb-$(CONFIG_TEGRA) += tegra20-harmony.dtb \
 	tegra20-medcom-wide.dtb \
 	tegra20-paz00.dtb \
arch/arm/dts/exynos5422-odroidxu3.dts new file +45 -0

45 lines not on this page · raw message

board/samsung/smdk5420/Kconfig +13 -0
index e7aafe5..ff28b1d 100644
@@ -1,3 +1,16 @@
+if TARGET_ODROID_XU3
+
+config SYS_BOARD
+	default "smdk5420"
+
+config SYS_VENDOR
+	default "samsung"
+
+config SYS_CONFIG_NAME
+	default "odroid_xu3"
+
+endif
+
 if TARGET_PEACH_PI
 
 config SYS_BOARD
configs/odroid-xu3_defconfig new file +4 -0
new file mode 100644
index 0000000..74aa0cf
@@ -0,0 +1,4 @@
+CONFIG_ARM=y
+CONFIG_ARCH_EXYNOS=y
+CONFIG_TARGET_ODROID_XU3=y
+CONFIG_DEFAULT_DEVICE_TREE="exynos5422-odroidxu3"
include/configs/odroid_xu3.h new file +54 -0

54 lines not on this page · raw message

Signature · 1 line
1.9.1
#5 lore raw

PATCH v10 2/4 Odroid-XU3: Add support for Odroid-XU3

Quoted 43 lines · Hyungwon Hwang

This patch adds support for Odroid-XU3.

Cc Minkyu Kang <[email protected]>
Cc Lukasz Majewski <[email protected]>
Cc Sjoerd Simons <[email protected]>
Cc Javier Martinez Canillas <[email protected]>
Cc Simon Glass <[email protected]>

---
Changes for v3:
- Remove unnecessary node from DT file
- Remove unnecessary features from config file
- Remove unnecessary macros from board-specific header file
- Fix some trivial typos in comments

Changes for v4:
- Add MMC FIFO buffer's configuration to DT file
- Make CONFIG_OF_CONTROL be set by the target information
- Add basic document to doc/README.odroid-xu3
- Add CONFIG_CMD_EXT4 to config file
- Add environment size and offset to config file
- Add extra default environment to make bootable without modification
- Remove unnecessary features from config file

Changes for v5:
- Convert /include/ to #include in DT file

Changes for v6:
- Separate out the documentation to new commit
- Remove unnecessary header file inclusions from the board-specific setup file
- Make the function board_clock_init be declared, only when CONFIG_BOARD_EARLY_INIT_F is defined

Changes for v7:
- Remove OF_CONTROL dependency from !SPL_BUILD

Changes for v8:
- Remove unnecessary properties in DT mmc node

Changes for v9:
- Remove useless variables in the default environment
- Replace the detailed information to the reference to the documentation

Quoted 6 lines

Changes for v10:
- Remove the config unsets which are added to make the result image small This is needless now, because the image is not small enough even though these unsets are added.
- Remove redundant DT node and properties
- Remove the odroid-xu3 board file and make odroid-xu3 a variant of smdk5420

Quoted 151 lines
 arch/arm/cpu/armv7/exynos/Kconfig     |  4 +++
 arch/arm/dts/Makefile                 |  3 +-
 arch/arm/dts/exynos5422-odroidxu3.dts | 45 +++++++++++++++++++++++++++++
 board/samsung/smdk5420/Kconfig        | 13 +++++++++
 configs/odroid-xu3_defconfig          |  4 +++
 include/configs/odroid_xu3.h          | 54 +++++++++++++++++++++++++++++++++++
 6 files changed, 122 insertions(+), 1 deletion(-)
 create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts
 create mode 100644 configs/odroid-xu3_defconfig
 create mode 100644 include/configs/odroid_xu3.h
arch/arm/cpu/armv7/exynos/Kconfig +4 -0
index f3eadb4..7fcb5d2 100644
@@ -24,6 +24,10 @@ config TARGET_TRATS2
 config TARGET_ODROID
        bool "Exynos4412 Odroid board"
 
+config TARGET_ODROID_XU3
+       bool "Exynos5422 Odroid board"
+       select OF_CONTROL
+
 config TARGET_ARNDALE
        bool "Exynos5250 Arndale board"
        select CPU_V7_HAS_NONSEC
arch/arm/dts/Makefile +2 -1
index e5846ea..a811b1b 100644
@@ -13,7 +13,8 @@ dtb-$(CONFIG_EXYNOS5) += exynos5250-arndale.dtb \
        exynos5250-smdk5250.dtb \
        exynos5420-smdk5420.dtb \
        exynos5420-peach-pit.dtb \
-       exynos5800-peach-pi.dtb
+       exynos5800-peach-pi.dtb \
+       exynos5422-odroidxu3.dtb
 dtb-$(CONFIG_TEGRA) += tegra20-harmony.dtb \
        tegra20-medcom-wide.dtb \
        tegra20-paz00.dtb \
arch/arm/dts/exynos5422-odroidxu3.dts new file +45 -0
new file mode 100644
index 0000000..cff32a9
@@ -0,0 +1,45 @@
+/*
+ * Odroid XU3 device tree source
+ *
+ * Copyright (c) 2014 Samsung Electronics Co., Ltd.
+ *             http://www.samsung.com
+ *
+ * SPDX-License-Identifier:    GPL-2.0+
+ */
+
+/dts-v1/;
+#include "exynos54xx.dtsi"
+
+/ {
+       model = "Odroid XU3 based on EXYNOS5422";
+       compatible = "samsung,odroidxu3", "samsung,exynos5";
+
+       aliases {
+               serial0 = "/serial at 12C00000";
+               console = "/serial at 12C20000";
+       };
+
+       memory {
+               device_type = "memory";
+               reg =  <0x40000000 0x10000000
+                       0x50000000 0x10000000
+                       0x60000000 0x10000000
+                       0x70000000 0x10000000
+                       0x80000000 0x10000000
+                       0x90000000 0x10000000
+                       0xa0000000 0x10000000
+                       0xb0000000 0xea00000>;
+       };
+
+       serial at 12C20000 {
+               status="okay";
+       };
+
+       mmc at 12200000 {
+               fifoth_val = <0x201f0020>;
+       };
+
+       mmc at 12220000 {
+               fifoth_val = <0x201f0020>;
+       };
+};
board/samsung/smdk5420/Kconfig +13 -0
index e7aafe5..ff28b1d 100644
@@ -1,3 +1,16 @@
+if TARGET_ODROID_XU3
+
+config SYS_BOARD
+       default "smdk5420"
+
+config SYS_VENDOR
+       default "samsung"
+
+config SYS_CONFIG_NAME
+       default "odroid_xu3"
+
+endif
+
 if TARGET_PEACH_PI
 
 config SYS_BOARD
configs/odroid-xu3_defconfig new file +4 -0
new file mode 100644
index 0000000..74aa0cf
@@ -0,0 +1,4 @@
+CONFIG_ARM=y
+CONFIG_ARCH_EXYNOS=y
+CONFIG_TARGET_ODROID_XU3=y
+CONFIG_DEFAULT_DEVICE_TREE="exynos5422-odroidxu3"
include/configs/odroid_xu3.h new file +22 -0
new file mode 100644
index 0000000..88bb98d
@@ -0,0 +1,54 @@
+/*
+ * Copyright (C) 2013 Samsung Electronics
+ * Hyungwon Hwang <[email protected]>
+ *
+ * SPDX-License-Identifier:     GPL-2.0+
+ */
+
+#ifndef __CONFIG_ODROID_XU3_H
+#define __CONFIG_ODROID_XU3_H
+
+#include "exynos5420-common.h"
+
+#define CONFIG_SYS_PROMPT              "ODROID-XU3 # "
+#define CONFIG_IDENT_STRING            " for ODROID-XU3"
+
+#define CONFIG_BOARD_COMMON
+
+#define CONFIG_SYS_SDRAM_BASE          0x40000000
+#define CONFIG_SYS_TEXT_BASE           0x43E00000
+
+/* select serial console configuration */
+#define CONFIG_SERIAL2                 /* use SERIAL 2 */

include/configs/odroid_xu3.h line 32

Quoted 10 lines
include/configs/odroid_xu3.h new file +10 -0
+
+#define TZPC_BASE_OFFSET               0x10000
+
+#define CONFIG_CMD_MMC
+#define CONFIG_CMD_EXT2
+#define CONFIG_CMD_EXT4
+#define CONFIG_CMD_FAT
+
+/*
+ * FIXME: The number of bank is actually 8. But there is no way to reserver the

reserve

Quoted 23 lines
include/configs/odroid_xu3.h new file +22 -1
+ * last 16 Mib in the last bank now. So I just excluded the last bank
+ * temporally.
+ */
+#define CONFIG_NR_DRAM_BANKS   7
+#define SDRAM_BANK_SIZE                (256UL << 20UL) /* 256 MB */
+
+#define CONFIG_ENV_IS_IN_MMC
+
+#undef CONFIG_ENV_SIZE
+#undef CONFIG_ENV_OFFSET
+#define CONFIG_ENV_SIZE                        4096
+#define CONFIG_ENV_OFFSET              (SZ_1K * 1280) /* 1.25 MiB offset */
+
+#define CONFIG_SYS_INIT_SP_ADDR        (CONFIG_SYS_LOAD_ADDR - 0x1000000)
+
+#define CONFIG_DEFAULT_CONSOLE         "console=ttySAC2,115200n8\0"
+
+/* FIXME: MUST BE REMOVED AFTER TMU IS TURNED ON */
+#undef CONFIG_EXYNOS_TMU
+#undef CONFIG_TMU_CMD_DTT
+
+#endif /* __CONFIG_H */
--
Quoted 2 lines

1.9.1

Regards,
Simon

#6 lore raw

PATCH v10 2/4 Odroid-XU3: Add support for Odroid-XU3

In reply to Hyungwon Hwang
Quoted 9 lines · Hyungwon Hwang

This patch adds support for Odroid-XU3.

configs/odroid-xu3_defconfig new file +0 -0
new file mode 100644
index 0000000..74aa0cf

configs/odroid-xu3_defconfig

Quoted 6 lines
configs/odroid-xu3_defconfig new file +5 -0
@@ -0,0 +1,4 @@
 
+
+#define CONFIG_CMD_MMC
+#define CONFIG_CMD_EXT2
+#define CONFIG_CMD_EXT4
+#define CONFIG_CMD_FAT

These four are already defined in include/config_distro_defaults.h which is included by the exynos headers

Other then that

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 6170 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141205/9a6e006b/attachment.bin>

#7 lore raw

PATCH v10 2/4 Odroid-XU3: Add support for Odroid-XU3

Dear Sjoerd,

Quoted 11 lines · On Fri, 05 Dec 2014 20:31:25 +0100 Sjoerd Simons…
Quoted 9 lines · Hyungwon Hwang

This patch adds support for Odroid-XU3.

(quoted diff) +0 -0

b/configs/odroid-xu3_defconfig new file mode 100644 index 0000000..74aa0cf

configs/odroid-xu3_defconfig new file +0 -0
@@ -0,0 +1,4 @@
Quoted 8 lines

configs/odroid-xu3_defconfig

Quoted 5 lines
configs/odroid-xu3_defconfig new file +5 -0
+
+#define CONFIG_CMD_MMC
+#define CONFIG_CMD_EXT2
+#define CONFIG_CMD_EXT4
+#define CONFIG_CMD_FAT

These four are already defined in include/config_distro_defaults.h which is included by the exynos headers

I will remove CONFIG_CMD_EXT2, CONFIG_CMD_EXT4, and CONFIG_CMD_FAT which defined needlessly from next version.

Thanks.

Quoted 5 lines

Other then that

Best regards,
Hyungwon Hwang

#8 lore raw

PATCH v10 3/4 Odroid-XU3: Add documentation for Odroid-XU3

In reply to Hyungwon Hwang

This patch adds documentation for Odroid-XU3. This documentation is based on that of Odroid (doc/README-odroid) made by Przemyslaw Marczak. The documentation includes basic information about boot media layout, environment, partition layout, and the instruction to burn the u-boot image to boot media.

Cc Minkyu Kang <[email protected]>
Cc Lukasz Majewski <[email protected]>
Cc Sjoerd Simons <[email protected]>
Cc Javier Martinez Canillas <[email protected]>
Cc Simon Glass <[email protected]>

---
Changes for v6:
- Newly added

Changes for v7:
- Fix several errata in the documentation

Changes for v8:
- None

Changes for v9:
- Add the new contents to the documentation of Odroid X2/U2, instead of making new document for Odorid XU3

Changes for v10:
- Fix a erratum

 doc/README.odroid | 46 ++++++++++++++++++++++++++++++----------------
 1 file changed, 30 insertions(+), 16 deletions(-)
doc/README.odroid +30 -16
index 25b962b..2e8dd27 100644
@@ -1,28 +1,39 @@
- U-boot for Odroid X2/U3
+ U-boot for Odroid X2/U3/XU3
 ========================
 
 1. Summary
 ==========
-This is a quick instruction for setup Odroid boards based on Exynos4412.
-Board config: odroid_config
+This is a quick instruction for setup Odroid boards.
+Board config: odroid_config for X2/U3
+Board config: odroid-xu3_config for XU3
 
 2. Supported devices
 ====================
-This U-BOOT config can be used on two boards:
+This U-BOOT config can be used on three boards:
 - Odroid U3
 - Odroid X2
 with CPU Exynos 4412 rev 2.0 and 2GB of RAM
+- Odroid XU3
+with CPU Exynos5422 and 2GB of RAM
 
 3. Boot sequence
 ================
 iROM->BL1->(BL2 + TrustZone)->U-BOOT
 
-This version of U-BOOT doesn't implement SPL but it is required(BL2)
-and can be found in "boot.tar.gz" from here:
+This version of U-BOOT doesn't implement SPL. So, BL1, BL2, and TrustZone
+binaries are needed to boot up.
+
+<< X2/U3 >>
+It can be found in "boot.tar.gz" from here:
 http://dev.odroid.com/projects/4412boot/wiki/FrontPage?action=download&value=boot.tar.gz
 or here:
 http://odroid.in/guides/ubuntu-lfs/boot.tar.gz
 
+<< XU3 >>
+It can be downloaded from:
+https://github.com/hardkernel/u-boot/tree/odroidxu3-v2012.07/sd_fuse/hardkernel
+
+
 4. Boot media layout
 ====================
 The table below shows SD/eMMC cards layout for U-boot.
@@ -35,18 +46,20 @@ The block offset is starting from 0 and the block size is 512B.
 | Bl2       | 31   | 30   |  1 (boot) |
 | U-boot    | 63   | 62   |  1 (boot) |
 | Tzsw      | 2111 | 2110 |  1 (boot) |
-| Uboot Env | 2500 | 2500 |  0 (user) |
+| Uboot Env | 2560 | 2560 |  0 (user) |
  -------------------------------------
 
 5. Prepare the SD boot card - with SD card reader
 =================================================
 To prepare bootable media you need boot binaries provided by hardkernel.
-File "boot.tar.gz" (link in point 3.) contains:
-- E4412_S.bl1.HardKernel.bin
-- E4412_S.tzsw.signed.bin
-- bl2.signed.bin
+From the downloaded files, You can find:
+- bl1.bin
+- tzsw.bin
+- bl2.bin
 - sd_fusing.sh
 - u-boot.bin
+(The file names can be slightly different, but you can distinguish what they are
+without problem)
 
 This is all you need to boot this board. But if you want to use your custom
 u-boot then you need to change u-boot.bin with your own u-boot binary*
@@ -56,7 +69,7 @@ and run the script "sd_fusing.sh" - this script is valid only for SD card.
 The proper binary file of current U-boot is u-boot-dtb.bin.
 
 quick steps for Linux:
-- extract boot.tar.gz
+- Download all files from the link at point 3 and extract it if needed.
 - put any SD card into the SD reader
 - check the device with "dmesg"
 - run ./sd_fusing.sh /dev/sdX - where X is SD card device (but not a partition)
@@ -66,7 +79,7 @@ Check if Hardkernel U-boot is booting, and next do the same with your U-boot.
    with a eMMC card reader (boot from eMMC card slot)
 =====================================================
 To boot the device from the eMMC slot you should use a special card reader
-which supports eMMC partiion switch. All of the boot binaries are stored
+which supports eMMC partition switch. All of the boot binaries are stored
 on the eMMC boot partition which is normally hidden.
 
 The "sd_fusing.sh" script can be used after updating offsets of binaries
@@ -81,8 +94,8 @@ But then the device can boot only from the SD card slot.
 
 8. Prepare the boot media using Hardkernel U-boot
 =================================================
-You can update the U-boot to the custom one if you have an working bootloader
-delivered with the board on a eMMC/SD card. Then follow the steps:
+You can update the U-boot to the custom one if you have a working bootloader
+delivered with the board on the eMMC/SD card. Then follow the steps:
 - install the android fastboot tool
 - connect a micro usb cable to the board
 - on the U-boot prompt, run command: fastboot (as a root)
@@ -91,7 +104,7 @@ delivered with the board on a eMMC/SD card. Then follow the steps:
 
 9. Partition layout
 ====================
-Default U-boot environment is setup for fixed partiion layout.
+Default U-boot environment is setup for fixed partition layout.
 
 Partition table: MSDOS. Disk layout and files as listed in the table below.
  ----- ------ ------ ------ -------- ---------------------------------
@@ -106,6 +119,7 @@ Partition table: MSDOS. Disk layout and files as listed in the table below.
 Supported fdt files are:
 - exynos4412-odroidx2.dtb
 - exynos4412-odroidu3.dtb
+- exynos5422-odroidxu3.dtb
 
 Supported kernel files are:
 - Image.itb
Signature · 1 line
1.9.1
#9 lore raw

PATCH v10 3/4 Odroid-XU3: Add documentation for Odroid-XU3

Hi,

Quoted 19 lines · Hyungwon Hwang

This patch adds documentation for Odroid-XU3. This documentation is based on that of Odroid (doc/README-odroid) made by Przemyslaw Marczak. The documentation includes basic information about boot media layout, environment, partition layout, and the instruction to burn the u-boot image to boot media.

Cc Minkyu Kang <[email protected]>
Cc Lukasz Majewski <[email protected]>
Cc Sjoerd Simons <[email protected]>
Cc Javier Martinez Canillas <[email protected]>
Cc Simon Glass <[email protected]>

---
Changes for v6:
- Newly added

Changes for v7:
- Fix several errata in the documentation

Quoted 6 lines

Changes for v8:
- None

Changes for v9:
- Add the new contents to the documentation of Odroid X2/U2, instead of making new document for Odorid XU3

Should you mention that networking is not available? Probably not that important.

Regards,
Simon

#10 lore raw

PATCH v10 3/4 Odroid-XU3: Add documentation for Odroid-XU3

Dear Simon,

Quoted 26 lines · On Tue, 02 Dec 2014 10:22:55 -0700 Simon Glass <[email protected]> wrote:

Hi,

Quoted 22 lines · Hyungwon Hwang

This patch adds documentation for Odroid-XU3. This documentation is based on that of Odroid (doc/README-odroid) made by Przemyslaw Marczak. The documentation includes basic information about boot media layout, environment, partition layout, and the instruction to burn the u-boot image to boot media.

Cc Minkyu Kang <[email protected]>
Cc Lukasz Majewski <[email protected]>
Cc Sjoerd Simons <[email protected]>
Cc Javier Martinez Canillas <[email protected]>
Cc Simon Glass <[email protected]>

---
Changes for v6:
- Newly added

Changes for v7:
- Fix several errata in the documentation

Changes for v8:
- None

Quoted 8 lines
Quoted 3 lines

Changes for v9:
- Add the new contents to the documentation of Odroid X2/U2, instead of making new document for Odorid XU3

Should you mention that networking is not available? Probably not that important.

Sorry for late reply. I will add comment for not available networking to the documentation.

Thanks for your review.

Quoted 3 lines

Regards,
Simon

Best regards,
Hyungwon Hwang

#11 lore raw

PATCH v10 4/4 Odroid-XU3: Add entry for DTS EHCI GPIO

In reply to Hyungwon Hwang

From: Sjoerd Simons <[email protected]>

Add samsung,vbus-gpio information for the XU3. This allows the usage of the EHCI controller on the XU3, which is connected to the SMSC LAN9514 chip (usb hub + network).

Note that this patch doesn't enable support for USB/USB networking in the default config as makes the u-boot binary too big for the current odroid setup.

Cc Minkyu Kang <[email protected]>
Cc Lukasz Majewski <[email protected]>
Cc Sjoerd Simons <[email protected]>
Cc Javier Martinez Canillas <[email protected]>
Cc Simon Glass <[email protected]>

---
Changes for v10:
- Newly added

 arch/arm/dts/exynos5422-odroidxu3.dts | 4 ++++
 1 file changed, 4 insertions(+)
arch/arm/dts/exynos5422-odroidxu3.dts +4 -0
index cff32a9..be20f1b 100644
@@ -31,6 +31,10 @@
 			0xb0000000 0xea00000>;
 	};
 
+	ehci at 12110000 {
+		samsung,vbus-gpio = <&gpio 0x316 0>; /* X26 */
+	};
+
 	serial at 12C20000 {
 		status="okay";
 	};
Signature · 1 line
1.9.1
#12 lore raw

PATCH v10 4/4 Odroid-XU3: Add entry for DTS EHCI GPIO

Hi,

Quoted 29 lines · Hyungwon Hwang

From: Sjoerd Simons <[email protected]>

Add samsung,vbus-gpio information for the XU3. This allows the usage of the EHCI controller on the XU3, which is connected to the SMSC LAN9514 chip (usb hub + network).

Note that this patch doesn't enable support for USB/USB networking in the default config as makes the u-boot binary too big for the current odroid setup.

Cc Minkyu Kang <[email protected]>
Cc Lukasz Majewski <[email protected]>
Cc Sjoerd Simons <[email protected]>
Cc Javier Martinez Canillas <[email protected]>
Cc Simon Glass <[email protected]>

---
Changes for v10:
- Newly added

 arch/arm/dts/exynos5422-odroidxu3.dts | 4 ++++
 1 file changed, 4 insertions(+)
arch/arm/dts/exynos5422-odroidxu3.dts +0 -0
index cff32a9..be20f1b 100644

arch/arm/dts/exynos5422-odroidxu3.dts line 35

Quoted 5 lines
arch/arm/dts/exynos5422-odroidxu3.dts +2 -0
@@ -31,6 +31,10 @@
                        0xb0000000 0xea00000>;
        };
 
+       ehci at 12110000 {
+               samsung,vbus-gpio = <&gpio 0x316 0>; /* X26 */

Are you sure this is right? By my count this should be 0x66, not 0x316. This might be historical.

It almost feels like you could drop this patch until you actually have working USB support.

Regards,
Simon

#13 lore raw

PATCH v10 4/4 Odroid-XU3: Add entry for DTS EHCI GPIO

+ajaykumar

Quoted 35 lines · Simon Glass

Hi,

Quoted 32 lines · Hyungwon Hwang

From: Sjoerd Simons <[email protected]>

Add samsung,vbus-gpio information for the XU3. This allows the usage of the EHCI controller on the XU3, which is connected to the SMSC LAN9514 chip (usb hub + network).

Note that this patch doesn't enable support for USB/USB networking in the default config as makes the u-boot binary too big for the current odroid setup.

Cc Minkyu Kang <[email protected]>
Cc Lukasz Majewski <[email protected]>
Cc Sjoerd Simons <[email protected]>
Cc Javier Martinez Canillas <[email protected]>
Cc Simon Glass <[email protected]>

---
Changes for v10:
- Newly added

 arch/arm/dts/exynos5422-odroidxu3.dts | 4 ++++
 1 file changed, 4 insertions(+)
arch/arm/dts/exynos5422-odroidxu3.dts +0 -0
index cff32a9..be20f1b 100644
@@ -31,6 +31,10 @@
                        0xb0000000 0xea00000>;
        };
 
Quoted 5 lines

arch/arm/dts/exynos5422-odroidxu3.dts line 35

Quoted 2 lines
arch/arm/dts/exynos5422-odroidxu3.dts +2 -0
+       ehci at 12110000 {
+               samsung,vbus-gpio = <&gpio 0x316 0>; /* X26 */

Are you sure this is right? By my count this should be 0x66, not 0x316. This might be historical.

Yes you're right! I copied the number from the exynos5250-smdk5250.dts without double-checking (even though i fixed the exact same isseu for snow recently, doh).

Please drop this patch from the set for now, and i'll send an updated one seperately. But that shouldn't block merging the other patches in this set.

Quoted 2 lines

It almost feels like you could drop this patch until you actually have working USB support.

USB does work, but it turns out that's because board_usb_vbus_init in smdk5420 hardcodes turning on GPIO X26 (which happens to be correct for XU3).. Which interestingly collides with exynos_lcd_power_on which hardcodes using GPIO X26 for the EDP_HPD (as used on the snow/peach boards).

So looks like some more things to fix :)

Signature · 9 lines
Sjoerd Simons <[email protected]>
Collabora Ltd.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 6170 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141203/5798e21c/attachment.bin>
#14 lore raw

v10 cover Adds support for Exynos5422 odroid xu3 board

In reply to Hyungwon Hwang

Hello all,

Quoted 128 lines · Hyungwon Hwang

This is v10 of the patchset adding support Odroud XU3 board.

link to the previous version:
v2: https://www.mail-archive.com/u-boot at lists.denx.de/msg152275.html
v3: https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html
v4: https://patchwork.ozlabs.org/patch/407411/
v5: https://patchwork.ozlabs.org/patch/407941/
v6: https://patchwork.ozlabs.org/patch/408647/
v7: https://patchwork.ozlabs.org/patch/410204/
v8: https://patchwork.ozlabs.org/patch/410709/
v9: https://patchwork.ozlabs.org/patch/415514/

The patchset by Akshay Saraswat is already merged into u-boot-samsung. Please ignore the instruction about the additional patchset in v8.

How to test this patch:
1. git clone http://git.denx.de/u-boot-samsung.git
2. Apply this patchset
3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- odroid-xu3_config
4. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- -j8
5. Now you can use u-boot-dtb.bin for your downloading

WARNING: (update in V10)
It was impossible to make the u-boot image smaller than 335872 bytes, and also I did not want to waste my time to make it fit for that restriction, because the discussion about this issue is in progress in malining list. I used BL2 by Suriyan Ramasami <[email protected]> which the restriction is removed for testing. If you need the BL2 for testing, please contact me or Suriyan.

Note: If you use micro SD card for your test you have to apply the below patch additionally. This patch is needed, because micro sd card is recognized as MMC1 instead of MMC0. Additional work is needed to make it work regardless of device id.

Thanks for Sjoerd Simons. Not only did he comment my patchset, but he also made real patch for me. I squashed some part of that patchset, and included the remainings into my patchset.

include/configs/exynos5-common.h +1 -1
index ba591e7..437eaae 100644
@@ -109,7 +109,7 @@
 
  #define CONFIG_SYS_MONITOR_BASE        0x00000000
 
-#define CONFIG_SYS_MMC_ENV_DEV         0
+#define CONFIG_SYS_MMC_ENV_DEV         1
 
  #define CONFIG_SECURE_BL1_ONLY
 

Changes for v2:
- Add a patch to add new common setup header file for Odroid X2/U3 and Odroid XU3

Changes for v3:
- Remove the patch which adds new common setup header file from v2
- Remove the wrong patch to fix GPIO information of Exynos 5800
- Remove unnecessary node from DT file
- Remove unnecessary features from config file
- Fix some trivial typos in comments

Changes for v4:
- Add MMC FIFO buffer's configuration to DT file
- Make CONFIG_OF_CONTROL be set by the target information
- Add basic document to doc/README.odroid-xu3
- Add CONFIG_CMD_EXT4 to config file
- Add environment size and offset to config file
- Add extra default environment to make bootable without modification
- Remove unnecessary features from config file

Changes for v5:
- Add the specific build instruction
- Update the information of patchset by Akshay Saraswat
- Convert /include/ to #include in DT file

Changes for v6:
- Separate out the documentation to new commit
- Remove unnecessary header file inclusuib from the board-specific setup file
- Make the function board_clock_init be declared, only when CONFIG_BOARD_EARLY_INIT_F is defined

Changes for v7:
- Fix several errata in the documentation
- Remove OF_CONTROL dependency from !SPL_BUILD

Changes for v8:
- Remove unnecessary properties in DT mmc node

Changes for v9:
- Remove the first patch which change GPIO entries order. It became useless, after DT support is added
- Remove useless variables in the default environment
- Add the new contents to the documentation of Odroid X2/U2, instead of making new document for Odorid XU3
- Remove the detailed information in the header file, and leave the reference to the documentation

Changes for v10:
- Move the non common configs in Exynos5420 config file to each board file
- Remove the config unsets which are added to make the result image small This is needless now, because the image is not small enough even though these unsets are added.
- Remove redundant DT node and properties
- Remove the odroid-xu3 board file and make odroid-xu3 a variant of smdk5420
- Fix some errata in documentation
- Include the patch by Sjoerd Simons as 4th patch

Hyungwon Hwang (3):
config: exynos5420: move non common configs to specific board files
Odroid-XU3: Add support for Odroid-XU3
Odroid-XU3: Add documentation for Odroid-XU3

Sjoerd Simons (1):
Odroid-XU3: Add entry for DTS EHCI GPIO

  arch/arm/cpu/armv7/exynos/Kconfig     |  4 +++
  arch/arm/dts/Makefile                 |  3 +-
  arch/arm/dts/exynos5422-odroidxu3.dts | 49 +++++++++++++++++++++++++++++++
  board/samsung/smdk5420/Kconfig        | 13 +++++++++
  configs/odroid-xu3_defconfig          |  4 +++
  doc/README.odroid                     | 46 ++++++++++++++++++-----------
  include/configs/exynos5420-common.h   | 18 ------------
  include/configs/odroid_xu3.h          | 54 +++++++++++++++++++++++++++++++++++
  include/configs/peach-pi.h            |  6 ++++
  include/configs/peach-pit.h           |  6 ++++
  include/configs/smdk5420.h            | 11 +++++++
  11 files changed, 179 insertions(+), 35 deletions(-)
Quoted 4 lines

create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts create mode 100644 configs/odroid-xu3_defconfig create mode 100644 include/configs/odroid_xu3.h

I talked to man from the hardkernel, and he said that there is no restrictions about adding BL1/BL2/TZSW as a binary blob to the Mainline U-Boot. So I think, in this case it should be provided also with those patches. Then we have a complete board setup.

To Hyungwon,
There is a size limit for each patch (100KiB), so if you would like to add such binary blob, you should add it using this site:
http://www.denx.de/wiki/attach/U-Boot/TooBigPatches
after a registration.

Best regards

Signature · 4 lines
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com
#15 lore raw

v10 cover Adds support for Exynos5422 odroid xu3 board

Hello all,

Quoted 155 lines · Przemyslaw Marczak

Hello all,

Quoted 141 lines · Hyungwon Hwang

This is v10 of the patchset adding support Odroud XU3 board.

link to the previous version:
v2: https://www.mail-archive.com/u-boot at lists.denx.de/msg152275.html
v3: https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html
v4: https://patchwork.ozlabs.org/patch/407411/
v5: https://patchwork.ozlabs.org/patch/407941/
v6: https://patchwork.ozlabs.org/patch/408647/
v7: https://patchwork.ozlabs.org/patch/410204/
v8: https://patchwork.ozlabs.org/patch/410709/
v9: https://patchwork.ozlabs.org/patch/415514/

The patchset by Akshay Saraswat is already merged into u-boot-samsung. Please ignore the instruction about the additional patchset in v8.

How to test this patch:
1. git clone http://git.denx.de/u-boot-samsung.git
2. Apply this patchset
3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- odroid-xu3_config
4. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- -j8
5. Now you can use u-boot-dtb.bin for your downloading

WARNING: (update in V10)
It was impossible to make the u-boot image smaller than 335872 bytes, and also
I did not want to waste my time to make it fit for that restriction, because
the discussion about this issue is in progress in malining list. I used BL2 by
Suriyan Ramasami <[email protected]> which the restriction is removed for
testing. If you need the BL2 for testing, please contact me or Suriyan.

Note: If you use micro SD card for your test you have to apply the below patch additionally. This patch is needed, because micro sd card is recognized as MMC1 instead of MMC0. Additional work is needed to make it work regardless of device id.

Thanks for Sjoerd Simons. Not only did he comment my patchset, but he also made
real patch for me. I squashed some part of that patchset, and included the remainings into my patchset.

include/configs/exynos5-common.h +1 -1
index ba591e7..437eaae 100644
@@ -109,7 +109,7 @@
 
  #define CONFIG_SYS_MONITOR_BASE        0x00000000
 
-#define CONFIG_SYS_MMC_ENV_DEV         0
+#define CONFIG_SYS_MMC_ENV_DEV         1
 
  #define CONFIG_SECURE_BL1_ONLY
 

Changes for v2:
- Add a patch to add new common setup header file for Odroid X2/U3 and Odroid XU3

Changes for v3:
- Remove the patch which adds new common setup header file from v2
- Remove the wrong patch to fix GPIO information of Exynos 5800
- Remove unnecessary node from DT file
- Remove unnecessary features from config file
- Fix some trivial typos in comments

Changes for v4:
- Add MMC FIFO buffer's configuration to DT file
- Make CONFIG_OF_CONTROL be set by the target information
- Add basic document to doc/README.odroid-xu3
- Add CONFIG_CMD_EXT4 to config file
- Add environment size and offset to config file
- Add extra default environment to make bootable without modification
- Remove unnecessary features from config file

Changes for v5:
- Add the specific build instruction
- Update the information of patchset by Akshay Saraswat
- Convert /include/ to #include in DT file

Changes for v6:
- Separate out the documentation to new commit
- Remove unnecessary header file inclusuib from the board-specific setup file
- Make the function board_clock_init be declared, only when CONFIG_BOARD_EARLY_INIT_F is defined

Changes for v7:
- Fix several errata in the documentation
- Remove OF_CONTROL dependency from !SPL_BUILD

Changes for v8:
- Remove unnecessary properties in DT mmc node

Changes for v9:
- Remove the first patch which change GPIO entries order. It became useless,
after DT support is added
- Remove useless variables in the default environment
- Add the new contents to the documentation of Odroid X2/U2, instead of making new document for Odorid XU3
- Remove the detailed information in the header file, and leave the reference to the documentation

Changes for v10:
- Move the non common configs in Exynos5420 config file to each board file
- Remove the config unsets which are added to make the result image small This is needless now, because the image is not small enough even though
these
unsets are added.
- Remove redundant DT node and properties
- Remove the odroid-xu3 board file and make odroid-xu3 a variant of smdk5420
- Fix some errata in documentation
- Include the patch by Sjoerd Simons as 4th patch

Hyungwon Hwang (3):
config: exynos5420: move non common configs to specific board files
Odroid-XU3: Add support for Odroid-XU3
Odroid-XU3: Add documentation for Odroid-XU3

Sjoerd Simons (1):
Odroid-XU3: Add entry for DTS EHCI GPIO

  arch/arm/cpu/armv7/exynos/Kconfig     |  4 +++
  arch/arm/dts/Makefile                 |  3 +-
  arch/arm/dts/exynos5422-odroidxu3.dts | 49
+++++++++++++++++++++++++++++++
  board/samsung/smdk5420/Kconfig        | 13 +++++++++
  configs/odroid-xu3_defconfig          |  4 +++
  doc/README.odroid                     | 46 ++++++++++++++++++-----------
  include/configs/exynos5420-common.h   | 18 ------------
  include/configs/odroid_xu3.h          | 54
+++++++++++++++++++++++++++++++++++
  include/configs/peach-pi.h            |  6 ++++
  include/configs/peach-pit.h           |  6 ++++
  include/configs/smdk5420.h            | 11 +++++++
  11 files changed, 179 insertions(+), 35 deletions(-)
Quoted 4 lines

create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts create mode 100644 configs/odroid-xu3_defconfig create mode 100644 include/configs/odroid_xu3.h

I talked to man from the hardkernel, and he said that there is no restrictions about adding BL1/BL2/TZSW as a binary blob to the Mainline U-Boot. So I think, in this case it should be provided also with those patches. Then we have a complete board setup.

Quoted 6 lines

To Hyungwon,
There is a size limit for each patch (100KiB), so if you would like to add such binary blob, you should add it using this site:
http://www.denx.de/wiki/attach/U-Boot/TooBigPatches
after a registration.

I do have a signed BL2 (1MB U-Boot image - same as for the Odroid U3) at http://forum.odroid.com/viewtopic.php?f=98&t=7388
It has the full package - BL1/BL2/TZ and the sd_fusing script which works for both SD as well as eMMC.
If we go with this BL2, then CONFIG_ENV_OFFSET (currently set at 1280
* 1024) will clash with the TZ area in SD card. ( I am assuming its OK for eMMC as the ENV resides in the user partition and not in boot0) This offset is OK for the Odroid-U3 as the TZ is smaller (156 K). For the Odroid U3: free area is after 2111 + (156 * 2) = 2423. The ENV starts at 1280K = offset block 2560.
For the Odroid-XU3 the TZ is 256K. Free area after the blobs starts from 2111 + (256 * 2) = 2623.
Hence, we will have to shift CONFIG_ENV_OFFSET to anything greater than 2623 (512 byte) blocks. Anything with CONFIG_ENV_OFFSET (SZ_1K * 1312) should work.

Thanks and Regards
- Suriyan

Quoted 6 lines

Best regards
--
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com

#16 lore raw

v10 cover Adds support for Exynos5422 odroid xu3 board

Hi All,

In Tizen mainline u-boot, CONFIG_ENV_OFFSET have already been fixed to SZ_1K * 3136 from SZ_1K * 1280.
It looks good to change CONFIG_ENV_OFFSET also in u-boot mainline to use the same offset with Tizen u-boot.

How about your opinion?

Best Regards,
Inha Song.

2014? 12? 04? 01:15, Suriyan Ramasami ? ?:

Quoted 193 lines

Hello all,

Quoted 153 lines · Przemyslaw Marczak

Hello all,

Quoted 140 lines · Hyungwon Hwang

This is v10 of the patchset adding support Odroud XU3 board.

link to the previous version:
v2: https://www.mail-archive.com/u-boot at lists.denx.de/msg152275.html
v3: https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html
v4: https://patchwork.ozlabs.org/patch/407411/
v5: https://patchwork.ozlabs.org/patch/407941/
v6: https://patchwork.ozlabs.org/patch/408647/
v7: https://patchwork.ozlabs.org/patch/410204/
v8: https://patchwork.ozlabs.org/patch/410709/
v9: https://patchwork.ozlabs.org/patch/415514/

The patchset by Akshay Saraswat is already merged into u-boot-samsung. Please ignore the instruction about the additional patchset in v8.

How to test this patch:
1. git clone http://git.denx.de/u-boot-samsung.git
2. Apply this patchset
3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- odroid-xu3_config
4. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- -j8
5. Now you can use u-boot-dtb.bin for your downloading

WARNING: (update in V10)
It was impossible to make the u-boot image smaller than 335872 bytes, and also
I did not want to waste my time to make it fit for that restriction, because
the discussion about this issue is in progress in malining list. I used BL2 by
Suriyan Ramasami <[email protected]> which the restriction is removed for
testing. If you need the BL2 for testing, please contact me or Suriyan.

Note: If you use micro SD card for your test you have to apply the below patch additionally. This patch is needed, because micro sd card is recognized as MMC1 instead of MMC0. Additional work is needed to make it work regardless of device id.

Thanks for Sjoerd Simons. Not only did he comment my patchset, but he also made
real patch for me. I squashed some part of that patchset, and included the remainings into my patchset.

include/configs/exynos5-common.h +1 -1
index ba591e7..437eaae 100644
@@ -109,7 +109,7 @@
 
   #define CONFIG_SYS_MONITOR_BASE        0x00000000
 
-#define CONFIG_SYS_MMC_ENV_DEV         0
+#define CONFIG_SYS_MMC_ENV_DEV         1
 
   #define CONFIG_SECURE_BL1_ONLY
 

Changes for v2:
- Add a patch to add new common setup header file for Odroid X2/U3 and Odroid XU3

Changes for v3:
- Remove the patch which adds new common setup header file from v2
- Remove the wrong patch to fix GPIO information of Exynos 5800
- Remove unnecessary node from DT file
- Remove unnecessary features from config file
- Fix some trivial typos in comments

Changes for v4:
- Add MMC FIFO buffer's configuration to DT file
- Make CONFIG_OF_CONTROL be set by the target information
- Add basic document to doc/README.odroid-xu3
- Add CONFIG_CMD_EXT4 to config file
- Add environment size and offset to config file
- Add extra default environment to make bootable without modification
- Remove unnecessary features from config file

Changes for v5:
- Add the specific build instruction
- Update the information of patchset by Akshay Saraswat
- Convert /include/ to #include in DT file

Changes for v6:
- Separate out the documentation to new commit
- Remove unnecessary header file inclusuib from the board-specific setup file
- Make the function board_clock_init be declared, only when CONFIG_BOARD_EARLY_INIT_F is defined

Changes for v7:
- Fix several errata in the documentation
- Remove OF_CONTROL dependency from !SPL_BUILD

Changes for v8:
- Remove unnecessary properties in DT mmc node

Changes for v9:
- Remove the first patch which change GPIO entries order. It became useless,
after DT support is added
- Remove useless variables in the default environment
- Add the new contents to the documentation of Odroid X2/U2, instead of making new document for Odorid XU3
- Remove the detailed information in the header file, and leave the reference to the documentation

Changes for v10:
- Move the non common configs in Exynos5420 config file to each board file
- Remove the config unsets which are added to make the result image small This is needless now, because the image is not small enough even though
these
unsets are added.
- Remove redundant DT node and properties
- Remove the odroid-xu3 board file and make odroid-xu3 a variant of smdk5420
- Fix some errata in documentation
- Include the patch by Sjoerd Simons as 4th patch

Hyungwon Hwang (3):
config: exynos5420: move non common configs to specific board files
Odroid-XU3: Add support for Odroid-XU3
Odroid-XU3: Add documentation for Odroid-XU3

Sjoerd Simons (1):
Odroid-XU3: Add entry for DTS EHCI GPIO

   arch/arm/cpu/armv7/exynos/Kconfig     |  4 +++
   arch/arm/dts/Makefile                 |  3 +-
   arch/arm/dts/exynos5422-odroidxu3.dts | 49
+++++++++++++++++++++++++++++++
   board/samsung/smdk5420/Kconfig        | 13 +++++++++
   configs/odroid-xu3_defconfig          |  4 +++
   doc/README.odroid                     | 46 ++++++++++++++++++-----------
   include/configs/exynos5420-common.h   | 18 ------------
   include/configs/odroid_xu3.h          | 54
+++++++++++++++++++++++++++++++++++
   include/configs/peach-pi.h            |  6 ++++
   include/configs/peach-pit.h           |  6 ++++
   include/configs/smdk5420.h            | 11 +++++++
   11 files changed, 179 insertions(+), 35 deletions(-)
Quoted 4 lines

create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts create mode 100644 configs/odroid-xu3_defconfig create mode 100644 include/configs/odroid_xu3.h

I talked to man from the hardkernel, and he said that there is no restrictions about adding BL1/BL2/TZSW as a binary blob to the Mainline U-Boot. So I think, in this case it should be provided also with those patches. Then we have a complete board setup.

Quoted 6 lines

To Hyungwon,
There is a size limit for each patch (100KiB), so if you would like to add such binary blob, you should add it using this site:
http://www.denx.de/wiki/attach/U-Boot/TooBigPatches
after a registration.

I do have a signed BL2 (1MB U-Boot image - same as for the Odroid U3) at http://forum.odroid.com/viewtopic.php?f=98&t=7388
It has the full package - BL1/BL2/TZ and the sd_fusing script which works for both SD as well as eMMC.
If we go with this BL2, then CONFIG_ENV_OFFSET (currently set at 1280
* 1024) will clash with the TZ area in SD card. ( I am assuming its OK for eMMC as the ENV resides in the user partition and not in boot0) This offset is OK for the Odroid-U3 as the TZ is smaller (156 K). For the Odroid U3: free area is after 2111 + (156 * 2) = 2423. The ENV starts at 1280K = offset block 2560.
For the Odroid-XU3 the TZ is 256K. Free area after the blobs starts from 2111 + (256 * 2) = 2623.
Hence, we will have to shift CONFIG_ENV_OFFSET to anything greater than 2623 (512 byte) blocks. Anything with CONFIG_ENV_OFFSET (SZ_1K * 1312) should work.

Thanks and Regards
- Suriyan

Quoted 6 lines

Best regards
--
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com

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

#17 lore raw

v10 cover Adds support for Exynos5422 odroid xu3 board

Dear all,

Quoted 8 lines · On Thu, 04 Dec 2014 10:27:04 +0900 Inha Song <[email protected]> wrote:

Hi All,

In Tizen mainline u-boot, CONFIG_ENV_OFFSET have already been fixed to SZ_1K * 3136 from SZ_1K * 1280.
It looks good to change CONFIG_ENV_OFFSET also in u-boot mainline to use the same offset with Tizen u-boot.

How about your opinion?

As I see the other boards' config files, there is no one principle to designate the environment offset, right? Then blk# 3136 seems reasonable for me. As Suriyan said, it must be bigger than blk# 2623. Can it be 256.5 Kbyte will be wasted, if blk# 3136 will be used for environment offset. Is it serious? If it is, I think blk# 2623 can be a good option for this.

Quoted 208 lines

Best Regards,
Inha Song.

2014? 12? 04? 01:15, Suriyan Ramasami ? ?:

Quoted 202 lines

Hello all,

Quoted 161 lines · Przemyslaw Marczak

Hello all,

Quoted 144 lines · Hyungwon Hwang

This is v10 of the patchset adding support Odroud XU3 board.

link to the previous version:
v2:
https://www.mail-archive.com/u-boot at lists.denx.de/msg152275.html v3:
https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html
v4: https://patchwork.ozlabs.org/patch/407411/ v5:
https://patchwork.ozlabs.org/patch/407941/ v6:
https://patchwork.ozlabs.org/patch/408647/ v7:
https://patchwork.ozlabs.org/patch/410204/ v8:
https://patchwork.ozlabs.org/patch/410709/ v9:
https://patchwork.ozlabs.org/patch/415514/

The patchset by Akshay Saraswat is already merged into u-boot-samsung. Please ignore the instruction about the additional patchset in v8.

How to test this patch:
1. git clone http://git.denx.de/u-boot-samsung.git
2. Apply this patchset
3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- odroid-xu3_config 4. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- -j8 5. Now you can use u-boot-dtb.bin for your downloading

WARNING: (update in V10)
It was impossible to make the u-boot image smaller than 335872 bytes, and also
I did not want to waste my time to make it fit for that restriction, because
the discussion about this issue is in progress in malining list. I used BL2 by
Suriyan Ramasami <[email protected]> which the restriction is removed for
testing. If you need the BL2 for testing, please contact me or Suriyan.

Note: If you use micro SD card for your test you have to apply the below patch additionally. This patch is needed, because micro sd card is recognized as MMC1 instead of MMC0. Additional work is needed to make it work regardless of device id.

Thanks for Sjoerd Simons. Not only did he comment my patchset, but he also made
real patch for me. I squashed some part of that patchset, and included the remainings into my patchset.

include/configs/exynos5-common.h +1 -1
index ba591e7..437eaae 100644
@@ -109,7 +109,7 @@
 
   #define CONFIG_SYS_MONITOR_BASE        0x00000000
 
-#define CONFIG_SYS_MMC_ENV_DEV         0
+#define CONFIG_SYS_MMC_ENV_DEV         1
 
   #define CONFIG_SECURE_BL1_ONLY
 

Changes for v2:
- Add a patch to add new common setup header file for Odroid X2/U3 and Odroid XU3

Changes for v3:
- Remove the patch which adds new common setup header file from v2
- Remove the wrong patch to fix GPIO information of Exynos 5800
- Remove unnecessary node from DT file
- Remove unnecessary features from config file
- Fix some trivial typos in comments

Changes for v4:
- Add MMC FIFO buffer's configuration to DT file
- Make CONFIG_OF_CONTROL be set by the target information
- Add basic document to doc/README.odroid-xu3
- Add CONFIG_CMD_EXT4 to config file
- Add environment size and offset to config file
- Add extra default environment to make bootable without modification
- Remove unnecessary features from config file

Changes for v5:
- Add the specific build instruction
- Update the information of patchset by Akshay Saraswat
- Convert /include/ to #include in DT file

Changes for v6:
- Separate out the documentation to new commit
- Remove unnecessary header file inclusuib from the board-specific setup file
- Make the function board_clock_init be declared, only when CONFIG_BOARD_EARLY_INIT_F is defined

Changes for v7:
- Fix several errata in the documentation
- Remove OF_CONTROL dependency from !SPL_BUILD

Changes for v8:
- Remove unnecessary properties in DT mmc node

Changes for v9:
- Remove the first patch which change GPIO entries order. It became useless,
after DT support is added
- Remove useless variables in the default environment
- Add the new contents to the documentation of Odroid X2/U2, instead of making new document for Odorid XU3
- Remove the detailed information in the header file, and leave the reference to the documentation

Changes for v10:
- Move the non common configs in Exynos5420 config file to each board file
- Remove the config unsets which are added to make the result image small This is needless now, because the image is not small enough even though these
unsets are added.
- Remove redundant DT node and properties
- Remove the odroid-xu3 board file and make odroid-xu3 a variant of smdk5420
- Fix some errata in documentation
- Include the patch by Sjoerd Simons as 4th patch

Hyungwon Hwang (3):
config: exynos5420: move non common configs to specific board
files Odroid-XU3: Add support for Odroid-XU3
Odroid-XU3: Add documentation for Odroid-XU3

Sjoerd Simons (1):
Odroid-XU3: Add entry for DTS EHCI GPIO

   arch/arm/cpu/armv7/exynos/Kconfig     |  4 +++
   arch/arm/dts/Makefile                 |  3 +-
   arch/arm/dts/exynos5422-odroidxu3.dts | 49
+++++++++++++++++++++++++++++++
   board/samsung/smdk5420/Kconfig        | 13 +++++++++
   configs/odroid-xu3_defconfig          |  4 +++
   doc/README.odroid                     | 46

++++++++++++++++++-----------
include/configs/exynos5420-common.h | 18 ------------
include/configs/odroid_xu3.h | 54
+++++++++++++++++++++++++++++++++++

Quoted 8 lines
include/configs/peach-pi.h            |  6 ++++
include/configs/peach-pit.h           |  6 ++++
include/configs/smdk5420.h            | 11 +++++++ 11 files

changed, 179 insertions(+), 35 deletions(-) create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts create mode 100644 configs/odroid-xu3_defconfig create mode 100644 include/configs/odroid_xu3.h

I talked to man from the hardkernel, and he said that there is no restrictions about adding BL1/BL2/TZSW as a binary blob to the Mainline U-Boot. So I think, in this case it should be provided also with those patches. Then we have a complete board setup.

Quoted 6 lines

To Hyungwon,
There is a size limit for each patch (100KiB), so if you would like to add such binary blob, you should add it using this site:
http://www.denx.de/wiki/attach/U-Boot/TooBigPatches
after a registration.

I do have a signed BL2 (1MB U-Boot image - same as for the Odroid U3) at http://forum.odroid.com/viewtopic.php?f=98&t=7388
It has the full package - BL1/BL2/TZ and the sd_fusing script which works for both SD as well as eMMC.
If we go with this BL2, then CONFIG_ENV_OFFSET (currently set at 1280
* 1024) will clash with the TZ area in SD card. ( I am assuming its OK for eMMC as the ENV resides in the user partition and not in boot0) This offset is OK for the Odroid-U3 as the TZ is smaller (156 K). For the Odroid U3: free area is after 2111 + (156 * 2) = 2423. The ENV starts at 1280K = offset block 2560.
For the Odroid-XU3 the TZ is 256K. Free area after the blobs starts from 2111 + (256 * 2) = 2623.
Hence, we will have to shift CONFIG_ENV_OFFSET to anything greater than 2623 (512 byte) blocks. Anything with CONFIG_ENV_OFFSET (SZ_1K
* 1312) should work.

Thanks and Regards
- Suriyan

Quoted 6 lines

Best regards
--
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com

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

Best regards,
Hyungwon Hwang

#18 lore raw

v10 cover Adds support for Exynos5422 odroid xu3 board

In reply to Przemyslaw Marczak

Hi,

Quoted 155 lines · Przemyslaw Marczak

Hello all,

Quoted 141 lines · Hyungwon Hwang

This is v10 of the patchset adding support Odroud XU3 board.

link to the previous version:
v2: https://www.mail-archive.com/u-boot at lists.denx.de/msg152275.html
v3: https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html
v4: https://patchwork.ozlabs.org/patch/407411/
v5: https://patchwork.ozlabs.org/patch/407941/
v6: https://patchwork.ozlabs.org/patch/408647/
v7: https://patchwork.ozlabs.org/patch/410204/
v8: https://patchwork.ozlabs.org/patch/410709/
v9: https://patchwork.ozlabs.org/patch/415514/

The patchset by Akshay Saraswat is already merged into u-boot-samsung. Please ignore the instruction about the additional patchset in v8.

How to test this patch:
1. git clone http://git.denx.de/u-boot-samsung.git
2. Apply this patchset
3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- odroid-xu3_config
4. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- -j8
5. Now you can use u-boot-dtb.bin for your downloading

WARNING: (update in V10)
It was impossible to make the u-boot image smaller than 335872 bytes, and also
I did not want to waste my time to make it fit for that restriction, because
the discussion about this issue is in progress in malining list. I used BL2 by
Suriyan Ramasami <[email protected]> which the restriction is removed for
testing. If you need the BL2 for testing, please contact me or Suriyan.

Note: If you use micro SD card for your test you have to apply the below patch additionally. This patch is needed, because micro sd card is recognized as MMC1 instead of MMC0. Additional work is needed to make it work regardless of device id.

Thanks for Sjoerd Simons. Not only did he comment my patchset, but he also made
real patch for me. I squashed some part of that patchset, and included the remainings into my patchset.

include/configs/exynos5-common.h +1 -1
index ba591e7..437eaae 100644
@@ -109,7 +109,7 @@
 
  #define CONFIG_SYS_MONITOR_BASE        0x00000000
 
-#define CONFIG_SYS_MMC_ENV_DEV         0
+#define CONFIG_SYS_MMC_ENV_DEV         1
 
  #define CONFIG_SECURE_BL1_ONLY
 

Changes for v2:
- Add a patch to add new common setup header file for Odroid X2/U3 and Odroid XU3

Changes for v3:
- Remove the patch which adds new common setup header file from v2
- Remove the wrong patch to fix GPIO information of Exynos 5800
- Remove unnecessary node from DT file
- Remove unnecessary features from config file
- Fix some trivial typos in comments

Changes for v4:
- Add MMC FIFO buffer's configuration to DT file
- Make CONFIG_OF_CONTROL be set by the target information
- Add basic document to doc/README.odroid-xu3
- Add CONFIG_CMD_EXT4 to config file
- Add environment size and offset to config file
- Add extra default environment to make bootable without modification
- Remove unnecessary features from config file

Changes for v5:
- Add the specific build instruction
- Update the information of patchset by Akshay Saraswat
- Convert /include/ to #include in DT file

Changes for v6:
- Separate out the documentation to new commit
- Remove unnecessary header file inclusuib from the board-specific setup file
- Make the function board_clock_init be declared, only when CONFIG_BOARD_EARLY_INIT_F is defined

Changes for v7:
- Fix several errata in the documentation
- Remove OF_CONTROL dependency from !SPL_BUILD

Changes for v8:
- Remove unnecessary properties in DT mmc node

Changes for v9:
- Remove the first patch which change GPIO entries order. It became useless,
after DT support is added
- Remove useless variables in the default environment
- Add the new contents to the documentation of Odroid X2/U2, instead of making new document for Odorid XU3
- Remove the detailed information in the header file, and leave the reference to the documentation

Changes for v10:
- Move the non common configs in Exynos5420 config file to each board file
- Remove the config unsets which are added to make the result image small This is needless now, because the image is not small enough even though
these
unsets are added.
- Remove redundant DT node and properties
- Remove the odroid-xu3 board file and make odroid-xu3 a variant of smdk5420
- Fix some errata in documentation
- Include the patch by Sjoerd Simons as 4th patch

Hyungwon Hwang (3):
config: exynos5420: move non common configs to specific board files
Odroid-XU3: Add support for Odroid-XU3
Odroid-XU3: Add documentation for Odroid-XU3

Sjoerd Simons (1):
Odroid-XU3: Add entry for DTS EHCI GPIO

  arch/arm/cpu/armv7/exynos/Kconfig     |  4 +++
  arch/arm/dts/Makefile                 |  3 +-
  arch/arm/dts/exynos5422-odroidxu3.dts | 49
+++++++++++++++++++++++++++++++
  board/samsung/smdk5420/Kconfig        | 13 +++++++++
  configs/odroid-xu3_defconfig          |  4 +++
  doc/README.odroid                     | 46 ++++++++++++++++++-----------
  include/configs/exynos5420-common.h   | 18 ------------
  include/configs/odroid_xu3.h          | 54
+++++++++++++++++++++++++++++++++++
  include/configs/peach-pi.h            |  6 ++++
  include/configs/peach-pit.h           |  6 ++++
  include/configs/smdk5420.h            | 11 +++++++
  11 files changed, 179 insertions(+), 35 deletions(-)
Quoted 4 lines

create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts create mode 100644 configs/odroid-xu3_defconfig create mode 100644 include/configs/odroid_xu3.h

I talked to man from the hardkernel, and he said that there is no restrictions about adding BL1/BL2/TZSW as a binary blob to the Mainline U-Boot. So I think, in this case it should be provided also with those patches. Then we have a complete board setup.

Quoted 5 lines

To Hyungwon,
There is a size limit for each patch (100KiB), so if you would like to add such binary blob, you should add it using this site:
http://www.denx.de/wiki/attach/U-Boot/TooBigPatches
after a registration.

If the intent is to upload a hexdump of a few of the files, that does sound like a convenient option. Very unfortunate though. At least for the trustzone file it is mostly zeroes, and seems to consist of 3 separate very small images. So perhaps we should have a tool that pieces them together?

Regards,
Simon

#19 lore raw

v10 cover Adds support for Exynos5422 odroid xu3 board

Hello,

Quoted 164 lines · Simon Glass

Hi,

Quoted 155 lines · Przemyslaw Marczak

Hello all,

Quoted 141 lines · Hyungwon Hwang

This is v10 of the patchset adding support Odroud XU3 board.

link to the previous version:
v2: https://www.mail-archive.com/u-boot at lists.denx.de/msg152275.html
v3: https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html
v4: https://patchwork.ozlabs.org/patch/407411/
v5: https://patchwork.ozlabs.org/patch/407941/
v6: https://patchwork.ozlabs.org/patch/408647/
v7: https://patchwork.ozlabs.org/patch/410204/
v8: https://patchwork.ozlabs.org/patch/410709/
v9: https://patchwork.ozlabs.org/patch/415514/

The patchset by Akshay Saraswat is already merged into u-boot-samsung. Please ignore the instruction about the additional patchset in v8.

How to test this patch:
1. git clone http://git.denx.de/u-boot-samsung.git
2. Apply this patchset
3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- odroid-xu3_config
4. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- -j8
5. Now you can use u-boot-dtb.bin for your downloading

WARNING: (update in V10)
It was impossible to make the u-boot image smaller than 335872 bytes, and also
I did not want to waste my time to make it fit for that restriction, because
the discussion about this issue is in progress in malining list. I used BL2 by
Suriyan Ramasami <[email protected]> which the restriction is removed for
testing. If you need the BL2 for testing, please contact me or Suriyan.

Note: If you use micro SD card for your test you have to apply the below patch additionally. This patch is needed, because micro sd card is recognized as MMC1 instead of MMC0. Additional work is needed to make it work regardless of device id.

Thanks for Sjoerd Simons. Not only did he comment my patchset, but he also made
real patch for me. I squashed some part of that patchset, and included the remainings into my patchset.

include/configs/exynos5-common.h +1 -1
index ba591e7..437eaae 100644
@@ -109,7 +109,7 @@
 
   #define CONFIG_SYS_MONITOR_BASE        0x00000000
 
-#define CONFIG_SYS_MMC_ENV_DEV         0
+#define CONFIG_SYS_MMC_ENV_DEV         1
 
   #define CONFIG_SECURE_BL1_ONLY
 

Changes for v2:
- Add a patch to add new common setup header file for Odroid X2/U3 and Odroid XU3

Changes for v3:
- Remove the patch which adds new common setup header file from v2
- Remove the wrong patch to fix GPIO information of Exynos 5800
- Remove unnecessary node from DT file
- Remove unnecessary features from config file
- Fix some trivial typos in comments

Changes for v4:
- Add MMC FIFO buffer's configuration to DT file
- Make CONFIG_OF_CONTROL be set by the target information
- Add basic document to doc/README.odroid-xu3
- Add CONFIG_CMD_EXT4 to config file
- Add environment size and offset to config file
- Add extra default environment to make bootable without modification
- Remove unnecessary features from config file

Changes for v5:
- Add the specific build instruction
- Update the information of patchset by Akshay Saraswat
- Convert /include/ to #include in DT file

Changes for v6:
- Separate out the documentation to new commit
- Remove unnecessary header file inclusuib from the board-specific setup file
- Make the function board_clock_init be declared, only when CONFIG_BOARD_EARLY_INIT_F is defined

Changes for v7:
- Fix several errata in the documentation
- Remove OF_CONTROL dependency from !SPL_BUILD

Changes for v8:
- Remove unnecessary properties in DT mmc node

Changes for v9:
- Remove the first patch which change GPIO entries order. It became useless,
after DT support is added
- Remove useless variables in the default environment
- Add the new contents to the documentation of Odroid X2/U2, instead of making new document for Odorid XU3
- Remove the detailed information in the header file, and leave the reference to the documentation

Changes for v10:
- Move the non common configs in Exynos5420 config file to each board file
- Remove the config unsets which are added to make the result image small This is needless now, because the image is not small enough even though
these
unsets are added.
- Remove redundant DT node and properties
- Remove the odroid-xu3 board file and make odroid-xu3 a variant of smdk5420
- Fix some errata in documentation
- Include the patch by Sjoerd Simons as 4th patch

Hyungwon Hwang (3):
config: exynos5420: move non common configs to specific board files
Odroid-XU3: Add support for Odroid-XU3
Odroid-XU3: Add documentation for Odroid-XU3

Sjoerd Simons (1):
Odroid-XU3: Add entry for DTS EHCI GPIO

   arch/arm/cpu/armv7/exynos/Kconfig     |  4 +++
   arch/arm/dts/Makefile                 |  3 +-
   arch/arm/dts/exynos5422-odroidxu3.dts | 49
+++++++++++++++++++++++++++++++
   board/samsung/smdk5420/Kconfig        | 13 +++++++++
   configs/odroid-xu3_defconfig          |  4 +++
   doc/README.odroid                     | 46 ++++++++++++++++++-----------
   include/configs/exynos5420-common.h   | 18 ------------
   include/configs/odroid_xu3.h          | 54
+++++++++++++++++++++++++++++++++++
   include/configs/peach-pi.h            |  6 ++++
   include/configs/peach-pit.h           |  6 ++++
   include/configs/smdk5420.h            | 11 +++++++
   11 files changed, 179 insertions(+), 35 deletions(-)
Quoted 4 lines

create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts create mode 100644 configs/odroid-xu3_defconfig create mode 100644 include/configs/odroid_xu3.h

I talked to man from the hardkernel, and he said that there is no restrictions about adding BL1/BL2/TZSW as a binary blob to the Mainline U-Boot. So I think, in this case it should be provided also with those patches. Then we have a complete board setup.

Quoted 5 lines

To Hyungwon,
There is a size limit for each patch (100KiB), so if you would like to add such binary blob, you should add it using this site:
http://www.denx.de/wiki/attach/U-Boot/TooBigPatches
after a registration.

Quoted 9 lines

If the intent is to upload a hexdump of a few of the files, that does sound like a convenient option. Very unfortunate though. At least for the trustzone file it is mostly zeroes, and seems to consist of 3 separate very small images. So perhaps we should have a tool that pieces them together?

Regards,
Simon

Yes, this makes sense.
And I think that we can modify the layout to have the order:
BL1|BL2|TZSW|U-BOOT,
then we can make one or two binnaries: signed blob and U-Boot.

Best regards,

Signature · 4 lines
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com
#20 lore raw

v10 cover Adds support for Exynos5422 odroid xu3 board

Quoted 187 lines · Przemyslaw Marczak

Hello,

Quoted 183 lines · Simon Glass

Hi,

Quoted 164 lines · Przemyslaw Marczak

Hello all,

Quoted 149 lines · Hyungwon Hwang

This is v10 of the patchset adding support Odroud XU3 board.

link to the previous version:
v2: https://www.mail-archive.com/u-boot at lists.denx.de/msg152275.html
v3: https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html
v4: https://patchwork.ozlabs.org/patch/407411/
v5: https://patchwork.ozlabs.org/patch/407941/
v6: https://patchwork.ozlabs.org/patch/408647/
v7: https://patchwork.ozlabs.org/patch/410204/
v8: https://patchwork.ozlabs.org/patch/410709/
v9: https://patchwork.ozlabs.org/patch/415514/

The patchset by Akshay Saraswat is already merged into u-boot-samsung. Please ignore the instruction about the additional patchset in v8.

How to test this patch:
1. git clone http://git.denx.de/u-boot-samsung.git
2. Apply this patchset
3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- odroid-xu3_config
4. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- -j8
5. Now you can use u-boot-dtb.bin for your downloading

WARNING: (update in V10)
It was impossible to make the u-boot image smaller than 335872 bytes, and
also
I did not want to waste my time to make it fit for that restriction, because
the discussion about this issue is in progress in malining list. I used BL2 by
Suriyan Ramasami <[email protected]> which the restriction is removed for
testing. If you need the BL2 for testing, please contact me or Suriyan.

Note: If you use micro SD card for your test you have to apply the below patch additionally. This patch is needed, because micro sd card is recognized as MMC1 instead of MMC0. Additional work is needed to make it work regardless of device id.

Thanks for Sjoerd Simons. Not only did he comment my patchset, but he also
made
real patch for me. I squashed some part of that patchset, and included the
remainings into my patchset.

include/configs/exynos5-common.h +1 -1
index ba591e7..437eaae 100644
@@ -109,7 +109,7 @@
 
   #define CONFIG_SYS_MONITOR_BASE        0x00000000
 
-#define CONFIG_SYS_MMC_ENV_DEV         0
+#define CONFIG_SYS_MMC_ENV_DEV         1
 
   #define CONFIG_SECURE_BL1_ONLY
 

Changes for v2:
- Add a patch to add new common setup header file for Odroid X2/U3 and Odroid XU3

Changes for v3:
- Remove the patch which adds new common setup header file from v2
- Remove the wrong patch to fix GPIO information of Exynos 5800
- Remove unnecessary node from DT file
- Remove unnecessary features from config file
- Fix some trivial typos in comments

Changes for v4:
- Add MMC FIFO buffer's configuration to DT file
- Make CONFIG_OF_CONTROL be set by the target information
- Add basic document to doc/README.odroid-xu3
- Add CONFIG_CMD_EXT4 to config file
- Add environment size and offset to config file
- Add extra default environment to make bootable without modification
- Remove unnecessary features from config file

Changes for v5:
- Add the specific build instruction
- Update the information of patchset by Akshay Saraswat
- Convert /include/ to #include in DT file

Changes for v6:
- Separate out the documentation to new commit
- Remove unnecessary header file inclusuib from the board-specific setup file
- Make the function board_clock_init be declared, only when CONFIG_BOARD_EARLY_INIT_F is defined

Changes for v7:
- Fix several errata in the documentation
- Remove OF_CONTROL dependency from !SPL_BUILD

Changes for v8:
- Remove unnecessary properties in DT mmc node

Changes for v9:
- Remove the first patch which change GPIO entries order. It became useless,
after DT support is added
- Remove useless variables in the default environment
- Add the new contents to the documentation of Odroid X2/U2, instead of making new document for Odorid XU3
- Remove the detailed information in the header file, and leave the reference to the documentation

Changes for v10:
- Move the non common configs in Exynos5420 config file to each board file
- Remove the config unsets which are added to make the result image small
This is needless now, because the image is not small enough even
though
these
unsets are added.
- Remove redundant DT node and properties
- Remove the odroid-xu3 board file and make odroid-xu3 a variant of smdk5420
- Fix some errata in documentation
- Include the patch by Sjoerd Simons as 4th patch

Hyungwon Hwang (3):
config: exynos5420: move non common configs to specific board files
Odroid-XU3: Add support for Odroid-XU3
Odroid-XU3: Add documentation for Odroid-XU3

Sjoerd Simons (1):
Odroid-XU3: Add entry for DTS EHCI GPIO

   arch/arm/cpu/armv7/exynos/Kconfig     |  4 +++
   arch/arm/dts/Makefile                 |  3 +-
   arch/arm/dts/exynos5422-odroidxu3.dts | 49
+++++++++++++++++++++++++++++++
   board/samsung/smdk5420/Kconfig        | 13 +++++++++
   configs/odroid-xu3_defconfig          |  4 +++
   doc/README.odroid                     | 46
++++++++++++++++++-----------
   include/configs/exynos5420-common.h   | 18 ------------
   include/configs/odroid_xu3.h          | 54
+++++++++++++++++++++++++++++++++++
   include/configs/peach-pi.h            |  6 ++++
   include/configs/peach-pit.h           |  6 ++++
   include/configs/smdk5420.h            | 11 +++++++
   11 files changed, 179 insertions(+), 35 deletions(-)
Quoted 4 lines

create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts create mode 100644 configs/odroid-xu3_defconfig create mode 100644 include/configs/odroid_xu3.h

I talked to man from the hardkernel, and he said that there is no restrictions about adding BL1/BL2/TZSW as a binary blob to the Mainline U-Boot. So I think, in this case it should be provided also with those patches. Then we have a complete board setup.

Quoted 6 lines

To Hyungwon,
There is a size limit for each patch (100KiB), so if you would like to add
such binary blob, you should add it using this site:
http://www.denx.de/wiki/attach/U-Boot/TooBigPatches
after a registration.

If the intent is to upload a hexdump of a few of the files, that does sound like a convenient option. Very unfortunate though. At least for the trustzone file it is mostly zeroes, and seems to consist of 3 separate very small images. So perhaps we should have a tool that pieces them together?

Quoted 6 lines
Quoted 3 lines

Regards,
Simon

Yes, this makes sense.
And I think that we can modify the layout to have the order:
BL1|BL2|TZSW|U-BOOT,

Yes! That is much more sane.

Quoted 1 line

then we can make one or two binnaries: signed blob and U-Boot.

Regards,
Simon

#21 lore raw

v10 cover Adds support for Exynos5422 odroid xu3 board

Dear all,

Quoted 196 lines · On Wed, 03 Dec 2014 09:31:44 -0800 Simon Glass <[email protected]> wrote:
Quoted 194 lines · Przemyslaw Marczak

Hello,

Quoted 190 lines · Simon Glass

Hi,

Quoted 171 lines · Przemyslaw Marczak

Hello all,

Quoted 156 lines · Hyungwon Hwang

This is v10 of the patchset adding support Odroud XU3 board.

link to the previous version:
v2:
https://www.mail-archive.com/u-boot at lists.denx.de/msg152275.html v3:
https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html
v4: https://patchwork.ozlabs.org/patch/407411/ v5:
https://patchwork.ozlabs.org/patch/407941/ v6:
https://patchwork.ozlabs.org/patch/408647/ v7:
https://patchwork.ozlabs.org/patch/410204/ v8:
https://patchwork.ozlabs.org/patch/410709/ v9:
https://patchwork.ozlabs.org/patch/415514/

The patchset by Akshay Saraswat is already merged into u-boot-samsung. Please ignore the instruction about the additional patchset in v8.

How to test this patch:
1. git clone http://git.denx.de/u-boot-samsung.git
2. Apply this patchset
3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- odroid-xu3_config 4. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- -j8 5. Now you can use u-boot-dtb.bin for your downloading

WARNING: (update in V10)
It was impossible to make the u-boot image smaller than 335872 bytes, and
also
I did not want to waste my time to make it fit for that restriction, because
the discussion about this issue is in progress in malining list. I used BL2 by
Suriyan Ramasami <[email protected]> which the restriction is removed for
testing. If you need the BL2 for testing, please contact me or Suriyan.

Note: If you use micro SD card for your test you have to apply the below patch additionally. This patch is needed, because micro sd card is recognized as MMC1 instead of MMC0. Additional work is needed to make it work regardless of device id.

Thanks for Sjoerd Simons. Not only did he comment my patchset, but he also
made
real patch for me. I squashed some part of that patchset, and included the
remainings into my patchset.

include/configs/exynos5-common.h +1 -1
index ba591e7..437eaae 100644
@@ -109,7 +109,7 @@
 
   #define CONFIG_SYS_MONITOR_BASE        0x00000000
 
-#define CONFIG_SYS_MMC_ENV_DEV         0
+#define CONFIG_SYS_MMC_ENV_DEV         1
 
   #define CONFIG_SECURE_BL1_ONLY
 

Changes for v2:
- Add a patch to add new common setup header file for Odroid X2/U3 and Odroid XU3

Changes for v3:
- Remove the patch which adds new common setup header file from v2
- Remove the wrong patch to fix GPIO information of Exynos 5800
- Remove unnecessary node from DT file
- Remove unnecessary features from config file
- Fix some trivial typos in comments

Changes for v4:
- Add MMC FIFO buffer's configuration to DT file
- Make CONFIG_OF_CONTROL be set by the target information
- Add basic document to doc/README.odroid-xu3
- Add CONFIG_CMD_EXT4 to config file
- Add environment size and offset to config file
- Add extra default environment to make bootable without modification
- Remove unnecessary features from config file

Changes for v5:
- Add the specific build instruction
- Update the information of patchset by Akshay Saraswat
- Convert /include/ to #include in DT file

Changes for v6:
- Separate out the documentation to new commit
- Remove unnecessary header file inclusuib from the board-specific setup file
- Make the function board_clock_init be declared, only when CONFIG_BOARD_EARLY_INIT_F is defined

Changes for v7:
- Fix several errata in the documentation
- Remove OF_CONTROL dependency from !SPL_BUILD

Changes for v8:
- Remove unnecessary properties in DT mmc node

Changes for v9:
- Remove the first patch which change GPIO entries order. It became useless,
after DT support is added
- Remove useless variables in the default environment
- Add the new contents to the documentation of Odroid X2/U2, instead of making new document for Odorid XU3
- Remove the detailed information in the header file, and leave the reference to the documentation

Changes for v10:
- Move the non common configs in Exynos5420 config file to each board file
- Remove the config unsets which are added to make the result image small
This is needless now, because the image is not small enough
even though
these
unsets are added.
- Remove redundant DT node and properties
- Remove the odroid-xu3 board file and make odroid-xu3 a variant of smdk5420
- Fix some errata in documentation
- Include the patch by Sjoerd Simons as 4th patch

Hyungwon Hwang (3):
config: exynos5420: move non common configs to specific
board files Odroid-XU3: Add support for Odroid-XU3
Odroid-XU3: Add documentation for Odroid-XU3

Sjoerd Simons (1):
Odroid-XU3: Add entry for DTS EHCI GPIO

   arch/arm/cpu/armv7/exynos/Kconfig     |  4 +++
   arch/arm/dts/Makefile                 |  3 +-
   arch/arm/dts/exynos5422-odroidxu3.dts | 49
+++++++++++++++++++++++++++++++
   board/samsung/smdk5420/Kconfig        | 13 +++++++++
   configs/odroid-xu3_defconfig          |  4 +++
   doc/README.odroid                     | 46
++++++++++++++++++-----------
   include/configs/exynos5420-common.h   | 18 ------------
   include/configs/odroid_xu3.h          | 54
+++++++++++++++++++++++++++++++++++
   include/configs/peach-pi.h            |  6 ++++
   include/configs/peach-pit.h           |  6 ++++
   include/configs/smdk5420.h            | 11 +++++++
   11 files changed, 179 insertions(+), 35 deletions(-)
Quoted 4 lines

create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts create mode 100644 configs/odroid-xu3_defconfig create mode 100644 include/configs/odroid_xu3.h

I talked to man from the hardkernel, and he said that there is no restrictions about adding BL1/BL2/TZSW as a binary blob to the Mainline U-Boot. So I think, in this case it should be provided also with those patches. Then we have a complete board setup.

Quoted 6 lines

To Hyungwon,
There is a size limit for each patch (100KiB), so if you would like to add
such binary blob, you should add it using this site:
http://www.denx.de/wiki/attach/U-Boot/TooBigPatches
after a registration.

If the intent is to upload a hexdump of a few of the files, that does sound like a convenient option. Very unfortunate though. At least for the trustzone file it is mostly zeroes, and seems to consist of 3 separate very small images. So perhaps we should have a tool that pieces them together?

Quoted 11 lines
Quoted 6 lines
Quoted 3 lines

Regards,
Simon

Yes, this makes sense.
And I think that we can modify the layout to have the order:
BL1|BL2|TZSW|U-BOOT,

Yes! That is much more sane.

Quoted 1 line

then we can make one or two binnaries: signed blob and U-Boot.

Hmm. It looks good and simple. But when using SD card for boot media, the developer must check the size of u-boot image whether the layout changes or not. As you know, the environment is saved in somewhere in the SD card (mostly fixed to some point of the front part of it). In this situation, is it worth to change the layout like that?

Quoted 2 lines

Regards,
Simon

Best regards,
Hyungwon Hwang