[PATCH] make struct config_device contain a struct device_attr instead of duplicating the mess.
This commit is contained in:
parent
b4c1dcf912
commit
cb32ac946f
101
namedev.c
101
namedev.c
|
@ -41,41 +41,6 @@
|
||||||
#define TYPE_TOPOLOGY "TOPOLOGY"
|
#define TYPE_TOPOLOGY "TOPOLOGY"
|
||||||
#define TYPE_REPLACE "REPLACE"
|
#define TYPE_REPLACE "REPLACE"
|
||||||
|
|
||||||
enum config_type {
|
|
||||||
KERNEL_NAME = 0, /* must be 0 to let memset() default to this value */
|
|
||||||
LABEL = 1,
|
|
||||||
NUMBER = 2,
|
|
||||||
TOPOLOGY = 3,
|
|
||||||
REPLACE = 4,
|
|
||||||
};
|
|
||||||
|
|
||||||
#define BUS_SIZE 30
|
|
||||||
#define FILE_SIZE 50
|
|
||||||
#define VALUE_SIZE 100
|
|
||||||
#define ID_SIZE 50
|
|
||||||
#define PLACE_SIZE 50
|
|
||||||
|
|
||||||
|
|
||||||
struct config_device {
|
|
||||||
struct list_head node;
|
|
||||||
|
|
||||||
enum config_type type;
|
|
||||||
|
|
||||||
char bus[BUS_SIZE];
|
|
||||||
char sysfs_file[FILE_SIZE];
|
|
||||||
char sysfs_value[VALUE_SIZE];
|
|
||||||
char id[ID_SIZE];
|
|
||||||
char place[PLACE_SIZE];
|
|
||||||
char kernel_name[NAME_SIZE];
|
|
||||||
|
|
||||||
/* what to set the device to */
|
|
||||||
int mode;
|
|
||||||
char name[NAME_SIZE];
|
|
||||||
char owner[OWNER_SIZE];
|
|
||||||
char group[GROUP_SIZE];
|
|
||||||
};
|
|
||||||
|
|
||||||
|
|
||||||
static LIST_HEAD(config_device_list);
|
static LIST_HEAD(config_device_list);
|
||||||
|
|
||||||
static void dump_dev(struct config_device *dev)
|
static void dump_dev(struct config_device *dev)
|
||||||
|
@ -84,32 +49,32 @@ static void dump_dev(struct config_device *dev)
|
||||||
case KERNEL_NAME:
|
case KERNEL_NAME:
|
||||||
dbg("KERNEL name ='%s'"
|
dbg("KERNEL name ='%s'"
|
||||||
" owner = '%s', group = '%s', mode = '%#o'",
|
" owner = '%s', group = '%s', mode = '%#o'",
|
||||||
dev->name,
|
dev->attr.name,
|
||||||
dev->owner, dev->group, dev->mode);
|
dev->attr.owner, dev->attr.group, dev->attr.mode);
|
||||||
break;
|
break;
|
||||||
case LABEL:
|
case LABEL:
|
||||||
dbg("LABEL name = '%s', bus = '%s', sysfs_file = '%s', sysfs_value = '%s'"
|
dbg("LABEL name = '%s', bus = '%s', sysfs_file = '%s', sysfs_value = '%s'"
|
||||||
" owner = '%s', group = '%s', mode = '%#o'",
|
" owner = '%s', group = '%s', mode = '%#o'",
|
||||||
dev->name, dev->bus, dev->sysfs_file, dev->sysfs_value,
|
dev->attr.name, dev->bus, dev->sysfs_file, dev->sysfs_value,
|
||||||
dev->owner, dev->group, dev->mode);
|
dev->attr.owner, dev->attr.group, dev->attr.mode);
|
||||||
break;
|
break;
|
||||||
case NUMBER:
|
case NUMBER:
|
||||||
dbg("NUMBER name = '%s', bus = '%s', id = '%s'"
|
dbg("NUMBER name = '%s', bus = '%s', id = '%s'"
|
||||||
" owner = '%s', group = '%s', mode = '%#o'",
|
" owner = '%s', group = '%s', mode = '%#o'",
|
||||||
dev->name, dev->bus, dev->id,
|
dev->attr.name, dev->bus, dev->id,
|
||||||
dev->owner, dev->group, dev->mode);
|
dev->attr.owner, dev->attr.group, dev->attr.mode);
|
||||||
break;
|
break;
|
||||||
case TOPOLOGY:
|
case TOPOLOGY:
|
||||||
dbg("TOPOLOGY name = '%s', bus = '%s', place = '%s'"
|
dbg("TOPOLOGY name = '%s', bus = '%s', place = '%s'"
|
||||||
" owner = '%s', group = '%s', mode = '%#o'",
|
" owner = '%s', group = '%s', mode = '%#o'",
|
||||||
dev->name, dev->bus, dev->place,
|
dev->attr.name, dev->bus, dev->place,
|
||||||
dev->owner, dev->group, dev->mode);
|
dev->attr.owner, dev->attr.group, dev->attr.mode);
|
||||||
break;
|
break;
|
||||||
case REPLACE:
|
case REPLACE:
|
||||||
dbg("REPLACE name = %s, kernel_name = %s"
|
dbg("REPLACE name = %s, kernel_name = %s"
|
||||||
" owner = '%s', group = '%s', mode = '%#o'",
|
" owner = '%s', group = '%s', mode = '%#o'",
|
||||||
dev->name, dev->kernel_name,
|
dev->attr.name, dev->kernel_name,
|
||||||
dev->owner, dev->group, dev->mode);
|
dev->attr.owner, dev->attr.group, dev->attr.mode);
|
||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
dbg("Unknown type of device!");
|
dbg("Unknown type of device!");
|
||||||
|
@ -133,18 +98,18 @@ static int add_dev(struct config_device *new_dev)
|
||||||
* this one... */
|
* this one... */
|
||||||
list_for_each(tmp, &config_device_list) {
|
list_for_each(tmp, &config_device_list) {
|
||||||
struct config_device *dev = list_entry(tmp, struct config_device, node);
|
struct config_device *dev = list_entry(tmp, struct config_device, node);
|
||||||
if (strcmp(dev->name, new_dev->name) == 0) {
|
if (strcmp(dev->attr.name, new_dev->attr.name) == 0) {
|
||||||
/* the same, copy the new info into this structure */
|
/* the same, copy the new info into this structure */
|
||||||
copy_var(dev, new_dev, type);
|
copy_var(dev, new_dev, type);
|
||||||
copy_var(dev, new_dev, mode);
|
copy_var(dev, new_dev, attr.mode);
|
||||||
copy_string(dev, new_dev, bus);
|
copy_string(dev, new_dev, bus);
|
||||||
copy_string(dev, new_dev, sysfs_file);
|
copy_string(dev, new_dev, sysfs_file);
|
||||||
copy_string(dev, new_dev, sysfs_value);
|
copy_string(dev, new_dev, sysfs_value);
|
||||||
copy_string(dev, new_dev, id);
|
copy_string(dev, new_dev, id);
|
||||||
copy_string(dev, new_dev, place);
|
copy_string(dev, new_dev, place);
|
||||||
copy_string(dev, new_dev, kernel_name);
|
copy_string(dev, new_dev, kernel_name);
|
||||||
copy_string(dev, new_dev, owner);
|
copy_string(dev, new_dev, attr.owner);
|
||||||
copy_string(dev, new_dev, group);
|
copy_string(dev, new_dev, attr.group);
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -286,9 +251,9 @@ static int namedev_init_config(void)
|
||||||
retval = get_value("NAME", &temp, &temp3);
|
retval = get_value("NAME", &temp, &temp3);
|
||||||
if (retval)
|
if (retval)
|
||||||
continue;
|
continue;
|
||||||
strcpy(dev.name, temp3);
|
strcpy(dev.attr.name, temp3);
|
||||||
|
|
||||||
dbg("LABEL name = '%s', bus = '%s', sysfs_file = '%s', sysfs_value = '%s'", dev.name, dev.bus, dev.sysfs_file, dev.sysfs_value);
|
dbg("LABEL name = '%s', bus = '%s', sysfs_file = '%s', sysfs_value = '%s'", dev.attr.name, dev.bus, dev.sysfs_file, dev.sysfs_value);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (strcasecmp(temp2, TYPE_NUMBER) == 0) {
|
if (strcasecmp(temp2, TYPE_NUMBER) == 0) {
|
||||||
|
@ -313,9 +278,9 @@ static int namedev_init_config(void)
|
||||||
retval = get_value("NAME", &temp, &temp3);
|
retval = get_value("NAME", &temp, &temp3);
|
||||||
if (retval)
|
if (retval)
|
||||||
continue;
|
continue;
|
||||||
strcpy(dev.name, temp3);
|
strcpy(dev.attr.name, temp3);
|
||||||
|
|
||||||
dbg("NUMBER name = '%s', bus = '%s', id = '%s'", dev.name, dev.bus, dev.id);
|
dbg("NUMBER name = '%s', bus = '%s', id = '%s'", dev.attr.name, dev.bus, dev.id);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (strcasecmp(temp2, TYPE_TOPOLOGY) == 0) {
|
if (strcasecmp(temp2, TYPE_TOPOLOGY) == 0) {
|
||||||
|
@ -340,9 +305,9 @@ static int namedev_init_config(void)
|
||||||
retval = get_value("NAME", &temp, &temp3);
|
retval = get_value("NAME", &temp, &temp3);
|
||||||
if (retval)
|
if (retval)
|
||||||
continue;
|
continue;
|
||||||
strcpy(dev.name, temp3);
|
strcpy(dev.attr.name, temp3);
|
||||||
|
|
||||||
dbg("TOPOLOGY name = '%s', bus = '%s', place = '%s'", dev.name, dev.bus, dev.place);
|
dbg("TOPOLOGY name = '%s', bus = '%s', place = '%s'", dev.attr.name, dev.bus, dev.place);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (strcasecmp(temp2, TYPE_REPLACE) == 0) {
|
if (strcasecmp(temp2, TYPE_REPLACE) == 0) {
|
||||||
|
@ -360,8 +325,8 @@ static int namedev_init_config(void)
|
||||||
retval = get_value("NAME", &temp, &temp3);
|
retval = get_value("NAME", &temp, &temp3);
|
||||||
if (retval)
|
if (retval)
|
||||||
continue;
|
continue;
|
||||||
strcpy(dev.name, temp3);
|
strcpy(dev.attr.name, temp3);
|
||||||
dbg("REPLACE name = %s, kernel_name = %s", dev.name, dev.kernel_name);
|
dbg("REPLACE name = %s, kernel_name = %s", dev.attr.name, dev.kernel_name);
|
||||||
}
|
}
|
||||||
|
|
||||||
retval = add_dev(&dev);
|
retval = add_dev(&dev);
|
||||||
|
@ -420,17 +385,17 @@ static int namedev_init_permissions(void)
|
||||||
|
|
||||||
/* parse the line */
|
/* parse the line */
|
||||||
temp2 = strsep(&temp, ":");
|
temp2 = strsep(&temp, ":");
|
||||||
strncpy(dev.name, temp2, sizeof(dev.name));
|
strncpy(dev.attr.name, temp2, sizeof(dev.attr.name));
|
||||||
|
|
||||||
temp2 = strsep(&temp, ":");
|
temp2 = strsep(&temp, ":");
|
||||||
strncpy(dev.owner, temp2, sizeof(dev.owner));
|
strncpy(dev.attr.owner, temp2, sizeof(dev.attr.owner));
|
||||||
|
|
||||||
temp2 = strsep(&temp, ":");
|
temp2 = strsep(&temp, ":");
|
||||||
strncpy(dev.group, temp2, sizeof(dev.owner));
|
strncpy(dev.attr.group, temp2, sizeof(dev.attr.owner));
|
||||||
|
|
||||||
dev.mode = strtol(temp, NULL, 8);
|
dev.attr.mode = strtol(temp, NULL, 8);
|
||||||
|
|
||||||
dbg("name = %s, owner = %s, group = %s, mode = %#o", dev.name, dev.owner, dev.group, dev.mode);
|
dbg("name = %s, owner = %s, group = %s, mode = %#o", dev.attr.name, dev.attr.owner, dev.attr.group, dev.attr.mode);
|
||||||
retval = add_dev(&dev);
|
retval = add_dev(&dev);
|
||||||
if (retval) {
|
if (retval) {
|
||||||
dbg("add_dev returned with error %d", retval);
|
dbg("add_dev returned with error %d", retval);
|
||||||
|
@ -457,13 +422,13 @@ static int get_attr(struct sysfs_class_device *class_dev, struct device_attr *at
|
||||||
|
|
||||||
list_for_each(tmp, &config_device_list) {
|
list_for_each(tmp, &config_device_list) {
|
||||||
struct config_device *dev = list_entry(tmp, struct config_device, node);
|
struct config_device *dev = list_entry(tmp, struct config_device, node);
|
||||||
if (strcmp(dev->name, class_dev->name) == 0) {
|
if (strcmp(dev->attr.name, class_dev->name) == 0) {
|
||||||
attr->mode = dev->mode;
|
attr->mode = dev->attr.mode;
|
||||||
strcpy(attr->owner, dev->owner);
|
strcpy(attr->owner, dev->attr.owner);
|
||||||
strcpy(attr->group, dev->group);
|
strcpy(attr->group, dev->attr.group);
|
||||||
/* FIXME put the proper name here!!! */
|
/* FIXME put the proper name here!!! */
|
||||||
strcpy(attr->name, dev->name);
|
strcpy(attr->name, dev->attr.name);
|
||||||
dbg("%s - owner = %s, group = %s, mode = %#o", dev->name, dev->owner, dev->group, dev->mode);
|
dbg("%s - owner = %s, group = %s, mode = %#o", dev->attr.name, dev->attr.owner, dev->attr.group, dev->attr.mode);
|
||||||
goto exit;
|
goto exit;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
34
namedev.h
34
namedev.h
|
@ -23,6 +23,8 @@
|
||||||
#ifndef NAMEDEV_H
|
#ifndef NAMEDEV_H
|
||||||
#define NAMEDEV_H
|
#define NAMEDEV_H
|
||||||
|
|
||||||
|
#include "list.h"
|
||||||
|
|
||||||
struct sysfs_class_device;
|
struct sysfs_class_device;
|
||||||
|
|
||||||
/* namedev config files */
|
/* namedev config files */
|
||||||
|
@ -31,6 +33,38 @@ struct sysfs_class_device;
|
||||||
#define NAMEDEV_CONFIG_PERMISSION_FILE "namedev.permissions"
|
#define NAMEDEV_CONFIG_PERMISSION_FILE "namedev.permissions"
|
||||||
#define NAMEDEV_CONFIG_FILE "namedev.config"
|
#define NAMEDEV_CONFIG_FILE "namedev.config"
|
||||||
|
|
||||||
|
enum config_type {
|
||||||
|
KERNEL_NAME = 0, /* must be 0 to let memset() default to this value */
|
||||||
|
LABEL = 1,
|
||||||
|
NUMBER = 2,
|
||||||
|
TOPOLOGY = 3,
|
||||||
|
REPLACE = 4,
|
||||||
|
};
|
||||||
|
|
||||||
|
#define BUS_SIZE 30
|
||||||
|
#define FILE_SIZE 50
|
||||||
|
#define VALUE_SIZE 100
|
||||||
|
#define ID_SIZE 50
|
||||||
|
#define PLACE_SIZE 50
|
||||||
|
|
||||||
|
|
||||||
|
struct config_device {
|
||||||
|
struct list_head node;
|
||||||
|
|
||||||
|
enum config_type type;
|
||||||
|
|
||||||
|
char bus[BUS_SIZE];
|
||||||
|
char sysfs_file[FILE_SIZE];
|
||||||
|
char sysfs_value[VALUE_SIZE];
|
||||||
|
char id[ID_SIZE];
|
||||||
|
char place[PLACE_SIZE];
|
||||||
|
char kernel_name[NAME_SIZE];
|
||||||
|
|
||||||
|
/* what to set the device to */
|
||||||
|
struct device_attr attr;
|
||||||
|
};
|
||||||
|
|
||||||
|
|
||||||
extern int namedev_init(void);
|
extern int namedev_init(void);
|
||||||
extern int namedev_name_device(struct sysfs_class_device *class_dev, struct device_attr *attr);
|
extern int namedev_name_device(struct sysfs_class_device *class_dev, struct device_attr *attr);
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue