提交 b4028437 编写于 作者: G Greg Kroah-Hartman

Driver core: move dev_get/set_drvdata to drivers/base/dd.c

No one should directly access the driver_data field, so remove the field
and make it private.  We dynamically create the private field now if it
is needed, to handle drivers that call get/set before they are
registered with the driver core.

Also update the copyright notices on these files while we are there.

Cc: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
上级 2023c610
...@@ -70,6 +70,8 @@ struct class_private { ...@@ -70,6 +70,8 @@ struct class_private {
* @knode_parent - node in sibling list * @knode_parent - node in sibling list
* @knode_driver - node in driver list * @knode_driver - node in driver list
* @knode_bus - node in bus list * @knode_bus - node in bus list
* @driver_data - private pointer for driver specific info. Will turn into a
* list soon.
* @device - pointer back to the struct class that this structure is * @device - pointer back to the struct class that this structure is
* associated with. * associated with.
* *
...@@ -80,6 +82,7 @@ struct device_private { ...@@ -80,6 +82,7 @@ struct device_private {
struct klist_node knode_parent; struct klist_node knode_parent;
struct klist_node knode_driver; struct klist_node knode_driver;
struct klist_node knode_bus; struct klist_node knode_bus;
void *driver_data;
struct device *device; struct device *device;
}; };
#define to_device_private_parent(obj) \ #define to_device_private_parent(obj) \
...@@ -89,6 +92,8 @@ struct device_private { ...@@ -89,6 +92,8 @@ struct device_private {
#define to_device_private_bus(obj) \ #define to_device_private_bus(obj) \
container_of(obj, struct device_private, knode_bus) container_of(obj, struct device_private, knode_bus)
extern int device_private_init(struct device *dev);
/* initialisation functions */ /* initialisation functions */
extern int devices_init(void); extern int devices_init(void);
extern int buses_init(void); extern int buses_init(void);
......
...@@ -843,6 +843,17 @@ static void device_remove_sys_dev_entry(struct device *dev) ...@@ -843,6 +843,17 @@ static void device_remove_sys_dev_entry(struct device *dev)
} }
} }
int device_private_init(struct device *dev)
{
dev->p = kzalloc(sizeof(*dev->p), GFP_KERNEL);
if (!dev->p)
return -ENOMEM;
dev->p->device = dev;
klist_init(&dev->p->klist_children, klist_children_get,
klist_children_put);
return 0;
}
/** /**
* device_add - add device to device hierarchy. * device_add - add device to device hierarchy.
* @dev: device. * @dev: device.
...@@ -868,14 +879,11 @@ int device_add(struct device *dev) ...@@ -868,14 +879,11 @@ int device_add(struct device *dev)
if (!dev) if (!dev)
goto done; goto done;
dev->p = kzalloc(sizeof(*dev->p), GFP_KERNEL);
if (!dev->p) { if (!dev->p) {
error = -ENOMEM; error = device_private_init(dev);
goto done; if (error)
goto done;
} }
dev->p->device = dev;
klist_init(&dev->p->klist_children, klist_children_get,
klist_children_put);
/* /*
* for statically allocated devices, which should all be converted * for statically allocated devices, which should all be converted
......
...@@ -11,8 +11,8 @@ ...@@ -11,8 +11,8 @@
* *
* Copyright (c) 2002-5 Patrick Mochel * Copyright (c) 2002-5 Patrick Mochel
* Copyright (c) 2002-3 Open Source Development Labs * Copyright (c) 2002-3 Open Source Development Labs
* Copyright (c) 2007 Greg Kroah-Hartman <gregkh@suse.de> * Copyright (c) 2007-2009 Greg Kroah-Hartman <gregkh@suse.de>
* Copyright (c) 2007 Novell Inc. * Copyright (c) 2007-2009 Novell Inc.
* *
* This file is released under the GPLv2 * This file is released under the GPLv2
*/ */
...@@ -391,3 +391,30 @@ void driver_detach(struct device_driver *drv) ...@@ -391,3 +391,30 @@ void driver_detach(struct device_driver *drv)
put_device(dev); put_device(dev);
} }
} }
/*
* These exports can't be _GPL due to .h files using this within them, and it
* might break something that was previously working...
*/
void *dev_get_drvdata(const struct device *dev)
{
if (dev && dev->p)
return dev->p->driver_data;
return NULL;
}
EXPORT_SYMBOL(dev_get_drvdata);
void dev_set_drvdata(struct device *dev, void *data)
{
int error;
if (!dev)
return;
if (!dev->p) {
error = device_private_init(dev);
if (error)
return;
}
dev->p->driver_data = data;
}
EXPORT_SYMBOL(dev_set_drvdata);
...@@ -2,7 +2,8 @@ ...@@ -2,7 +2,8 @@
* device.h - generic, centralized driver model * device.h - generic, centralized driver model
* *
* Copyright (c) 2001-2003 Patrick Mochel <mochel@osdl.org> * Copyright (c) 2001-2003 Patrick Mochel <mochel@osdl.org>
* Copyright (c) 2004-2007 Greg Kroah-Hartman <gregkh@suse.de> * Copyright (c) 2004-2009 Greg Kroah-Hartman <gregkh@suse.de>
* Copyright (c) 2008-2009 Novell Inc.
* *
* This file is released under the GPLv2 * This file is released under the GPLv2
* *
...@@ -381,7 +382,6 @@ struct device { ...@@ -381,7 +382,6 @@ struct device {
struct bus_type *bus; /* type of bus device is on */ struct bus_type *bus; /* type of bus device is on */
struct device_driver *driver; /* which driver has allocated this struct device_driver *driver; /* which driver has allocated this
device */ device */
void *driver_data; /* data private to the driver */
void *platform_data; /* Platform specific data, device void *platform_data; /* Platform specific data, device
core doesn't touch it */ core doesn't touch it */
struct dev_pm_info power; struct dev_pm_info power;
...@@ -447,16 +447,6 @@ static inline void set_dev_node(struct device *dev, int node) ...@@ -447,16 +447,6 @@ static inline void set_dev_node(struct device *dev, int node)
} }
#endif #endif
static inline void *dev_get_drvdata(const struct device *dev)
{
return dev->driver_data;
}
static inline void dev_set_drvdata(struct device *dev, void *data)
{
dev->driver_data = data;
}
static inline unsigned int dev_get_uevent_suppress(const struct device *dev) static inline unsigned int dev_get_uevent_suppress(const struct device *dev)
{ {
return dev->kobj.uevent_suppress; return dev->kobj.uevent_suppress;
...@@ -490,6 +480,8 @@ extern int device_rename(struct device *dev, char *new_name); ...@@ -490,6 +480,8 @@ extern int device_rename(struct device *dev, char *new_name);
extern int device_move(struct device *dev, struct device *new_parent, extern int device_move(struct device *dev, struct device *new_parent,
enum dpm_order dpm_order); enum dpm_order dpm_order);
extern const char *device_get_nodename(struct device *dev, const char **tmp); extern const char *device_get_nodename(struct device *dev, const char **tmp);
extern void *dev_get_drvdata(const struct device *dev);
extern void dev_set_drvdata(struct device *dev, void *data);
/* /*
* Root device objects for grouping under /sys/devices * Root device objects for grouping under /sys/devices
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册