http://bugzilla.kernel.org/show_bug.cgi?id=13529





--- Comment #21 from ykzhao <yakui.z...@intel.com>  2009-06-17 09:53:28 ---
Agree with what Len said in comment #14.
    It seems that the frequency in _PSS package is in ascending order. But
according to ACPI spec it should be the descending order.
    From the dmesg log it seems that two P-states are reported by the _PSS
package. But unfortunately the second P-state is removed by the following code,
which causes that only one P-state is remained in the frequency table. So it
will always use the lowest frequency. 
     >/* table init */
        for (i = 0; i < perf->state_count; i++) {
                if (i > 0 && perf->states[i].core_frequency >=
                    data->freq_table[valid_states-1].frequency / 1000)
                        continue;
                /* If the current frequency is larger or equal to the previous
frequency, it will be skipped */

                data->freq_table[valid_states].index = i;

     Maybe we should initialize the frequency table correctly even when the
frequency in _PSS is in ascending order.
     Thanks.

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching the assignee of the bug.

------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
acpi-bugzilla mailing list
acpi-bugzilla@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to