* Tony Lindgren <tony@xxxxxxxxxxx> [180508 08:22]: > * Johan Hovold <johan@xxxxxxxxxx> [180508 07:00]: > > With the negative autosuspend set in both omap drivers probe functions, > > this is the expected behaviour. Which I think we must fix. > > Yes indeed. I've been using my script for years now and have > completely missed the fact that the unused ports are not idled > at all on start-up when unused. This might be all that's needed, care to try it and if it works I'll send out two separate proper patches? I'm seeing this now on my bbb after temporarily disabling my UART idle init script: # rwmem -s32 0x44e004b4 # uart 1 on l4_wkup 0x44e004b4 = 0x00000002 # rwmem -s32 0x44e0006c+0x10 # uart 2 - 5 on l4_per 0x44e0006c = 0x00030000 0x44e00070 = 0x00030000 0x44e00074 = 0x00030000 0x44e00078 = 0x00030000 # rwmem -s32 0x44e00038 # uart 6 on l4_per 0x44e00038 = 0x00030000 Regards, Tony 8< ---------------------- diff --git a/drivers/tty/serial/8250/8250_omap.c b/drivers/tty/serial/8250/8250_omap.c --- a/drivers/tty/serial/8250/8250_omap.c +++ b/drivers/tty/serial/8250/8250_omap.c @@ -685,8 +685,7 @@ static void omap_8250_shutdown(struct uart_port *port) serial_out(up, UART_LCR, up->lcr & ~UART_LCR_SBC); serial_out(up, UART_FCR, UART_FCR_CLEAR_RCVR | UART_FCR_CLEAR_XMIT); - pm_runtime_mark_last_busy(port->dev); - pm_runtime_put_autosuspend(port->dev); + pm_runtime_put_sync(port->dev); free_irq(port->irq, port); dev_pm_clear_wake_irq(port->dev); } @@ -1265,8 +1264,7 @@ static int omap8250_probe(struct platform_device *pdev) } priv->line = ret; platform_set_drvdata(pdev, priv); - pm_runtime_mark_last_busy(&pdev->dev); - pm_runtime_put_autosuspend(&pdev->dev); + pm_runtime_put_sync(&pdev->dev); return 0; err: pm_runtime_dont_use_autosuspend(&pdev->dev); diff --git a/drivers/tty/serial/omap-serial.c b/drivers/tty/serial/omap-serial.c --- a/drivers/tty/serial/omap-serial.c +++ b/drivers/tty/serial/omap-serial.c @@ -821,8 +821,7 @@ static void serial_omap_shutdown(struct uart_port *port) if (serial_in(up, UART_LSR) & UART_LSR_DR) (void) serial_in(up, UART_RX); - pm_runtime_mark_last_busy(up->dev); - pm_runtime_put_autosuspend(up->dev); + pm_runtime_put_sync(up->dev); free_irq(up->port.irq, up); dev_pm_clear_wake_irq(up->dev); } @@ -1751,8 +1750,7 @@ static int serial_omap_probe(struct platform_device *pdev) if (ret != 0) goto err_add_port; - pm_runtime_mark_last_busy(up->dev); - pm_runtime_put_autosuspend(up->dev); + pm_runtime_put_sync(&pdev->dev); return 0; err_add_port: -- 2.17.0 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
