On Mon, Mar 02, 2009 at 08:52:02PM +0100, Magnus Lilja wrote: > Shouldn't you move local_irq_save(flags) to before the "val = > __raw_readl(S3C_CLK_DIV0);" line? Just in case the function gets > interrupted by some other entity doing a read-modify-write of > S3C_CLK_DIV0? Yes, that's safer - thanks! -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html