diff options
| author |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2018-05-29 17:30:48 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2018-05-31 12:54:21 +0200 |
| commit | 8a1ef171ffc3de16d4f34e03a2b8c296771f7015 (patch) | |
| tree | 51acfda55aaab7cde1f0a1e67d109008c05f6a23 /drivers/usb/musb/musb_core.c | |
| parent | USB: typec: tcpm: no need to check return value of debugfs_create_dir() (diff) | |
| download | wireguard-linux-8a1ef171ffc3de16d4f34e03a2b8c296771f7015.tar.xz wireguard-linux-8a1ef171ffc3de16d4f34e03a2b8c296771f7015.zip | |
USB: musb: clean up debugfs file and directory creation
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.
Because of this, lots of init functions do not need to have return
values, so this cleans up a lot of unused error handling code that never
could have triggered in the past.
Cc: Bin Liu <b-liu@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/usb/musb/musb_core.c')
| -rw-r--r-- | drivers/usb/musb/musb_core.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/usb/musb/musb_core.c b/drivers/usb/musb/musb_core.c index 5cc64980058b..b7d56272f9d1 100644 --- a/drivers/usb/musb/musb_core.c +++ b/drivers/usb/musb/musb_core.c @@ -2387,9 +2387,7 @@ musb_init_controller(struct device *dev, int nIrq, void __iomem *ctrl) if (status < 0) goto fail3; - status = musb_init_debugfs(musb); - if (status < 0) - goto fail4; + musb_init_debugfs(musb); status = sysfs_create_group(&musb->controller->kobj, &musb_attr_group); if (status) @@ -2404,7 +2402,6 @@ musb_init_controller(struct device *dev, int nIrq, void __iomem *ctrl) fail5: musb_exit_debugfs(musb); -fail4: musb_gadget_cleanup(musb); musb_host_cleanup(musb); |