cvs commit: src/sys/compat/ndis kern_ndis.c subr_ndis.c

[M. Warner Losh]

In message: <20040325234024.GJ26131 at elvis.mu.org>
            Alfred Perlstein <alfred at freebsd.org> writes:
: * M. Warner Losh <imp at bsdimp.com> [040325 14:11] wrote:
: > In message: <xzpvfksfztc.fsf at dwp.des.no>
: >             des at des.no (Dag-Erling Sm?rgrav) writes:
: > : Bill Paul <wpaul at FreeBSD.org> writes:
: > : 
: > : des at dwp ~% gf BUS_GET_RESOURCE_LIST
: > : qsrc/sys/compat/ndis/kern_ndis.c:       brl = BUS_GET_RESOURCE_LIST(device_get_parent(dev), dev);
: > : src/sys/dev/if_ndis/if_ndis_pci.c:      rl = BUS_GET_RESOURCE_LIST(device_get_parent(dev), dev);
: > : src/sys/kern/subr_bus.c:        rl = BUS_GET_RESOURCE_LIST(dev, child);
: > : src/sys/kern/subr_bus.c:        rl = BUS_GET_RESOURCE_LIST(dev, child);
: > : src/sys/kern/subr_bus.c:        rl = BUS_GET_RESOURCE_LIST(dev, child);
: > : src/sys/kern/subr_bus.c:        rl = BUS_GET_RESOURCE_LIST(dev, child);
: > : src/sys/kern/subr_bus.c:        rl = BUS_GET_RESOURCE_LIST(dev, child);
: > 
: > Acutally, there are alot of downstream users of rl.
: 
: :(
: 
: Shouldn't we have macros:
: 
: BUS_GET_RESOURCE_NEXT ?
: 
: (instead of SLIST_NEXT)

We already do.  bus_generic_rl_get_resource, et al.  The problem here
is that the pci bus never intended its resources to be exposed outside
of the bus layer.  But windows drivers need it, so there needs to be
some tweaks need to be made.

It looks like I was incorrect about the downstream uses of it.  I
thought that pci and pccard did that, but it appears that they now use
the common functions that matt dodd moved into subr_bus a while ago.

Warner