When calling debugfs functions, there is no need to ever check the
return value.  The function can work or not, but the code logic should
never do something different based on this.

Cc: Arnd Bergmann <a...@arndb.de>
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Greg Kroah-Hartman <gre...@linuxfoundation.org>
---
 drivers/misc/ti-st/st_kim.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/misc/ti-st/st_kim.c b/drivers/misc/ti-st/st_kim.c
index e7cfdbd1f66d..93821c11bff9 100644
--- a/drivers/misc/ti-st/st_kim.c
+++ b/drivers/misc/ti-st/st_kim.c
@@ -761,10 +761,6 @@ static int kim_probe(struct platform_device *pdev)
        pr_info("sysfs entries created\n");
 
        kim_debugfs_dir = debugfs_create_dir("ti-st", NULL);
-       if (!kim_debugfs_dir) {
-               pr_err(" debugfs entries creation failed ");
-               return 0;
-       }
 
        debugfs_create_file("version", S_IRUGO, kim_debugfs_dir,
                                kim_gdata, &version_fops);
-- 
2.22.0

Reply via email to