staging: mei: renaming delayed work field and function to a meaningful name.
authorOren Weil <oren.jer.weil@intel.com>
Wed, 7 Sep 2011 06:03:13 +0000 (09:03 +0300)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 9 Sep 2011 20:28:21 +0000 (13:28 -0700)
the wd_work and mei_wd_timer() function was not just for watchdog.
mei managing the AMT watchdog going to be replace by the kernel watchdog
interface.

the mei_timer() will still manage the mei driver timeouts.

Signed-off-by: Oren Weil <oren.jer.weil@intel.com>
Acked-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/mei/interrupt.c
drivers/staging/mei/main.c
drivers/staging/mei/mei_dev.h
drivers/staging/mei/wd.c

index 62b8001ba1d703e53188be53728c5421ade6aec3..d1da3aa1cd5db12a5cb65b013e35e0dbbc2ac97a 100644 (file)
@@ -1393,7 +1393,7 @@ static int mei_irq_thread_write_handler(struct mei_io_list *cmpl_list,
  *
  * NOTE: This function is called by timer interrupt work
  */
-void mei_wd_timer(struct work_struct *work)
+void mei_timer(struct work_struct *work)
 {
        unsigned long timeout;
        struct mei_cl *cl_pos = NULL;
@@ -1403,7 +1403,7 @@ void mei_wd_timer(struct work_struct *work)
        struct mei_cl_cb  *cb_next = NULL;
 
        struct mei_device *dev = container_of(work,
-                                       struct mei_device, wd_work.work);
+                                       struct mei_device, timer_work.work);
 
 
        mutex_lock(&dev->device_lock);
@@ -1495,7 +1495,7 @@ void mei_wd_timer(struct work_struct *work)
                }
        }
 out:
-        schedule_delayed_work(&dev->wd_work, 2 * HZ);
+        schedule_delayed_work(&dev->timer_work, 2 * HZ);
         mutex_unlock(&dev->device_lock);
 }
 
index 670feb7f91a0ec1242857cc46a3fe1ae81464f09..925de38adc8f6f4969956d764117addbd4f901ab 100644 (file)
@@ -170,7 +170,7 @@ static int __devinit mei_probe(struct pci_dev *pdev,
                       pdev->irq);
                goto unmap_memory;
        }
-       INIT_DELAYED_WORK(&dev->wd_work, mei_wd_timer);
+       INIT_DELAYED_WORK(&dev->timer_work, mei_timer);
        if (mei_hw_init(dev)) {
                printk(KERN_ERR "mei: Init hw failure.\n");
                err = -ENODEV;
@@ -178,7 +178,7 @@ static int __devinit mei_probe(struct pci_dev *pdev,
        }
        mei_device = pdev;
        pci_set_drvdata(pdev, dev);
-       schedule_delayed_work(&dev->wd_work, HZ);
+       schedule_delayed_work(&dev->timer_work, HZ);
 
        mutex_unlock(&mei_mutex);
 
@@ -1154,7 +1154,7 @@ static int mei_pci_resume(struct device *device)
        /* Start watchdog if stopped in suspend */
        if (dev->wd_timeout) {
                dev->wd_due_counter = 1;
-               schedule_delayed_work(&dev->wd_work, HZ);
+               schedule_delayed_work(&dev->timer_work, HZ);
        }
        return err;
 }
index 43d7e1c044108de082ac50164545810e562f7399..6487be1a4bfac544ee7f909d1fd20ad5cc074c9c 100644 (file)
@@ -209,7 +209,7 @@ struct mei_device {
         * lock for the device
         */
        struct mutex device_lock; /* device lock */
-       struct delayed_work wd_work;    /* watch dog deleye work */
+       struct delayed_work timer_work; /* MEI timer delayed work (timeouts) */
        bool recvd_msg;
        /*
         * hw states of host and fw(ME)
@@ -336,7 +336,7 @@ void mei_host_client_properties(struct mei_device *dev);
  */
 irqreturn_t mei_interrupt_quick_handler(int irq, void *dev_id);
 irqreturn_t mei_interrupt_thread_handler(int irq, void *dev_id);
-void mei_wd_timer(struct work_struct *work);
+void mei_timer(struct work_struct *work);
 
 /*
  *  MEI input output function prototype
index 6643f7aa97db2a1c92cc6f0282c527c5bb146ee7..68c177be7e406539e2768702c379105bbb880d31 100644 (file)
@@ -142,7 +142,7 @@ int mei_wd_stop(struct mei_device *dev, bool preserve)
        int ret;
        u16 wd_timeout = dev->wd_timeout;
 
-       cancel_delayed_work(&dev->wd_work);
+       cancel_delayed_work(&dev->timer_work);
        if (dev->wd_cl.state != MEI_FILE_CONNECTED || !dev->wd_timeout)
                return 0;