Commit 8882b394 authored by Greg Kroah-Hartman's avatar Greg Kroah-Hartman

Driver core: add device_create_vargs and device_create_drvdata

We want to have the drvdata field set properly when creating the device
as sysfs callbacks can assume it is present and it can race the later
setting of this field.

So, create two new functions, deviec_create_vargs() and
device_create_drvdata() that take this new field.

device_create_drvdata() will go away in 2.6.27 as the drvdata field will
just be moved to the device_create() call as it should be.

Cc: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent c110a2bd
...@@ -1084,11 +1084,13 @@ static void device_create_release(struct device *dev) ...@@ -1084,11 +1084,13 @@ static void device_create_release(struct device *dev)
} }
/** /**
* device_create - creates a device and registers it with sysfs * device_create_vargs - creates a device and registers it with sysfs
* @class: pointer to the struct class that this device should be registered to * @class: pointer to the struct class that this device should be registered to
* @parent: pointer to the parent struct device of this new device, if any * @parent: pointer to the parent struct device of this new device, if any
* @devt: the dev_t for the char device to be added * @devt: the dev_t for the char device to be added
* @drvdata: the data to be added to the device for callbacks
* @fmt: string for the device's name * @fmt: string for the device's name
* @args: va_list for the device's name
* *
* This function can be used by char device classes. A struct device * This function can be used by char device classes. A struct device
* will be created in sysfs, registered to the specified class. * will be created in sysfs, registered to the specified class.
...@@ -1104,10 +1106,10 @@ static void device_create_release(struct device *dev) ...@@ -1104,10 +1106,10 @@ static void device_create_release(struct device *dev)
* Note: the struct class passed to this function must have previously * Note: the struct class passed to this function must have previously
* been created with a call to class_create(). * been created with a call to class_create().
*/ */
struct device *device_create(struct class *class, struct device *parent, struct device *device_create_vargs(struct class *class, struct device *parent,
dev_t devt, const char *fmt, ...) dev_t devt, void *drvdata, const char *fmt,
va_list args)
{ {
va_list args;
struct device *dev = NULL; struct device *dev = NULL;
int retval = -ENODEV; int retval = -ENODEV;
...@@ -1124,10 +1126,9 @@ struct device *device_create(struct class *class, struct device *parent, ...@@ -1124,10 +1126,9 @@ struct device *device_create(struct class *class, struct device *parent,
dev->class = class; dev->class = class;
dev->parent = parent; dev->parent = parent;
dev->release = device_create_release; dev->release = device_create_release;
dev_set_drvdata(dev, drvdata);
va_start(args, fmt);
vsnprintf(dev->bus_id, BUS_ID_SIZE, fmt, args); vsnprintf(dev->bus_id, BUS_ID_SIZE, fmt, args);
va_end(args);
retval = device_register(dev); retval = device_register(dev);
if (retval) if (retval)
goto error; goto error;
...@@ -1138,6 +1139,78 @@ struct device *device_create(struct class *class, struct device *parent, ...@@ -1138,6 +1139,78 @@ struct device *device_create(struct class *class, struct device *parent,
kfree(dev); kfree(dev);
return ERR_PTR(retval); return ERR_PTR(retval);
} }
EXPORT_SYMBOL_GPL(device_create_vargs);
/**
* device_create_drvdata - creates a device and registers it with sysfs
* @class: pointer to the struct class that this device should be registered to
* @parent: pointer to the parent struct device of this new device, if any
* @devt: the dev_t for the char device to be added
* @drvdata: the data to be added to the device for callbacks
* @fmt: string for the device's name
*
* This function can be used by char device classes. A struct device
* will be created in sysfs, registered to the specified class.
*
* A "dev" file will be created, showing the dev_t for the device, if
* the dev_t is not 0,0.
* If a pointer to a parent struct device is passed in, the newly created
* struct device will be a child of that device in sysfs.
* The pointer to the struct device will be returned from the call.
* Any further sysfs files that might be required can be created using this
* pointer.
*
* Note: the struct class passed to this function must have previously
* been created with a call to class_create().
*/
struct device *device_create_drvdata(struct class *class,
struct device *parent,
dev_t devt,
void *drvdata,
const char *fmt, ...)
{
va_list vargs;
struct device *dev;
va_start(vargs, fmt);
dev = device_create_vargs(class, parent, devt, drvdata, fmt, vargs);
va_end(vargs);
return dev;
}
EXPORT_SYMBOL_GPL(device_create_drvdata);
/**
* device_create - creates a device and registers it with sysfs
* @class: pointer to the struct class that this device should be registered to
* @parent: pointer to the parent struct device of this new device, if any
* @devt: the dev_t for the char device to be added
* @fmt: string for the device's name
*
* This function can be used by char device classes. A struct device
* will be created in sysfs, registered to the specified class.
*
* A "dev" file will be created, showing the dev_t for the device, if
* the dev_t is not 0,0.
* If a pointer to a parent struct device is passed in, the newly created
* struct device will be a child of that device in sysfs.
* The pointer to the struct device will be returned from the call.
* Any further sysfs files that might be required can be created using this
* pointer.
*
* Note: the struct class passed to this function must have previously
* been created with a call to class_create().
*/
struct device *device_create(struct class *class, struct device *parent,
dev_t devt, const char *fmt, ...)
{
va_list vargs;
struct device *dev;
va_start(vargs, fmt);
dev = device_create_vargs(class, parent, devt, NULL, fmt, vargs);
va_end(vargs);
return dev;
}
EXPORT_SYMBOL_GPL(device_create); EXPORT_SYMBOL_GPL(device_create);
static int __match_devt(struct device *dev, void *data) static int __match_devt(struct device *dev, void *data)
......
...@@ -449,9 +449,21 @@ extern int __must_check device_reprobe(struct device *dev); ...@@ -449,9 +449,21 @@ extern int __must_check device_reprobe(struct device *dev);
/* /*
* Easy functions for dynamically creating devices on the fly * Easy functions for dynamically creating devices on the fly
*/ */
extern struct device *device_create_vargs(struct class *cls,
struct device *parent,
dev_t devt,
void *drvdata,
const char *fmt,
va_list vargs);
extern struct device *device_create(struct class *cls, struct device *parent, extern struct device *device_create(struct class *cls, struct device *parent,
dev_t devt, const char *fmt, ...) dev_t devt, const char *fmt, ...)
__attribute__((format(printf, 4, 5))); __attribute__((format(printf, 4, 5)));
extern struct device *device_create_drvdata(struct class *cls,
struct device *parent,
dev_t devt,
void *drvdata,
const char *fmt, ...)
__attribute__((format(printf, 5, 6)));
extern void device_destroy(struct class *cls, dev_t devt); extern void device_destroy(struct class *cls, dev_t devt);
/* /*
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment