]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
misc: keba: Use variable ret for return values
authorGerhard Engleder <eg@keba.com>
Fri, 11 Oct 2024 19:12:50 +0000 (21:12 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 13 Oct 2024 15:16:57 +0000 (17:16 +0200)
One function of the cp500 driver uses the variable name retval for
return values but all others use the variable name ret. Use ret for
return values in all functions.

Signed-off-by: Gerhard Engleder <eg@keba.com>
Link: https://lore.kernel.org/r/20241011191257.19702-2-gerhard@engleder-embedded.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/misc/keba/cp500.c

index ae09228178810c7710c57888c9f9b5b6866b6aab..b5327feb2847a65a3dfff410faeffbad3a32cad3 100644 (file)
@@ -229,7 +229,7 @@ static void cp500_i2c_release(struct device *dev)
 
 static int cp500_register_i2c(struct cp500 *cp500)
 {
-       int retval;
+       int ret;
 
        cp500->i2c = kzalloc(sizeof(*cp500->i2c), GFP_KERNEL);
        if (!cp500->i2c)
@@ -251,19 +251,19 @@ static int cp500_register_i2c(struct cp500 *cp500)
        cp500->i2c->info_size = ARRAY_SIZE(cp500_i2c_info);
        cp500->i2c->info = cp500_i2c_info;
 
-       retval = auxiliary_device_init(&cp500->i2c->auxdev);
-       if (retval) {
+       ret = auxiliary_device_init(&cp500->i2c->auxdev);
+       if (ret) {
                kfree(cp500->i2c);
                cp500->i2c = NULL;
 
-               return retval;
+               return ret;
        }
-       retval = __auxiliary_device_add(&cp500->i2c->auxdev, "keba");
-       if (retval) {
+       ret = __auxiliary_device_add(&cp500->i2c->auxdev, "keba");
+       if (ret) {
                auxiliary_device_uninit(&cp500->i2c->auxdev);
                cp500->i2c = NULL;
 
-               return retval;
+               return ret;
        }
 
        return 0;