[PATCH v3 1/2] usb: dwc3: make dwc3_debugfs_init return value be void

[changbin . du]

From: "Du, Changbin" <changbin...@intel.com>

Debugfs init failure is not so important. We can continue our job on
this failure. Also no need to check debugfs_create_file call results.

Signed-off-by: Du, Changbin <changbin...@intel.com>
---
changes from v2:
  no changes

---
 drivers/usb/dwc3/core.c    | 10 +---------
 drivers/usb/dwc3/debug.h   |  6 +++---
 drivers/usb/dwc3/debugfs.c | 50 ++++++++++------------------------------------
 3 files changed, 14 insertions(+), 52 deletions(-)

diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c
index 17fd814..30f825c 100644
--- a/drivers/usb/dwc3/core.c
+++ b/drivers/usb/dwc3/core.c
@@ -1062,19 +1062,11 @@ static int dwc3_probe(struct platform_device *pdev)
        if (ret)
                goto err5;
 
-       ret = dwc3_debugfs_init(dwc);
-       if (ret) {
-               dev_err(dev, "failed to initialize debugfs\n");
-               goto err6;
-       }
-
+       dwc3_debugfs_init(dwc);
        pm_runtime_allow(dev);
 
        return 0;
 
-err6:
-       dwc3_core_exit_mode(dwc);
-
 err5:
        dwc3_event_buffers_cleanup(dwc);
 
diff --git a/drivers/usb/dwc3/debug.h b/drivers/usb/dwc3/debug.h
index 07fbc2d..71e3180 100644
--- a/drivers/usb/dwc3/debug.h
+++ b/drivers/usb/dwc3/debug.h
@@ -217,11 +217,11 @@ static inline const char 
*dwc3_gadget_event_type_string(u8 event)
 void dwc3_trace(void (*trace)(struct va_format *), const char *fmt, ...);
 
 #ifdef CONFIG_DEBUG_FS
-extern int dwc3_debugfs_init(struct dwc3 *);
+extern void dwc3_debugfs_init(struct dwc3 *);
 extern void dwc3_debugfs_exit(struct dwc3 *);
 #else
-static inline int dwc3_debugfs_init(struct dwc3 *d)
-{  return 0;  }
+static inline void dwc3_debugfs_init(struct dwc3 *d)
+{  }
 static inline void dwc3_debugfs_exit(struct dwc3 *d)
 {  }
 #endif
diff --git a/drivers/usb/dwc3/debugfs.c b/drivers/usb/dwc3/debugfs.c
index 9ac37fe..071b286 100644
--- a/drivers/usb/dwc3/debugfs.c
+++ b/drivers/usb/dwc3/debugfs.c
@@ -618,69 +618,39 @@ static const struct file_operations dwc3_link_state_fops 
= {
        .release                = single_release,
 };
 
-int dwc3_debugfs_init(struct dwc3 *dwc)
+void dwc3_debugfs_init(struct dwc3 *dwc)
 {
        struct dentry           *root;
-       struct dentry           *file;
-       int                     ret;
 
        root = debugfs_create_dir(dev_name(dwc->dev), NULL);
-       if (!root) {
-               ret = -ENOMEM;
-               goto err0;
-       }
+       if (IS_ERR_OR_NULL(root))
+               return;
 
        dwc->root = root;
 
        dwc->regset = kzalloc(sizeof(*dwc->regset), GFP_KERNEL);
        if (!dwc->regset) {
-               ret = -ENOMEM;
-               goto err1;
+               debugfs_remove_recursive(root);
+               return;
        }
 
        dwc->regset->regs = dwc3_regs;
        dwc->regset->nregs = ARRAY_SIZE(dwc3_regs);
        dwc->regset->base = dwc->regs;
 
-       file = debugfs_create_regset32("regdump", S_IRUGO, root, dwc->regset);
-       if (!file) {
-               ret = -ENOMEM;
-               goto err1;
-       }
+       debugfs_create_regset32("regdump", S_IRUGO, root, dwc->regset);
 
-       if (IS_ENABLED(CONFIG_USB_DWC3_DUAL_ROLE)) {
-               file = debugfs_create_file("mode", S_IRUGO | S_IWUSR, root,
+       if (IS_ENABLED(CONFIG_USB_DWC3_DUAL_ROLE))
+               debugfs_create_file("mode", S_IRUGO | S_IWUSR, root,
                                dwc, &dwc3_mode_fops);
-               if (!file) {
-                       ret = -ENOMEM;
-                       goto err1;
-               }
-       }
 
        if (IS_ENABLED(CONFIG_USB_DWC3_DUAL_ROLE) ||
                        IS_ENABLED(CONFIG_USB_DWC3_GADGET)) {
-               file = debugfs_create_file("testmode", S_IRUGO | S_IWUSR, root,
+               debugfs_create_file("testmode", S_IRUGO | S_IWUSR, root,
                                dwc, &dwc3_testmode_fops);
-               if (!file) {
-                       ret = -ENOMEM;
-                       goto err1;
-               }
-
-               file = debugfs_create_file("link_state", S_IRUGO | S_IWUSR, 
root,
+               debugfs_create_file("link_state", S_IRUGO | S_IWUSR, root,
                                dwc, &dwc3_link_state_fops);
-               if (!file) {
-                       ret = -ENOMEM;
-                       goto err1;
-               }
        }
-
-       return 0;
-
-err1:
-       debugfs_remove_recursive(root);
-
-err0:
-       return ret;
 }
 
 void dwc3_debugfs_exit(struct dwc3 *dwc)
-- 
2.5.0