On Wed, Nov 26 2014 at 12:48 -0700, Stephen Boyd wrote:

On 11/21/2014 10:03 AM, Lina Iyer wrote:

quoted

@@ -37,3 +54,17 @@ int scm_set_boot_addr(phys_addr_t addr, int flags)
 			&cmd, sizeof(cmd), NULL, 0);
 }
 EXPORT_SYMBOL(scm_set_boot_addr);
+
+int scm_set_warm_boot_addr(void *entry, int cpu)
+{
+	int ret;
+
+	if (entry == scm_flags[cpu].entry)
+		return 0;

Is the entry going to be different at some point? I wonder why we 
can't just call this function once when the SPM device probes to setup 
the entry point to cpu_resume()?

Hotplug would use a different re-entry point. Hence the check.

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project