From 041a275af5b0d12f4435e999d75c3c27cf2c04ca Mon Sep 17 00:00:00 2001 From: Andy Shevchenko Date: Fri, 14 Aug 2026 13:16:29 +0200 Subject: [PATCH] serial: 8250_port: Remove redundant pm_runtime_mark_last_busy() call The pm_runtime_mark_last_busy() call is redundant in the wrapper function as pm_runtime_put_autosuspend() already calls pm_runtime_mark_last_busy() internally to update the last access time of the device before queuing autosuspend. Signed-off-by: Andy Shevchenko Link: https://patch.msgid.link/20260814112435.3290545-2-andriy.shevchenko@linux.intel.com Signed-off-by: Greg Kroah-Hartman --- drivers/tty/serial/8250/8250_port.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/tty/serial/8250/8250_port.c b/drivers/tty/serial/8250/8250_port.c index 6f88b7dd2bc2..3eac456fe189 100644 --- a/drivers/tty/serial/8250/8250_port.c +++ b/drivers/tty/serial/8250/8250_port.c @@ -542,7 +542,6 @@ void serial8250_rpm_put(struct uart_8250_port *p) { if (!(p->capabilities & UART_CAP_RPM)) return; - pm_runtime_mark_last_busy(p->port.dev); pm_runtime_put_autosuspend(p->port.dev); } EXPORT_SYMBOL_GPL(serial8250_rpm_put); @@ -683,7 +682,6 @@ static void serial8250_rpm_put_tx(struct uart_8250_port *p) rpm_active = xchg(&p->rpm_tx_active, 0); if (!rpm_active) return; - pm_runtime_mark_last_busy(p->port.dev); pm_runtime_put_autosuspend(p->port.dev); }