Skip to content

Commit

Permalink
mei: remove write only wariable wd_due_counter
Browse files Browse the repository at this point in the history
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
  • Loading branch information
Tomas Winkler authored and Greg Kroah-Hartman committed Jul 10, 2012
1 parent 4a0d6a7 commit d242a0a
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 13 deletions.
1 change: 0 additions & 1 deletion drivers/misc/mei/init.c
Original file line number Diff line number Diff line change
Expand Up @@ -306,7 +306,6 @@ void mei_reset(struct mei_device *dev, int interrupts_enabled)
dev->iamthif_cl.host_client_id);

mei_reset_iamthif_params(dev);
dev->wd_due_counter = 0;
dev->extra_write_index = 0;
}

Expand Down
12 changes: 4 additions & 8 deletions drivers/misc/mei/interrupt.c
Original file line number Diff line number Diff line change
Expand Up @@ -575,10 +575,9 @@ static void mei_client_disconnect_request(struct mei_device *dev,
disconnect_req->me_addr);
cl_pos->state = MEI_FILE_DISCONNECTED;
cl_pos->timer_count = 0;
if (cl_pos == &dev->wd_cl) {
dev->wd_due_counter = 0;
if (cl_pos == &dev->wd_cl)
dev->wd_pending = false;
} else if (cl_pos == &dev->iamthif_cl)
else if (cl_pos == &dev->iamthif_cl)
dev->iamthif_timer = 0;

/* prepare disconnect response */
Expand Down Expand Up @@ -1266,15 +1265,12 @@ static int mei_irq_thread_write_handler(struct mei_io_list *cmpl_list,

dev->wd_pending = false;

if (dev->wd_timeout) {
if (dev->wd_timeout)
*slots -= (sizeof(struct mei_msg_hdr) +
MEI_START_WD_DATA_SIZE + 3) / 4;
dev->wd_due_counter = 2;
} else {
else
*slots -= (sizeof(struct mei_msg_hdr) +
MEI_WD_PARAMS_SIZE + 3) / 4;
dev->wd_due_counter = 0;
}

}
}
Expand Down
1 change: 0 additions & 1 deletion drivers/misc/mei/mei_dev.h
Original file line number Diff line number Diff line change
Expand Up @@ -246,7 +246,6 @@ struct mei_device {
bool wd_stopped;
bool wd_bypass; /* if false, don't refresh watchdog ME client */
u16 wd_timeout; /* seconds ((wd_data[1] << 8) + wd_data[0]) */
u16 wd_due_counter;
unsigned char wd_data[MEI_START_WD_DATA_SIZE];


Expand Down
3 changes: 0 additions & 3 deletions drivers/misc/mei/wd.c
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,6 @@ int mei_wd_stop(struct mei_device *dev, bool preserve)
return 0;

dev->wd_timeout = 0;
dev->wd_due_counter = 0;
memcpy(dev->wd_data, mei_stop_wd_params, MEI_WD_PARAMS_SIZE);
dev->stop = true;

Expand Down Expand Up @@ -357,8 +356,6 @@ void mei_watchdog_register(struct mei_device *dev)
{
dev_dbg(&dev->pdev->dev, "dev->wd_timeout =%d.\n", dev->wd_timeout);

dev->wd_due_counter = !!dev->wd_timeout;

if (watchdog_register_device(&amt_wd_dev)) {
dev_err(&dev->pdev->dev,
"wd: unable to register watchdog device.\n");
Expand Down

0 comments on commit d242a0a

Please sign in to comment.