[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1219394447.20723.51.camel@lgn.site>
Date: Fri, 22 Aug 2008 10:40:46 +0200
From: Kay Sievers <kay.sievers@...y.org>
To: Bjorn Helgaas <bjorn.helgaas@...com>
Cc: ambx1@....rr.com, elendil@...net.nl, trenn@...e.de,
akpm@...ux-foundation.org, linux-kernel@...r.kernel.org,
tpm@...horst.net, rjw@...k.pl, greg@...ah.com
Subject: Re: char/tpm: tpm_infineon no longer loaded for HP 2510p laptop
On Thu, 2008-08-21 at 15:58 -0600, Bjorn Helgaas wrote:
> On Thursday 21 August 2008 03:18:33 pm Kay Sievers wrote:
> > Bjorn Helgaas wrote:
> > >Is there a "complete idiot's guide to modules
> > >and udev"? There's precious little in Documentation/ other than a bunch
> > >of sample rules for various subsystems.
> >
> > I don't know of any specific documentation, but it's pretty easy:
> > ...
>
> Thanks for the tutorial.
>
> > >Could I fix this by implementing pnp_device_uevent()?
> >
> > Only if we change the format of the current pnp device aliases
> > to something like:
> > pnp*:XYZ2324:*
> > pnp*:ABC1234:*
> >
> > and create a "modalias" file at every pnp device, and add MODALIAS to
> > the uevent. The modalias must contains all ID's which belong to that
> > device in one single string, separated and terminated by a special
> > character, something like:
> > pnp:ABC1234:XYZ2324:RST3445:
>
> This all sounds like good stuff that I'd like PNP to have. Is there
> any reason I shouldn't implement pnp_device_uevent()? Any backwards-
> compatibility issues?
Yeah, people probably use the current aliases in modprobe.conf files,
I'm not sure, if we can change them.
> I think that sounds like a better solution than doing this PNP ID
> mangling.
Sure, that will go with the acpi aliases + acpi modalias, or if you
change pnp to have proper pnp aliases + pnp modalias, then with them,
yes.
Thanks,
Kay
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists