On Fri, Jul 19, 2019 at 11:26:14PM +0200, Eugeniu Rosca wrote: > These have been reported by Simon in [1] and fixed in [2]. > However, since [1] has already been pushed to u-boot/master, the > improvements incorporated in [2] are now extracted and resubmitted. > > The changes are in the area of coding style and best practices: > * s/field/fieldp/, s/size/sizep/, to convey that the variables return > an output to the caller > * s/err_1/err_read_fail/, s/err_2/err_too_small/, to be more descriptive > * Made sure 'static int do_bcb_load' appears on the same line > * Placed a `/*` on top of multi-line comment > > [1] https://patchwork.ozlabs.org/patch/1104244/#2200259 > [2] https://patchwork.ozlabs.org/cover/1128661/ > ("[v4,0/4] Add 'bcb' command to read/modify/write Android BCB") > > Fixes: db7b7a05b267 ("cmd: Add 'bcb' command to read/modify/write BCB fields") > Reported-by: Simon Glass <s...@chromium.org> > Signed-off-by: Eugeniu Rosca <ero...@de.adit-jv.com> > Reviewed-by: Sam Protsenko <semen.protse...@linaro.org>
Applied to u-boot/master, thanks! -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
