diff options
| author | Jedrzej Jagielski <[email protected]> | 2024-01-31 12:04:19 +0100 |
|---|---|---|
| committer | Tony Nguyen <[email protected]> | 2024-02-06 13:17:14 -0800 |
| commit | ef3dd596504777266619252efff67149aa921ed3 (patch) | |
| tree | e4a825b399c919d30ffaf6e04326f8c857849ea3 /lib/string_helpers.c | |
| parent | b678b63a2454983cf22ffc8f8e5366ac9f376972 (diff) | |
ixgbe: Clarify the values of the returning status
Converting s32 functions to regular int in the previous patch of the series
caused triggering smatch warnings about missing error code.
New smatch warnings:
drivers/net/ethernet/intel/ixgbe/ixgbe_x550.c:2884 ixgbe_get_lcd_t_x550em() warn: missing error code? 'status'
drivers/net/ethernet/intel/ixgbe/ixgbe_x550.c:3130 ixgbe_enter_lplu_t_x550em() warn: missing error code? 'status'
Old smatch warnings:
drivers/net/ethernet/intel/ixgbe/ixgbe_x550.c:2890 ixgbe_get_lcd_t_x550em() warn: missing error code? 'status'
Fix it by clearly stating returning error code as 0.
Reported-by: kernel test robot <[email protected]>
Reported-by: Dan Carpenter <[email protected]>
Closes: https://lore.kernel.org/r/[email protected]/
Reviewed-by: Simon Horman <[email protected]>
Signed-off-by: Jedrzej Jagielski <[email protected]>
Tested-by: Pucha Himasekhar Reddy <[email protected]> (A Contingent worker at Intel)
Signed-off-by: Tony Nguyen <[email protected]>
Diffstat (limited to 'lib/string_helpers.c')
0 files changed, 0 insertions, 0 deletions