Hello, On Wed, Jun 28, 2017 at 03:59:36PM +0200, Romain Perier wrote: > + temp |= UCR1_RRDYEN | UCR1_UARTEN; > + if (sport->have_rtscts) > + temp |= UCR1_RTSDEN; There is a tab too much. > writel(temp, sport->port.membase + UCR1); Other than that: Acked-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ | -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html