[U-Boot] [PATCH] mx6slevk: Use PARTUUID to specify the rootfs location

[Tom Rini]

On Mon, Oct 02, 2017 at 10:11:37AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam at nxp.com>
> 
> mx6slevk can run different kernel versions, such as NXP 4.1 or mainline.
>     
> Currently the rootfs location is passed via mmcblk number and the
> problem with this approach is that the mmcblk number for the SD
> card changes depending on the kernel version.
>     
> In order to avoid such issue, use the UUID method to specify the
> rootfs location.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam at nxp.com>
> ---
>  configs/mx6slevk_defconfig     | 1 +
>  configs/mx6slevk_spl_defconfig | 1 +
>  include/configs/mx6slevk.h     | 5 +++--
>  3 files changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/configs/mx6slevk_defconfig b/configs/mx6slevk_defconfig
> index 1c28942..641fa85 100644
> --- a/configs/mx6slevk_defconfig
> +++ b/configs/mx6slevk_defconfig
> @@ -13,6 +13,7 @@ CONFIG_CMD_BOOTZ=y
>  CONFIG_CMD_GPIO=y
>  CONFIG_CMD_I2C=y
>  CONFIG_CMD_MMC=y
> +CONFIG_CMD_PART=y
>  CONFIG_CMD_SF=y
>  CONFIG_CMD_USB=y
>  # CONFIG_CMD_SETEXPR is not set
> diff --git a/configs/mx6slevk_spl_defconfig b/configs/mx6slevk_spl_defconfig
> index 4d4f273..8fadbcc 100644
> --- a/configs/mx6slevk_spl_defconfig
> +++ b/configs/mx6slevk_spl_defconfig
> @@ -21,6 +21,7 @@ CONFIG_CMD_BOOTZ=y
>  CONFIG_CMD_GPIO=y
>  CONFIG_CMD_I2C=y
>  CONFIG_CMD_MMC=y
> +CONFIG_CMD_PART=y
>  CONFIG_CMD_SF=y
>  CONFIG_CMD_USB=y
>  CONFIG_CMD_DHCP=y
> diff --git a/include/configs/mx6slevk.h b/include/configs/mx6slevk.h
> index 25c83e8..9ddb143 100644
> --- a/include/configs/mx6slevk.h
> +++ b/include/configs/mx6slevk.h
> @@ -53,9 +53,9 @@
>  	"ip_dyn=yes\0" \
>  	"mmcdev=1\0" \
>  	"mmcpart=1\0" \
> -	"mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
> +	"finduuid=part uuid mmc 1:2 uuid\0" \
>  	"mmcargs=setenv bootargs console=${console},${baudrate} " \
> -		"root=${mmcroot}\0" \
> +		"root=PARTUUID=${uuid} rootwait rw\0" \

Shouldn't mmcpart be changed to 2 here as well and finduud be part uuid
${mmcdev}:${mmcpart} ?  This would also make things a bit more
scriptable for use in other distros.

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