Re: [PATCH] net: Fix the displayed value of bytes transferred

2023-08-13 Thread Siddharth Vadapalli
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 th

Re: [PATCH] net: Fix the displayed value of bytes transferred

2023-08-13 Thread Siddharth Vadapalli
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: >>> Siddharth, >>> >>> On 8/10/23 2:45 PM, Siddharth Vadapalli wrote: In the case of NETLOOP_SUCCESS, the decimal valu

Re: [PATCH] net: Fix the displayed value of bytes transferred

2023-08-11 Thread Tom Rini
On Fri, Aug 11, 2023 at 10:49:23AM +0530, Siddharth Vadapalli wrote: > Ravi, > > On 10/08/23 17:00, Ravi Gunasekaran wrote: > > Siddharth, > > > > On 8/10/23 2:45 PM, Siddharth Vadapalli wrote: > >> In the case of NETLOOP_SUCCESS, the decimal value of the u32 variable > >> "net_boot_file_size" is

Re: [PATCH] net: Fix the displayed value of bytes transferred

2023-08-10 Thread Siddharth Vadapalli
Ravi, On 10/08/23 17:00, Ravi Gunasekaran wrote: > Siddharth, > > On 8/10/23 2:45 PM, Siddharth Vadapalli wrote: >> In the case of NETLOOP_SUCCESS, the decimal value of the u32 variable >> "net_boot_file_size" is printed using "%d", resulting in negative values >> being reported for large file si

Re: [PATCH] net: Fix the displayed value of bytes transferred

2023-08-10 Thread Ravi Gunasekaran
Siddharth, On 8/10/23 2:45 PM, Siddharth Vadapalli wrote: > In the case of NETLOOP_SUCCESS, the decimal value of the u32 variable > "net_boot_file_size" is printed using "%d", resulting in negative values > being reported for large file sizes. Fix this by using "%lu" to print > the decimal value c

[PATCH] net: Fix the displayed value of bytes transferred

2023-08-10 Thread Siddharth Vadapalli
In the case of NETLOOP_SUCCESS, the decimal value of the u32 variable "net_boot_file_size" is printed using "%d", resulting in negative values being reported for large file sizes. Fix this by using "%lu" to print the decimal value corresponding to the bytes transferred. Fixes: 1411157d8578 ("net: