linux-kernel - RE: [PATCH 2/2] ASoC: amd: acp: add pm ops support for renoir platform Add pm ops for renoir platform.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 10 Nov 2023 13:20:32 +0000
From:   "Saba Kareem, Syed" <Syed.SabaKareem@....com>
To:     "Saba Kareem, Syed" <Syed.SabaKareem@....com>,
        "broonie@...nel.org" <broonie@...nel.org>,
        "alsa-devel@...a-project.org" <alsa-devel@...a-project.org>
CC:     "Mukunda, Vijendar" <Vijendar.Mukunda@....com>,
        "Hiregoudar, Basavaraj" <Basavaraj.Hiregoudar@....com>,
        "Dommati, Sunil-kumar" <Sunil-kumar.Dommati@....com>,
        "Limonciello, Mario" <Mario.Limonciello@....com>,
        "Gong, Richard" <Richard.Gong@....com>,
        "posteuca@...ex.one" <posteuca@...ex.one>,
        Liam Girdwood <lgirdwood@...il.com>,
        Jaroslav Kysela <perex@...ex.cz>,
        Takashi Iwai <tiwai@...e.com>,
        Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>,
        Nicolas Ferre <nicolas.ferre@...rochip.com>,
        Uwe Kleine-König 
        <u.kleine-koenig@...gutronix.de>,
        open list <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH 2/2] ASoC: amd: acp: add pm ops support for renoir
 platform Add pm ops for renoir platform.

[AMD Official Use Only - General]

Commit message got mixed with commit title will resend the patch again.

-----Original Message-----
From: Syed Saba Kareem <Syed.SabaKareem@....com>
Sent: Friday, November 10, 2023 6:22 PM
To: broonie@...nel.org; alsa-devel@...a-project.org
Cc: Mukunda, Vijendar <Vijendar.Mukunda@....com>; Hiregoudar, Basavaraj <Basavaraj.Hiregoudar@....com>; Dommati, Sunil-kumar <Sunil-kumar.Dommati@....com>; Limonciello, Mario <Mario.Limonciello@....com>; Gong, Richard <Richard.Gong@....com>; posteuca@...ex.one; Saba Kareem, Syed <Syed.SabaKareem@....com>; Liam Girdwood <lgirdwood@...il.com>; Jaroslav Kysela <perex@...ex.cz>; Takashi Iwai <tiwai@...e.com>; Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>; Nicolas Ferre <nicolas.ferre@...rochip.com>; Uwe Kleine-König <u.kleine-koenig@...gutronix.de>; open list <linux-kernel@...r.kernel.org>
Subject: [PATCH 2/2] ASoC: amd: acp: add pm ops support for renoir platform Add pm ops for renoir platform.

Signed-off-by: Syed Saba Kareem <Syed.SabaKareem@....com>
---
 sound/soc/amd/acp/acp-renoir.c | 38 +++++++++++++++++++++++++++++++++-
 1 file changed, 37 insertions(+), 1 deletion(-)

diff --git a/sound/soc/amd/acp/acp-renoir.c b/sound/soc/amd/acp/acp-renoir.c index a591482a0726..8539fbacdf4c 100644
--- a/sound/soc/amd/acp/acp-renoir.c
+++ b/sound/soc/amd/acp/acp-renoir.c
@@ -20,6 +20,7 @@
 #include <sound/soc.h>
 #include <sound/soc-dai.h>
 #include <linux/dma-mapping.h>
+#include <linux/pm_runtime.h>

 #include "amd.h"
 #include "acp-mach.h"
@@ -195,7 +196,11 @@ static int renoir_audio_probe(struct platform_device *pdev)
        dev_set_drvdata(dev, adata);
        acp_enable_interrupts(adata);
        acp_platform_register(dev);
-
+       pm_runtime_set_autosuspend_delay(&pdev->dev, ACP_SUSPEND_DELAY_MS);
+       pm_runtime_use_autosuspend(&pdev->dev);
+       pm_runtime_mark_last_busy(&pdev->dev);
+       pm_runtime_set_active(&pdev->dev);
+       pm_runtime_enable(&pdev->dev);
        return 0;
 }

@@ -208,11 +213,42 @@ static void renoir_audio_remove(struct platform_device *pdev)
        acp_platform_unregister(dev);
 }

+static int __maybe_unused rn_pcm_resume(struct device *dev) {
+       struct acp_dev_data *adata = dev_get_drvdata(dev);
+       struct acp_stream *stream;
+       struct snd_pcm_substream *substream;
+       snd_pcm_uframes_t buf_in_frames;
+       u64 buf_size;
+
+       spin_lock(&adata->acp_lock);
+       list_for_each_entry(stream, &adata->stream_list, list) {
+               substream = stream->substream;
+               if (substream && substream->runtime) {
+                       buf_in_frames = (substream->runtime->buffer_size);
+                       buf_size = frames_to_bytes(substream->runtime, buf_in_frames);
+                       config_pte_for_stream(adata, stream);
+                       config_acp_dma(adata, stream, buf_size);
+                       if (stream->dai_id)
+                               restore_acp_i2s_params(substream, adata, stream);
+                       else
+                               restore_acp_pdm_params(substream, adata);
+               }
+       }
+       spin_unlock(&adata->acp_lock);
+       return 0;
+}
+
+static const struct dev_pm_ops rn_dma_pm_ops = {
+       SET_SYSTEM_SLEEP_PM_OPS(NULL, rn_pcm_resume) };
+
 static struct platform_driver renoir_driver = {
        .probe = renoir_audio_probe,
        .remove_new = renoir_audio_remove,
        .driver = {
                .name = "acp_asoc_renoir",
+               .pm = &rn_dma_pm_ops,
        },
 };

--
2.25.1