]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
macintosh/ams: Remove unneeded result variable
authorMinghao Chi <chi.minghao@zte.com.cn>
Tue, 18 Jan 2022 07:52:52 +0000 (07:52 +0000)
committerMichael Ellerman <mpe@ellerman.id.au>
Wed, 4 May 2022 09:37:45 +0000 (19:37 +1000)
Return the value from i2c_add_driver() directly instead of storing it in
another redundant variable.

Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn>
Signed-off-by: CGEL ZTE <cgel.zte@gmail.com>
Signed-off-by: Christophe Leroy <christophe.leroy@csgroup.eu>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20220118075252.925616-1-chi.minghao@zte.com.cn
drivers/macintosh/ams/ams-i2c.c

index 21271b2e9259e048d37169bbe3838362d45688ad..d2f0cde6f9c742325417b4abf1dbabedea424afe 100644 (file)
@@ -256,8 +256,6 @@ static void ams_i2c_exit(void)
 
 int __init ams_i2c_init(struct device_node *np)
 {
-       int result;
-
        /* Set implementation stuff */
        ams_info.of_node = np;
        ams_info.exit = ams_i2c_exit;
@@ -266,7 +264,5 @@ int __init ams_i2c_init(struct device_node *np)
        ams_info.clear_irq = ams_i2c_clear_irq;
        ams_info.bustype = BUS_I2C;
 
-       result = i2c_add_driver(&ams_i2c_driver);
-
-       return result;
+       return i2c_add_driver(&ams_i2c_driver);
 }