linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Dave Gerlach <d-gerlach@ti.com>
To: Nishanth Menon <nm@ti.com>, Tero Kristo <kristo@kernel.org>,
	Santosh Shilimkar <ssantosh@kernel.org>
Cc: Dave Gerlach <d-gerlach@ti.com>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>,
	Vignesh Raghavendra <vigneshr@ti.com>
Subject: [PATCH 0/1] firmware: ti_sci: Use polled mode of operation during suspend
Date: Thu, 10 Feb 2022 11:58:57 -0600	[thread overview]
Message-ID: <20220210175858.11247-1-d-gerlach@ti.com> (raw)

Hi,
This series allows the ti_sci driver to be able operate safely in noirq
mode.  It depends on the ti-msgmgr series here for updates to the
ti-msgmgr header [1], and together with that series will stop timeouts
that are seen when attempting to suspend platforms using TI SCI.

This allows communication with TISCI firmware without the need of
interrupts, which is critical during the system suspend path. Currently,
the ti_sci_pm_domains genpd driver will communicate with firmware during
the noirq phase, so this must be done using a polled communication
layer, however it still makes sense to use IRQ mode during normal
operation, so continue to support that as well, and only switch to
polled mode during low power path.

This is accomplished in the ti_sci driver by setting a flag to indicate
we are in the suspend path in the system suspend handler that then
causes the message handling logic to manually poll the completion used
to indicate message response using try_wait_for_completion rather than
the original wait_for_completion_timeout, which cannot be used in noirq
phase.

Regards,
Dave

[1] https://lore.kernel.org/lkml/20220210041631.26767-1-d-gerlach@ti.com/

Dave Gerlach (1):
  firmware: ti_sci: Switch transport to polled mode during system
    suspend

 drivers/firmware/ti_sci.c | 61 +++++++++++++++++++++++++++++++++++----
 1 file changed, 55 insertions(+), 6 deletions(-)

-- 
2.35.0


             reply	other threads:[~2022-02-10 17:59 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-02-10 17:58 Dave Gerlach [this message]
2022-02-10 17:58 ` [PATCH 1/1] firmware: ti_sci: Switch transport to polled mode during system suspend Dave Gerlach

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=20220210175858.11247-1-d-gerlach@ti.com \
    --to=d-gerlach@ti.com \
    --cc=kristo@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nm@ti.com \
    --cc=ssantosh@kernel.org \
    --cc=vigneshr@ti.com \
    /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 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).