From: Cezary Rojewski <cezary.rojewski@intel.com>
To: alsa-devel@alsa-project.org, broonie@kernel.org
Cc: Cezary Rojewski <cezary.rojewski@intel.com>,
pierre-louis.bossart@linux.intel.com, tiwai@suse.com,
hdegoede@redhat.com, amadeuszx.slawinski@linux.intel.com
Subject: [PATCH v4 0/9] ASoC: Intel: avs: PCM power management
Date: Thu, 27 Oct 2022 14:46:53 +0200 [thread overview]
Message-ID: <20221027124702.1761002-1-cezary.rojewski@intel.com> (raw)
Goal of the series is implementation of suspend/resume operations for a
PCM stream along with all the collaterals connected to the subject.
Start with splitting avs_dai_fe_hw_free() as ideally we would like to
reuse as much of existing code as possible but snd_pcm_lib_free_pages()
is not desired part of the function when speaking of suspend operation.
The actual implementation of suspend/resume() for component drivers
follows. For most scenarios, the PM flow is similar to standard
streaming one, except for the part where the position register are being
saved and the lack of PCM pages freeing. To reduce code duplication, all
avs_dai_suspend_XXX() and avs_dai_resume_XXX() functions reuse their
non-PM equivalents.
Order of operations is affected by the fact that path binding/unbinding
happens only in FE part of the stream.
Above essentially unlocks SX+streaming scenarios i.e.: power transitions
with an ongoing stream.
As some streams are allowed to run in low power state, support is
provided for S0iX state. The handlers check ACPI capabilities and the
number of active low-power paths before deciding between SX and S0iX
flows.
The last portion of the patchset is addition of power/clock gating
overrides. There is no single set of registers that ensures AudioDSP
firmware loads 100% of time on every single configuration. By having
them exposed, user can have the loading procedure behavior adjusted for
their configuration without having to recompile the kernel.
Changes in v4:
- replaced readb_poll_timeout() with read_poll_timeout() for the DRSM
polling function as the register is u32 wide, not u8
Changes in v3:
- fixed unused-but-set-variable warnings reported by the test robot
Changes in v2:
- moved DRSM reg polling to separate function,
snd_hdac_stream_wait_drsm() as suggested by Pierre
- moved the acpi_gbl_FADT under AVS_S0IX_SUPPORTED macro and relocated
the checks from pcm.c to topology.c so that the low-power streams are
filtered as early as possible
- fixed compilation when CONFIG_ACPI is disabled as reported by the test
robot
- simplified all the avs_dai_resume/suspend_xxx() by moving
->ignore_suspend check one level up, to avs_component_pm_op()
(patch 0002)
Amadeusz Sławiński (1):
ASoC: Intel: avs: Handle SUSPEND and RESUME triggers
Cezary Rojewski (7):
ASoC: Intel: avs: Split pcm pages freeing operation from hw_free()
ASoC: Intel: avs: Introduce PCM power management routines
ALSA: hda: Introduce snd_hdac_stream_wait_drsm()
ASoC: Intel: avs: Restart instead of resuming HDA capture streams
ASoC: Intel: avs: Count low power streams
ASoC: Intel: avs: Power and clock gating policy overriding
ASoC: Intel: avs: Enact power gating policy
Piotr Maziarz (1):
ASoC: Intel: avs: Standby power-state support
include/sound/hdaudio.h | 1 +
include/sound/hdaudio_ext.h | 5 +
sound/hda/hdac_stream.c | 22 ++
sound/soc/intel/avs/avs.h | 8 +
sound/soc/intel/avs/core.c | 97 +++++++--
sound/soc/intel/avs/loader.c | 10 +
sound/soc/intel/avs/pcm.c | 368 +++++++++++++++++++++++++++++----
sound/soc/intel/avs/topology.c | 10 +
8 files changed, 461 insertions(+), 60 deletions(-)
--
2.25.1
next reply other threads:[~2022-10-27 12:31 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-27 12:46 Cezary Rojewski [this message]
2022-10-27 12:46 ` [PATCH v4 1/9] ASoC: Intel: avs: Split pcm pages freeing operation from hw_free() Cezary Rojewski
2022-10-27 12:46 ` [PATCH v4 2/9] ASoC: Intel: avs: Introduce PCM power management routines Cezary Rojewski
2022-10-27 12:46 ` [PATCH v4 3/9] ALSA: hda: Introduce snd_hdac_stream_wait_drsm() Cezary Rojewski
2022-10-27 13:31 ` Takashi Iwai
2022-10-27 12:46 ` [PATCH v4 4/9] ASoC: Intel: avs: Handle SUSPEND and RESUME triggers Cezary Rojewski
2022-10-27 12:46 ` [PATCH v4 5/9] ASoC: Intel: avs: Restart instead of resuming HDA capture streams Cezary Rojewski
2022-10-27 12:46 ` [PATCH v4 6/9] ASoC: Intel: avs: Count low power streams Cezary Rojewski
2022-10-27 12:47 ` [PATCH v4 7/9] ASoC: Intel: avs: Standby power-state support Cezary Rojewski
2022-10-27 12:47 ` [PATCH v4 8/9] ASoC: Intel: avs: Power and clock gating policy overriding Cezary Rojewski
2022-10-27 12:47 ` [PATCH v4 9/9] ASoC: Intel: avs: Enact power gating policy Cezary Rojewski
2022-10-28 16:40 ` [PATCH v4 0/9] ASoC: Intel: avs: PCM power management Mark Brown
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=20221027124702.1761002-1-cezary.rojewski@intel.com \
--to=cezary.rojewski@intel.com \
--cc=alsa-devel@alsa-project.org \
--cc=amadeuszx.slawinski@linux.intel.com \
--cc=broonie@kernel.org \
--cc=hdegoede@redhat.com \
--cc=pierre-louis.bossart@linux.intel.com \
--cc=tiwai@suse.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