cvs commit: src/sys/amd64/isa atpic.c src/sys/contrib/dev/oltr
if_oltr.c src/sys/dev/aac aac_pci.c src/sys/dev/acpica/Os
Bill Paul
wpaul at FreeBSD.ORG
Sun Mar 21 11:46:56 PST 2004
> njl 2004/03/17 09:50:55 PST
>
> FreeBSD src repository
>
> Modified files:
[...]
> sys/dev/if_ndis if_ndis_pccard.c if_ndis_pci.c
[...]
> Log:
> Convert callers to the new bus_alloc_resource_any(9) API.
>
> Submitted by: Mark Santcroos <marks at ripe.net>
> Reviewed by: imp, dfr, bde
:(
By doing this, you have fixed it to people can't use the NDISulator
code on 5.2-RELEASE or 5.2.1-RELEASE. I think it is hardly worth it
to break source code compatibility in this case just to avoid having
to type a few extra arguments. At the very least, bus_alloc_resource_any()
could have been implemented as a macro: then I could just test for its
existence and #define it myself so the code would run on older systems.
I'm very susprised that bde never raised this point.
I have no choice now but to revert if_ndis_pci.c and if_ndis_pccard.c
back to the old API to un-break it for 5.2-REL and 5.2.1-REL users.
Thanks a whole lot, people.
-Bill
--
=============================================================================
-Bill Paul (510) 749-2329 | Senior Engineer, Master of Unix-Fu
wpaul at windriver.com | Wind River Systems
=============================================================================
<adamw> you're just BEGGING to face the moose
=============================================================================
More information about the cvs-src
mailing list