[U-Boot] [PATCH] dfu: thor: fix: Modify dfu_get_alt() function to support absolute paths

Marek Vasut marex at denx.de
Tue Nov 4 18:23:39 CET 2014


On Tuesday, November 04, 2014 at 05:25:43 PM, Lukasz Majewski wrote:
> Recently the ext4 file system imposed passing absolute path with its file
> name parameter.
> As a result dfu_alt_info env variable has been modified to provide absolute
> path when ext4 file system is accessed (e.g. /uImage ext4 0 2;).
> 
> Unfortunately, lthor flashing program provides plain file name (like
> uImage) and hence those two file names do not match anymore.
> 
> Presented commit also allows lthor to write files to sub directories (like
> /boot/bin/uImage).
> 
> Signed-off-by: Lukasz Majewski <l.majewski at samsung.com>
> ---
>  drivers/dfu/dfu.c | 29 +++++++++++++++++++++++++++--
>  1 file changed, 27 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/dfu/dfu.c b/drivers/dfu/dfu.c
> index 55e6a83..1e9ccdc 100644
> --- a/drivers/dfu/dfu.c
> +++ b/drivers/dfu/dfu.c
> @@ -544,10 +544,35 @@ struct dfu_entity *dfu_get_entity(int alt)
>  int dfu_get_alt(char *name)
>  {
>  	struct dfu_entity *dfu;
> +	char *str;
> 
>  	list_for_each_entry(dfu, &dfu_list, list) {
> -		if (!strncmp(dfu->name, name, strlen(dfu->name)))
> -			return dfu->alt;
> +		if (dfu->name[0] != '/') {
> +			if (!strncmp(dfu->name, name, strlen(dfu->name)))
> +				return dfu->alt;
> +		} else {
> +			/*
> +			 * One must also consider absolute path
> +			 * (/boot/bin/uImage) available at dfu->name when
> +			 * compared "plain" file name (uImage)
> +			 *
> +			 * It is the case for e.g. thor gadget where lthor SW
> +			 * sends only the file name, so only the very last part
> +			 * of patch must be checked for equality

I think this patch needs patching using s/patch/path/ ;-)

Otherwise I don't see much issues.

Best regards,
Marek Vasut


More information about the U-Boot mailing list