Messages in this thread | | | Date | Fri, 13 Jun 2008 14:45:43 +0200 | From | "Klaus S. Madsen" <> | Subject | Re: [PATCH] hdaps add support for various IBM thinkpads R61/T61/T61P |
| |
On Fri, Jun 13, 2008 at 14:27:51 +0200, maximilian attems wrote: > On Fri, Jun 13, 2008 at 01:40:02PM +0200, Klaus S. Madsen wrote: > > On Fri, Jun 13, 2008 at 13:26:47 +0200, maximilian attems wrote: > > > diff --git a/drivers/hwmon/hdaps.c b/drivers/hwmon/hdaps.c > > > index 26df06f..e07c712 100644 > > > --- a/drivers/hwmon/hdaps.c > > > +++ b/drivers/hwmon/hdaps.c > > > @@ -515,6 +515,7 @@ static struct dmi_system_id __initdata hdaps_whitelist[] = { > > > HDAPS_DMI_MATCH_NORMAL("IBM", "ThinkPad R50"), > > > HDAPS_DMI_MATCH_NORMAL("IBM", "ThinkPad R51"), > > > HDAPS_DMI_MATCH_NORMAL("IBM", "ThinkPad R52"), > > > + HDAPS_DMI_MATCH_NORMAL("IBM", "ThinkPad R61"), > > > HDAPS_DMI_MATCH_INVERT("LENOVO", "ThinkPad R61i"), > > > HDAPS_DMI_MATCH_INVERT("IBM", "ThinkPad T41p"), > > > HDAPS_DMI_MATCH_NORMAL("IBM", "ThinkPad T41"), > > > @@ -523,6 +524,8 @@ static struct dmi_system_id __initdata hdaps_whitelist[] = { > > > HDAPS_DMI_MATCH_NORMAL("IBM", "ThinkPad T43"), > > > HDAPS_DMI_MATCH_INVERT("LENOVO", "ThinkPad T60"), > > > HDAPS_DMI_MATCH_INVERT("LENOVO", "ThinkPad T61"), > > > + HDAPS_DMI_MATCH_NORMAL("IBM", "ThinkPad T61"), > > > + HDAPS_DMI_MATCH_NORMAL("IBM", "ThinkPad T61P"), > > I would suspect that this part of the patch is unnecessary. It would be > > strange if there are any T61's where the manufacturer is IBM, instead of > > LENOVO, since the T60 is older and has the manufacturer listed as > > LENOVO, and no entry exists for IBM. > > > > But according to the comment at the top of the struct, the second line > > (the T61P part) is unnecessary, as a T61p already would be matched by > > the T61 line. > > according to hardy git there is no bug reference, but the T61p got added > before the T61 both by chuck. could you enlighten us please on the T61 > IBM match need? I believe that this patch is a response to https://bugs.launchpad.net/ubuntu/+source/linux-source-2.6.22/+bug/133636
But as I stated in that bug description, the fix did not work for me. Also both R61, T60, T61 and T61p owners responded that their dmi-manufacturer was listed as LENOVO. So I'm quite dubious if these entries will match any machines at all?
-- Kind regards, Klaus S. Madsen
| |