summaryrefslogtreecommitdiffstats
path: root/drivers/misc/mei/client.c
diff options
context:
space:
mode:
authorAlexander Usyskin2014-10-02 12:39:31 +0200
committerGreg Kroah-Hartman2014-11-04 00:52:48 +0100
commit140c7553231dcd119ebda8c27dec7f06aef7e323 (patch)
treedeebf68909102b7a828ca8b92cb354383c72004a /drivers/misc/mei/client.c
parentLinux 3.18-rc3 (diff)
downloadkernel-qcow2-linux-140c7553231dcd119ebda8c27dec7f06aef7e323.tar.gz
kernel-qcow2-linux-140c7553231dcd119ebda8c27dec7f06aef7e323.tar.xz
kernel-qcow2-linux-140c7553231dcd119ebda8c27dec7f06aef7e323.zip
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.c2
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,
/* enable removing everything if no cl is specified */
list_for_each_entry_safe(cb, next, &list->list, list) {
- if (!cl || (cb->cl && mei_cl_cmp_id(cl, cb->cl))) {
+ if (!cl || mei_cl_cmp_id(cl, cb->cl)) {
list_del(&cb->list);
if (free)
mei_io_cb_free(cb);