i2c: gpio: remove error checks with debugfs
authorWolfram Sang <wsa+renesas@sang-engineering.com>
Thu, 21 Sep 2023 08:40:15 +0000 (10:40 +0200)
committerWolfram Sang <wsa@kernel.org>
Fri, 22 Sep 2023 09:56:57 +0000 (11:56 +0200)
debugfs can handle error pointers in subsequent calls. So, remove the
error checks as suggested by kerneldoc of this function.

Reported-by: Minjie Du <duminjie@vivo.com>
Closes: http://patchwork.ozlabs.org/project/linux-i2c/patch/20230713101829.15548-1-duminjie@vivo.com/
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Reviewed-by: Andi Shyti <andi.shyti@kernel.org>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Wolfram Sang <wsa@kernel.org>
drivers/i2c/busses/i2c-gpio.c

index e5a5b9e8bf2c760719eafa5298cf22755c9f25a0..fb35a75fe0e32f54bac273b4a24404c16bf7e663 100644 (file)
@@ -263,15 +263,10 @@ static void i2c_gpio_fault_injector_init(struct platform_device *pdev)
         * 'fault-injector' dir there. Until then, we have a global dir with
         * all adapters as subdirs.
         */
-       if (!i2c_gpio_debug_dir) {
+       if (!i2c_gpio_debug_dir)
                i2c_gpio_debug_dir = debugfs_create_dir("i2c-fault-injector", NULL);
-               if (!i2c_gpio_debug_dir)
-                       return;
-       }
 
        priv->debug_dir = debugfs_create_dir(pdev->name, i2c_gpio_debug_dir);
-       if (!priv->debug_dir)
-               return;
 
        init_completion(&priv->scl_irq_completion);