Staging: hv: mousevsc_drv: Cleanup mousevsc_drv_exit() function
To conform to the Linux device model, the device should persist even when there is no driver bound to it. Cleanup the mousevsc_drv_exit() routine keeping this in mind. Signed-off-by: K. Y. Srinivasan <kys@microsoft.com> Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by: Abhishek Kane <v-abkane@microsoft.com> Signed-off-by: Hank Janssen <hjanssen@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This commit is contained in:
parent
2bda87cb6c
commit
96053efd3c
|
@ -915,13 +915,6 @@ static void reportdesc_callback(struct hv_device *dev, void *packet, u32 len)
|
||||||
kfree(hid_dev);
|
kfree(hid_dev);
|
||||||
}
|
}
|
||||||
|
|
||||||
static int mousevsc_drv_exit_cb(struct device *dev, void *data)
|
|
||||||
{
|
|
||||||
struct device **curr = (struct device **)data;
|
|
||||||
*curr = dev;
|
|
||||||
|
|
||||||
return 1;
|
|
||||||
}
|
|
||||||
|
|
||||||
static struct hv_driver mousevsc_drv = {
|
static struct hv_driver mousevsc_drv = {
|
||||||
.probe = mousevsc_probe,
|
.probe = mousevsc_probe,
|
||||||
|
@ -930,31 +923,7 @@ static struct hv_driver mousevsc_drv = {
|
||||||
|
|
||||||
static void mousevsc_drv_exit(void)
|
static void mousevsc_drv_exit(void)
|
||||||
{
|
{
|
||||||
struct hv_driver *drv = &mousevsc_drv;
|
vmbus_child_driver_unregister(&mousevsc_drv.driver);
|
||||||
int ret;
|
|
||||||
|
|
||||||
struct device *current_dev = NULL;
|
|
||||||
|
|
||||||
while (1) {
|
|
||||||
current_dev = NULL;
|
|
||||||
|
|
||||||
/* Get the device */
|
|
||||||
ret = driver_for_each_device(&drv->driver, NULL,
|
|
||||||
(void *)¤t_dev,
|
|
||||||
mousevsc_drv_exit_cb);
|
|
||||||
if (ret)
|
|
||||||
printk(KERN_ERR "Can't find mouse device!\n");
|
|
||||||
|
|
||||||
if (current_dev == NULL)
|
|
||||||
break;
|
|
||||||
|
|
||||||
/* Initiate removal from the top-down */
|
|
||||||
device_unregister(current_dev);
|
|
||||||
}
|
|
||||||
|
|
||||||
vmbus_child_driver_unregister(&drv->driver);
|
|
||||||
|
|
||||||
return;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static int __init mousevsc_init(void)
|
static int __init mousevsc_init(void)
|
||||||
|
|
Loading…
Reference in New Issue