Re: [PATCH] PnP Layer Rewrite V0.7 - 2.4.42

Jaroslav Kysela (perex@perex.cz)
Mon, 14 Oct 2002 21:10:04 +0200 (CEST)


On Mon, 14 Oct 2002, Adam Belay wrote:

> Linux Plug and Play Rewrite V0.7
>
> After much testing the Linux PnP Rewrite is ready to be included. For
> those who would like to try it, be sure to enable debugging or else it
> will operate silently. Also enable both PnP protocols.

A few notes. Please, could you leave the raw proc interface for ISA PnP?
I mean isapnp_proc_bus_read() function. Also, encoding device/vendor to
7-byte string seems like wasting bytes and CPU cycles. If you use 2/2 byte
format, you'll spare 3 bytes and comparing of two short values (or one
int value) is always less expensive.

Anyway, I like this code. It seems that you don't use standard pci_dev /
pci_bus structures as I was forced by Linus at ISA PnP code inclusion
time. But it's true that we have new device model, so these things might
be private. Also, don't forget to remove additional ISA PnP members from
pci structures when Linus approves pnp_dev and pnp_card structures.

Jaroslav

-----
Jaroslav Kysela <perex@suse.cz>
Linux Kernel Sound Maintainer
ALSA Project http://www.alsa-project.org
SuSE Linux http://www.suse.com

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/