pinctrl: Remove redundant null pointer checks in pinctrl_remove_device_debugfs()

Since the debugfs_create_dir() never returns a null pointer, checking
the return value for a null pointer is redundant, and using IS_ERR is
safe enough.

Signed-off-by: Li Zetao <lizetao1@huawei.com>
Link: https://lore.kernel.org/20240903143812.2005071-1-lizetao1@huawei.com
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

diff --git a/drivers/pinctrl/core.c b/drivers/pinctrl/core.c
index 314ab93d769182ae65c25380d6211c570ee33e16..4061890a174835ccd667c388f44208ddb326f4f3 100644 (file)
--- a/drivers/pinctrl/core.c
+++ b/drivers/pinctrl/core.c
@@ -1971,7 +1971,7 @@ static void pinctrl_remove_device_debugfs(struct pinctrl_dev *pctldev)
 static void pinctrl_init_debugfs(void)
 {
        debugfs_root = debugfs_create_dir("pinctrl", NULL);
-       if (IS_ERR(debugfs_root) || !debugfs_root) {
+       if (IS_ERR(debugfs_root)) {
                pr_warn("failed to create debugfs directory\n");
                debugfs_root = NULL;
                return;