Hi,
On Mon, 21 Aug 2017 21:53:41 -0700 Bin Meng wrote:
> There are already two macros LBAF and LBAFU to aid formatted print
> with lbaint_t variables. Now let's add another one LBAFD and use it
> in the common block command codes.
>
> Signed-off-by: Bin Meng <bmeng...@gmail.com>
> ---
>
> cmd/blk_common.c | 10 ++++------
> include/blk.h | 1 +
> 2 files changed, 5 insertions(+), 6 deletions(-)
>
> diff --git a/cmd/blk_common.c b/cmd/blk_common.c
> index 86c75e7..2626eef 100644
> --- a/cmd/blk_common.c
> +++ b/cmd/blk_common.c
> @@ -68,9 +68,8 @@ int blk_common_cmd(int argc, char * const argv[], enum
> if_type if_type,
> ulong cnt = simple_strtoul(argv[4], NULL, 16);
> ulong n;
>
> - printf("\n%s read: device %d block # %lld, count %ld
> ... ",
> - if_name, *cur_devnump, (unsigned long long)blk,
> - cnt);
> + printf("\n%s read: device %d block # "LBAFD", count %ld
> ... ",
> + if_name, *cur_devnump, blk, cnt);
>
> n = blk_read_devnum(if_type, *cur_devnump, blk, cnt,
> (ulong *)addr);
> @@ -84,9 +83,8 @@ int blk_common_cmd(int argc, char * const argv[], enum
> if_type if_type,
> ulong cnt = simple_strtoul(argv[4], NULL, 16);
> ulong n;
>
> - printf("\n%s write: device %d block # %lld, count %ld
> ... ",
> - if_name, *cur_devnump, (unsigned long long)blk,
> - cnt);
> + printf("\n%s write: device %d block # "LBAFD", count
> %ld ... ",
> + if_name, *cur_devnump, blk, cnt);
>
> n = blk_write_devnum(if_type, *cur_devnump, blk, cnt,
> (ulong *)addr);
>
Are there devices that have valid negative block numbers?
Otherwise using LBAFU would be fine here.
Lothar Waßmann
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
