LKML Archive on
help / color / mirror / Atom feed
From: Tony Lindgren <>
To: Johan Hovold <>
Cc: Greg Kroah-Hartman <>,
	Rob Herring <>, Sebastian Reichel <>,
	"H. Nikolaus Schaller" <>,
	Andreas Kemnade <>,
	Mark Rutland <>,
	Arnd Bergmann <>, Pavel Machek <>,,,
Subject: Re: [PATCH 1/2] serdev: add controller runtime PM support
Date: Thu, 10 May 2018 09:48:31 -0700	[thread overview]
Message-ID: <> (raw)
In-Reply-To: <>

* Johan Hovold <> [180509 09:46]:
> Add support for controller runtime power management to serdev core. This
> is needed to allow slave drivers to manage the runtime PM state of the
> underlying serial controller when its driver, in turn, implements more
> aggressive runtime power management (e.g. using autosuspend).
> For some applications, for example, where loss off initial data after a
> remote-wakeup event is acceptable or where rx is not used at all,
> aggressive serial controller runtime PM may be used without further
> involvement of the slave driver. But when this is not the case, the
> slave driver must be able to indicate when incoming data is expected in
> order to avoid data loss.
> To facilitate the common case, where the serial controller power state
> is active whenever the port is open (which is the case with just about
> every serial driver), and where data loss is not acceptable and cannot
> even be prevented by explicit controller runtime power management, an
> RPM reference is taken in serdev open and put again at close. This
> reference can later be balanced by any serdev driver which wants and/or
> can handle aggressive controller runtime PM.
> Note that the .ignore_children flag is set for the serdev controller to
> allow the underlying hardware to idle when no I/O is expected, regardless
> of the slave device RPM state.

If this solution works for GPS then this should also work for modems
that might produce data. And as long as the serdev consumer driver
can wake up the UART with pm_runtime_get(&serdev->ctrl->dev) then
the out of band GPIO wake interrupts will work to. And for TX,
the serdev consumer driver can toggle the wake GPIO, and then call
pm_runtime_get(&serdev->ctrl->dev). So yeah, looks nice to me:

Acked-by: Tony Lindgren <>

  parent reply	other threads:[~2018-05-10 16:48 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-05-09  9:44 Johan Hovold
2018-05-09  9:44 ` [PATCH EXAMPLE 2/2] dbg: gnss: sirf: allow aggressive controller runtime PM Johan Hovold
2018-05-10 16:48 ` Tony Lindgren [this message]
2018-05-11  8:07   ` [PATCH 1/2] serdev: add controller runtime PM support Johan Hovold
2018-05-11 12:56     ` Tony Lindgren
2018-05-11 13:12       ` Johan Hovold
2018-05-11 16:00         ` Tony Lindgren
2018-05-11 12:35 ` Sebastian Reichel

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:

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

  git send-email \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
    --subject='Re: [PATCH 1/2] serdev: add controller runtime PM support' \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).