[PATCH] net: Fix the displayed value of bytes transferred
Siddharth Vadapalli
s-vadapalli at ti.com
Mon Aug 14 06:59:50 CEST 2023
On 14/08/23 10:06, Siddharth Vadapalli wrote:
> Hello Tom,
>
> On 11/08/23 21:45, Tom Rini wrote:
>> On Fri, Aug 11, 2023 at 10:49:23AM +0530, Siddharth Vadapalli wrote:
>>> Ravi,
>>>
>>> On 10/08/23 17:00, Ravi Gunasekaran wrote:
...
>>
>> Uh, maybe I'm just missing something, but I think there's two things.
>> First, this should be "%u" for "unsigned decimal". Second,
>> doc/develop/printf.rst needs to be fixed since:
>> int %d, %x
>> unsigned int %d, %x
>>
>> Should is wrong and should say %u, %x, because, well, that's what would
>> be correct, yes?
>
> Thank you for reviewing the patch. Yes, %u works well and can print u32 variable
> accurately. I tested it for 0xffffffff. %d prints -1 for the same. So, %lu isn't
> necessary and %u is sufficient. I will replace %lu with %u and post the v2
> patch. Additionally, I will include a patch in the v2 series to update the
> Documentation as pointed out by you.
I have posted the v2 series at:
https://patchwork.ozlabs.org/project/uboot/list/?series=368661&state=%2A&archive=both
>
>>
>
--
Regards,
Siddharth.
More information about the U-Boot
mailing list