misc: Add attribute groups

Add groups field to struct miscdevice for passing the attribute groups
at device creation.  In this way, the driver can avoid the manual call
of device_create_file() after the device registration, which is
basically a racy operation, in addition to the reduction of manual
device_remove_file() calls.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Takashi Iwai 2015-02-02 15:44:54 +01:00 committed by Greg Kroah-Hartman
parent 2f9763190d
commit bd73599530
2 changed files with 5 additions and 2 deletions

View File

@ -207,8 +207,9 @@ int misc_register(struct miscdevice * misc)
dev = MKDEV(MISC_MAJOR, misc->minor); dev = MKDEV(MISC_MAJOR, misc->minor);
misc->this_device = device_create(misc_class, misc->parent, dev, misc->this_device =
misc, "%s", misc->name); device_create_with_groups(misc_class, misc->parent, dev,
misc, misc->groups, "%s", misc->name);
if (IS_ERR(misc->this_device)) { if (IS_ERR(misc->this_device)) {
int i = DYNAMIC_MINORS - misc->minor - 1; int i = DYNAMIC_MINORS - misc->minor - 1;
if (i < DYNAMIC_MINORS && i >= 0) if (i < DYNAMIC_MINORS && i >= 0)

View File

@ -52,6 +52,7 @@
#define MISC_DYNAMIC_MINOR 255 #define MISC_DYNAMIC_MINOR 255
struct device; struct device;
struct attribute_group;
struct miscdevice { struct miscdevice {
int minor; int minor;
@ -60,6 +61,7 @@ struct miscdevice {
struct list_head list; struct list_head list;
struct device *parent; struct device *parent;
struct device *this_device; struct device *this_device;
const struct attribute_group **groups;
const char *nodename; const char *nodename;
umode_t mode; umode_t mode;
}; };