On Tue, Nov 21, 2006 at 11:30:42AM +0100, Bruno Ducrot wrote: > Hi all, > > processor_perflib.c::acpi_processor_ppc_notifier() check if the value > returned by the processor's _PPC method is 0 and return failed if so. > This is wrong since 0 indicate that the bios think the processor can go > to the highest frequency. This patch for example fix the HP NX 6125 to > allow its highest frequency to be available. > > Signed-off-by: Bruno Ducrot <ducrot@xxxxxxxxxxxx> > > --- linux-2.6.19-rc6/drivers/acpi/processor_perflib.c 2006/11/21 10:18:39 1.1 > +++ linux-2.6.19-rc6/drivers/acpi/processor_perflib.c 2006/11/21 10:18:55 > @@ -83,8 +83,6 @@ static int acpi_processor_ppc_notifier(s > goto out; > > ppc = (unsigned int)pr->performance_platform_limit; > - if (!ppc) > - goto out; > > if (ppc > pr->performance->state_count) BTW I'm wondering if this check should be : if (ppc >= pr->performance->state_count) > goto out; -- Bruno Ducrot -- Which is worse: ignorance or apathy? -- Don't know. Don't care. - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
