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 <[email protected]>
Applied to u-boot/next, thanks! -- Tom
signature.asc
Description: PGP signature
