* [U-Boot] [PATCH v2 2/6] pico-imx6ul: Use PARTUUID to specify the rootfs location
2017-11-16 11:59 [U-Boot] [PATCH v2 1/6] pico-imx6ul: Move dfu_alt_info to CONFIG_DFU_ENV_SETTINGS variable Otavio Salvador
@ 2017-11-16 11:59 ` Otavio Salvador
2017-11-16 18:11 ` Fabio Estevam
2017-11-16 11:59 ` [U-Boot] [PATCH v2 3/6] pico-imx6ul: Add boot and rootfs dfu_alt_info Otavio Salvador
` (5 subsequent siblings)
6 siblings, 1 reply; 10+ messages in thread
From: Otavio Salvador @ 2017-11-16 11:59 UTC (permalink / raw)
To: u-boot
From: Fabio Berton <fabio.berton@ossystems.com.br>
Currently the rootfs location is passed via mmcblk number and using
the UUID method to specify the rootfs location is a better approach
working even if mmcblk number for the eMMC changes depending on the
kernel versions.
Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
Changes in v2:
- fix typo (s/eMMM/eMMC/) - Fabio Estevam
configs/pico-imx6ul_defconfig | 1 +
include/configs/pico-imx6ul.h | 6 +++---
2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/configs/pico-imx6ul_defconfig b/configs/pico-imx6ul_defconfig
index 09b36ccb09..e00f9f34a3 100644
--- a/configs/pico-imx6ul_defconfig
+++ b/configs/pico-imx6ul_defconfig
@@ -13,6 +13,7 @@ CONFIG_CMD_DFU=y
# CONFIG_CMD_FLASH is not set
CONFIG_CMD_GPIO=y
CONFIG_CMD_MMC=y
+CONFIG_CMD_PART=y
CONFIG_CMD_USB=y
CONFIG_CMD_USB_MASS_STORAGE=y
CONFIG_CMD_DHCP=y
diff --git a/include/configs/pico-imx6ul.h b/include/configs/pico-imx6ul.h
index d4b29e3950..c775c32690 100644
--- a/include/configs/pico-imx6ul.h
+++ b/include/configs/pico-imx6ul.h
@@ -61,14 +61,15 @@
"fdt_addr=0x83000000\0" \
"mmcdev="__stringify(CONFIG_SYS_MMC_ENV_DEV)"\0" \
"mmcpart=" __stringify(CONFIG_SYS_MMC_IMG_LOAD_PART) "\0" \
- "mmcroot=" CONFIG_MMCROOT " rootwait rw\0" \
"mmcautodetect=yes\0" \
CONFIG_DFU_ENV_SETTINGS \
+ "finduuid=part uuid mmc 0:2 uuid\0" \
"mmcargs=setenv bootargs console=${console},${baudrate} " \
- "root=${mmcroot}\0" \
+ "root=PARTUUID=${uuid} rootwait rw\0" \
"loadimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${image}\0" \
"loadfdt=fatload mmc ${mmcdev}:${mmcpart} ${fdt_addr} ${fdt_file}\0" \
"mmcboot=echo Booting from mmc ...; " \
+ "run finduuid; " \
"run mmcargs; " \
"if run loadfdt; then " \
"bootz ${loadaddr} - ${fdt_addr}; " \
@@ -147,6 +148,5 @@
#define CONFIG_SYS_MMC_ENV_DEV 0
#define CONFIG_SYS_MMC_ENV_PART 0
-#define CONFIG_MMCROOT "/dev/mmcblk0p2"
#endif /* __PICO_IMX6UL_CONFIG_H */
--
2.15.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [U-Boot] [PATCH v2 2/6] pico-imx6ul: Use PARTUUID to specify the rootfs location
2017-11-16 11:59 ` [U-Boot] [PATCH v2 2/6] pico-imx6ul: Use PARTUUID to specify the rootfs location Otavio Salvador
@ 2017-11-16 18:11 ` Fabio Estevam
0 siblings, 0 replies; 10+ messages in thread
From: Fabio Estevam @ 2017-11-16 18:11 UTC (permalink / raw)
To: u-boot
On Thu, Nov 16, 2017 at 9:59 AM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> From: Fabio Berton <fabio.berton@ossystems.com.br>
>
> Currently the rootfs location is passed via mmcblk number and using
> the UUID method to specify the rootfs location is a better approach
> working even if mmcblk number for the eMMC changes depending on the
> kernel versions.
>
> Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
Reviewed-by: Fabio Estevam <fabio.estevam@nxp.com>
(You should have kept my Reviewed-by tag on the other patches that you
resent without changes)
^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [PATCH v2 3/6] pico-imx6ul: Add boot and rootfs dfu_alt_info
2017-11-16 11:59 [U-Boot] [PATCH v2 1/6] pico-imx6ul: Move dfu_alt_info to CONFIG_DFU_ENV_SETTINGS variable Otavio Salvador
2017-11-16 11:59 ` [U-Boot] [PATCH v2 2/6] pico-imx6ul: Use PARTUUID to specify the rootfs location Otavio Salvador
@ 2017-11-16 11:59 ` Otavio Salvador
2017-11-16 11:59 ` [U-Boot] [PATCH v2 4/6] pico-imx6ul: Define partition layout in the environment Otavio Salvador
` (4 subsequent siblings)
6 siblings, 0 replies; 10+ messages in thread
From: Otavio Salvador @ 2017-11-16 11:59 UTC (permalink / raw)
To: u-boot
From: Fabio Berton <fabio.berton@ossystems.com.br>
Change dfu_alt_info variable to use the following altsetting:
uboot: To flash raw U-Boot
/zImage: boot: To copy kernel image
/imx6ul-pico-hobbit.dtb: To copy dtb file
rootfs: To copy rootfs
List the currently attached DFU capable USB devices running:
sudo dfu-util -l
Flash U-Boot:
sudo dfu-util -D u-boot.imx -a uboot
Copy boot files:
sudo dfu-util -D zImage -a /zImage
sudo dfu-util -D imx6ul-pico-hobbit.dtb -a /imx6ul-pico-hobbit.dtb
Flash rootfs:
sudo dfu-util -D rootfs.ext4 -a rootfs
To copy boot files you need to have a formated mmc 0:1 partition. To
format with ext4 filesystem you can use ums.
Run on target:
ums 0 mmc 0
and on host:
sudo mkfs.ext4 /dev/sdx1
Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
Changes in v2: None
include/configs/pico-imx6ul.h | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/include/configs/pico-imx6ul.h b/include/configs/pico-imx6ul.h
index c775c32690..6a19861da7 100644
--- a/include/configs/pico-imx6ul.h
+++ b/include/configs/pico-imx6ul.h
@@ -48,7 +48,11 @@
#define DFU_DEFAULT_POLL_TIMEOUT 300
#define CONFIG_DFU_ENV_SETTINGS \
- "dfu_alt_info=boot raw 0x2 0x400 mmcpart 1\0" \
+ "dfu_alt_info=uboot raw 0x2 0x400 mmcpart 1;" \
+ "boot part 0 1;" \
+ "/zImage ext4 0 1;" \
+ "/imx6ul-pico-hobbit.dtb ext4 0 1;" \
+ "rootfs part 0 2\0" \
#define CONFIG_SYS_MMC_IMG_LOAD_PART 1
--
2.15.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [U-Boot] [PATCH v2 4/6] pico-imx6ul: Define partition layout in the environment
2017-11-16 11:59 [U-Boot] [PATCH v2 1/6] pico-imx6ul: Move dfu_alt_info to CONFIG_DFU_ENV_SETTINGS variable Otavio Salvador
2017-11-16 11:59 ` [U-Boot] [PATCH v2 2/6] pico-imx6ul: Use PARTUUID to specify the rootfs location Otavio Salvador
2017-11-16 11:59 ` [U-Boot] [PATCH v2 3/6] pico-imx6ul: Add boot and rootfs dfu_alt_info Otavio Salvador
@ 2017-11-16 11:59 ` Otavio Salvador
2017-11-16 18:11 ` Fabio Estevam
2017-11-16 11:59 ` [U-Boot] [PATCH v2 5/6] pico-imx6ul: Add function to create gpt partitions Otavio Salvador
` (3 subsequent siblings)
6 siblings, 1 reply; 10+ messages in thread
From: Otavio Salvador @ 2017-11-16 11:59 UTC (permalink / raw)
To: u-boot
From: Fabio Berton <fabio.berton@ossystems.com.br>
Create layout with a boot 16MiB partition and rootfs with remain
space.
Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
Changes in v2:
- fix typo (s/Mib/MiB/) - Fabio Estevam
include/configs/pico-imx6ul.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/include/configs/pico-imx6ul.h b/include/configs/pico-imx6ul.h
index 6a19861da7..1ea79b9c18 100644
--- a/include/configs/pico-imx6ul.h
+++ b/include/configs/pico-imx6ul.h
@@ -68,6 +68,9 @@
"mmcautodetect=yes\0" \
CONFIG_DFU_ENV_SETTINGS \
"finduuid=part uuid mmc 0:2 uuid\0" \
+ "partitions=" \
+ "uuid_disk=${uuid_gpt_disk};" \
+ "name=boot,size=16MiB;name=rootfs,size=0,uuid=${uuid_gpt_rootfs}\0" \
"mmcargs=setenv bootargs console=${console},${baudrate} " \
"root=PARTUUID=${uuid} rootwait rw\0" \
"loadimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${image}\0" \
--
2.15.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [U-Boot] [PATCH v2 5/6] pico-imx6ul: Add function to create gpt partitions
2017-11-16 11:59 [U-Boot] [PATCH v2 1/6] pico-imx6ul: Move dfu_alt_info to CONFIG_DFU_ENV_SETTINGS variable Otavio Salvador
` (2 preceding siblings ...)
2017-11-16 11:59 ` [U-Boot] [PATCH v2 4/6] pico-imx6ul: Define partition layout in the environment Otavio Salvador
@ 2017-11-16 11:59 ` Otavio Salvador
2017-11-16 11:59 ` [U-Boot] [PATCH v2 6/6] pico-imx6ul: Use FS_GENERIC load command Otavio Salvador
` (2 subsequent siblings)
6 siblings, 0 replies; 10+ messages in thread
From: Otavio Salvador @ 2017-11-16 11:59 UTC (permalink / raw)
To: u-boot
From: Fabio Berton <fabio.berton@ossystems.com.br>
Command run setup_emmc will create gpt partitions for mmc 0 device
based on patitions variable.
Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
Changes in v2: None
configs/pico-imx6ul_defconfig | 1 +
include/configs/pico-imx6ul.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/configs/pico-imx6ul_defconfig b/configs/pico-imx6ul_defconfig
index e00f9f34a3..f28f9b7a4e 100644
--- a/configs/pico-imx6ul_defconfig
+++ b/configs/pico-imx6ul_defconfig
@@ -12,6 +12,7 @@ CONFIG_CMD_MEMTEST=y
CONFIG_CMD_DFU=y
# CONFIG_CMD_FLASH is not set
CONFIG_CMD_GPIO=y
+CONFIG_CMD_GPT=y
CONFIG_CMD_MMC=y
CONFIG_CMD_PART=y
CONFIG_CMD_USB=y
diff --git a/include/configs/pico-imx6ul.h b/include/configs/pico-imx6ul.h
index 1ea79b9c18..310fceb14b 100644
--- a/include/configs/pico-imx6ul.h
+++ b/include/configs/pico-imx6ul.h
@@ -71,6 +71,7 @@
"partitions=" \
"uuid_disk=${uuid_gpt_disk};" \
"name=boot,size=16MiB;name=rootfs,size=0,uuid=${uuid_gpt_rootfs}\0" \
+ "setup_emmc=gpt write mmc 0 $partitions; reset;\0" \
"mmcargs=setenv bootargs console=${console},${baudrate} " \
"root=PARTUUID=${uuid} rootwait rw\0" \
"loadimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${image}\0" \
--
2.15.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [U-Boot] [PATCH v2 6/6] pico-imx6ul: Use FS_GENERIC load command
2017-11-16 11:59 [U-Boot] [PATCH v2 1/6] pico-imx6ul: Move dfu_alt_info to CONFIG_DFU_ENV_SETTINGS variable Otavio Salvador
` (3 preceding siblings ...)
2017-11-16 11:59 ` [U-Boot] [PATCH v2 5/6] pico-imx6ul: Add function to create gpt partitions Otavio Salvador
@ 2017-11-16 11:59 ` Otavio Salvador
2017-11-17 9:34 ` [U-Boot] [PATCH v2 1/6] pico-imx6ul: Move dfu_alt_info to CONFIG_DFU_ENV_SETTINGS variable Lukasz Majewski
2017-11-20 8:56 ` Stefano Babic
6 siblings, 0 replies; 10+ messages in thread
From: Otavio Salvador @ 2017-11-16 11:59 UTC (permalink / raw)
To: u-boot
From: Fabio Berton <fabio.berton@ossystems.com.br>
Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
Changes in v2: None
include/configs/pico-imx6ul.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/configs/pico-imx6ul.h b/include/configs/pico-imx6ul.h
index 310fceb14b..f821b67c72 100644
--- a/include/configs/pico-imx6ul.h
+++ b/include/configs/pico-imx6ul.h
@@ -74,8 +74,8 @@
"setup_emmc=gpt write mmc 0 $partitions; reset;\0" \
"mmcargs=setenv bootargs console=${console},${baudrate} " \
"root=PARTUUID=${uuid} rootwait rw\0" \
- "loadimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${image}\0" \
- "loadfdt=fatload mmc ${mmcdev}:${mmcpart} ${fdt_addr} ${fdt_file}\0" \
+ "loadimage=load mmc ${mmcdev}:${mmcpart} ${loadaddr} ${image}\0" \
+ "loadfdt=load mmc ${mmcdev}:${mmcpart} ${fdt_addr} ${fdt_file}\0" \
"mmcboot=echo Booting from mmc ...; " \
"run finduuid; " \
"run mmcargs; " \
--
2.15.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [U-Boot] [PATCH v2 1/6] pico-imx6ul: Move dfu_alt_info to CONFIG_DFU_ENV_SETTINGS variable
2017-11-16 11:59 [U-Boot] [PATCH v2 1/6] pico-imx6ul: Move dfu_alt_info to CONFIG_DFU_ENV_SETTINGS variable Otavio Salvador
` (4 preceding siblings ...)
2017-11-16 11:59 ` [U-Boot] [PATCH v2 6/6] pico-imx6ul: Use FS_GENERIC load command Otavio Salvador
@ 2017-11-17 9:34 ` Lukasz Majewski
2017-11-20 8:56 ` Stefano Babic
6 siblings, 0 replies; 10+ messages in thread
From: Lukasz Majewski @ 2017-11-17 9:34 UTC (permalink / raw)
To: u-boot
On Thu, 16 Nov 2017 09:59:44 -0200
Otavio Salvador <otavio@ossystems.com.br> wrote:
> From: Fabio Berton <fabio.berton@ossystems.com.br>
>
> Create CONFIG_DFU_ENV_SETTINGS to set dfu environment settings to
> improve human readable code.
>
> Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
>
> Changes in v2: None
>
> include/configs/pico-imx6ul.h | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/include/configs/pico-imx6ul.h
> b/include/configs/pico-imx6ul.h index 75658fd489..d4b29e3950 100644
> --- a/include/configs/pico-imx6ul.h
> +++ b/include/configs/pico-imx6ul.h
> @@ -47,6 +47,9 @@
> #define CONFIG_SYS_DFU_DATA_BUF_SIZE SZ_16M
> #define DFU_DEFAULT_POLL_TIMEOUT 300
>
> +#define CONFIG_DFU_ENV_SETTINGS \
> + "dfu_alt_info=boot raw 0x2 0x400 mmcpart 1\0" \
> +
> #define CONFIG_SYS_MMC_IMG_LOAD_PART 1
>
> #define CONFIG_EXTRA_ENV_SETTINGS \
> @@ -60,7 +63,7 @@
> "mmcpart=" __stringify(CONFIG_SYS_MMC_IMG_LOAD_PART) "\0" \
> "mmcroot=" CONFIG_MMCROOT " rootwait rw\0" \
> "mmcautodetect=yes\0" \
> - "dfu_alt_info=boot raw 0x2 0x400 mmcpart 1\0" \
> + CONFIG_DFU_ENV_SETTINGS \
> "mmcargs=setenv bootargs console=${console},${baudrate} " \
> "root=${mmcroot}\0" \
> "loadimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr}
> ${image}\0" \
Reviewed-by: Lukasz Majewski <lukma@denx.de>
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171117/697484aa/attachment.sig>
^ permalink raw reply [flat|nested] 10+ messages in thread* [U-Boot] [PATCH v2 1/6] pico-imx6ul: Move dfu_alt_info to CONFIG_DFU_ENV_SETTINGS variable
2017-11-16 11:59 [U-Boot] [PATCH v2 1/6] pico-imx6ul: Move dfu_alt_info to CONFIG_DFU_ENV_SETTINGS variable Otavio Salvador
` (5 preceding siblings ...)
2017-11-17 9:34 ` [U-Boot] [PATCH v2 1/6] pico-imx6ul: Move dfu_alt_info to CONFIG_DFU_ENV_SETTINGS variable Lukasz Majewski
@ 2017-11-20 8:56 ` Stefano Babic
6 siblings, 0 replies; 10+ messages in thread
From: Stefano Babic @ 2017-11-20 8:56 UTC (permalink / raw)
To: u-boot
Hi Otavio,
On 16/11/2017 12:59, Otavio Salvador wrote:
> From: Fabio Berton <fabio.berton@ossystems.com.br>
>
> Create CONFIG_DFU_ENV_SETTINGS to set dfu environment settings to
> improve human readable code.
>
> Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
>
Whole series applied to u-boot-imx, thanks !
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 10+ messages in thread