]> www.infradead.org Git - users/hch/misc.git/commitdiff
ionic: use int type for err in ionic_get_module_eeprom_by_page
authorAlok Tiwari <alok.a.tiwari@oracle.com>
Fri, 12 Sep 2025 14:14:24 +0000 (07:14 -0700)
committerJakub Kicinski <kuba@kernel.org>
Sun, 14 Sep 2025 20:19:21 +0000 (13:19 -0700)
The variable 'err' is declared as u32, but it is used to store
negative error codes such as -EINVAL.

Changing the type of 'err' to int ensures proper representation of
negative error codes and aligns with standard kernel error handling
conventions.

Also, there is no need to initialize 'err' since it is always set
before being used.

Signed-off-by: Alok Tiwari <alok.a.tiwari@oracle.com>
Reviewed-by: Shannon Nelson <sln@onemain.com>
Reviewed-by: Brett Creeley <brett.creeley@amd.com>
Link: https://patch.msgid.link/20250912141426.3922545-1-alok.a.tiwari@oracle.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/pensando/ionic/ionic_ethtool.c

index 92f30ff2d63167bbc9debbe22885f9c03744d121..2d9efadb5d2ae1b19ce3c485a53775eb553e0c02 100644 (file)
@@ -978,7 +978,7 @@ static int ionic_get_module_eeprom_by_page(struct net_device *netdev,
 {
        struct ionic_lif *lif = netdev_priv(netdev);
        struct ionic_dev *idev = &lif->ionic->idev;
-       u32 err = -EINVAL;
+       int err;
        u8 *src;
 
        if (!page_data->length)