[PATCH] staging: unisys: don't care about debugfs errors

gregkh at linuxfoundation.org gregkh at linuxfoundation.org
Fri Nov 3 14:47:04 UTC 2017

A caller should never care about a debugfs error return value, and it
should never abort its normal operation if something "odd" goes on.  Fix
up the unisys init code to not care if the root debugfs directory for
the subsystem is created or not, as no place it is used will matter.

Cc: David Kershner <david.kershner at unisys.com>
Cc: Tim Sell <Timothy.Sell at unisys.com>
Cc: Sameer Wadgaonkar <sameer.wadgaonkar at unisys.com>
Cc: David Binder <david.binder at unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh at linuxfoundation.org>

diff --git a/drivers/staging/unisys/visorbus/visorbus_main.c b/drivers/staging/unisys/visorbus/visorbus_main.c
index 2f1e8d36aedf..b604d0cccef1 100644
--- a/drivers/staging/unisys/visorbus/visorbus_main.c
+++ b/drivers/staging/unisys/visorbus/visorbus_main.c
@@ -1217,8 +1217,6 @@ int visorbus_init(void)
 	int err;
 	visorbus_debugfs_dir = debugfs_create_dir("visorbus", NULL);
-	if (!visorbus_debugfs_dir)
-		return -ENOMEM;
 	bus_device_info_init(&clientbus_driverinfo, "clientbus", "visorbus");
 	err = bus_register(&visorbus_type);
 	if (err < 0)

More information about the devel mailing list