ASoC: fsl_audmix: Don't use devm_regmap_init_mmio_clk
authorShengjiu Wang <shengjiu.wang@nxp.com>
Wed, 24 Mar 2021 09:58:47 +0000 (17:58 +0800)
committerMark Brown <broonie@kernel.org>
Wed, 24 Mar 2021 19:49:31 +0000 (19:49 +0000)
When there is power domain bind with ipg clock,

The call flow:
devm_regmap_init_mmio_clk
   - clk_prepare()
       - clk_pm_runtime_get()

cause the power domain of clock always be enabled after
regmap_init(). which impact the power consumption.

So use devm_regmap_init_mmio instead of
devm_regmap_init_mmio_clk,but explicitly enable
clock when it is used.

Signed-off-by: Shengjiu Wang <shengjiu.wang@nxp.com>
Link: https://lore.kernel.org/r/1616579928-22428-6-git-send-email-shengjiu.wang@nxp.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/fsl/fsl_audmix.c

index 8dc44de..f931288 100644 (file)
@@ -476,8 +476,7 @@ static int fsl_audmix_probe(struct platform_device *pdev)
        if (IS_ERR(regs))
                return PTR_ERR(regs);
 
-       priv->regmap = devm_regmap_init_mmio_clk(dev, "ipg", regs,
-                                                &fsl_audmix_regmap_config);
+       priv->regmap = devm_regmap_init_mmio(dev, regs, &fsl_audmix_regmap_config);
        if (IS_ERR(priv->regmap)) {
                dev_err(dev, "failed to init regmap\n");
                return PTR_ERR(priv->regmap);