Use driver_find() in ds.c instead of legacy implementation, and make use of
its reference counting awareness.
Signed-off-by: Dominik Brodowski <linux@brodo.de>
drivers/pcmcia/ds.c | 49 ++++++++++++++++++++++---------------------------
1 files changed, 22 insertions(+), 27 deletions(-)
diff -ruN linux-original/drivers/pcmcia/ds.c linux/drivers/pcmcia/ds.c
--- linux-original/drivers/pcmcia/ds.c 2004-11-13 19:52:32.000000000 +0100
+++ linux/drivers/pcmcia/ds.c 2004-11-13 21:44:28.734152480 +0100
@@ -570,11 +570,16 @@
(char *)bind_info->dev_info);
p_drv = get_pcmcia_driver(&bind_info->dev_info);
- if ((!p_drv) || (!try_module_get(p_drv->owner))) {
+ if (!p_drv) {
ret = -EINVAL;
goto err_put;
}
+ if (!try_module_get(p_drv->owner)) {
+ ret = -EINVAL;
+ goto err_put_driver;
+ }
+
/* Currently, the userspace pcmcia cardmgr detects pcmcia devices.
* Here this information is translated into a kernel
* struct pcmcia_device.
@@ -635,16 +640,21 @@
goto err_unregister;
}
}
-
+
+ put_driver(&p_drv->drv);
+
return 0;
err_unregister:
device_unregister(&p_dev->dev);
module_put(p_drv->owner);
+ put_driver(&p_drv->drv);
return (ret);
err_put_module:
module_put(p_drv->owner);
+ err_put_driver:
+ put_driver(&p_drv->drv);
err_put:
pcmcia_put_bus_socket(s);
return (ret);
@@ -1421,33 +1426,18 @@
/* backwards-compatible accessing of driver --- by name! */
-struct cmp_data {
- void *dev_info;
- struct pcmcia_driver *drv;
-};
-
-static int cmp_drv_callback(struct device_driver *drv, void *data)
-{
- struct cmp_data *cmp = data;
- if (strncmp((char *)cmp->dev_info, (char *)drv->name,
- DEV_NAME_LEN) == 0) {
- cmp->drv = container_of(drv, struct pcmcia_driver, drv);
- return -EINVAL;
- }
- return 0;
-}
-
static struct pcmcia_driver * get_pcmcia_driver (dev_info_t *dev_info)
{
- int ret;
- struct cmp_data cmp = {
- .dev_info = dev_info,
- };
-
- ret = bus_for_each_drv(&pcmcia_bus_type, NULL, &cmp, cmp_drv_callback);
- if (ret)
- return cmp.drv;
- return NULL;
+ struct device_driver *drv;
+ struct pcmcia_driver *p_drv;
+
+ drv = driver_find((char *) dev_info, &pcmcia_bus_type);
+ if (!drv)
+ return NULL;
+
+ p_drv = container_of(drv, struct pcmcia_driver, drv);
+
+ return (p_drv);
}
MODULE_ALIAS("ds");