staging: comedi: amplc_dio200: split dio200_boards[] into ISA & PCI
Split `dio200_boards[]` into `dio200_isa_boards[]` for ISA cards and `dio200_pci_boards[]` for PCI cards. Only initialize the board-name look-up members of `struct comedi_driver amplc_dio200_driver` if the ISA part of the driver is enabled in the kernel config (`CONFIG_COMEDI_AMPLC_DIO200_ISA`) using the array of ISA boards (`dio200_isa_boards[]`). The driver doesn't allow manual configuration of PCI devices, so there is no point having the comedi core match the names of the PCI boards before it calls our driver's legacy attach routine (`dio200_attach()`). Signed-off-by: Ian Abbott <abbotti@mev.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
d52361c6bd
commit
3d9bfccd13
|
@ -416,95 +416,98 @@ struct dio200_board {
|
||||||
unsigned int mainsize;
|
unsigned int mainsize;
|
||||||
};
|
};
|
||||||
|
|
||||||
static const struct dio200_board dio200_boards[] = {
|
|
||||||
#if DO_ISA
|
#if DO_ISA
|
||||||
|
static const struct dio200_board dio200_isa_boards[] = {
|
||||||
{
|
{
|
||||||
.name = "pc212e",
|
.name = "pc212e",
|
||||||
.bustype = isa_bustype,
|
.bustype = isa_bustype,
|
||||||
.model = pc212e_model,
|
.model = pc212e_model,
|
||||||
.layout = pc212_layout,
|
.layout = pc212_layout,
|
||||||
.mainsize = DIO200_IO_SIZE,
|
.mainsize = DIO200_IO_SIZE,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
.name = "pc214e",
|
.name = "pc214e",
|
||||||
.bustype = isa_bustype,
|
.bustype = isa_bustype,
|
||||||
.model = pc214e_model,
|
.model = pc214e_model,
|
||||||
.layout = pc214_layout,
|
.layout = pc214_layout,
|
||||||
.mainsize = DIO200_IO_SIZE,
|
.mainsize = DIO200_IO_SIZE,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
.name = "pc215e",
|
.name = "pc215e",
|
||||||
.bustype = isa_bustype,
|
.bustype = isa_bustype,
|
||||||
.model = pc215e_model,
|
.model = pc215e_model,
|
||||||
.layout = pc215_layout,
|
.layout = pc215_layout,
|
||||||
.mainsize = DIO200_IO_SIZE,
|
.mainsize = DIO200_IO_SIZE,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
.name = "pc218e",
|
.name = "pc218e",
|
||||||
.bustype = isa_bustype,
|
.bustype = isa_bustype,
|
||||||
.model = pc218e_model,
|
.model = pc218e_model,
|
||||||
.layout = pc218_layout,
|
.layout = pc218_layout,
|
||||||
.mainsize = DIO200_IO_SIZE,
|
.mainsize = DIO200_IO_SIZE,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
.name = "pc272e",
|
.name = "pc272e",
|
||||||
.bustype = isa_bustype,
|
.bustype = isa_bustype,
|
||||||
.model = pc272e_model,
|
.model = pc272e_model,
|
||||||
.layout = pc272_layout,
|
.layout = pc272_layout,
|
||||||
.mainsize = DIO200_IO_SIZE,
|
.mainsize = DIO200_IO_SIZE,
|
||||||
},
|
},
|
||||||
#endif
|
|
||||||
#if DO_PCI
|
|
||||||
{
|
|
||||||
.name = "pci215",
|
|
||||||
.devid = PCI_DEVICE_ID_AMPLICON_PCI215,
|
|
||||||
.bustype = pci_bustype,
|
|
||||||
.model = pci215_model,
|
|
||||||
.layout = pc215_layout,
|
|
||||||
.mainbar = 2,
|
|
||||||
.mainsize = DIO200_IO_SIZE,
|
|
||||||
},
|
|
||||||
{
|
|
||||||
.name = "pci272",
|
|
||||||
.devid = PCI_DEVICE_ID_AMPLICON_PCI272,
|
|
||||||
.bustype = pci_bustype,
|
|
||||||
.model = pci272_model,
|
|
||||||
.layout = pc272_layout,
|
|
||||||
.mainbar = 2,
|
|
||||||
.mainsize = DIO200_IO_SIZE,
|
|
||||||
},
|
|
||||||
{
|
|
||||||
.name = "pcie215",
|
|
||||||
.devid = PCI_DEVICE_ID_AMPLICON_PCIE215,
|
|
||||||
.bustype = pci_bustype,
|
|
||||||
.model = pcie215_model,
|
|
||||||
.layout = pcie215_layout,
|
|
||||||
.mainbar = 1,
|
|
||||||
.mainshift = 3,
|
|
||||||
.mainsize = DIO200_PCIE_IO_SIZE,
|
|
||||||
},
|
|
||||||
{
|
|
||||||
.name = "pcie236",
|
|
||||||
.devid = PCI_DEVICE_ID_AMPLICON_PCIE236,
|
|
||||||
.bustype = pci_bustype,
|
|
||||||
.model = pcie236_model,
|
|
||||||
.layout = pcie236_layout,
|
|
||||||
.mainbar = 1,
|
|
||||||
.mainshift = 3,
|
|
||||||
.mainsize = DIO200_PCIE_IO_SIZE,
|
|
||||||
},
|
|
||||||
{
|
|
||||||
.name = "pcie296",
|
|
||||||
.devid = PCI_DEVICE_ID_AMPLICON_PCIE296,
|
|
||||||
.bustype = pci_bustype,
|
|
||||||
.model = pcie296_model,
|
|
||||||
.layout = pcie296_layout,
|
|
||||||
.mainbar = 1,
|
|
||||||
.mainshift = 3,
|
|
||||||
.mainsize = DIO200_PCIE_IO_SIZE,
|
|
||||||
},
|
|
||||||
#endif
|
|
||||||
};
|
};
|
||||||
|
#endif
|
||||||
|
|
||||||
|
#if DO_PCI
|
||||||
|
static const struct dio200_board dio200_pci_boards[] = {
|
||||||
|
{
|
||||||
|
.name = "pci215",
|
||||||
|
.devid = PCI_DEVICE_ID_AMPLICON_PCI215,
|
||||||
|
.bustype = pci_bustype,
|
||||||
|
.model = pci215_model,
|
||||||
|
.layout = pc215_layout,
|
||||||
|
.mainbar = 2,
|
||||||
|
.mainsize = DIO200_IO_SIZE,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
.name = "pci272",
|
||||||
|
.devid = PCI_DEVICE_ID_AMPLICON_PCI272,
|
||||||
|
.bustype = pci_bustype,
|
||||||
|
.model = pci272_model,
|
||||||
|
.layout = pc272_layout,
|
||||||
|
.mainbar = 2,
|
||||||
|
.mainsize = DIO200_IO_SIZE,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
.name = "pcie215",
|
||||||
|
.devid = PCI_DEVICE_ID_AMPLICON_PCIE215,
|
||||||
|
.bustype = pci_bustype,
|
||||||
|
.model = pcie215_model,
|
||||||
|
.layout = pcie215_layout,
|
||||||
|
.mainbar = 1,
|
||||||
|
.mainshift = 3,
|
||||||
|
.mainsize = DIO200_PCIE_IO_SIZE,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
.name = "pcie236",
|
||||||
|
.devid = PCI_DEVICE_ID_AMPLICON_PCIE236,
|
||||||
|
.bustype = pci_bustype,
|
||||||
|
.model = pcie236_model,
|
||||||
|
.layout = pcie236_layout,
|
||||||
|
.mainbar = 1,
|
||||||
|
.mainshift = 3,
|
||||||
|
.mainsize = DIO200_PCIE_IO_SIZE,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
.name = "pcie296",
|
||||||
|
.devid = PCI_DEVICE_ID_AMPLICON_PCIE296,
|
||||||
|
.bustype = pci_bustype,
|
||||||
|
.model = pcie296_model,
|
||||||
|
.layout = pcie296_layout,
|
||||||
|
.mainbar = 1,
|
||||||
|
.mainshift = 3,
|
||||||
|
.mainsize = DIO200_PCIE_IO_SIZE,
|
||||||
|
},
|
||||||
|
};
|
||||||
|
#endif
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Layout descriptions - some ISA and PCI board descriptions share the same
|
* Layout descriptions - some ISA and PCI board descriptions share the same
|
||||||
|
@ -737,10 +740,10 @@ dio200_find_pci_board(struct pci_dev *pci_dev)
|
||||||
{
|
{
|
||||||
unsigned int i;
|
unsigned int i;
|
||||||
|
|
||||||
for (i = 0; i < ARRAY_SIZE(dio200_boards); i++)
|
for (i = 0; i < ARRAY_SIZE(dio200_pci_boards); i++)
|
||||||
if (is_pci_board(&dio200_boards[i]) &&
|
if (is_pci_board(&dio200_pci_boards[i]) &&
|
||||||
pci_dev->device == dio200_boards[i].devid)
|
pci_dev->device == dio200_pci_boards[i].devid)
|
||||||
return &dio200_boards[i];
|
return &dio200_pci_boards[i];
|
||||||
return NULL;
|
return NULL;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -2046,9 +2049,11 @@ static struct comedi_driver amplc_dio200_driver = {
|
||||||
.attach = dio200_attach,
|
.attach = dio200_attach,
|
||||||
.auto_attach = dio200_auto_attach,
|
.auto_attach = dio200_auto_attach,
|
||||||
.detach = dio200_detach,
|
.detach = dio200_detach,
|
||||||
.board_name = &dio200_boards[0].name,
|
#if DO_ISA
|
||||||
|
.board_name = &dio200_isa_boards[0].name,
|
||||||
.offset = sizeof(struct dio200_board),
|
.offset = sizeof(struct dio200_board),
|
||||||
.num_names = ARRAY_SIZE(dio200_boards),
|
.num_names = ARRAY_SIZE(dio200_isa_boards),
|
||||||
|
#endif
|
||||||
};
|
};
|
||||||
|
|
||||||
#if DO_PCI
|
#if DO_PCI
|
||||||
|
|
Loading…
Reference in New Issue