From nobody Thu Sep 18 04:23:45 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0C2B1C4167B for ; Mon, 12 Dec 2022 13:10:33 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231741AbiLLNKb (ORCPT ); Mon, 12 Dec 2022 08:10:31 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39746 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231638AbiLLNKZ (ORCPT ); Mon, 12 Dec 2022 08:10:25 -0500 Received: from esa.microchip.iphmx.com (esa.microchip.iphmx.com [68.232.153.233]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 61D2111473; Mon, 12 Dec 2022 05:10:24 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1670850624; x=1702386624; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=bslg/447s4CvuvWyjtCJZnYd7qg99VqMxwy62j2qIto=; b=T5qEmEKTv2J5tOIAzEfYQnfIVUz7j9Tdh7HMp58JS7F/CkYxJ8RHeFsX HEMyV4KblCdJV6zGq4vhvwMoYIFC9GPLR1xURYYT3B1HzL12bWGtgxbkj wvCTn7Rr+gF+bAfoJDy3cVnGxCJXVTv5JEvZnpVOleyDXny/WZ4ihGrR1 kuzYRQacw2/UB68tefuob83pqOGNYATEaZRe5/qHgzKkVcgz+XOdzKGTQ 93O8vSlT236+VRvpykZO+FlVhIu1mvceiHfWtXVBhGS4mL6pwpTOBqkZG qStNZnuRvfDFjhq5U4sNKVUAfAm8NjtGQJXfj5QzpNC/vafG2AkJkuDNU Q==; X-IronPort-AV: E=Sophos;i="5.96,238,1665471600"; d="scan'208";a="192689974" Received: from unknown (HELO email.microchip.com) ([170.129.1.10]) by esa3.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 12 Dec 2022 06:10:21 -0700 Received: from chn-vm-ex01.mchp-main.com (10.10.85.143) by chn-vm-ex04.mchp-main.com (10.10.85.152) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.16; Mon, 12 Dec 2022 06:10:21 -0700 Received: from localhost.localdomain (10.10.115.15) by chn-vm-ex01.mchp-main.com (10.10.85.143) with Microsoft SMTP Server id 15.1.2507.16 via Frontend Transport; Mon, 12 Dec 2022 06:10:18 -0700 From: Claudiu Beznea To: , , , , , , , CC: , , , , Claudiu Beznea Subject: [PATCH 2/4] ASoC: mchp-pdmc: use runtime pm for clock power saving Date: Mon, 12 Dec 2022 15:15:33 +0200 Message-ID: <20221212131535.76111-3-claudiu.beznea@microchip.com> X-Mailer: git-send-email 2.33.0 In-Reply-To: <20221212131535.76111-1-claudiu.beznea@microchip.com> References: <20221212131535.76111-1-claudiu.beznea@microchip.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Implement clock power saving by taking advantage of runtime PM infrastructure. This simplifies the code and allow using the same infrastructure for suspend to RAM functionalities. Signed-off-by: Claudiu Beznea --- sound/soc/atmel/mchp-pdmc.c | 126 ++++++++++++++++++++++++------------ 1 file changed, 84 insertions(+), 42 deletions(-) diff --git a/sound/soc/atmel/mchp-pdmc.c b/sound/soc/atmel/mchp-pdmc.c index 44aefbd5b62c..f184404e74e5 100644 --- a/sound/soc/atmel/mchp-pdmc.c +++ b/sound/soc/atmel/mchp-pdmc.c @@ -11,6 +11,7 @@ #include #include #include +#include #include =20 #include @@ -115,7 +116,6 @@ struct mchp_pdmc { int mic_no; int sinc_order; bool audio_filter_en; - u8 gclk_enabled:1; }; =20 static const char *const mchp_pdmc_sinc_filter_order_text[] =3D { @@ -454,13 +454,6 @@ static int mchp_pdmc_startup(struct snd_pcm_substream = *substream, struct snd_soc_dai *dai) { struct mchp_pdmc *dd =3D snd_soc_dai_get_drvdata(dai); - int ret; - - ret =3D clk_prepare_enable(dd->pclk); - if (ret) { - dev_err(dd->dev, "failed to enable the peripheral clock: %d\n", ret); - return ret; - } =20 regmap_write(dd->regmap, MCHP_PDMC_CR, MCHP_PDMC_CR_SWRST); =20 @@ -470,14 +463,6 @@ static int mchp_pdmc_startup(struct snd_pcm_substream = *substream, return 0; } =20 -static void mchp_pdmc_shutdown(struct snd_pcm_substream *substream, - struct snd_soc_dai *dai) -{ - struct mchp_pdmc *dd =3D snd_soc_dai_get_drvdata(dai); - - clk_disable_unprepare(dd->pclk); -} - static int mchp_pdmc_dai_probe(struct snd_soc_dai *dai) { struct mchp_pdmc *dd =3D snd_soc_dai_get_drvdata(dai); @@ -594,11 +579,6 @@ static int mchp_pdmc_hw_params(struct snd_pcm_substrea= m *substream, cfgr_val |=3D MCHP_PDMC_CFGR_BSSEL(i); } =20 - if (dd->gclk_enabled) { - clk_disable_unprepare(dd->gclk); - dd->gclk_enabled =3D 0; - } - for (osr_start =3D dd->audio_filter_en ? 64 : 8; osr_start <=3D 256 && best_diff_rate; osr_start *=3D 2) { long round_rate; @@ -620,8 +600,12 @@ static int mchp_pdmc_hw_params(struct snd_pcm_substrea= m *substream, return -EINVAL; } =20 + /* CLK is enabled by runtime PM. */ + clk_disable_unprepare(dd->gclk); + /* set the rate */ ret =3D clk_set_rate(dd->gclk, gclk_rate); + clk_prepare_enable(dd->gclk); if (ret) { dev_err(comp->dev, "unable to set rate %lu to GCLK: %d\n", gclk_rate, ret); @@ -636,9 +620,6 @@ static int mchp_pdmc_hw_params(struct snd_pcm_substream= *substream, mr_val |=3D MCHP_PDMC_MR_CHUNK(dd->addr.maxburst); dev_dbg(comp->dev, "maxburst set to %d\n", dd->addr.maxburst); =20 - clk_prepare_enable(dd->gclk); - dd->gclk_enabled =3D 1; - snd_soc_component_update_bits(comp, MCHP_PDMC_MR, MCHP_PDMC_MR_OSR_MASK | MCHP_PDMC_MR_SINCORDER_MASK | @@ -650,19 +631,6 @@ static int mchp_pdmc_hw_params(struct snd_pcm_substrea= m *substream, return 0; } =20 -static int mchp_pdmc_hw_free(struct snd_pcm_substream *substream, - struct snd_soc_dai *dai) -{ - struct mchp_pdmc *dd =3D snd_soc_dai_get_drvdata(dai); - - if (dd->gclk_enabled) { - clk_disable_unprepare(dd->gclk); - dd->gclk_enabled =3D 0; - } - - return 0; -} - static int mchp_pdmc_trigger(struct snd_pcm_substream *substream, int cmd, struct snd_soc_dai *dai) { @@ -711,9 +679,7 @@ static int mchp_pdmc_trigger(struct snd_pcm_substream *= substream, static const struct snd_soc_dai_ops mchp_pdmc_dai_ops =3D { .set_fmt =3D mchp_pdmc_set_fmt, .startup =3D mchp_pdmc_startup, - .shutdown =3D mchp_pdmc_shutdown, .hw_params =3D mchp_pdmc_hw_params, - .hw_free =3D mchp_pdmc_hw_free, .trigger =3D mchp_pdmc_trigger, }; =20 @@ -864,6 +830,7 @@ static const struct regmap_config mchp_pdmc_regmap_conf= ig =3D { .readable_reg =3D mchp_pdmc_readable_reg, .writeable_reg =3D mchp_pdmc_writeable_reg, .precious_reg =3D mchp_pdmc_precious_reg, + .cache_type =3D REGCACHE_FLAT, }; =20 static int mchp_pdmc_dt_init(struct mchp_pdmc *dd) @@ -970,6 +937,49 @@ static struct snd_dmaengine_pcm_config mchp_pdmc_confi= g =3D { .prepare_slave_config =3D snd_dmaengine_pcm_prepare_slave_config, }; =20 +static int mchp_pdmc_runtime_suspend(struct device *dev) +{ + struct mchp_pdmc *dd =3D dev_get_drvdata(dev); + + regcache_cache_only(dd->regmap, true); + + clk_disable_unprepare(dd->gclk); + clk_disable_unprepare(dd->pclk); + + return 0; +} + +static int mchp_pdmc_runtime_resume(struct device *dev) +{ + struct mchp_pdmc *dd =3D dev_get_drvdata(dev); + int ret; + + ret =3D clk_prepare_enable(dd->pclk); + if (ret) { + dev_err(dd->dev, + "failed to enable the peripheral clock: %d\n", ret); + return ret; + } + ret =3D clk_prepare_enable(dd->gclk); + if (ret) { + dev_err(dd->dev, + "failed to enable generic clock: %d\n", ret); + goto disable_pclk; + } + + regcache_cache_only(dd->regmap, false); + regcache_mark_dirty(dd->regmap); + ret =3D regcache_sync(dd->regmap); + if (ret) { + regcache_cache_only(dd->regmap, true); + clk_disable_unprepare(dd->gclk); +disable_pclk: + clk_disable_unprepare(dd->pclk); + } + + return ret; +} + static int mchp_pdmc_probe(struct platform_device *pdev) { struct device *dev =3D &pdev->dev; @@ -1039,18 +1049,25 @@ static int mchp_pdmc_probe(struct platform_device *= pdev) dd->addr.addr =3D (dma_addr_t)res->start + MCHP_PDMC_RHR; platform_set_drvdata(pdev, dd); =20 + pm_runtime_enable(dd->dev); + if (!pm_runtime_enabled(dd->dev)) { + ret =3D mchp_pdmc_runtime_resume(dd->dev); + if (ret) + return ret; + } + /* register platform */ ret =3D devm_snd_dmaengine_pcm_register(dev, &mchp_pdmc_config, 0); if (ret) { dev_err(dev, "could not register platform: %d\n", ret); - return ret; + goto pm_runtime_suspend; } =20 ret =3D devm_snd_soc_register_component(dev, &mchp_pdmc_dai_component, &mchp_pdmc_dai, 1); if (ret) { dev_err(dev, "could not register CPU DAI: %d\n", ret); - return ret; + goto pm_runtime_suspend; } =20 /* print IP version */ @@ -1059,6 +1076,25 @@ static int mchp_pdmc_probe(struct platform_device *p= dev) version & MCHP_PDMC_VER_VERSION); =20 return 0; + +pm_runtime_suspend: + if (!pm_runtime_status_suspended(dd->dev)) + mchp_pdmc_runtime_suspend(dd->dev); + pm_runtime_disable(dd->dev); + + return ret; +} + +static int mchp_pdmc_remove(struct platform_device *pdev) +{ + struct mchp_pdmc *dd =3D platform_get_drvdata(pdev); + + if (!pm_runtime_status_suspended(dd->dev)) + mchp_pdmc_runtime_suspend(dd->dev); + + pm_runtime_disable(dd->dev); + + return 0; } =20 static const struct of_device_id mchp_pdmc_of_match[] =3D { @@ -1070,13 +1106,19 @@ static const struct of_device_id mchp_pdmc_of_match= [] =3D { }; MODULE_DEVICE_TABLE(of, mchp_pdmc_of_match); =20 +static const struct dev_pm_ops mchp_pdmc_pm_ops =3D { + RUNTIME_PM_OPS(mchp_pdmc_runtime_suspend, mchp_pdmc_runtime_resume, + NULL) +}; + static struct platform_driver mchp_pdmc_driver =3D { .driver =3D { .name =3D "mchp-pdmc", .of_match_table =3D of_match_ptr(mchp_pdmc_of_match), - .pm =3D &snd_soc_pm_ops, + .pm =3D pm_ptr(&mchp_pdmc_pm_ops), }, .probe =3D mchp_pdmc_probe, + .remove =3D mchp_pdmc_remove, }; module_platform_driver(mchp_pdmc_driver); =20 --=20 2.34.1