On Friday 22 May 2015 11:19:52 Timur Tabi wrote:
On 05/22/2015 10:24 AM, Arnd Bergmann wrote:
That will break the ACPI case, but ACPI could use platform_data to pass the clock rate into the driver, to make it independent of low-level APIs.
The clock rate isn't the only problem. You still need to current clock timestamp, and that's ARM-specific.
Ah, I didn't realize that. I also just noticed that the two arch_timer functions we call here are inline functions that just access control registers.
If we want to enable COMPILE_TEST then, it would have to be something like
#ifdef CONFIG_ARM || CONFIG_ARM64 #include <asm/arch_timer.h> #else ... /* stub functions */ #endif
That might still be helpful to get coverage from things like the public coverity builds that always build x86 allmodconfig, but it's also a bit ugly.
Arnd