Use pci_compatible_match().

use pci_intr_establish_xname() everywhere

branches: 1.20.16; 1.20.18;
KNF. Remove extra spaces. No functional change.

branches: 1.19.6;
make pci_intr_string and eisa_intr_string take a buffer and a length
instead of relying in local static storage.

branches: 1.18.2;
split device_t/softc for all remaining drivers.
replace "struct device *" with "device_t".
use device_xname(), device_unit(), etc.

branches: 1.17.6;
Use pci_aprint_devinfo(9) instead of pci_devinfo+aprint_{normal,naive}
where it looks straightforward, and pci_aprint_devinfo_fancy in a few
others where drivers want to supply their own device names instead
of the pcidevs generated one. More complicated cases, where names
are composed at runtime, are left alone for now. It certainly makes
sense to simplify the drivers here rather than inventing a catch-all API.
This should serve as as example for new drivers, and also ensure
consistent output in the AB_QUIET ("boot -q") case. Also, it avoids
excessive stack usage where drivers attach child devices because the
buffer for the device name is not kept on the local stack anymore.

branches: 1.16.2; 1.16.6;
Replace anonymous constants, 0x10, 0x14, ..., with PCI_BAR(0),
PCI_BAR(1), .... There was no change in the generated assembly. I used
this semantic patch:

@ mapsit @
identifier bar;
expression pact;
@@

(
pci_mapreg_map
|
Cardbus_mapreg_map
)(pact, bar, ...)

@ depends on mapsit @
identifier mapsit.bar;
@@
(
- #define bar 0x10
+ #define bar PCI_BAR(0)
|
- #define bar 0x14
+ #define bar PCI_BAR(1)
|
- #define bar 0x18
+ #define bar PCI_BAR(2)
|
- #define bar 0x1C
+ #define bar PCI_BAR(3)
|
- #define bar 0x20
+ #define bar PCI_BAR(4)
)

Cleanup interrupt establish error messages. Do not mix
aprint_error/aprint_normal/printf calls for a single line.

struct cfdata * -> cfdata_t, no functional changes intended.

branches: 1.13.14;
Remove clause 3 and 4 from TNF licenses

branches: 1.12.2; 1.12.4;
use aprint_*_dev and device_xname

Use device_t and accessors.

branches: 1.10.12; 1.10.16;
machine/{bus,cpu,intr}.h -> sys/{bus,cpu,intr}.h

branches: 1.9.8; 1.9.22; 1.9.24; 1.9.28;
__unused removal on arguments; approved by core.

- sprinkle __unused on function decls.
- fix a couple of unused bugs
- no more -Wno-unused for i386

branches: 1.7.20; 1.7.22;
merge ktrace-lwp.

branches: 1.6.4;
nuke trailing whitespace

branches: 1.5.4; 1.5.6;
Use ANSI function decls and make use of static.

pass string length (= boundary info) to pci_devinfo so that we do not run over
the end of memory region

Fix obvious typos in a call to pci_mapreg_map().

add missing __KERNEL_RCSID()

branches: 1.1.2; 1.1.8;
Add a driver for the Myson Technology MTD803 3-in-1 Fast Ethernet Controller,
provided by Peter Bex in PR 18675.

use pci_intr_establish_xname() everywhere

branches: 1.20.16;
KNF. Remove extra spaces. No functional change.

branches: 1.19.6;
make pci_intr_string and eisa_intr_string take a buffer and a length
instead of relying in local static storage.

branches: 1.18.2;
split device_t/softc for all remaining drivers.
replace "struct device *" with "device_t".
use device_xname(), device_unit(), etc.

branches: 1.17.6;
Use pci_aprint_devinfo(9) instead of pci_devinfo+aprint_{normal,naive}
where it looks straightforward, and pci_aprint_devinfo_fancy in a few
others where drivers want to supply their own device names instead
of the pcidevs generated one. More complicated cases, where names
are composed at runtime, are left alone for now. It certainly makes
sense to simplify the drivers here rather than inventing a catch-all API.
This should serve as as example for new drivers, and also ensure
consistent output in the AB_QUIET ("boot -q") case. Also, it avoids
excessive stack usage where drivers attach child devices because the
buffer for the device name is not kept on the local stack anymore.

branches: 1.16.2; 1.16.6;
Replace anonymous constants, 0x10, 0x14, ..., with PCI_BAR(0),
PCI_BAR(1), .... There was no change in the generated assembly. I used
this semantic patch:

@ mapsit @
identifier bar;
expression pact;
@@

(
pci_mapreg_map
|
Cardbus_mapreg_map
)(pact, bar, ...)

@ depends on mapsit @
identifier mapsit.bar;
@@
(
- #define bar 0x10
+ #define bar PCI_BAR(0)
|
- #define bar 0x14
+ #define bar PCI_BAR(1)
|
- #define bar 0x18
+ #define bar PCI_BAR(2)
|
- #define bar 0x1C
+ #define bar PCI_BAR(3)
|
- #define bar 0x20
+ #define bar PCI_BAR(4)
)

Cleanup interrupt establish error messages. Do not mix
aprint_error/aprint_normal/printf calls for a single line.

struct cfdata * -> cfdata_t, no functional changes intended.

branches: 1.13.14;
Remove clause 3 and 4 from TNF licenses

branches: 1.12.2; 1.12.4;
use aprint_*_dev and device_xname

Use device_t and accessors.

branches: 1.10.12; 1.10.16;
machine/{bus,cpu,intr}.h -> sys/{bus,cpu,intr}.h

branches: 1.9.8; 1.9.22; 1.9.24; 1.9.28;
__unused removal on arguments; approved by core.

- sprinkle __unused on function decls.
- fix a couple of unused bugs
- no more -Wno-unused for i386

branches: 1.7.20; 1.7.22;
merge ktrace-lwp.

branches: 1.6.4;
nuke trailing whitespace

branches: 1.5.4; 1.5.6;
Use ANSI function decls and make use of static.

pass string length (= boundary info) to pci_devinfo so that we do not run over
the end of memory region

Fix obvious typos in a call to pci_mapreg_map().

add missing __KERNEL_RCSID()

branches: 1.1.2; 1.1.8;
Add a driver for the Myson Technology MTD803 3-in-1 Fast Ethernet Controller,
provided by Peter Bex in PR 18675.

KNF. Remove extra spaces. No functional change.

branches: 1.19.6;
make pci_intr_string and eisa_intr_string take a buffer and a length
instead of relying in local static storage.

branches: 1.18.2;
split device_t/softc for all remaining drivers.
replace "struct device *" with "device_t".
use device_xname(), device_unit(), etc.

branches: 1.17.6;
Use pci_aprint_devinfo(9) instead of pci_devinfo+aprint_{normal,naive}
where it looks straightforward, and pci_aprint_devinfo_fancy in a few
others where drivers want to supply their own device names instead
of the pcidevs generated one. More complicated cases, where names
are composed at runtime, are left alone for now. It certainly makes
sense to simplify the drivers here rather than inventing a catch-all API.
This should serve as as example for new drivers, and also ensure
consistent output in the AB_QUIET ("boot -q") case. Also, it avoids
excessive stack usage where drivers attach child devices because the
buffer for the device name is not kept on the local stack anymore.

branches: 1.16.2; 1.16.6;
Replace anonymous constants, 0x10, 0x14, ..., with PCI_BAR(0),
PCI_BAR(1), .... There was no change in the generated assembly. I used
this semantic patch:

@ mapsit @
identifier bar;
expression pact;
@@

(
pci_mapreg_map
|
Cardbus_mapreg_map
)(pact, bar, ...)

@ depends on mapsit @
identifier mapsit.bar;
@@
(
- #define bar 0x10
+ #define bar PCI_BAR(0)
|
- #define bar 0x14
+ #define bar PCI_BAR(1)
|
- #define bar 0x18
+ #define bar PCI_BAR(2)
|
- #define bar 0x1C
+ #define bar PCI_BAR(3)
|
- #define bar 0x20
+ #define bar PCI_BAR(4)
)

Cleanup interrupt establish error messages. Do not mix
aprint_error/aprint_normal/printf calls for a single line.

struct cfdata * -> cfdata_t, no functional changes intended.

branches: 1.13.14;
Remove clause 3 and 4 from TNF licenses

branches: 1.12.2; 1.12.4;
use aprint_*_dev and device_xname

Use device_t and accessors.

branches: 1.10.12; 1.10.16;
machine/{bus,cpu,intr}.h -> sys/{bus,cpu,intr}.h

branches: 1.9.8; 1.9.22; 1.9.24; 1.9.28;
__unused removal on arguments; approved by core.

- sprinkle __unused on function decls.
- fix a couple of unused bugs
- no more -Wno-unused for i386

branches: 1.7.20; 1.7.22;
merge ktrace-lwp.

branches: 1.6.4;
nuke trailing whitespace

branches: 1.5.4; 1.5.6;
Use ANSI function decls and make use of static.

pass string length (= boundary info) to pci_devinfo so that we do not run over
the end of memory region

Fix obvious typos in a call to pci_mapreg_map().

add missing __KERNEL_RCSID()

branches: 1.1.2; 1.1.8;
Add a driver for the Myson Technology MTD803 3-in-1 Fast Ethernet Controller,
provided by Peter Bex in PR 18675.