diff options
author | Alexander Usyskin <alexander.usyskin@intel.com> | 2014-10-02 06:39:31 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-11-03 18:52:48 -0500 |
commit | 140c7553231dcd119ebda8c27dec7f06aef7e323 (patch) | |
tree | deebf68909102b7a828ca8b92cb354383c72004a /drivers/misc/mei/client.c | |
parent | 0df1f2487d2f0d04703f142813d53615d62a1da4 (diff) |
mei: drop unneeded client NULL check in cb structure
The pointer to client in the callback structure (cb->cl)
can't be NULL with current locking.
We can drop check and warnings as in some cases this just
uselessly complicates the code flow.
Signed-off-by: Alexander Usyskin <alexander.usyskin@intel.com>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/misc/mei/client.c')
-rw-r--r-- | drivers/misc/mei/client.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/misc/mei/client.c b/drivers/misc/mei/client.c index bc9ba5359bc6..1382d551d7ed 100644 --- a/drivers/misc/mei/client.c +++ b/drivers/misc/mei/client.c | |||
@@ -146,7 +146,7 @@ static void __mei_io_list_flush(struct mei_cl_cb *list, | |||
146 | 146 | ||
147 | /* enable removing everything if no cl is specified */ | 147 | /* enable removing everything if no cl is specified */ |
148 | list_for_each_entry_safe(cb, next, &list->list, list) { | 148 | list_for_each_entry_safe(cb, next, &list->list, list) { |
149 | if (!cl || (cb->cl && mei_cl_cmp_id(cl, cb->cl))) { | 149 | if (!cl || mei_cl_cmp_id(cl, cb->cl)) { |
150 | list_del(&cb->list); | 150 | list_del(&cb->list); |
151 | if (free) | 151 | if (free) |
152 | mei_io_cb_free(cb); | 152 | mei_io_cb_free(cb); |