Use module_param() instead of MODULE_PARM() -- cannot be mixed. Signed-off-by: Randy Dunlap diffstat:= drivers/parport/parport_pc.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff -Naurp ./drivers/parport/parport_pc.c~parport_modprm ./drivers/parport/parport_pc.c --- ./drivers/parport/parport_pc.c~parport_modprm 2004-12-24 13:35:39.000000000 -0800 +++ ./drivers/parport/parport_pc.c 2004-12-25 21:03:35.123264504 -0800 @@ -3176,7 +3176,6 @@ static int __init parport_init_mode_setu #ifdef MODULE static const char *irq[PARPORT_PC_MAX_PORTS]; static const char *dma[PARPORT_PC_MAX_PORTS]; -static char *init_mode; MODULE_PARM_DESC(io, "Base I/O address (SPP regs)"); module_param_array(io, int, NULL, 0); @@ -3192,8 +3191,9 @@ MODULE_PARM_DESC(verbose_probing, "Log c module_param(verbose_probing, int, 0644); #endif #ifdef CONFIG_PCI +static char *init_mode; MODULE_PARM_DESC(init_mode, "Initialise mode for VIA VT8231 port (spp, ps2, epp, ecp or ecpepp)"); -MODULE_PARM(init_mode, "s"); +module_param(init_mode, charp, 0); #endif static int __init parse_parport_params(void)