Re: [PATCH v11 4/6] ARM: Exynos: switch to using generic cpufreq driver for Exynos4210/5250/5420

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 24 November 2014 at 19:14, Sudeep Holla <sudeep.holla@xxxxxxx> wrote:
> On 20/11/14 03:48, Viresh Kumar wrote:
>> - I believe CPUs boot in the order they are present in DT except for the
>> boot CPU. So, the first node for every cluster should have it.
>>
>> Correct ? Then we can update the fixme.
>>
>
> No, I disagree as you trying to implicitly depend on the logic Linux uses to
> assign logical index. It can be changed any time, and depending
> on it might break things in future which can't be fixed easily later
> especially if it's DT related.

Yeah, correct.

>> Hmm, I never thought about it. But yes the CPU with the OPPs should
>> be the first one to come back.
>>
>
> How can you assume that ? I can write a simple test which hot-plugs out
> all the CPUs in the (esp. multi-cluster) system in ascending order of
> logical index and hot-plug back in descending order. Then the above
> logic fails.

What I said is: Yes the CPU with the OPPs should be the first one to come
back, if this has to work correctly. Otherwise it will break..

And you repeated the same :)

> The platforms needing it have to get involved in such discussions to
> make any progress.

Yes there were people involved. I hope I cc'd you as well...
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux