[PATCH] common: dfu: Remove leading space characters

Tom Rini trini at konsulko.com
Thu Jun 1 17:25:15 CEST 2023


On Thu, May 18, 2023 at 12:06:23PM +0530, Ravi Gunasekaran wrote:

> As per [1], dfu_alt_info is mentioned to be as semicolon separated
> string of information on each alternate and the parsing logic in
> the dfu.c is based on this.
> 
> Typically, the dfu_alt_info_* is defined in .h files as preprocessor
> macros with 'alt' info separated by semicolon.
> 
> But when dfu_alt_info_* is added in the environment files(.env)
> the script at "scripts/env2string.awk" converts a newline to space.
> Thus adding a space character after semicolon. This results in
> incorrect parsing in dfu.c which is based on the information that
> 'alt' info are only semicolon separated.
> 
> One option is to add dfu_alt_info_* variable in .env in single line.
> But there is possiblity for it to exceed the line length limit.
> So update the parsing logic to remove leading space characters
> before adding to the dfu list.
> 
> [1]: https://u-boot.readthedocs.io/en/latest/usage/dfu.html
> 
> Signed-off-by: Ravi Gunasekaran <r-gunasekaran at ti.com>

Applied to u-boot/next, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230601/0c8dbcea/attachment.sig>


More information about the U-Boot mailing list