Commit 07563c71 authored by Michael Tokarev's avatar Michael Tokarev Committed by Linus Torvalds

[PATCH] EISA bus MODALIAS attributes support

Add modalias attribute support for the almost forgotten now EISA bus and
(at least some) EISA-aware modules.

The modalias entry looks like (for an 3c509 NIC):

 eisa:sTCM5093

and the in-module alias like:

 eisa:sTCM5093*

The patch moves struct eisa_device_id declaration from include/linux/eisa.h
to include/linux/mod_devicetable.h (so that the former now #includes the
latter), adds proper MODULE_DEVICE_TABLE(eisa, ...) statements for all
drivers with EISA IDs I found (some drivers already have that DEVICE_TABLE
declared), and adds recognision of __mod_eisa_device_table to
scripts/mod/file2alias.c so that proper modules.alias will be generated.

There's no support for /lib/modules/$kver/modules.eisamap, as it's not used
by any existing tools, and because with in-kernel modalias mechanism those
maps are obsolete anyway.

The rationale for this patch is:

 a) to make EISA bus to act as other busses with modalias
    support, to unify driver loading

 b) to foget about EISA finally - with this patch, kernel
    (who still supports EISA) will be the only one who knows
    how to choose the necessary drivers for this bus ;)

[akpm@osdl.org: fix the kbuild bit]
Signed-off-by: default avatarMichael Tokarev <mjt@tls.msk.ru>
Cc: Rusty Russell <rusty@rustcorp.com.au>
Cc: Randy Dunlap <rdunlap@xenotime.net>
Acked-the-net-bits-by: default avatarJeff Garzik <jeff@garzik.org>
Acked-the-tulip-bit-by: default avatarValerie Henson <val_henson@linux.intel.com>
Cc: James Bottomley <James.Bottomley@steeleye.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent ebba5f9f
...@@ -128,9 +128,23 @@ static int eisa_bus_match (struct device *dev, struct device_driver *drv) ...@@ -128,9 +128,23 @@ static int eisa_bus_match (struct device *dev, struct device_driver *drv)
return 0; return 0;
} }
static int eisa_bus_uevent(struct device *dev, char **envp, int num_envp,
char *buffer, int buffer_size)
{
struct eisa_device *edev = to_eisa_device(dev);
int i = 0;
int length = 0;
add_uevent_var(envp, num_envp, &i, buffer, buffer_size, &length,
"MODALIAS=" EISA_DEVICE_MODALIAS_FMT, edev->id.sig);
envp[i] = NULL;
return 0;
}
struct bus_type eisa_bus_type = { struct bus_type eisa_bus_type = {
.name = "eisa", .name = "eisa",
.match = eisa_bus_match, .match = eisa_bus_match,
.uevent = eisa_bus_uevent,
}; };
int eisa_driver_register (struct eisa_driver *edrv) int eisa_driver_register (struct eisa_driver *edrv)
...@@ -160,6 +174,14 @@ static ssize_t eisa_show_state (struct device *dev, struct device_attribute *att ...@@ -160,6 +174,14 @@ static ssize_t eisa_show_state (struct device *dev, struct device_attribute *att
static DEVICE_ATTR(enabled, S_IRUGO, eisa_show_state, NULL); static DEVICE_ATTR(enabled, S_IRUGO, eisa_show_state, NULL);
static ssize_t eisa_show_modalias (struct device *dev, struct device_attribute *attr, char *buf)
{
struct eisa_device *edev = to_eisa_device (dev);
return sprintf (buf, EISA_DEVICE_MODALIAS_FMT "\n", edev->id.sig);
}
static DEVICE_ATTR(modalias, S_IRUGO, eisa_show_modalias, NULL);
static int __init eisa_init_device (struct eisa_root_device *root, static int __init eisa_init_device (struct eisa_root_device *root,
struct eisa_device *edev, struct eisa_device *edev,
int slot) int slot)
...@@ -209,6 +231,7 @@ static int __init eisa_register_device (struct eisa_device *edev) ...@@ -209,6 +231,7 @@ static int __init eisa_register_device (struct eisa_device *edev)
device_create_file (&edev->dev, &dev_attr_signature); device_create_file (&edev->dev, &dev_attr_signature);
device_create_file (&edev->dev, &dev_attr_enabled); device_create_file (&edev->dev, &dev_attr_enabled);
device_create_file (&edev->dev, &dev_attr_modalias);
return 0; return 0;
} }
......
...@@ -225,6 +225,7 @@ static struct eisa_device_id el3_eisa_ids[] = { ...@@ -225,6 +225,7 @@ static struct eisa_device_id el3_eisa_ids[] = {
{ "TCM5095" }, { "TCM5095" },
{ "" } { "" }
}; };
MODULE_DEVICE_TABLE(eisa, el3_eisa_ids);
static int el3_eisa_probe (struct device *device); static int el3_eisa_probe (struct device *device);
......
...@@ -851,6 +851,7 @@ static struct eisa_device_id vortex_eisa_ids[] = { ...@@ -851,6 +851,7 @@ static struct eisa_device_id vortex_eisa_ids[] = {
{ "TCM5970", CH_3C597 }, { "TCM5970", CH_3C597 },
{ "" } { "" }
}; };
MODULE_DEVICE_TABLE(eisa, vortex_eisa_ids);
static int vortex_eisa_probe(struct device *device); static int vortex_eisa_probe(struct device *device);
static int vortex_eisa_remove(struct device *device); static int vortex_eisa_remove(struct device *device);
......
...@@ -343,6 +343,7 @@ static struct eisa_device_id ne3210_ids[] = { ...@@ -343,6 +343,7 @@ static struct eisa_device_id ne3210_ids[] = {
{ "NVL1801" }, { "NVL1801" },
{ "" }, { "" },
}; };
MODULE_DEVICE_TABLE(eisa, ne3210_ids);
static struct eisa_driver ne3210_eisa_driver = { static struct eisa_driver ne3210_eisa_driver = {
.id_table = ne3210_ids, .id_table = ne3210_ids,
......
...@@ -2114,6 +2114,7 @@ static struct eisa_device_id de4x5_eisa_ids[] = { ...@@ -2114,6 +2114,7 @@ static struct eisa_device_id de4x5_eisa_ids[] = {
{ "DEC4250", 0 }, /* 0 is the board name index... */ { "DEC4250", 0 }, /* 0 is the board name index... */
{ "" } { "" }
}; };
MODULE_DEVICE_TABLE(eisa, de4x5_eisa_ids);
static struct eisa_driver de4x5_eisa_driver = { static struct eisa_driver de4x5_eisa_driver = {
.id_table = de4x5_eisa_ids, .id_table = de4x5_eisa_ids,
......
...@@ -681,6 +681,7 @@ static struct eisa_device_id aha1740_ids[] = { ...@@ -681,6 +681,7 @@ static struct eisa_device_id aha1740_ids[] = {
{ "ADP0400" }, /* 1744 */ { "ADP0400" }, /* 1744 */
{ "" } { "" }
}; };
MODULE_DEVICE_TABLE(eisa, aha1740_ids);
static struct eisa_driver aha1740_driver = { static struct eisa_driver aha1740_driver = {
.id_table = aha1740_ids, .id_table = aha1740_ids,
......
...@@ -132,6 +132,7 @@ static struct eisa_device_id aic7770_ids[] = { ...@@ -132,6 +132,7 @@ static struct eisa_device_id aic7770_ids[] = {
{ "ADP7770", 5 }, /* AIC7770 generic */ { "ADP7770", 5 }, /* AIC7770 generic */
{ "" } { "" }
}; };
MODULE_DEVICE_TABLE(eisa, aic7770_ids);
static struct eisa_driver aic7770_driver = { static struct eisa_driver aic7770_driver = {
.id_table = aic7770_ids, .id_table = aic7770_ids,
......
...@@ -282,6 +282,7 @@ static struct eisa_device_id sim710_eisa_ids[] = { ...@@ -282,6 +282,7 @@ static struct eisa_device_id sim710_eisa_ids[] = {
{ "HWP0C80" }, { "HWP0C80" },
{ "" } { "" }
}; };
MODULE_DEVICE_TABLE(eisa, sim710_eisa_ids);
static __init int static __init int
sim710_eisa_probe(struct device *dev) sim710_eisa_probe(struct device *dev)
......
...@@ -3,8 +3,8 @@ ...@@ -3,8 +3,8 @@
#include <linux/ioport.h> #include <linux/ioport.h>
#include <linux/device.h> #include <linux/device.h>
#include <linux/mod_devicetable.h>
#define EISA_SIG_LEN 8
#define EISA_MAX_SLOTS 8 #define EISA_MAX_SLOTS 8
#define EISA_MAX_RESOURCES 4 #define EISA_MAX_RESOURCES 4
...@@ -27,12 +27,6 @@ ...@@ -27,12 +27,6 @@
#define EISA_CONFIG_ENABLED 1 #define EISA_CONFIG_ENABLED 1
#define EISA_CONFIG_FORCED 2 #define EISA_CONFIG_FORCED 2
/* The EISA signature, in ASCII form, null terminated */
struct eisa_device_id {
char sig[EISA_SIG_LEN];
unsigned long driver_data;
};
/* There is not much we can say about an EISA device, apart from /* There is not much we can say about an EISA device, apart from
* signature, slot number, and base address. dma_mask is set by * signature, slot number, and base address. dma_mask is set by
* default to parent device mask..*/ * default to parent device mask..*/
......
...@@ -308,4 +308,16 @@ struct input_device_id { ...@@ -308,4 +308,16 @@ struct input_device_id {
kernel_ulong_t driver_info; kernel_ulong_t driver_info;
}; };
/* EISA */
#define EISA_SIG_LEN 8
/* The EISA signature, in ASCII form, null terminated */
struct eisa_device_id {
char sig[EISA_SIG_LEN];
kernel_ulong_t driver_data;
};
#define EISA_DEVICE_MODALIAS_FMT "eisa:s%s"
#endif /* LINUX_MOD_DEVICETABLE_H */ #endif /* LINUX_MOD_DEVICETABLE_H */
...@@ -444,6 +444,14 @@ static int do_input_entry(const char *filename, struct input_device_id *id, ...@@ -444,6 +444,14 @@ static int do_input_entry(const char *filename, struct input_device_id *id,
return 1; return 1;
} }
static int do_eisa_entry(const char *filename, struct eisa_device_id *eisa,
char *alias)
{
if (eisa->sig[0])
sprintf(alias, EISA_DEVICE_MODALIAS_FMT "*", eisa->sig);
return 1;
}
/* Ignore any prefix, eg. v850 prepends _ */ /* Ignore any prefix, eg. v850 prepends _ */
static inline int sym_is(const char *symbol, const char *name) static inline int sym_is(const char *symbol, const char *name)
{ {
...@@ -547,6 +555,10 @@ void handle_moddevtable(struct module *mod, struct elf_info *info, ...@@ -547,6 +555,10 @@ void handle_moddevtable(struct module *mod, struct elf_info *info,
do_table(symval, sym->st_size, do_table(symval, sym->st_size,
sizeof(struct input_device_id), "input", sizeof(struct input_device_id), "input",
do_input_entry, mod); do_input_entry, mod);
else if (sym_is(symname, "__mod_eisa_device_table"))
do_table(symval, sym->st_size,
sizeof(struct eisa_device_id), "eisa",
do_eisa_entry, mod);
} }
/* Now add out buffered information to the generated C source */ /* Now add out buffered information to the generated C source */
......
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