diff options
| author | Benoit Taine <benoit.taine@lip6.fr> | 2014-08-08 09:56:03 -0400 |
|---|---|---|
| committer | Bjorn Helgaas <bhelgaas@google.com> | 2014-08-12 14:15:14 -0400 |
| commit | 9baa3c34ac4e27f7e062f266f50cc5dbea26a6c1 (patch) | |
| tree | 2c6f874e1eb4dbb415ad96b6a39f1f7982655412 /drivers/block | |
| parent | c8d6637d0497d62093dbba0694c7b3a80b79bfe1 (diff) | |
PCI: Remove DEFINE_PCI_DEVICE_TABLE macro use
We should prefer `struct pci_device_id` over `DEFINE_PCI_DEVICE_TABLE` to
meet kernel coding style guidelines. This issue was reported by checkpatch.
A simplified version of the semantic patch that makes this change is as
follows (http://coccinelle.lip6.fr/):
// <smpl>
@@
identifier i;
declarer name DEFINE_PCI_DEVICE_TABLE;
initializer z;
@@
- DEFINE_PCI_DEVICE_TABLE(i)
+ const struct pci_device_id i[]
= z;
// </smpl>
[bhelgaas: add semantic patch]
Signed-off-by: Benoit Taine <benoit.taine@lip6.fr>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Diffstat (limited to 'drivers/block')
| -rw-r--r-- | drivers/block/mtip32xx/mtip32xx.c | 2 | ||||
| -rw-r--r-- | drivers/block/rsxx/core.c | 2 | ||||
| -rw-r--r-- | drivers/block/skd_main.c | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/drivers/block/mtip32xx/mtip32xx.c b/drivers/block/mtip32xx/mtip32xx.c index 295f3afbbef5..db1e9560d8a7 100644 --- a/drivers/block/mtip32xx/mtip32xx.c +++ b/drivers/block/mtip32xx/mtip32xx.c | |||
| @@ -4632,7 +4632,7 @@ static void mtip_pci_shutdown(struct pci_dev *pdev) | |||
| 4632 | } | 4632 | } |
| 4633 | 4633 | ||
| 4634 | /* Table of device ids supported by this driver. */ | 4634 | /* Table of device ids supported by this driver. */ |
| 4635 | static DEFINE_PCI_DEVICE_TABLE(mtip_pci_tbl) = { | 4635 | static const struct pci_device_id mtip_pci_tbl[] = { |
| 4636 | { PCI_DEVICE(PCI_VENDOR_ID_MICRON, P320H_DEVICE_ID) }, | 4636 | { PCI_DEVICE(PCI_VENDOR_ID_MICRON, P320H_DEVICE_ID) }, |
| 4637 | { PCI_DEVICE(PCI_VENDOR_ID_MICRON, P320M_DEVICE_ID) }, | 4637 | { PCI_DEVICE(PCI_VENDOR_ID_MICRON, P320M_DEVICE_ID) }, |
| 4638 | { PCI_DEVICE(PCI_VENDOR_ID_MICRON, P320S_DEVICE_ID) }, | 4638 | { PCI_DEVICE(PCI_VENDOR_ID_MICRON, P320S_DEVICE_ID) }, |
diff --git a/drivers/block/rsxx/core.c b/drivers/block/rsxx/core.c index a8de2eec6ff3..820b4009d5f7 100644 --- a/drivers/block/rsxx/core.c +++ b/drivers/block/rsxx/core.c | |||
| @@ -1137,7 +1137,7 @@ static const struct pci_error_handlers rsxx_err_handler = { | |||
| 1137 | .slot_reset = rsxx_slot_reset, | 1137 | .slot_reset = rsxx_slot_reset, |
| 1138 | }; | 1138 | }; |
| 1139 | 1139 | ||
| 1140 | static DEFINE_PCI_DEVICE_TABLE(rsxx_pci_ids) = { | 1140 | static const struct pci_device_id rsxx_pci_ids[] = { |
| 1141 | {PCI_DEVICE(PCI_VENDOR_ID_IBM, PCI_DEVICE_ID_FS70_FLASH)}, | 1141 | {PCI_DEVICE(PCI_VENDOR_ID_IBM, PCI_DEVICE_ID_FS70_FLASH)}, |
| 1142 | {PCI_DEVICE(PCI_VENDOR_ID_IBM, PCI_DEVICE_ID_FS80_FLASH)}, | 1142 | {PCI_DEVICE(PCI_VENDOR_ID_IBM, PCI_DEVICE_ID_FS80_FLASH)}, |
| 1143 | {0,}, | 1143 | {0,}, |
diff --git a/drivers/block/skd_main.c b/drivers/block/skd_main.c index f0a089df85cc..8fcdcfb4b472 100644 --- a/drivers/block/skd_main.c +++ b/drivers/block/skd_main.c | |||
| @@ -4766,7 +4766,7 @@ static const struct block_device_operations skd_blockdev_ops = { | |||
| 4766 | ***************************************************************************** | 4766 | ***************************************************************************** |
| 4767 | */ | 4767 | */ |
| 4768 | 4768 | ||
| 4769 | static DEFINE_PCI_DEVICE_TABLE(skd_pci_tbl) = { | 4769 | static const struct pci_device_id skd_pci_tbl[] = { |
| 4770 | { PCI_VENDOR_ID_STEC, PCI_DEVICE_ID_S1120, | 4770 | { PCI_VENDOR_ID_STEC, PCI_DEVICE_ID_S1120, |
| 4771 | PCI_ANY_ID, PCI_ANY_ID, 0, 0, }, | 4771 | PCI_ANY_ID, PCI_ANY_ID, 0, 0, }, |
| 4772 | { 0 } /* terminate list */ | 4772 | { 0 } /* terminate list */ |
