diff options
author | Eugeniu Rosca <roscaeugeniu@gmail.com> | 2019-07-19 23:26:14 +0200 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2019-07-24 13:16:29 -0400 |
commit | 37f1811a0b31005284e8514955e7bf7e174cc506 (patch) | |
tree | fcafadd87d3d827bfbb7fbdf7b4d1d0eeff49dfe /disk | |
parent | 739168cfdc18690273e24a34f0360a61e6658424 (diff) |
cmd: bcb: Apply non-functional refinements
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 <sjg@chromium.org>
Signed-off-by: Eugeniu Rosca <erosca@de.adit-jv.com>
Reviewed-by: Sam Protsenko <semen.protsenko@linaro.org>
Diffstat (limited to 'disk')
0 files changed, 0 insertions, 0 deletions