Message ID | 20190712112534.25684-1-erosca@de.adit-jv.com |
---|---|
State | Superseded |
Delegated to: | Tom Rini |
Headers | show |
Series | Fixes and improvements in BCB and Android docs | expand |
On Fri, Jul 12, 2019 at 2:26 PM Eugeniu Rosca <erosca@de.adit-jv.com> 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. minor: could you please add more details about particular issues that were addressed in this commit, because current commit message contains only some insights of a glitch in the merge workflow instead of overview of changes. > > [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> > --- > cmd/bcb.c | 23 ++++++++++++----------- > 1 file changed, 12 insertions(+), 11 deletions(-) > > diff --git a/cmd/bcb.c b/cmd/bcb.c > index c7138a5179a9..9626f2c69e34 100644 > --- a/cmd/bcb.c > +++ b/cmd/bcb.c > @@ -83,7 +83,7 @@ err: > return -1; > } > > -static int bcb_field_get(char *name, char **field, int *size) > +static int bcb_field_get(char *name, char **fieldp, int *sizep) > { > if (!strcmp(name, "command")) { > *fieldp = bcb.command; > @@ -108,8 +108,8 @@ static int bcb_field_get(char *name, char **field, int *size) > return 0; > } > > -static int > -do_bcb_load(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) > +static int do_bcb_load(cmd_tbl_t *cmdtp, int flag, int argc, > + char * const argv[]) > { > struct blk_desc *desc; > disk_partition_t info; > @@ -119,28 +119,28 @@ do_bcb_load(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) > > ret = blk_get_device_by_str("mmc", argv[1], &desc); > if (ret < 0) > - goto err_1; > + goto err_read_fail; > > part = simple_strtoul(argv[2], &endp, 0); > if (*endp == '\0') { > ret = part_get_info(desc, part, &info); > if (ret) > - goto err_1; > + goto err_read_fail; > } else { > part = part_get_info_by_name(desc, argv[2], &info); > if (part < 0) { > ret = part; > - goto err_1; > + goto err_read_fail; > } > } > > cnt = DIV_ROUND_UP(sizeof(struct bootloader_message), info.blksz); > if (cnt > info.size) > - goto err_2; > + goto err_too_small; > > if (blk_dread(desc, info.start, cnt, &bcb) != cnt) { > ret = -EIO; > - goto err_1; > + goto err_read_fail; > } > > bcb_dev = desc->devnum; > @@ -148,10 +148,10 @@ do_bcb_load(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) > debug("%s: Loaded from mmc %d:%d\n", __func__, bcb_dev, bcb_part); > > return CMD_RET_SUCCESS; > -err_1: > +err_read_fail: > printf("Error: mmc %s:%s read failed (%d)\n", argv[1], argv[2], ret); > goto err; > -err_2: > +err_too_small: > printf("Error: mmc %s:%s too small!", argv[1], argv[2]); > goto err; > err: > @@ -304,7 +304,8 @@ static int do_bcb(cmd_tbl_t *cmdtp, int flag, int argc, char *const argv[]) > return CMD_RET_USAGE; > > if (bcb_is_misused(argc, argv)) { > - /* We try to improve the user experience by reporting the > + /* > + * We try to improve the user experience by reporting the > * root-cause of misusage, so don't return CMD_RET_USAGE, > * since the latter prints out the full-blown help text > */ > -- > 2.22.0 > With my comment addressed: Reviewed-by: Igor Opaniuk <igor.opaniuk@gmail.com>
On Fri, Jul 12, 2019 at 03:05:59PM +0300, Igor Opaniuk wrote: > On Fri, Jul 12, 2019 at 2:26 PM Eugeniu Rosca <erosca@de.adit-jv.com> 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. > > minor: could you please add more details about particular issues that > were addressed in this commit, because current commit message > contains only some insights of a glitch in the merge workflow instead of > overview of changes. Fixed in https://patchwork.ozlabs.org/patch/1131359/ ("[v2,5/5] cmd: bcb: Apply non-functional refinements"). Thanks!
Superseded by https://patchwork.ozlabs.org/patch/1131359/ ("[v2,5/5] cmd: bcb: Apply non-functional refinements")
diff --git a/cmd/bcb.c b/cmd/bcb.c index c7138a5179a9..9626f2c69e34 100644 --- a/cmd/bcb.c +++ b/cmd/bcb.c @@ -83,7 +83,7 @@ err: return -1; } -static int bcb_field_get(char *name, char **field, int *size) +static int bcb_field_get(char *name, char **fieldp, int *sizep) { if (!strcmp(name, "command")) { *fieldp = bcb.command; @@ -108,8 +108,8 @@ static int bcb_field_get(char *name, char **field, int *size) return 0; } -static int -do_bcb_load(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) +static int do_bcb_load(cmd_tbl_t *cmdtp, int flag, int argc, + char * const argv[]) { struct blk_desc *desc; disk_partition_t info; @@ -119,28 +119,28 @@ do_bcb_load(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) ret = blk_get_device_by_str("mmc", argv[1], &desc); if (ret < 0) - goto err_1; + goto err_read_fail; part = simple_strtoul(argv[2], &endp, 0); if (*endp == '\0') { ret = part_get_info(desc, part, &info); if (ret) - goto err_1; + goto err_read_fail; } else { part = part_get_info_by_name(desc, argv[2], &info); if (part < 0) { ret = part; - goto err_1; + goto err_read_fail; } } cnt = DIV_ROUND_UP(sizeof(struct bootloader_message), info.blksz); if (cnt > info.size) - goto err_2; + goto err_too_small; if (blk_dread(desc, info.start, cnt, &bcb) != cnt) { ret = -EIO; - goto err_1; + goto err_read_fail; } bcb_dev = desc->devnum; @@ -148,10 +148,10 @@ do_bcb_load(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) debug("%s: Loaded from mmc %d:%d\n", __func__, bcb_dev, bcb_part); return CMD_RET_SUCCESS; -err_1: +err_read_fail: printf("Error: mmc %s:%s read failed (%d)\n", argv[1], argv[2], ret); goto err; -err_2: +err_too_small: printf("Error: mmc %s:%s too small!", argv[1], argv[2]); goto err; err: @@ -304,7 +304,8 @@ static int do_bcb(cmd_tbl_t *cmdtp, int flag, int argc, char *const argv[]) return CMD_RET_USAGE; if (bcb_is_misused(argc, argv)) { - /* We try to improve the user experience by reporting the + /* + * We try to improve the user experience by reporting the * root-cause of misusage, so don't return CMD_RET_USAGE, * since the latter prints out the full-blown help text */
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. [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> --- cmd/bcb.c | 23 ++++++++++++----------- 1 file changed, 12 insertions(+), 11 deletions(-)