Hi Lothar,
On Tue, Aug 22, 2017 at 2:41 PM, Lothar Waßmann <l...@karo-electronics.de>
wrote:
> 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.
Makes sense. Will do in v2.
Regards,
Bin
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
