Judith Mendez <jm@xxxxxx> writes: > Currently in omap_8250_shutdown, the dma->rx_running > flag is set to zero in omap_8250_rx_dma_flush. Next > pm_runtime_get_sync is called, which is a runtime > resume call stack which can re-set the flag. When the > call omap_8250_shutdown returns, the flag is expected > to be UN-SET, but this is not the case. This is causing > issues the next time UART is re-opened and omap_8250_rx_dma > is called. Fix by moving pm_runtime_get_sync before the > omap_8250_rx_dma_flush. > > Signed-off-by: Bin Liu <b-liu@xxxxxx> > Signed-off-by: Judith Mendez <jm@xxxxxx> Reviewed-by: Kevin Hilman <khilman@xxxxxxxxxxxx> Tested-by: Kevin Hilman <khilman@xxxxxxxxxxxx> Gave this a quick boot test on am335x-boneblack and am57xx-beagle-x15. I realize that doesn't really test the DMA paths involved here, but at least it doesn't break basic boot to serial console, and the change looks coorect. Thanks for sending a fix for this. Kevin
