]> git.itanic.dy.fi Git - linux-stable/commitdiff
hwmon: (smsc47m1) Rename global platform device variable
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Thu, 7 Dec 2023 14:09:32 +0000 (15:09 +0100)
committerGuenter Roeck <linux@roeck-us.net>
Tue, 2 Jan 2024 16:44:57 +0000 (08:44 -0800)
pdev is a bad name for a global variable. Still more as the driver has
functions where pdev is a local variable. Rename it to smsc47m1_pdev.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Link: https://lore.kernel.org/r/68a959b56da7f9452557d08c72249182364b0dd0.1701957841.git.u.kleine-koenig@pengutronix.de
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/smsc47m1.c

index a17fbe969317cf8e033e6ae985ef1ce2f5d0a8e9..0d46edbcb144ba27417dde9996eeb041b52007c7 100644 (file)
@@ -33,7 +33,7 @@ static unsigned short force_id;
 module_param(force_id, ushort, 0);
 MODULE_PARM_DESC(force_id, "Override the detected device ID");
 
-static struct platform_device *pdev;
+static struct platform_device *smsc47m1_pdev;
 
 #define DRVNAME "smsc47m1"
 enum chips { smsc47m1, smsc47m2 };
@@ -884,10 +884,10 @@ static int __init smsc47m1_device_add(unsigned short address,
        if (err)
                return err;
 
-       pdev = platform_device_register_full(&pdevinfo);
-       if (IS_ERR(pdev)) {
+       smsc47m1_pdev = platform_device_register_full(&pdevinfo);
+       if (IS_ERR(smsc47m1_pdev)) {
                pr_err("Device allocation failed\n");
-               return PTR_ERR(pdev);
+               return PTR_ERR(smsc47m1_pdev);
        }
 
        return 0;
@@ -904,7 +904,7 @@ static int __init sm_smsc47m1_init(void)
                return err;
        address = err;
 
-       /* Sets global pdev as a side effect */
+       /* Sets global smsc47m1_pdev as a side effect */
        err = smsc47m1_device_add(address, &sio_data);
        if (err)
                return err;
@@ -916,7 +916,7 @@ static int __init sm_smsc47m1_init(void)
        return 0;
 
 exit_device:
-       platform_device_unregister(pdev);
+       platform_device_unregister(smsc47m1_pdev);
        smsc47m1_restore(&sio_data);
        return err;
 }
@@ -924,8 +924,8 @@ static int __init sm_smsc47m1_init(void)
 static void __exit sm_smsc47m1_exit(void)
 {
        platform_driver_unregister(&smsc47m1_driver);
-       smsc47m1_restore(dev_get_platdata(&pdev->dev));
-       platform_device_unregister(pdev);
+       smsc47m1_restore(dev_get_platdata(&smsc47m1_pdev->dev));
+       platform_device_unregister(smsc47m1_pdev);
 }
 
 MODULE_AUTHOR("Mark D. Studebaker <mdsxyz123@yahoo.com>");