Re: [PATCH 3/3] ARM: OMAP: AM35xx: fix UART4 softreset

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

 



On Thu, May 10, 2012 at 11:29:19AM -0600, Paul Walmsley wrote:
> During kernel init, the AM3505/AM3517 UART4 cannot complete its softreset:
> 
> omap_hwmod: uart4: softreset failed (waited 10000 usec)
> 
> This also results in another warning later in the boot process:
> 
> omap_hwmod: uart4: enabled state can only be entered from initialized, idle, or disabled state
> 
> >From empirical observation, the AM35xx UART4 IP block requires either
> uart1_fck or uart2_fck to be enabled while UART4 resets.  Otherwise
> the reset will never complete.  So this patch adds uart1_fck as an
> optional clock for UART4 and adds the appropriate hwmod flag to cause
> uart1_fck to be enabled during the reset process.  (The choice of
> uart1_fck over uart2_fck was arbitrary.)
> 
> Unfortunately this observation raises many questions.  Is it necessary
> for uart1_fck or uart2_fck to be controlled with uart4_fck for the
> UART4 to work correctly?  What exactly do the AM35xx UART4 clock
> tree and the related PRCM idle management FSMs look like?  If anyone
> has the ability to answer these questions through empirical functional
> testing, or hardware information from the AM35xx designers, it would
> be greatly appreciated.
> 
> Cc: Benoît Cousson <b-cousson@xxxxxx>
> Cc: Kyle Manna <kyle.manna@xxxxxxxxx>
> Cc: Mark A. Greer <mgreer@xxxxxxxxxxxxxxx>
> Cc: Ranjith Lohithakshan <ranjithl@xxxxxx>
> Signed-off-by: Paul Walmsley <paul@xxxxxxxxx>

Acked-by: Mark A. Greer <mgreer@xxxxxxxxxxxxxxx>
(on an am3517evm)

Mark
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux