From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?ISO-8859-2?Q?Rafa=B3_Bilski?= Subject: [PATCH 2/2] Longhaul - Add voltage scaling to driver Date: Tue, 15 Aug 2006 23:38:24 +0200 Message-ID: <44E23ED0.4090607@interia.pl> Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Return-path: 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=m.gmane.org+glkc-cpufreq=m.gmane.org@lists.linux.org.uk Content-Type: text/plain; charset="iso-8859-1" To: Dave Jones Cc: cpufreq@lists.linux.org.uk Rename option "dont_scale_voltage" to "scale_voltage" because=20 don't will be default. Use "pos" for calculating voltage. In this way driver don't need=20 to know mV value or low level value. Simply min U is one pos and=20 max U is second pos. All pos between these two are used. Assume that min U is for min f and max U for max f. For frequency=20 between min and max calculate pos based on difference between=20 current frequency and min f. Signed-off-by: Rafa=B3 Bilski --- diff --git a/arch/i386/kernel/cpu/cpufreq/longhaul.c b/arch/i386/kernel/cpu= /cpufreq/longhaul.c --- a/arch/i386/kernel/cpu/cpufreq/longhaul.c +++ b/arch/i386/kernel/cpu/cpufreq/longhaul.c @@ -53,18 +53,25 @@ #define CPU_EZRA_T 4 #define CPU_NEHEMIAH 5 =20 static int cpu_model; -static unsigned int numscales=3D16, numvscales; +static unsigned int numscales=3D16; static unsigned int fsb; -static int minvid, maxvid; + +static struct mV_pos *vrm_mV_table; +static unsigned char *mV_vrm_table; +struct f_msr { + unsigned char vrm; +}; +static struct f_msr f_msr_table[32]; + +static unsigned int highest_speed, lowest_speed; /* kHz */ static unsigned int minmult, maxmult; -static int can_scale_voltage; -static int vrmrev; +static int can_scale_voltage =3D 0; static struct acpi_processor *pr =3D NULL; static struct acpi_processor_cx *cx =3D NULL; static int port22_en =3D 0; =20 /* Module parameters */ -static int dont_scale_voltage; +static int scale_voltage =3D 0; static int ignore_latency =3D 0; =20 #define dprintk(msg...) cpufreq_debug_printk(CPUFREQ_DEBUG_DRIVER, "longha= ul", msg) @@ -73,7 +80,6 @@ #define dprintk(msg...) cpufreq_debug_pr /* Clock ratios multiplied by 10 */ static int clock_ratio[32]; static int eblcr_table[32]; -static int voltage_table[32]; static unsigned int highest_speed, lowest_speed; /* kHz */ static int longhaul_version; static struct cpufreq_frequency_table *longhaul_table; @@ -163,6 +169,11 @@ static void do_powersaver(int cx_address longhaul.bits.SoftBusRatio4 =3D (clock_ratio_index & 0x10) >> 4; longhaul.bits.EnableSoftBusRatio =3D 1; =20 + if (can_scale_voltage) { + longhaul.bits.SoftVID =3D f_msr_table[clock_ratio_index].vrm; + longhaul.bits.EnableSoftVID =3D 1; + } + /* Sync to timer tick */ safe_halt(); /* Change frequency on next halt or sleep */ @@ -454,53 +465,62 @@ static int __init longhaul_get_ranges(vo static void __init longhaul_setup_voltagescaling(void) { union msr_longhaul longhaul; + struct mV_pos minvid, + maxvid; + unsigned int j, + speed, + pos, + kHz_step, + numvscales; =20 - rdmsrl (MSR_VIA_LONGHAUL, longhaul.val); - - if (!(longhaul.bits.RevisionID & 1)) + rdmsrl(MSR_VIA_LONGHAUL, longhaul.val); + if ( !(longhaul.bits.RevisionID & 1) ) { + printk(KERN_INFO PFX "Voltage scaling not supported by CPU.\n"); return; + } + + if (!longhaul.bits.VRMRev) { + printk (KERN_INFO PFX "VRM 8.5\n"); + vrm_mV_table =3D &vrm85_mV[0]; + mV_vrm_table =3D &mV_vrm85[0]; + } else { + printk (KERN_INFO PFX "Mobile VRM\n"); + vrm_mV_table =3D &mobilevrm_mV[0]; + mV_vrm_table =3D &mV_mobilevrm[0]; + } =20 - minvid =3D longhaul.bits.MinimumVID; - maxvid =3D longhaul.bits.MaximumVID; - vrmrev =3D longhaul.bits.VRMRev; + minvid =3D vrm_mV_table[longhaul.bits.MinimumVID]; + maxvid =3D vrm_mV_table[longhaul.bits.MaximumVID]; + numvscales =3D maxvid.pos - minvid.pos + 1; + kHz_step =3D (highest_speed - lowest_speed) / numvscales; =20 - if (minvid =3D=3D 0 || maxvid =3D=3D 0) { + if (minvid.mV =3D=3D 0 || maxvid.mV =3D=3D 0 || minvid.mV > maxvid.mV) { printk (KERN_INFO PFX "Bogus values Min:%d.%03d Max:%d.%03d. " "Voltage scaling disabled.\n", - minvid/1000, minvid%1000, maxvid/1000, maxvid%1000); + minvid.mV/1000, minvid.mV%1000, maxvid.mV/1000, maxvid.mV%1000); return; } =20 - if (minvid =3D=3D maxvid) { + if (minvid.mV =3D=3D maxvid.mV) { printk (KERN_INFO PFX "Claims to support voltage scaling but min & max a= re " "both %d.%03d. Voltage scaling disabled\n", - maxvid/1000, maxvid%1000); + maxvid.mV/1000, maxvid.mV%1000); return; } =20 - if (vrmrev=3D=3D0) { - dprintk ("VRM 8.5\n"); - memcpy (voltage_table, vrm85scales, sizeof(voltage_table)); - numvscales =3D (voltage_table[maxvid]-voltage_table[minvid])/25; - } else { - dprintk ("Mobile VRM\n"); - memcpy (voltage_table, mobilevrmscales, sizeof(voltage_table)); - numvscales =3D (voltage_table[maxvid]-voltage_table[minvid])/5; + printk(KERN_INFO PFX "Max VID=3D%d.%03d Min VID=3D%d.%03d, %d possible v= oltage scales\n", + maxvid.mV/1000, maxvid.mV%1000, + minvid.mV/1000, minvid.mV%1000, + numvscales); +=09 + j =3D 0; + while (longhaul_table[j].frequency !=3D CPUFREQ_TABLE_END) { + speed =3D longhaul_table[j].frequency; + pos =3D (speed - lowest_speed) / kHz_step + minvid.pos; + f_msr_table[ longhaul_table[j].index ].vrm =3D mV_vrm_table[pos]; + j++; } =20 - /* Current voltage isn't readable at first, so we need to - set it to a known value. The spec says to use maxvid */ - longhaul.bits.RevisionKey =3D longhaul.bits.RevisionID; /* FIXME: This is= bad. */ - longhaul.bits.EnableSoftVID =3D 1; - longhaul.bits.SoftVID =3D maxvid; - wrmsrl (MSR_VIA_LONGHAUL, longhaul.val); - - minvid =3D voltage_table[minvid]; - maxvid =3D voltage_table[maxvid]; - - dprintk ("Min VID=3D%d.%03d Max VID=3D%d.%03d, %d possible voltage scales= \n", - maxvid/1000, maxvid%1000, minvid/1000, minvid%1000, numvscales); - can_scale_voltage =3D 1; } =20 @@ -685,7 +705,7 @@ static int __init longhaul_cpu_init(stru return ret; =20 if ((longhaul_version=3D=3DTYPE_LONGHAUL_V2 || longhaul_version=3D=3DTYPE= _POWERSAVER) && - (dont_scale_voltage=3D=3D0)) + (scale_voltage !=3D 0)) longhaul_setup_voltagescaling(); =20 policy->governor =3D CPUFREQ_DEFAULT_GOVERNOR; @@ -773,8 +793,8 @@ static void __exit longhaul_exit(void) kfree(longhaul_table); } =20 -module_param (dont_scale_voltage, int, 0644); -MODULE_PARM_DESC(dont_scale_voltage, "Don't scale voltage of processor"); +module_param (scale_voltage, int, 0644); +MODULE_PARM_DESC(scale_voltage, "Scale voltage of processor"); module_param(ignore_latency, int, 0644); MODULE_PARM_DESC(ignore_latency, "Skip ACPI C3 latency test");