Messages in this thread | | | Date | Thu, 21 Sep 2017 20:15:03 +0200 | From | Uwe Kleine-König <> | Subject | Re: [PATCH v3 2/6] serial: imx: only set dma_is_rxing when DMA starts |
| |
On Thu, Sep 21, 2017 at 05:18:13PM +0100, Martyn Welch wrote: > From: Romain Perier <romain.perier@collabora.com> > > The variable dma_is_rxing is currently set to 1 in imx_disable_rx_int(). > This is problematic as: > > - whilst imx_disable_rx_int() is currently always called before > start_rx_dma() this dependency isn't obvious. > - start_rx_dma() does error checking and might exit without > enabling DMA. Currently this will result in dma_is_rxing suggesting > that DMA is being used for recieving. > > To avoid these issues, move the setting of dma_is_rxing to > start_rx_dma() when appropriate. > > Signed-off-by: Romain Perier <romain.perier@collabora.com> > Signed-off-by: Martyn Welch <martyn.welch@collabora.co.uk> Reviewed-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Thanks Uwe
-- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ |
| |