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 <[email protected]>
Cc: Tim Sell <[email protected]>
Cc: Sameer Wadgaonkar <[email protected]>
Cc: David Binder <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>


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)
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to