diff options
author | mdodd <mdodd@FreeBSD.org> | 1999-07-29 01:03:04 +0000 |
---|---|---|
committer | mdodd <mdodd@FreeBSD.org> | 1999-07-29 01:03:04 +0000 |
commit | 1b3328c30076a4872fb76549719ee41c4f633e6a (patch) | |
tree | 08fbd565ea1bf201e2bfcfcd2a4119c82ef665a7 /sys/isa/isa_common.c | |
parent | 0d84c8efb0a2e4b063d0c818f89eb17ee4bc6e47 (diff) | |
download | FreeBSD-src-1b3328c30076a4872fb76549719ee41c4f633e6a.zip FreeBSD-src-1b3328c30076a4872fb76549719ee41c4f633e6a.tar.gz |
Alter the behavior of sys/kern/subr_bus.c:device_print_child()
- device_print_child() either lets the BUS_PRINT_CHILD
method produce the entire device announcement message or
it prints "foo0: not found\n"
Alter sys/kern/subr_bus.c:bus_generic_print_child() to take on
the previous behavior of device_print_child() (printing the
"foo0: <FooDevice 1.1>" bit of the announce message.)
Provide bus_print_child_header() and bus_print_child_footer()
to actually print the output for bus_generic_print_child().
These functions should be used whenever possible (unless you can
just use bus_generic_print_child())
The BUS_PRINT_CHILD method now returns int instead of void.
Modify everything else that defines or uses a BUS_PRINT_CHILD
method to comply with the above changes.
- Devices are 'on' a bus, not 'at' it.
- If a custom BUS_PRINT_CHILD method does the same thing
as bus_generic_print_child(), use bus_generic_print_child()
- Use device_get_nameunit() instead of both
device_get_name() and device_get_unit()
- All BUS_PRINT_CHILD methods return the number of
characters output.
Reviewed by: dfr, peter
Diffstat (limited to 'sys/isa/isa_common.c')
-rw-r--r-- | sys/isa/isa_common.c | 24 |
1 files changed, 14 insertions, 10 deletions
diff --git a/sys/isa/isa_common.c b/sys/isa/isa_common.c index d2207ac..35bd28a 100644 --- a/sys/isa/isa_common.c +++ b/sys/isa/isa_common.c @@ -23,7 +23,7 @@ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF * SUCH DAMAGE. * - * $Id: isa_common.c,v 1.2 1999/05/28 09:24:58 dfr Exp $ + * $Id: isa_common.c,v 1.3 1999/05/30 11:02:09 dfr Exp $ */ /* * Modifications for Intel architecture by Garrett A. Wollman. @@ -152,24 +152,28 @@ isa_print_resources(struct resource_list *rl, const char *name, int type, } } -static void +static int isa_print_child(device_t bus, device_t dev) { struct isa_device *idev = DEVTOISA(dev); struct resource_list *rl = &idev->id_resources; + int retval = 0; + + retval += bus_print_child_header(bus, dev); if (SLIST_FIRST(rl) || idev->id_flags) - printf(" at"); + retval += printf(" at"); - isa_print_resources(rl, "port", SYS_RES_IOPORT, "%#lx"); - isa_print_resources(rl, "iomem", SYS_RES_MEMORY, "%#lx"); - isa_print_resources(rl, "irq", SYS_RES_IRQ, "%ld"); - isa_print_resources(rl, "drq", SYS_RES_DRQ, "%ld"); + retval += isa_print_resources(rl, "port", SYS_RES_IOPORT, "%#lx"); + retval += isa_print_resources(rl, "iomem", SYS_RES_MEMORY, "%#lx"); + retval += isa_print_resources(rl, "irq", SYS_RES_IRQ, "%ld"); + retval += isa_print_resources(rl, "drq", SYS_RES_DRQ, "%ld"); if (idev->id_flags) - printf(" flags %#x", idev->id_flags); + retval += printf(" flags %#x", idev->id_flags); + + retval += bus_print_child_footer(bus, dev); - printf(" on %s%d", - device_get_name(bus), device_get_unit(bus)); + return (retval); } static int |