Re: [PATCH v3 01/12] net/ice: use correct format specifiers for unsigned ints

[Bruce Richardson <bruce.richardson@intel.com>]

On Fri, Aug 23, 2024 at 09:56:39AM +0000, Soumyadeep Hore wrote:
> From: Yogesh Bhosale <yogesh.bhosale@intel.com>
> 
> Firmware was giving a number for the MSIX vectors that was
> way too big and obviously not right. Because of the wrong
> format specifier, this big number ended up looking like a
> tiny negative number in the logs. This was fixed by using
> the right format specifier everywhere it's needed.
> 
> Signed-off-by: Yogesh Bhosale <yogesh.bhosale@intel.com>
> Signed-off-by: Soumyadeep Hore <soumyadeep.hore@intel.com>
> ---
>  drivers/net/ice/base/ice_common.c | 54 +++++++++++++++----------------
>  1 file changed, 27 insertions(+), 27 deletions(-)
> 

Since this is a base code update, the commit log prefix should be
"net/ice/base" rather than just "net/ice". Base code patches tend to get
reviewed slightly differently since it's understood that they come from a
separate shared source, so we are checking for things like patch split as
much as for code correctness. Therefore we try to keep such patches
explicitly marked as affecting base code.

Also, please run checkpatches.sh and check-git-log.sh from the devtools
directory. It allows picking up more patch issues, such as in this case a
missing mailmap entry for the original patch author. In cases like this,
the mailmap entry is added in the first patch from that author.

Thanks,
/Bruce