On Mon, Oct 02, 2017 at 10:11:37AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.este...@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.este...@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

Attachment: signature.asc
Description: PGP signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to