Kever, On Mon, Oct 13, 2014 at 1:12 PM, Kever Yang <kever.yang at rock-chips.com> wrote: > + /* > + * We need to soft reset the cpu when we turn off the cpu power domain, > + * or else the active processors might be stalled when the individual > + * processor is powered down. > + */ > + if (read_cpuid_part_number() != ARM_CPU_PART_CORTEX_A9) { I haven't done a full review of this patch, but it seems unlikely that your uses of read_cpuid_part_number() and read_cpuid_part() in this patch are correct. You use both functions and in both cases compare the results to ARM_CPU_PART_CORTEX_A9.