[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