diff options
| author | Alexander Usyskin | 2014-10-02 12:39:31 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman | 2014-11-04 00:52:48 +0100 |
| commit | 140c7553231dcd119ebda8c27dec7f06aef7e323 (patch) | |
| tree | deebf68909102b7a828ca8b92cb354383c72004a /drivers/misc/mei/hbm.c | |
| parent | Linux 3.18-rc3 (diff) | |
| download | kernel-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/hbm.c')
| -rw-r--r-- | drivers/misc/mei/hbm.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/misc/mei/hbm.c b/drivers/misc/mei/hbm.c index 49a2653d91a5..57c1bde9b2b8 100644 --- a/drivers/misc/mei/hbm.c +++ b/drivers/misc/mei/hbm.c @@ -637,11 +637,6 @@ static void mei_hbm_cl_res(struct mei_device *dev, list_for_each_entry_safe(cb, next, &dev->ctrl_rd_list.list, list) { cl = cb->cl; - /* this should not happen */ - if (WARN_ON(!cl)) { - list_del_init(&cb->list); - continue; - } if (cb->fop_type != fop_type) continue; |