aboutsummaryrefslogtreecommitdiff
path: root/sys/dev/aac
diff options
context:
space:
mode:
authorConrad Meyer <cem@FreeBSD.org>2018-08-19 00:22:21 +0000
committerConrad Meyer <cem@FreeBSD.org>2018-08-19 00:22:21 +0000
commitfaa319436f0d548cff97c2c8c763538997b63f34 (patch)
tree83d929a61077c1eee88b03c7698a3fcbfab6df73 /sys/dev/aac
parent94d0f0877db6bf0511dc0198e78b870618f42d3e (diff)
downloadsrc-faa319436f0d548cff97c2c8c763538997b63f34.tar.gz
src-faa319436f0d548cff97c2c8c763538997b63f34.zip
Remove unused and easy to misuse PNP macro parameter
Inspired by r338025, just remove the element size parameter to the MODULE_PNP_INFO macro entirely. The 'table' parameter is now required to have correct pointer (or array) type. Since all invocations of the macro already had this property and the emitted PNP data continues to include the element size, there is no functional change. Mostly done with the coccinelle 'spatch' tool: $ cat modpnpsize0.cocci @normaltables@ identifier b,c; expression a,d,e; declarer MODULE_PNP_INFO; @@ MODULE_PNP_INFO(a,b,c,d, -sizeof(d[0]), e); @singletons@ identifier b,c,d; expression a; declarer MODULE_PNP_INFO; @@ MODULE_PNP_INFO(a,b,c,&d, -sizeof(d), 1); $ rg -l MODULE_PNP_INFO -- sys | \ xargs spatch --in-place --sp-file modpnpsize0.cocci (Note that coccinelle invokes diff(1) via a PATH search and expects diff to tolerate the -B flag, which BSD diff does not. So I had to link gdiff into PATH as diff to use spatch.) Tinderbox'd (-DMAKE_JUST_KERNELS).
Notes
Notes: svn path=/head/; revision=338035
Diffstat (limited to 'sys/dev/aac')
-rw-r--r--sys/dev/aac/aac_pci.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/dev/aac/aac_pci.c b/sys/dev/aac/aac_pci.c
index 399e55a9a47f..738bfbf79167 100644
--- a/sys/dev/aac/aac_pci.c
+++ b/sys/dev/aac/aac_pci.c
@@ -494,7 +494,7 @@ static driver_t aacch_driver = {
static devclass_t aacch_devclass;
DRIVER_MODULE(aacch, pci, aacch_driver, aacch_devclass, NULL, NULL);
MODULE_PNP_INFO("U16:vendor;U16:device;", pci, aac,
- aac_identifiers, sizeof(aac_identifiers[0]), nitems(aac_identifiers) - 1);
+ aac_identifiers, nitems(aac_identifiers) - 1);
static int
aacch_probe(device_t dev)