[PATCH] arm: dts: exynos5: Remove multi core timer
Doug Anderson
dianders at chromium.org
Thu May 29 14:41:14 PDT 2014
Vincent,
On Thu, May 29, 2014 at 1:42 PM, Vincent Guittot
<vincent.guittot at linaro.org> wrote:
>> In summary, we've got the following MCT patches proposed to go upstream:
>>
>> 1. MCT scheduler clock: <http://crosreview.com/56363> and
>> <http://crosreview.com/56364>
>> 2. Speed MCT access: <http://crosreview.com/56365>. I wonder if we
>> could also speed it up further with a 64-bit read.
>> 3. Use MCT for udelay: yet to be written.
>>
>> ...does someone want to claim the task of sending those things up?
>>
>>
>> Oh, actually it looks like (93bfb76 clocksource: exynos_mct: register
>> sched_clock callback) in linuxnext adds a partial version of the first
>> patch but isn't as complete as what's in our tree (it's missing the
>> KConfig change we have locally as well as the notrace so it probably
>> breaks ftrace?). Adding Vincent.
>
> Hi Doug,
>
> Thanks for adding me in the loop.
>
> The only difference i see are:
> -HAVE_SCHED_CLOCK which is no more needed
> -and the use of 32bit vs 64bit in the for-next
> -notrace is present in the for-next AFAICT
Ah, my bad! Yes, you're right that things look OK. I looked too
quickly and didn't see the notrace. ...and I wasn't aware that
HAVE_SCHED_CLOCK was no longer needed.
One thing that might be interesting is to consider using 32-bit
instead of 64-bit. We know that this clock is slow to access, so
reducing 3 reads down to 1 would be worth it. A 32-bit clock should
be sufficient for the scheduler anyway.
-Doug
More information about the linux-arm-kernel
mailing list