[U-Boot] [PATCH v2 4/4] distro_bootcmd: Add support for booting from ubifs

[Stephen Warren]

On 08/22/2015 11:04 AM, Hans de Goede wrote:
> From: Roy Spliet <r.spliet at ultimaker.com>
> 
> Under the assumptions of having a UBI volume called boot, containing
> a ubifs filesystem.
> 
> Signed-off-by: Hans de Goede <hdegoede at redhat.com>

I'd expect the person in the "From:" line above to have an s-o-b line
too. If the patch has changed so much that isn't appropriate, perhaps
change the patch's git author to you and just mentioned "Based on work
by: Roy ..."?

> diff --git a/include/config_distro_bootcmd.h b/include/config_distro_bootcmd.h

> +#ifdef CONFIG_CMD_UBIFS
> +#define BOOTENV_SHARED_UBIFS \
> +	"ubifs_boot=" \
> +		"if ubi part UBI && ubifsmount ubi${devnum}:boot; then "  \

I don't know ubifs well enough to know the implications of that
particular set of ubi commands, but as far as the
impact-on/integration-into the bootcmd.h file, this patch looks fine.

Acked-by: Stephen Warren <swarren at nvidia.com>