Linux Sound subsystem development
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Mark Brown <broonie@kernel.org>
Cc: linux-sound@vger.kernel.org
Subject: [PATCH 01/89] ASoC: au1x: Convert to DEFINE_SIPMLE_DEV_PM_OPS()
Date: Thu, 13 Mar 2025 18:40:06 +0100	[thread overview]
Message-ID: <20250313174139.29942-2-tiwai@suse.de> (raw)
In-Reply-To: <20250313174139.29942-1-tiwai@suse.de>

Use the new DEFINE_SIPMLE_DEV_PM_OPS() macro instead of open code
together with pm_ptr(), which allows us dropping CONFIG_PM ifdefs.

Merely a cleanup, there should be no actual code change.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
---
 sound/soc/au1x/i2sc.c     | 17 +++--------------
 sound/soc/au1x/psc-ac97.c | 17 +++--------------
 sound/soc/au1x/psc-i2s.c  | 17 +++--------------
 3 files changed, 9 insertions(+), 42 deletions(-)

diff --git a/sound/soc/au1x/i2sc.c b/sound/soc/au1x/i2sc.c
index 7d296f29dade..f94269814544 100644
--- a/sound/soc/au1x/i2sc.c
+++ b/sound/soc/au1x/i2sc.c
@@ -279,7 +279,6 @@ static void au1xi2s_drvremove(struct platform_device *pdev)
 	WR(ctx, I2S_ENABLE, EN_D);	/* clock off, disable */
 }
 
-#ifdef CONFIG_PM
 static int au1xi2s_drvsuspend(struct device *dev)
 {
 	struct au1xpsc_audio_data *ctx = dev_get_drvdata(dev);
@@ -294,23 +293,13 @@ static int au1xi2s_drvresume(struct device *dev)
 	return 0;
 }
 
-static const struct dev_pm_ops au1xi2sc_pmops = {
-	.suspend	= au1xi2s_drvsuspend,
-	.resume		= au1xi2s_drvresume,
-};
-
-#define AU1XI2SC_PMOPS (&au1xi2sc_pmops)
-
-#else
-
-#define AU1XI2SC_PMOPS NULL
-
-#endif
+static DEFINE_SIPMLE_DEV_PM_OPS(au1xi2sc_pmops, au1xi2s_drvsuspend,
+				au1xi2s_drvresume);
 
 static struct platform_driver au1xi2s_driver = {
 	.driver	= {
 		.name	= "alchemy-i2sc",
-		.pm	= AU1XI2SC_PMOPS,
+		.pm	= pm_ptr(&au1xi2sc_pmops),
 	},
 	.probe		= au1xi2s_drvprobe,
 	.remove		= au1xi2s_drvremove,
diff --git a/sound/soc/au1x/psc-ac97.c b/sound/soc/au1x/psc-ac97.c
index 8a59a50978b9..94698e08a513 100644
--- a/sound/soc/au1x/psc-ac97.c
+++ b/sound/soc/au1x/psc-ac97.c
@@ -436,7 +436,6 @@ static void au1xpsc_ac97_drvremove(struct platform_device *pdev)
 	au1xpsc_ac97_workdata = NULL;	/* MDEV */
 }
 
-#ifdef CONFIG_PM
 static int au1xpsc_ac97_drvsuspend(struct device *dev)
 {
 	struct au1xpsc_audio_data *wd = dev_get_drvdata(dev);
@@ -467,23 +466,13 @@ static int au1xpsc_ac97_drvresume(struct device *dev)
 	return 0;
 }
 
-static const struct dev_pm_ops au1xpscac97_pmops = {
-	.suspend	= au1xpsc_ac97_drvsuspend,
-	.resume		= au1xpsc_ac97_drvresume,
-};
-
-#define AU1XPSCAC97_PMOPS &au1xpscac97_pmops
-
-#else
-
-#define AU1XPSCAC97_PMOPS NULL
-
-#endif
+static DEFINE_SIMPLE_DEV_PM_OPS(au1xpscac97_pmops, au1xpsc_ac97_drvsuspend,
+				au1xpsc_ac97_drvresume);
 
 static struct platform_driver au1xpsc_ac97_driver = {
 	.driver	= {
 		.name	= "au1xpsc_ac97",
-		.pm	= AU1XPSCAC97_PMOPS,
+		.pm	= pm_ptr(&au1xpscac97_pmops),
 	},
 	.probe		= au1xpsc_ac97_drvprobe,
 	.remove		= au1xpsc_ac97_drvremove,
diff --git a/sound/soc/au1x/psc-i2s.c b/sound/soc/au1x/psc-i2s.c
index bee013555e7a..bf59105fcb7a 100644
--- a/sound/soc/au1x/psc-i2s.c
+++ b/sound/soc/au1x/psc-i2s.c
@@ -354,7 +354,6 @@ static void au1xpsc_i2s_drvremove(struct platform_device *pdev)
 	wmb(); /* drain writebuffer */
 }
 
-#ifdef CONFIG_PM
 static int au1xpsc_i2s_drvsuspend(struct device *dev)
 {
 	struct au1xpsc_audio_data *wd = dev_get_drvdata(dev);
@@ -385,23 +384,13 @@ static int au1xpsc_i2s_drvresume(struct device *dev)
 	return 0;
 }
 
-static const struct dev_pm_ops au1xpsci2s_pmops = {
-	.suspend	= au1xpsc_i2s_drvsuspend,
-	.resume		= au1xpsc_i2s_drvresume,
-};
-
-#define AU1XPSCI2S_PMOPS &au1xpsci2s_pmops
-
-#else
-
-#define AU1XPSCI2S_PMOPS NULL
-
-#endif
+static DEFINE_SIMPLE_DEV_PM_OPS(au1xpsci2s_pmops, au1xpsc_i2s_drvsuspend,
+				au1xpsc_i2s_drvresume);
 
 static struct platform_driver au1xpsc_i2s_driver = {
 	.driver		= {
 		.name	= "au1xpsc_i2s",
-		.pm	= AU1XPSCI2S_PMOPS,
+		.pm	= pm_ptr(&au1xpsci2s_pmops),
 	},
 	.probe		= au1xpsc_i2s_drvprobe,
 	.remove		= au1xpsc_i2s_drvremove,
-- 
2.43.0


  reply	other threads:[~2025-03-13 17:42 UTC|newest]

Thread overview: 97+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-03-13 17:40 [PATCH 00/89] ASoC: Convert to modern PM macros Takashi Iwai
2025-03-13 17:40 ` Takashi Iwai [this message]
2025-03-13 17:40 ` [PATCH 02/89] ASoC: ak4375: Convert to RUNTIME_PM_OPS() & co Takashi Iwai
2025-03-13 17:40 ` [PATCH 03/89] ASoC: ak4458: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 04/89] ASoC: ak5558: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 05/89] ASoC: cs35l32: Convert to RUNTIME_PM_OPS() Takashi Iwai
2025-03-13 17:40 ` [PATCH 06/89] ASoC: cs35l33: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 07/89] ASoC: cs35l34: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 08/89] ASoC: cs35l56: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 09/89] ASoC: cs4234: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 10/89] ASoC: cs42l42: Convert to RUNTIME_PM_OPS() & co Takashi Iwai
2025-03-13 17:40 ` [PATCH 11/89] ASoC: cs42l51: Convert to SYSTEM_SLEEP_PM_OPS() Takashi Iwai
2025-03-13 17:40 ` [PATCH 12/89] ASoC: cs42l83: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 13/89] ASoC: cs42xx8: Convert to EXPORT_GPL_DEV_PM_OPS() Takashi Iwai
2025-03-13 17:40 ` [PATCH 14/89] ASoC: cs43130: Convert to RUNTIME_PM_OPS() Takashi Iwai
2025-03-13 17:40 ` [PATCH 15/89] ASoC: cs4349: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 16/89] ASoC: cs53l30: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 17/89] ASoC: cx2072x: Convert to RUNTIME_PM_OPS() & co Takashi Iwai
2025-03-13 17:40 ` [PATCH 18/89] ASoC: da7213: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 19/89] ASoC: hdac_hdmi: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 20/89] ASoC: lpass: Convert to RUNTIME_PM_OPS() Takashi Iwai
2025-03-13 17:40 ` [PATCH 21/89] ASoC: max98090: Convert to RUNTIME_PM_OPS() & co Takashi Iwai
2025-03-13 17:40 ` [PATCH 22/89] ASoC: max98373: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 23/89] ASoC: max98390: Convert to SYSTEM_SLEEP_PM_OPS() Takashi Iwai
2025-03-13 17:40 ` [PATCH 24/89] ASoC: max98396: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 25/89] ASoC: max98520: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 26/89] ASoC: max9860: Convert to RUNTIME_PM_OPS() Takashi Iwai
2025-03-13 17:40 ` [PATCH 27/89] ASoC: max98927: Convert to SYSTEM_SLEEP_PM_OPS() Takashi Iwai
2025-03-13 17:40 ` [PATCH 28/89] ASoC: mt6660: Convert to RUNTIME_PM_OPS() Takashi Iwai
2025-03-13 17:40 ` [PATCH 29/89] ASoC: pcm512x: Convert to EXPORT_GPL_DEV_PM_OPS() Takashi Iwai
2025-03-13 17:40 ` [PATCH 30/89] ASoC: rt1017-sdca-sdw: Convert to RUNTIME_PM_OPS() & co Takashi Iwai
2025-03-13 17:40 ` [PATCH 31/89] ASoC: rt1308-sdw: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 32/89] ASoC: rt1316-sdw: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 33/89] ASoC: rt1318-sdw: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 34/89] ASoC: rt1320-sdw: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 35/89] ASoC: rt5514: Convert to SYSTEM_SLEEP_PM_OPS() Takashi Iwai
2025-03-13 17:40 ` [PATCH 36/89] ASoC: rt5645: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 37/89] ASoC: rt5682-sdw: Convert to RUNTIME_PM_OPS() & co Takashi Iwai
2025-03-13 17:40 ` [PATCH 38/89] ASoC: rt700-sdw: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 39/89] ASoC: rt711: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 40/89] ASoC: rt712: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 41/89] ASoC: rt715: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 42/89] ASoC: rt721: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 43/89] ASoC: rt722: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 44/89] ASoC: rt9120: Convert to RUNTIME_PM_OPS() Takashi Iwai
2025-03-13 17:40 ` [PATCH 45/89] ASoC: rtq9128: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 46/89] ASoC: tas2552: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 47/89] ASoC: ts3a227e: Convert to SYSTEM_SLEEP_PM_OPS() Takashi Iwai
2025-03-13 17:40 ` [PATCH 48/89] ASoC: wcd937x: Convert to RUNTIME_PM_OPS() Takashi Iwai
2025-03-13 17:40 ` [PATCH 49/89] ASoC: wcd938x: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 50/89] ASoC: wcd939x: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 51/89] ASoC: wm2200: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 52/89] ASoC: wm5100: " Takashi Iwai
2025-03-13 17:40 ` [PATCH 53/89] ASoC: wm8804: Convert to EXPORT_GPL_DEV_PM_OPS() Takashi Iwai
2025-03-13 17:40 ` [PATCH 54/89] ASoC: wm8962: Convert to RUNTIME_PM_OPS() & co Takashi Iwai
2025-03-13 17:41 ` [PATCH 55/89] ASoC: wm8994: Convert to SYSTEM_SLEEP_PM_OPS() Takashi Iwai
2025-03-13 17:41 ` [PATCH 56/89] ASoC: wsa881x: Convert to RUNTIME_PM_OPS() Takashi Iwai
2025-03-13 17:41 ` [PATCH 57/89] ASoC: wsa883x: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 58/89] ASoC: wsa884x: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 59/89] ASoC: dwc: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 60/89] ASoC: fsl: Convert to RUNTIME_PM_OPS() and co Takashi Iwai
2025-03-13 17:41 ` [PATCH 61/89] ASoC: img: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 62/89] ASoC: intel: avs: Convert to RUNTIE_PM_OPS() Takashi Iwai
2025-03-13 17:41 ` [PATCH 63/89] ASoC: intel: catpt: Convert to RUNTIE_PM_OPS() & co Takashi Iwai
2025-03-13 17:41 ` [PATCH 64/89] ASoC: mediatek: mt2701: Convert to RUNTIME_PM_OPS() Takashi Iwai
2025-03-13 17:41 ` [PATCH 65/89] ASoC: mediatek: mt6797: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 66/89] ASoC: mediatek: mt7986: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 67/89] ASoC: mediatek: mt8173: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 68/89] ASoC: mediatek: mt8183: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 69/89] ASoC: mediatek: mt8186: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 70/89] ASoC: mediatek: mt8188: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 71/89] ASoC: mediatek: mt8192: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 72/89] ASoC: mediatek: mt8195: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 73/89] ASoC: mediatek: mt8365: Convert to RUNTIME_PM_OPS() & co Takashi Iwai
2025-03-13 17:41 ` [PATCH 74/89] ASoC: qcom: Convert to SYSTEM_SLEEP_PM_OPS() Takashi Iwai
2025-03-13 17:41 ` [PATCH 75/89] ASoC: rcar: " Takashi Iwai
2025-03-14  1:43   ` Kuninori Morimoto
2025-03-13 17:41 ` [PATCH 76/89] ASoC: rockchip: Convert to RUNTIME_PM_OPS() & co Takashi Iwai
2025-03-13 17:41 ` [PATCH 77/89] ASoC: samsung: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 78/89] ASoC: SOF: acpi: Convert to EXPORT_NS_DEV_PM_OPS() Takashi Iwai
2025-03-13 17:41 ` [PATCH 79/89] ASoC: SOF: pci: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 80/89] ASoC: SOF: of: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 81/89] ASoC: stm: Convert to SYSTEM_SLEEP_PM_OPS() Takashi Iwai
2025-03-13 17:41 ` [PATCH 82/89] ASoC: sunxi: Convert to RUNTIME_PM_OPS() Takashi Iwai
2025-03-13 17:41 ` [PATCH 83/89] ASoC: tegra186: Convert to RUNTIME_PM_OPS() & co Takashi Iwai
2025-03-13 17:41 ` [PATCH 84/89] ASoC: tegra210: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 85/89] ASoC: tegra20: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 86/89] ASoC: tegra30: " Takashi Iwai
2025-03-13 17:41 ` [PATCH 87/89] ASoC: xtensa: Convert to RUNTIME_PM_OPS() Takashi Iwai
2025-03-13 18:01   ` Max Filippov
2025-03-13 17:41 ` [PATCH 88/89] ASoC: amd: Convert to RUNTIME_PM_OPS() & co Takashi Iwai
2025-03-13 17:41 ` [PATCH 89/89] ASoC: pcm3168a: Convert to EXPORT_GPL_DEV_PM_OPS() Takashi Iwai
2025-03-14  7:02 ` [PATCH 00/89] ASoC: Convert to modern PM macros Geraldo Nascimento
2025-03-14  8:36   ` Takashi Iwai
2025-03-14  9:39     ` Geraldo Nascimento
2025-03-14 15:36 ` Charles Keepax
2025-03-17 21:54 ` 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=20250313174139.29942-2-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=broonie@kernel.org \
    --cc=linux-sound@vger.kernel.org \
    /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