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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html