Commit 15ea1910 authored by Tomas Winkler's avatar Tomas Winkler Committed by Greg Kroah-Hartman

mei: mei_clear_list: kill file_temp

file_temp is used only once, so there is no any benefit of creating
a temporary variable
Signed-off-by: default avatarTomas Winkler <tomas.winkler@intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 19838fb8
...@@ -103,18 +103,16 @@ static DEFINE_MUTEX(mei_mutex); ...@@ -103,18 +103,16 @@ static DEFINE_MUTEX(mei_mutex);
* returns true if callback removed from the list, false otherwise * returns true if callback removed from the list, false otherwise
*/ */
static bool mei_clear_list(struct mei_device *dev, static bool mei_clear_list(struct mei_device *dev,
struct file *file, struct list_head *mei_cb_list) const struct file *file, struct list_head *mei_cb_list)
{ {
struct mei_cl_cb *cb_pos = NULL; struct mei_cl_cb *cb_pos = NULL;
struct mei_cl_cb *cb_next = NULL; struct mei_cl_cb *cb_next = NULL;
struct file *file_temp;
bool removed = false; bool removed = false;
/* list all list member */ /* list all list member */
list_for_each_entry_safe(cb_pos, cb_next, mei_cb_list, list) { list_for_each_entry_safe(cb_pos, cb_next, mei_cb_list, list) {
file_temp = (struct file *)cb_pos->file_object;
/* check if list member associated with a file */ /* check if list member associated with a file */
if (file_temp == file) { if (file == cb_pos->file_object) {
/* remove member from the list */ /* remove member from the list */
list_del(&cb_pos->list); list_del(&cb_pos->list);
/* check if cb equal to current iamthif cb */ /* check if cb equal to current iamthif cb */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment