Commit 4a79f730 authored by H Hartley Sweeten's avatar H Hartley Sweeten Committed by Greg Kroah-Hartman

staging: comedi: remove 'comedi_autoconfig' module parameter

This module parameter is used to enable the auto config mechanism
in the comedi core. Most of the PCI, PCMCIA, and USB drivers have
been converted to use the auto config mechanism and will not attach
if it is disabled.

Since the 'comedi_autoconfig' parameter is defaulted to true, just
remove it so that the comedi drivers that use auto config will
always be able to attach.
Signed-off-by: default avatarH Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent bd7807f9
...@@ -58,11 +58,6 @@ MODULE_PARM_DESC(comedi_debug, ...@@ -58,11 +58,6 @@ MODULE_PARM_DESC(comedi_debug,
); );
#endif #endif
bool comedi_autoconfig = true;
module_param(comedi_autoconfig, bool, S_IRUGO);
MODULE_PARM_DESC(comedi_autoconfig,
"enable drivers to auto-configure comedi devices (default 1)");
static int comedi_num_legacy_minors; static int comedi_num_legacy_minors;
module_param(comedi_num_legacy_minors, int, S_IRUGO); module_param(comedi_num_legacy_minors, int, S_IRUGO);
MODULE_PARM_DESC(comedi_num_legacy_minors, MODULE_PARM_DESC(comedi_num_legacy_minors,
...@@ -2437,14 +2432,6 @@ static int __init comedi_init(void) ...@@ -2437,14 +2432,6 @@ static int __init comedi_init(void)
return -EINVAL; return -EINVAL;
} }
/*
* comedi is unusable if both comedi_autoconfig and
* comedi_num_legacy_minors are zero, so we might as well adjust the
* defaults in that case
*/
if (!comedi_autoconfig && comedi_num_legacy_minors == 0)
comedi_num_legacy_minors = 16;
memset(comedi_file_info_table, 0, memset(comedi_file_info_table, 0,
sizeof(struct comedi_file_info *) * COMEDI_NUM_MINORS); sizeof(struct comedi_file_info *) * COMEDI_NUM_MINORS);
......
...@@ -21,7 +21,6 @@ unsigned int comedi_buf_write_n_allocated(struct comedi_async *async); ...@@ -21,7 +21,6 @@ unsigned int comedi_buf_write_n_allocated(struct comedi_async *async);
extern unsigned int comedi_default_buf_size_kb; extern unsigned int comedi_default_buf_size_kb;
extern unsigned int comedi_default_buf_maxsize_kb; extern unsigned int comedi_default_buf_maxsize_kb;
extern bool comedi_autoconfig;
/* drivers.c */ /* drivers.c */
......
...@@ -410,9 +410,6 @@ int comedi_auto_config(struct device *hardware_device, ...@@ -410,9 +410,6 @@ int comedi_auto_config(struct device *hardware_device,
struct comedi_device *comedi_dev; struct comedi_device *comedi_dev;
int ret; int ret;
if (!comedi_autoconfig)
return 0;
if (!driver->auto_attach) { if (!driver->auto_attach) {
dev_warn(hardware_device, dev_warn(hardware_device,
"BUG! comedi driver '%s' has no auto_attach handler\n", "BUG! comedi driver '%s' has no auto_attach handler\n",
......
...@@ -38,12 +38,6 @@ Author: H Hartley Sweeten <hsweeten@visionengravers.com> ...@@ -38,12 +38,6 @@ Author: H Hartley Sweeten <hsweeten@visionengravers.com>
Updated: Thu, 02 Aug 2012 14:27:46 -0700 Updated: Thu, 02 Aug 2012 14:27:46 -0700
Status: untested Status: untested
This driver only attaches using the PCI PnP auto config support
in the comedi core. The module parameter 'comedi_autoconfig'
must be 1 (default) to enable this feature. The COMEDI_DEVCONFIG
ioctl, used by the comedi_config utility, is not supported by
this driver.
The PCI-7230, PCI-7432 and PCI-7433 boards also support external The PCI-7230, PCI-7432 and PCI-7433 boards also support external
interrupt signals on digital input channels 0 and 1. The PCI-7233 interrupt signals on digital input channels 0 and 1. The PCI-7233
has dual-interrupt sources for change-of-state (COS) on any 16 has dual-interrupt sources for change-of-state (COS) on any 16
...@@ -51,7 +45,7 @@ digital input channels of LSB and for COS on any 16 digital input ...@@ -51,7 +45,7 @@ digital input channels of LSB and for COS on any 16 digital input
lines of MSB. Interrupts are not currently supported by this lines of MSB. Interrupts are not currently supported by this
driver. driver.
Configuration Options: not applicable Configuration Options: not applicable, uses comedi PCI auto config
*/ */
#include <linux/pci.h> #include <linux/pci.h>
......
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