diff options
| author | Benjamin Romer <benjamin.romer@unisys.com> | 2014-12-05 17:09:13 -0500 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-01-09 17:32:34 -0800 |
| commit | 94070ed4dd28a451c2ab7c5f90840270ac4cd61a (patch) | |
| tree | b8b65f8894db47db104fc732e574d2512b546cf9 | |
| parent | 717dd09fe75f5e165410caab197b2f7ec74e46e7 (diff) | |
staging: unisys: refactor uisctrl_unregister_req_handler_ex()
Get rid of the unnecessary goto statement and just return directly.
Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com>
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/unisys/uislib/uisutils.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/drivers/staging/unisys/uislib/uisutils.c b/drivers/staging/unisys/uislib/uisutils.c index 9ca10e923d89..e8c868ea2206 100644 --- a/drivers/staging/unisys/uislib/uisutils.c +++ b/drivers/staging/unisys/uislib/uisutils.c @@ -155,20 +155,14 @@ EXPORT_SYMBOL_GPL(uisctrl_register_req_handler_ex); int uisctrl_unregister_req_handler_ex(uuid_le switch_uuid) { - int rc = 0; /* assume failure */ - LOGINF("type=%pUL.\n", &switch_uuid); if (req_handler_del(switch_uuid) < 0) { LOGERR("failed to remove %pUL from server list\n", &switch_uuid); - goto Away; + return 0; } atomic_dec(&uisutils_registered_services); - rc = 1; /* success */ -Away: - if (!rc) - LOGERR("failed to unregister type %pUL.\n", &switch_uuid); - return rc; + return 1; } EXPORT_SYMBOL_GPL(uisctrl_unregister_req_handler_ex); |