Date: Sat, 16 Jul 2005 10:30:28 -0600 (MDT) From: "M. Warner Losh" <imp@bsdimp.com> To: nate@root.org Cc: freebsd-current@freebsd.org, harrycoin@qconline.com Subject: Re: mss.c pcm fix to ' attach returned 6 ' load failure for v5.x acpi and up Message-ID: <20050716.103028.04060625.imp@bsdimp.com> In-Reply-To: <42D82D59.9060605@root.org> References: <200507121027.14113.jhb@FreeBSD.org> <4.3.2.7.2.20050715164008.01f0fdd8@mail.qconline.com> <42D82D59.9060605@root.org>
next in thread | previous in thread | raw e-mail | index | archive | help
In message: <42D82D59.9060605@root.org> Nate Lawson <nate@root.org> writes: : Harry Coin wrote: : > At 02:28 PM 7/15/2005 -0700, Nate Lawson wrote: : > : >> Drivers should not rely on isa_get_logicalid() to determine a boolean : >> "is PNP?" : > : > : > The architecture manual specifies ISA_PNP_PROBE in non pnp ISA drivers : > for that purpose. As I understand it, John doesn't like the ugly nature : > of passing in a null device list for non-pnp ISA drivers. Hard to : > argue with that. : > : > So why not gin up a tiny little boolean kernel function : > 'device_is_pnp(dev)) ' that does the right thing for non-pnp isa drivers : > - once -,right after wherever ISA_PNP_PROBE is defined in the kernel? : : I don't understand how this is needed. ACPI devices are always a : superset of PNP. If a probe method is not PNP capable, it should never : attach to the ACPI bus. I think that's what his fix changes, and I : think it's sufficient. Nate's right here. It isn't needed. The ISA_PNP_PROBE is for all devices that have a PNP ID. Thse includes ISA PnP cards, PNP BIOS devices and ACPI devices. So the device_is_pnp() isn't needed at all. Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20050716.103028.04060625.imp>