[PATCH] ARM: mxs: Change duart device to use amba-pl011

Russell King - ARM Linux linux at arm.linux.org.uk
Mon Jan 3 05:39:13 EST 2011


On Mon, Dec 27, 2010 at 07:49:40PM +0800, Shawn Guo wrote:
> I did not see this problem. It's true that the clock is turned off
> in amba_device_register. But later, the clock will be turned on
> again by amba_probe -> amba_get_enable_pclk very soon.
> 
> But we should add something like below in mx28_clocks_init to reflect
> the clock status and get usecount updated.
> 
> 	clk_enable(&uart_clk)
> 
> I ran into similar problem with fec driver, and would cover that in
> fec patch set.

This sounds like entirely the wrong approach.  Please take some time to
understand that there are different clocks which do different things.

The PCLK is the name ARM Ltd give to the APB bus clock.  This we name
apb_pclk, and we expect platforms to provide it where possible.  This
name will be looked up for every primecell device on the system.

If you don't provide an apb_pclk, it will find the device specific
function clock.  This in itself is no bad thing, unless you use the
device outside of the AMBA driver (iow, before probe or after remove)
and helps identify when you've forgotten to provide an apb_pclk.



More information about the linux-arm-kernel mailing list