[PATCH linux v3] arm: aspeed: zaius: Disable LPC reset for UART1

Andrew Jeffery andrew at aj.id.au
Fri Feb 3 12:15:22 AEDT 2017


On Tue, 2017-01-31 at 17:12 -0800, Xo Wang wrote:
> Currently, UART1 on Zaius BMC is unusable until brought out of reset by
> powering the host on. In that default reset state, ttyS0 can still be
> opened and UART1 silently drops bytes, which is not obviously expected
> behavior.
> 
> Clear the LPC block control bit that enables LPCRST# as a reset source
> for UART1, making UART1 immediately useful in that it won't discard
> bytes.
> 
> > Signed-off-by: Xo Wang <xow at google.com>
> ---
>  arch/arm/mach-aspeed/aspeed.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/arch/arm/mach-aspeed/aspeed.c b/arch/arm/mach-aspeed/aspeed.c
> index 4bd3680d742d..b7022eaa7b5d 100644
> --- a/arch/arm/mach-aspeed/aspeed.c
> +++ b/arch/arm/mach-aspeed/aspeed.c
> @@ -185,6 +185,14 @@ static void __init do_zaius_setup(void)
>  
> >  	/* Set SPI1 CE0 decoding window to 0x30000000 */
> >  	writel(0x68600000, AST_IO(AST_BASE_SPI | 0x30));
> +
> > +	/* Disable default behavior of UART1 being held in reset by LPCRST#.
> > +	 * By releasing UART1 from being controlled by LPC reset, it becomes
> > +	 * immediately available regardless of the host being up.
> +	 * */

Joel: Want to clean up the formatting here when you apply it? If so:

Reviewed-by: Andrew Jeffery <andrew at aj.id.au>

> +	reg = readl(AST_IO(AST_BASE_LPC | 0x98));
> > +	/* Clear "Enable UART1 reset source from LPC" */
> > +	writel(reg & ~BIT(4), AST_IO(AST_BASE_LPC | 0x98));
>  }
>  
>  static void __init do_witherspoon_setup(void)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: This is a digitally signed message part
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20170203/77e12c95/attachment.sig>


More information about the openbmc mailing list