diff options
author | marius <marius@FreeBSD.org> | 2011-11-22 21:28:20 +0000 |
---|---|---|
committer | marius <marius@FreeBSD.org> | 2011-11-22 21:28:20 +0000 |
commit | 17e14c6132be998c8e8d0c2fc89ff0b66d24999c (patch) | |
tree | 4941db3a873a5f9b2ccc4a64c23bf5cd65516206 /sys/pci/intpm.c | |
parent | 6479d45e1526c99672febf3a29a6a99305871131 (diff) | |
download | FreeBSD-src-17e14c6132be998c8e8d0c2fc89ff0b66d24999c.zip FreeBSD-src-17e14c6132be998c8e8d0c2fc89ff0b66d24999c.tar.gz |
- There's no need to overwrite the default device method with the default
one. Interestingly, these are actually the default for quite some time
(bus_generic_driver_added(9) since r52045 and bus_generic_print_child(9)
since r52045) but even recently added device drivers do this unnecessarily.
Discussed with: jhb, marcel
- While at it, use DEVMETHOD_END.
Discussed with: jhb
- Also while at it, use __FBSDID.
Diffstat (limited to 'sys/pci/intpm.c')
-rw-r--r-- | sys/pci/intpm.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/sys/pci/intpm.c b/sys/pci/intpm.c index 1cbe75d..565b5a2 100644 --- a/sys/pci/intpm.c +++ b/sys/pci/intpm.c @@ -785,9 +785,6 @@ static device_method_t intsmb_methods[] = { DEVMETHOD(device_attach, intsmb_attach), DEVMETHOD(device_detach, intsmb_detach), - /* Bus interface */ - DEVMETHOD(bus_print_child, bus_generic_print_child), - /* SMBus interface */ DEVMETHOD(smbus_callback, intsmb_callback), DEVMETHOD(smbus_quick, intsmb_quick), @@ -801,7 +798,7 @@ static device_method_t intsmb_methods[] = { DEVMETHOD(smbus_bwrite, intsmb_bwrite), DEVMETHOD(smbus_bread, intsmb_bread), - { 0, 0 } + DEVMETHOD_END }; static driver_t intsmb_driver = { |