Commit 48f70474 authored by Greg Kroah-Hartman's avatar Greg Kroah-Hartman

greybus: debugfs: we shouldn't care if debugfs is working or not

This removes the error checking for debugfs initialization as we really
don't care if it failed or not.
Signed-off-by: default avatarGreg Kroah-Hartman <greg@kroah.com>
Reviewed-by: default avatarAlex Elder <elder@linaro.org>
parent 142f8ddf
...@@ -212,11 +212,7 @@ static int __init gb_init(void) ...@@ -212,11 +212,7 @@ static int __init gb_init(void)
BUILD_BUG_ON(HOST_DEV_CPORT_ID_MAX >= (long)CPORT_ID_BAD); BUILD_BUG_ON(HOST_DEV_CPORT_ID_MAX >= (long)CPORT_ID_BAD);
retval = gb_debugfs_init(); gb_debugfs_init();
if (retval) {
pr_err("debugfs failed\n");
return retval;
}
retval = bus_register(&greybus_bus_type); retval = bus_register(&greybus_bus_type);
if (retval) { if (retval) {
......
...@@ -15,13 +15,9 @@ ...@@ -15,13 +15,9 @@
static struct dentry *gb_debug_root; static struct dentry *gb_debug_root;
int gb_debugfs_init(void) void gb_debugfs_init(void)
{ {
gb_debug_root = debugfs_create_dir("greybus", NULL); gb_debug_root = debugfs_create_dir("greybus", NULL);
if (!gb_debug_root)
return -ENOENT;
return 0;
} }
void gb_debugfs_cleanup(void) void gb_debugfs_cleanup(void)
......
...@@ -158,7 +158,7 @@ int greybus_disabled(void); ...@@ -158,7 +158,7 @@ int greybus_disabled(void);
int greybus_svc_in(struct greybus_host_device *hd, u8 *data, int length); int greybus_svc_in(struct greybus_host_device *hd, u8 *data, int length);
int gb_ap_init(void); int gb_ap_init(void);
void gb_ap_exit(void); void gb_ap_exit(void);
int gb_debugfs_init(void); void gb_debugfs_init(void);
void gb_debugfs_cleanup(void); void gb_debugfs_cleanup(void);
struct dentry *gb_debugfs_get(void); struct dentry *gb_debugfs_get(void);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment