]> www.infradead.org Git - users/hch/misc.git/commitdiff
iavf: fix proper type for error code in iavf_resume()
authorAleksandr Loktionov <aleksandr.loktionov@intel.com>
Fri, 12 Sep 2025 08:02:08 +0000 (08:02 +0000)
committerTony Nguyen <anthony.l.nguyen@intel.com>
Fri, 19 Sep 2025 15:42:08 +0000 (08:42 -0700)
The variable 'err' in iavf_resume() is used to store the return value
of different functions, which return an int. Currently, 'err' is
declared as u32, which is semantically incorrect and misleading.

In the Linux kernel, u32 is typically reserved for fixed-width data
used in hardware interfaces or protocol structures. Using it for a
generic error code may confuse reviewers or developers into thinking
the value is hardware-related or size-constrained.

Replace u32 with int to reflect the actual usage and improve code
clarity and semantic correctness.

No functional change.

Signed-off-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@intel.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Reviewed-by: Paul Menzel <pmenzel@molgen.mpg.de>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
drivers/net/ethernet/intel/iavf/iavf_main.c

index 69054af4689a3075d76d9d5fb0de2f78ba9165ec..c2fbe443ef853a1d20da16aa9bf9ee4e85a4ddf3 100644 (file)
@@ -5491,7 +5491,7 @@ static int iavf_resume(struct device *dev_d)
 {
        struct pci_dev *pdev = to_pci_dev(dev_d);
        struct iavf_adapter *adapter;
-       u32 err;
+       int err;
 
        adapter = iavf_pdev_to_adapter(pdev);