Olof Johansson wrote: > > On Wed, Nov 24, 2010 at 10:28:22AM -0800, Stephen Warren wrote: > > Ping. > > > > Does this change look reasonable? In particular, I wonder if > > the clock enable code should actually be part of > > mach-tegra/common.c's common_clk_init_table[]? > > Sounds reasonable to me. Only drawback is that if CONFIG_TEGRA_SYSTEM_DMA > isn't enabled, the clock still will be running. On the other hand, I'm > not sure why we even make that configurable, it's unlikely that someone > will want to turn it off. We could always conditionalize the common_clk_init_table[] entry under the same config option that enables the DMA driver in the first place. -- nvpublic -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html