Messages in this thread | | | Date | Wed, 1 Mar 2006 20:45:32 +0100 | From | Kay Sievers <> | Subject | Re: [PATCH] [PNP] 'modalias' sysfs export |
| |
On Mon, Feb 27, 2006 at 10:40:19PM +0100, Pierre Ossman wrote: > User space hardware detection need the 'modalias' attributes in the > sysfs tree. This patch adds support to the PNP bus.
> + > + /* FIXME: modalias can only do one alias */ > + return sprintf(buf, "pnp:c%s\n", pos->id);
Without the FIXME actually "fixed", this does not make sense. You want to match always on _all_ aliases. In most cases where you have more than one, the first one is the vendor specific one which isn't interesting at all on Linux. If you have more than one entry usually the second one is the one you are looking for.
So eighter we find a way to encode _all_ id's in one modalias string which can be matched by a wildcard or keep the current solution which iterates over the sysfs "id" file and calls modprobe for every entry.
Thanks, Kay - 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/
| |