CPUFreq PowerOP integratoin, CPUFreq core 1/3

drivers/cpufreq/Makefile +1 -1

-obj-$(CONFIG_CPU_FREQ)			+= cpufreq.o
+obj-$(CONFIG_CPU_FREQ)			+= cpufreq.o freq_helpers.o

drivers/cpufreq/cpufreq.c +1 -1

-	freqs->flags = cpufreq_driver->flags;
 	dprintk("notification %u of frequency transition to %u kHz\n",
 		state, freqs->new);
 
+	/* FIXME: don't we need a lock here? */

drivers/cpufreq/cpufreq.c +6 -3

-					freqs->old, policy->cur);
-				freqs->old = policy->cur;
-			}
+		if ((policy) && (policy->cpu == freqs->cpu) &&
+		    (policy->cur) && (policy->cur != freqs->old)) {
+			dprintk("Warning: CPU frequency is"
+				" %u, cpufreq assumed %u kHz.\n",
+				freqs->old, policy->cur);
+			freqs->old = policy->cur;
 		}

drivers/cpufreq/cpufreq.c +5 -0

+/**
+ * show_scaling_governor - show the current policy for the specified CPU
+ */
+static ssize_t 
+show_available_freqs(struct cpufreq_policy *policy, char *buf)

/**
- * show_scaling_driver - show the cpufreq driver currently loaded
- */

drivers/cpufreq/cpufreq.c +0 -5

-static ssize_t show_scaling_driver (struct cpufreq_policy * policy, char *buf)
-{
-	return scnprintf(buf, CPUFREQ_NAME_LEN, "%s\n", cpufreq_driver->name);
-}
-

drivers/cpufreq/cpufreq.c +1 -1

@@ -617,10 +635,10 @@ static int cpufreq_add_dev (struct sys_d
 	int ret = 0;
 	struct cpufreq_policy new_policy;
 	struct cpufreq_policy *policy;
-	struct freq_attr **drv_attr;
 	struct sys_device *cpu_sys_dev;
 	unsigned long flags;
 	unsigned int j;
+	char freq_n[CPUFREQ_FREQ_STR_SIZE];

drivers/cpufreq/cpufreq.c +1 -1

 #ifdef CONFIG_SMP
 	struct cpufreq_policy *managed_policy;
 #endif

@@ -630,7 +648,7 @@ #endif
 
 	cpufreq_debug_disable_ratelimit();
 	dprintk("adding CPU %u\n", cpu);
-
+	

drivers/cpufreq/cpufreq.c +1 -1

 #ifdef CONFIG_SMP
 	/* check whether a different CPU already registered this
 	 * CPU because it is in the same boat. */

@@ -638,15 +656,9 @@ #ifdef CONFIG_SMP
 	if (unlikely(policy)) {
 		cpufreq_cpu_put(policy);
 		cpufreq_debug_enable_ratelimit();
-		return 0;
+		return ret;
 	}
 #endif

drivers/cpufreq/cpufreq.c +6 -1

@@ -654,23 +666,38 @@ #endif
 	}
 
 	policy->cpu = cpu;
-	policy->cpus = cpumask_of_cpu(cpu);
+
+	/* 
+	 * FIXME: this will go away once notifications moved to lower layers 
+	 * presumably to clock/voltage framework level
+	 */
+	policy->cpus = arch_get_dependent_cpus_mask(cpu);

drivers/cpufreq/cpufreq.c +4 -5

 
-	/* call driver. From then on the cpufreq must be able
-	 * to accept all calls to ->verify and ->setpolicy for this CPU
-	 */
-	ret = cpufreq_driver->init(policy);
-	if (ret) {
+	policy->governor = CPUFREQ_DEFAULT_GOVERNOR;
+
+	/* FIXME: soon this will come from PowerOP Core as well */
+	policy->cpuinfo.transition_latency = 10000; /* 10uS latency */

drivers/cpufreq/cpufreq.c +10 -0

+			/* 
+			 * FIXME: interesting... cpufreq handles one policy 
+			 * entry for all affected CPUs... it works since 
+			 * currently same set of freq/voltage pairs are always 
+			 * registered for all CPUs. That bacomes not the case
+			 * with PowerOP approach when we can have different
+			 * set of operating points even for affected CPUs
+			 */ 
+			kfree(policy);
+			return 0;

drivers/cpufreq/cpufreq.c +1 -0

@@ -944,6 +972,7 @@ unsigned int cpufreq_quick_get(unsigned 
 EXPORT_SYMBOL(cpufreq_quick_get);
 
 
+/* FIXME: REVISIT: this routine returns 0 on error! */
 /**
  * cpufreq_get - get the current CPU frequency (in kHz)

drivers/cpufreq/cpufreq.c +2 -0

+module_param(smart_cpu, uint, 0644);
+MODULE_PARM_DESC(smart_cpu, "Define if cpu handles frequency intervals insted oftarget frequencies");