From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rich Townsend Subject: Re: Dothan VID# identification script (was Re: [ACPI] Speedstep-centrino problems) Date: Sun, 13 Feb 2005 09:11:59 -0500 Message-ID: <420F602F.3000303@bartol.udel.edu> References: <0FEA36BB2ABDF44FAAFEB7A75367C07A0117DCE4@alaska.condis.es> <420B5CB1.2090804@bartol.udel.edu> <20050210172904.GB6824@isilmar.linta.de> <420E911B.1060405@bartol.udel.edu> <20050213094029.GC8081@dominikbrodowski.de> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit In-Reply-To: <20050213094029.GC8081@dominikbrodowski.de> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: cpufreq-bounces@lists.linux.org.uk Errors-To: cpufreq-bounces+glkc-cpufreq=gmane.org@lists.linux.org.uk Content-Type: text/plain; charset="us-ascii"; format="flowed" To: Dominik Brodowski Cc: cpufreq@ZenII.linux.org.uk, Simon Moore Dominik Brodowski wrote: > On Sat, Feb 12, 2005 at 06:28:27PM -0500, Rich Townsend wrote: > >>Of course, we can't guarantee that there will always be a match; or that >>there won't be duplicate matches. Also, this approach relies on having >>ACPI P-states, to build up the frequency/voltage table. However, it >>still seems to me to be a pretty robust way of determining the VID# in a >>system, and one that could be incorporated into speedstep-centrino. >>Dominik, what do you think? > > > It indeed seems somewhat robust to me, I'd like to get Venkatesh's input on > this, though. "Transient voltages" may pose a problem, though... Gak, hadn't thought of that. > > I wouldn't favor for inclusion into speedstep-centrino, though -- one > cpufreq driver requiring at first one other to run, then to unload the other > with speedstep-centrino taking over... no, that doesn't _feel_ right. I > wouldn't vote against including the Dothan tables but only using them if the > user passes a parameter "vid=[a-f]", though. I concurr. Unless a subset of acpi_cpufreq could be implemented within speedstep-centrino without nasty hackery, you end up with an unpleasant dependency problem. I'll look into adding the vid= option over the Dothan tables patch. cheers, Rich