[U-Boot] [PATCH v2] dfu: thor: fix: Modify dfu_get_alt() function to support absolute paths
Lukasz Majewski
l.majewski at samsung.com
Fri Nov 14 11:39:48 CET 2014
Hi Lukasz,
> 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>
> Reviewed-by: Marek Vasut <marex at denx.de>
> ---
> Changes for v2:
> - Fix patch to path in comment
> ---
> 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..deafda2 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 path must be checked for equality
> + */
> +
> + str = strstr(dfu->name, name);
> + if (!str)
> + continue;
> +
> + /*
> + * Check if matching substring is the last
> element of
> + * dfu->name (uImage)
> + */
> + if (strlen(dfu->name) ==
> + ((str - dfu->name) + strlen(name)))
> + return dfu->alt;
> + }
> }
>
> return -ENODEV;
Applied to u-boot-dfu repository. Thanks
--
Best regards,
Lukasz Majewski
Samsung R&D Institute Poland (SRPOL) | Linux Platform Group
More information about the U-Boot
mailing list