[PATCH 08/10] driver core: convert to use class_find_device api

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: <linux-kernel@...>
Cc: Dave Young <hidave.darkstar@...>, Greg Kroah-Hartman <gregkh@...>
Date: Saturday, February 2, 2008 - 7:56 pm

From: Dave Young <hidave.darkstar@gmail.com>

Convert to use class_find_device api in drivers/base/core.c

Signed-off-by: Dave Young <hidave.darkstar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
---
 drivers/base/core.c |   32 +++++++++++---------------------
 1 files changed, 11 insertions(+), 21 deletions(-)

diff --git a/drivers/base/core.c b/drivers/base/core.c
index b172787..a0cfda5 100644
--- a/drivers/base/core.c
+++ b/drivers/base/core.c
@@ -1144,25 +1144,11 @@ error:
 }
 EXPORT_SYMBOL_GPL(device_create);
 
-/**
- * find_device - finds a device that was created with device_create()
- * @class: pointer to the struct class that this device was registered with
- * @devt: the dev_t of the device that was previously registered
- */
-static struct device *find_device(struct class *class, dev_t devt)
+static int __match_devt(struct device *dev, void *data)
 {
-	struct device *dev = NULL;
-	struct device *dev_tmp;
+	dev_t *devt = data;
 
-	down(&class->sem);
-	list_for_each_entry(dev_tmp, &class->devices, node) {
-		if (dev_tmp->devt == devt) {
-			dev = dev_tmp;
-			break;
-		}
-	}
-	up(&class->sem);
-	return dev;
+	return dev->devt == *devt;
 }
 
 /**
@@ -1177,9 +1163,11 @@ void device_destroy(struct class *class, dev_t devt)
 {
 	struct device *dev;
 
-	dev = find_device(class, devt);
-	if (dev)
+	dev = class_find_device(class, &devt, __match_devt);
+	if (dev) {
+		put_device(dev);
 		device_unregister(dev);
+	}
 }
 EXPORT_SYMBOL_GPL(device_destroy);
 
@@ -1203,9 +1191,11 @@ void destroy_suspended_device(struct class *class, dev_t devt)
 {
 	struct device *dev;
 
-	dev = find_device(class, devt);
-	if (dev)
+	dev = class_find_device(class, &devt, __match_devt);
+	if (dev) {
 		device_pm_schedule_removal(dev);
+		put_device(dev);
+	}
 }
 EXPORT_SYMBOL_GPL(destroy_suspended_device);
 #endif /* CONFIG_PM_SLEEP */
-- 
1.5.3.8

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 10/10] Driver core: Remove unneeded get_{device,drive..., Greg Kroah-Hartman, (Sat Feb 2, 7:56 pm)
[PATCH 09/10] Driver core: Update some prototypes in platfor..., Greg Kroah-Hartman, (Sat Feb 2, 7:56 pm)
[PATCH 08/10] driver core: convert to use class_find_device ..., Greg Kroah-Hartman, (Sat Feb 2, 7:56 pm)
[PATCH 07/10] PM: Export device_pm_schedule_removal, Greg Kroah-Hartman, (Sat Feb 2, 7:56 pm)
[PATCH 06/10] nozomi: finish constification, Greg Kroah-Hartman, (Sat Feb 2, 7:56 pm)
[PATCH 05/10] nozomi: constify driver, Greg Kroah-Hartman, (Sat Feb 2, 7:55 pm)
[PATCH 04/10] nozomi driver update, Greg Kroah-Hartman, (Sat Feb 2, 7:55 pm)
[PATCH 03/10] Add ja_JP translation of stable_kernel_rules.txt, Greg Kroah-Hartman, (Sat Feb 2, 7:55 pm)
[PATCH 02/10] kobject: kerneldoc comment fix, Greg Kroah-Hartman, (Sat Feb 2, 7:55 pm)
[PATCH 01/10] kobject: Always build in kernel/ksysfs.o., Greg Kroah-Hartman, (Sat Feb 2, 7:55 pm)