RE: [PATCH] ARM: Tegra: APB DMA: Enable clock and remove reset.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux