All of lore.kernel.org
 help / color / mirror / Atom feed
From: sashiko-bot@kernel.org
To: "James Hilliard" <james.hilliard1@gmail.com>
Cc: linux-serial@vger.kernel.org
Subject: Re: [PATCH] serial: 8250: release the transmit PM reference on shutdown
Date: Thu, 22 Oct 2026 17:25:01 +0000	[thread overview]
Message-ID: <sashiko-outbox-181856@kernel.org> (raw)
In-Reply-To: <20261022-submit-serial-8250-tx-pm-shutdown-v1-v1-1-0e894afa3bb8@gmail.com>

> Transmit holds a runtime PM reference until the transmitter-empty path
> clears THRI. A hangup can shut down the port while that reference is still
> held. Shutdown clears the interrupt enables and stops DMA, leaving no TX
> interrupt to release rpm_tx_active. A suppressed DMA completion cannot
> release it either.
> 
> Drop the transmit reference after shutting down TX in both the generic
> and OMAP shutdown paths. Their separate shutdown PM references keep the
> UART accessible throughout cleanup. The exchange in the helper also
> handles shutdown after the normal TX path already released its reference.
> 
> Fixes: d74d5d1b7288 ("tty: serial: 8250_core: add run time pm")
> Reported-by: Raul E Rangel <rrangel@chromium.org>
> Closes: https://lore.kernel.org/r/20261022152019.1621716-1-rrangel@chromium.org
> Signed-off-by: James Hilliard <james.hilliard1@gmail.com>

Sashiko has reviewed this patch and found no issues.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261022-submit-serial-8250-tx-pm-shutdown-v1-v1-1-0e894afa3bb8@gmail.com?part=1


      reply	other threads:[~2026-10-22 17:25 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-22 17:16 [PATCH] serial: 8250: release the transmit PM reference on shutdown James Hilliard
2026-10-22 17:25 ` sashiko-bot [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=sashiko-outbox-181856@kernel.org \
    --to=sashiko-bot@kernel.org \
    --cc=james.hilliard1@gmail.com \
    --cc=linux-serial@vger.kernel.org \
    --cc=sashiko-reviews@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.