Re: CVS commit: src/sys/arch

[Matt Thomas <matt%3am-software.com@localhost>]

> On Aug 29, 2014, at 4:46 AM, Reinoud Zandijk <reinoud%NetBSD.org@localhost> 
> wrote:
> 
> Mi Matthew,
> 
> On Fri, Aug 29, 2014 at 09:53:31AM +1000, matthew green wrote:
>> 
>> "Reinoud Zandijk" writes:
>>> Module Name:        src
>>> Committed By:       reinoud
>>> Date:               Thu Aug 28 18:02:37 UTC 2014
>>> 
>>> Modified Files:
>>>     src/sys/arch/arm/samsung: exynos_io.c exynos_soc.c exynos_var.h
>>>     src/sys/arch/evbarm/odroid: odroid_machdep.c
>>> 
>>> Log Message:
>>> Implement CPU speed control for Exynos4 and Exynos5 CPUs using APLL 
>>> frequency
>>> adjustment.
>> 
>> this change seems broken as it modifies cpu_counter specific
>> information in cpu_data.
> 
> It might be wrong, but all the evbarm architectures i've seen refect the cpu
> freq. in that variable. Not sure even if it needs to be updated on frequency
> changes. I'll dig into this though i'd love to see some more information!

Of course it need to be updated.