ASoC: amd: Added ACP3x system resume and runtime pm

When system wide suspend happens, ACP will be powered off
and when system resumes,for audio usecase to continue,all
the runtime configuration data needs to be programmed again.
Added resume pm call back to ACP pm ops and also added runtime
PM operations for ACP3x PCM platform device.
Device will enter into D3 state when there is no activity
on audio I2S lines.

Signed-off-by: Ravulapati Vishnu vardhan rao <Vishnuvardhanrao.Ravulapati@amd.com>
Link: https://lore.kernel.org/r/1577540460-21438-6-git-send-email-Vishnuvardhanrao.Ravulapati@amd.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/amd/raven/acp3x-pcm-dma.c b/sound/soc/amd/raven/acp3x-pcm-dma.c
index c5d7f3558867921d6e20455d04889cfcf9d2e437..151daa996343a4a46f45633ba8b030ae6458c36f 100644 (file)
--- a/sound/soc/amd/raven/acp3x-pcm-dma.c
+++ b/sound/soc/amd/raven/acp3x-pcm-dma.c
@@ -57,106 +57,6 @@ static const struct snd_pcm_hardware acp3x_pcm_hardware_capture = {
        .periods_max = CAPTURE_MAX_NUM_PERIODS,
 };
 
-static int acp3x_power_on(void __iomem *acp3x_base, bool on)
-{
-       u16 val, mask;
-       u32 timeout;
-
-       if (on == true) {
-               val = 1;
-               mask = ACP3x_POWER_ON;
-       } else {
-               val = 0;
-               mask = ACP3x_POWER_OFF;
-       }
-
-       rv_writel(val, acp3x_base + mmACP_PGFSM_CONTROL);
-       timeout = 0;
-       while (true) {
-               val = rv_readl(acp3x_base + mmACP_PGFSM_STATUS);
-               if ((val & ACP3x_POWER_OFF_IN_PROGRESS) == mask)
-                       break;
-               if (timeout > 100) {
-                       pr_err("ACP3x power state change failure\n");
-                       return -ENODEV;
-               }
-               timeout++;
-               cpu_relax();
-       }
-       return 0;
-}
-
-static int acp3x_reset(void __iomem *acp3x_base)
-{
-       u32 val, timeout;
-
-       rv_writel(1, acp3x_base + mmACP_SOFT_RESET);
-       timeout = 0;
-       while (true) {
-               val = rv_readl(acp3x_base + mmACP_SOFT_RESET);
-               if ((val & ACP3x_SOFT_RESET__SoftResetAudDone_MASK) ||
-                    timeout > 100) {
-                       if (val & ACP3x_SOFT_RESET__SoftResetAudDone_MASK)
-                               break;
-                       return -ENODEV;
-               }
-               timeout++;
-               cpu_relax();
-       }
-
-       rv_writel(0, acp3x_base + mmACP_SOFT_RESET);
-       timeout = 0;
-       while (true) {
-               val = rv_readl(acp3x_base + mmACP_SOFT_RESET);
-               if (!val || timeout > 100) {
-                       if (!val)
-                               break;
-                       return -ENODEV;
-               }
-               timeout++;
-               cpu_relax();
-       }
-       return 0;
-}
-
-static int acp3x_init(void __iomem *acp3x_base)
-{
-       int ret;
-
-       /* power on */
-       ret = acp3x_power_on(acp3x_base, true);
-       if (ret) {
-               pr_err("ACP3x power on failed\n");
-               return ret;
-       }
-       /* Reset */
-       ret = acp3x_reset(acp3x_base);
-       if (ret) {
-               pr_err("ACP3x reset failed\n");
-               return ret;
-       }
-       return 0;
-}
-
-static int acp3x_deinit(void __iomem *acp3x_base)
-{
-       int ret;
-
-       /* Reset */
-       ret = acp3x_reset(acp3x_base);
-       if (ret) {
-               pr_err("ACP3x reset failed\n");
-               return ret;
-       }
-       /* power off */
-       ret = acp3x_power_on(acp3x_base, false);
-       if (ret) {
-               pr_err("ACP3x power off failed\n");
-               return ret;
-       }
-       return 0;
-}
-
 static irqreturn_t i2s_irq_handler(int irq, void *dev_id)
 {
        struct i2s_dev_data *rv_i2s_data;
@@ -479,7 +379,7 @@ static int acp3x_audio_probe(struct platform_device *pdev)
        struct resource *res;
        struct i2s_dev_data *adata;
        unsigned int irqflags;
-       int status, ret;
+       int status;
 
        if (!pdev->dev.platform_data) {
                dev_err(&pdev->dev, "platform_data not retrieved\n");
@@ -511,53 +411,29 @@ static int acp3x_audio_probe(struct platform_device *pdev)
        adata->i2s_irq = res->start;
 
        dev_set_drvdata(&pdev->dev, adata);
-       /* Initialize ACP */
-       status = acp3x_init(adata->acp3x_base);
-       if (status)
-               return -ENODEV;
-
        status = devm_snd_soc_register_component(&pdev->dev,
                                                 &acp3x_i2s_component,
                                                 NULL, 0);
        if (status) {
                dev_err(&pdev->dev, "Fail to register acp i2s component\n");
-               ret = -ENODEV;
-               goto dev_err;
+               return -ENODEV;
        }
        status = devm_request_irq(&pdev->dev, adata->i2s_irq, i2s_irq_handler,
                                  irqflags, "ACP3x_I2S_IRQ", adata);
        if (status) {
                dev_err(&pdev->dev, "ACP3x I2S IRQ request failed\n");
-               ret = -ENODEV;
-               goto dev_err;
+               return -ENODEV;
        }
 
-       pm_runtime_set_autosuspend_delay(&pdev->dev, 5000);
+       pm_runtime_set_autosuspend_delay(&pdev->dev, 2000);
        pm_runtime_use_autosuspend(&pdev->dev);
        pm_runtime_enable(&pdev->dev);
+       pm_runtime_allow(&pdev->dev);
        return 0;
-
-dev_err:
-       status = acp3x_deinit(adata->acp3x_base);
-       if (status)
-               dev_err(&pdev->dev, "ACP de-init failed\n");
-       else
-               dev_dbg(&pdev->dev, "ACP de-initialized\n");
-       return ret;
 }
 
 static int acp3x_audio_remove(struct platform_device *pdev)
 {
-       struct i2s_dev_data *adata;
-       int ret;
-
-       adata = dev_get_drvdata(&pdev->dev);
-       ret = acp3x_deinit(adata->acp3x_base);
-       if (ret)
-               dev_err(&pdev->dev, "ACP de-init failed\n");
-       else
-               dev_dbg(&pdev->dev, "ACP de-initialized\n");
-
        pm_runtime_disable(&pdev->dev);
        return 0;
 }
@@ -565,15 +441,11 @@ static int acp3x_audio_remove(struct platform_device *pdev)
 static int acp3x_resume(struct device *dev)
 {
        struct i2s_dev_data *adata;
-       int status;
        u32 val, reg_val, frmt_val;
 
        reg_val = 0;
        frmt_val = 0;
        adata = dev_get_drvdata(dev);
-       status = acp3x_init(adata->acp3x_base);
-       if (status)
-               return -ENODEV;
 
        if (adata->play_stream && adata->play_stream->runtime) {
                struct i2s_stream_instance *rtd =
@@ -620,14 +492,8 @@ static int acp3x_resume(struct device *dev)
 static int acp3x_pcm_runtime_suspend(struct device *dev)
 {
        struct i2s_dev_data *adata;
-       int status;
 
        adata = dev_get_drvdata(dev);
-       status = acp3x_deinit(adata->acp3x_base);
-       if (status)
-               dev_err(dev, "ACP de-init failed\n");
-       else
-               dev_dbg(dev, "ACP de-initialized\n");
 
        rv_writel(0, adata->acp3x_base + mmACP_EXTERNAL_INTR_ENB);
 
@@ -637,12 +503,9 @@ static int acp3x_pcm_runtime_suspend(struct device *dev)
 static int acp3x_pcm_runtime_resume(struct device *dev)
 {
        struct i2s_dev_data *adata;
-       int status;
 
        adata = dev_get_drvdata(dev);
-       status = acp3x_init(adata->acp3x_base);
-       if (status)
-               return -ENODEV;
+
        rv_writel(1, adata->acp3x_base + mmACP_EXTERNAL_INTR_ENB);
        return 0;
 }

diff --git a/sound/soc/amd/raven/acp3x.h b/sound/soc/amd/raven/acp3x.h
index 43213aec7f59f79ea43b78a764dcd30dfc8807b1..b6a80dc0b641e4d61357b9b8351daad3bf79f33b 100644 (file)
--- a/sound/soc/amd/raven/acp3x.h
+++ b/sound/soc/amd/raven/acp3x.h
@@ -68,6 +68,13 @@
 #define SLOT_WIDTH_16 0x10
 #define SLOT_WIDTH_24 0x18
 #define SLOT_WIDTH_32 0x20
+#define ACP_PGFSM_CNTL_POWER_ON_MASK   0x01
+#define ACP_PGFSM_CNTL_POWER_OFF_MASK  0x00
+#define ACP_PGFSM_STATUS_MASK          0x03
+#define ACP_POWERED_ON                 0x00
+#define ACP_POWER_ON_IN_PROGRESS       0x01
+#define ACP_POWERED_OFF                        0x02
+#define ACP_POWER_OFF_IN_PROGRESS      0x03
 
 struct acp3x_platform_info {
        u16 play_i2s_instance;

diff --git a/sound/soc/amd/raven/pci-acp3x.c b/sound/soc/amd/raven/pci-acp3x.c
index 94f5f21d9a538ba6b39df881a234f922837e21fb..2f9f52905853ae781c6e183658a1aae0edf53e55 100644 (file)
--- a/sound/soc/amd/raven/pci-acp3x.c
+++ b/sound/soc/amd/raven/pci-acp3x.c
@@ -9,6 +9,8 @@
 #include <linux/io.h>
 #include <linux/platform_device.h>
 #include <linux/interrupt.h>
+#include <linux/pm_runtime.h>
+#include <linux/delay.h>
 
 #include "acp3x.h"
 
@@ -19,6 +21,109 @@ struct acp3x_dev_data {
        struct platform_device *pdev[ACP3x_DEVS];
 };
 
+static int acp3x_power_on(void __iomem *acp3x_base)
+{
+       u32 val;
+       int timeout;
+
+       val = rv_readl(acp3x_base + mmACP_PGFSM_STATUS);
+
+       if (val == 0)
+               return val;
+
+       if (!((val & ACP_PGFSM_STATUS_MASK) ==
+                               ACP_POWER_ON_IN_PROGRESS))
+               rv_writel(ACP_PGFSM_CNTL_POWER_ON_MASK,
+                       acp3x_base + mmACP_PGFSM_CONTROL);
+       timeout = 0;
+       while (++timeout < 500) {
+               val = rv_readl(acp3x_base + mmACP_PGFSM_STATUS);
+               if (!val)
+                       return 0;
+               udelay(1);
+       }
+       return -ETIMEDOUT;
+}
+
+static int acp3x_power_off(void __iomem *acp3x_base)
+{
+       u32 val;
+       int timeout;
+
+       rv_writel(ACP_PGFSM_CNTL_POWER_OFF_MASK,
+                       acp3x_base + mmACP_PGFSM_CONTROL);
+       timeout = 0;
+       while (++timeout < 500) {
+               val = rv_readl(acp3x_base + mmACP_PGFSM_STATUS);
+               if ((val & ACP_PGFSM_STATUS_MASK) == ACP_POWERED_OFF)
+                       return 0;
+               udelay(1);
+       }
+       return -ETIMEDOUT;
+}
+
+static int acp3x_reset(void __iomem *acp3x_base)
+{
+       u32 val;
+       int timeout;
+
+       rv_writel(1, acp3x_base + mmACP_SOFT_RESET);
+       timeout = 0;
+       while (++timeout < 500) {
+               val = rv_readl(acp3x_base + mmACP_SOFT_RESET);
+               if (val & ACP3x_SOFT_RESET__SoftResetAudDone_MASK)
+                       break;
+               cpu_relax();
+       }
+       rv_writel(0, acp3x_base + mmACP_SOFT_RESET);
+       timeout = 0;
+       while (++timeout < 500) {
+               val = rv_readl(acp3x_base + mmACP_SOFT_RESET);
+               if (!val)
+                       return 0;
+               cpu_relax();
+       }
+       return -ETIMEDOUT;
+}
+
+static int acp3x_init(void __iomem *acp3x_base)
+{
+       int ret;
+
+       /* power on */
+       ret = acp3x_power_on(acp3x_base);
+       if (ret) {
+               pr_err("ACP3x power on failed\n");
+               return ret;
+       }
+       /* Reset */
+       ret = acp3x_reset(acp3x_base);
+       if (ret) {
+               pr_err("ACP3x reset failed\n");
+               return ret;
+       }
+       return 0;
+}
+
+static int acp3x_deinit(void __iomem *acp3x_base)
+{
+       int ret;
+
+       /* Reset */
+       ret = acp3x_reset(acp3x_base);
+       if (ret) {
+               pr_err("ACP3x reset failed\n");
+               return ret;
+       }
+       /* power off */
+       ret = acp3x_power_off(acp3x_base);
+       if (ret) {
+               pr_err("ACP3x power off failed\n");
+               return ret;
+       }
+       return 0;
+}
+
 static int snd_acp3x_probe(struct pci_dev *pci,
                           const struct pci_device_id *pci_id)
 {
@@ -64,6 +169,9 @@ static int snd_acp3x_probe(struct pci_dev *pci,
        }
        pci_set_master(pci);
        pci_set_drvdata(pci, adata);
+       ret = acp3x_init(adata->acp3x_base);
+       if (ret)
+               goto disable_msi;
 
        val = rv_readl(adata->acp3x_base + mmACP_I2S_PIN_CONFIG);
        switch (val) {
@@ -73,7 +181,7 @@ static int snd_acp3x_probe(struct pci_dev *pci,
                                          GFP_KERNEL);
                if (!adata->res) {
                        ret = -ENOMEM;
-                       goto disable_msi;
+                       goto de_init;
                }
 
                adata->res[0].name = "acp3x_i2s_iomem";
@@ -118,7 +226,7 @@ static int snd_acp3x_probe(struct pci_dev *pci,
                pdevinfo[2].parent = &pci->dev;
                pdevinfo[2].num_res = 1;
                pdevinfo[2].res = &adata->res[2];
-               for (i = 0; i < ACP3x_DEVS ; i++) {
+               for (i = 0; i < ACP3x_DEVS; i++) {
                        adata->pdev[i] =
                                platform_device_register_full(&pdevinfo[i]);
                        if (IS_ERR(adata->pdev[i])) {
@@ -134,12 +242,21 @@ static int snd_acp3x_probe(struct pci_dev *pci,
                ret = -ENODEV;
                goto disable_msi;
        }
+       pm_runtime_set_autosuspend_delay(&pci->dev, 2000);
+       pm_runtime_use_autosuspend(&pci->dev);
+       pm_runtime_set_active(&pci->dev);
+       pm_runtime_put_noidle(&pci->dev);
+       pm_runtime_enable(&pci->dev);
+       pm_runtime_allow(&pci->dev);
        return 0;
 
 unregister_devs:
        if (val == I2S_MODE)
-               for (i = 0 ; i < ACP3x_DEVS ; i++)
+               for (i = 0; i < ACP3x_DEVS; i++)
                        platform_device_unregister(adata->pdev[i]);
+de_init:
+       if (acp3x_deinit(adata->acp3x_base))
+               dev_err(&pci->dev, "ACP de-init failed\n");
 disable_msi:
        pci_disable_msi(pci);
 release_regions:
@@ -150,15 +267,56 @@ disable_pci:
        return ret;
 }
 
+static int snd_acp3x_suspend(struct device *dev)
+{
+       int ret;
+       struct acp3x_dev_data *adata;
+
+       adata = dev_get_drvdata(dev);
+       ret = acp3x_deinit(adata->acp3x_base);
+       if (ret)
+               dev_err(dev, "ACP de-init failed\n");
+       else
+               dev_dbg(dev, "ACP de-initialized\n");
+
+       return 0;
+}
+
+static int snd_acp3x_resume(struct device *dev)
+{
+       int ret;
+       struct acp3x_dev_data *adata;
+
+       adata = dev_get_drvdata(dev);
+       ret = acp3x_init(adata->acp3x_base);
+       if (ret) {
+               dev_err(dev, "ACP init failed\n");
+               return ret;
+       }
+       return 0;
+}
+
+static const struct dev_pm_ops acp3x_pm = {
+       .runtime_suspend = snd_acp3x_suspend,
+       .runtime_resume =  snd_acp3x_resume,
+       .resume =       snd_acp3x_resume,
+};
+
 static void snd_acp3x_remove(struct pci_dev *pci)
 {
-       struct acp3x_dev_data *adata = pci_get_drvdata(pci);
-       int i;
+       struct acp3x_dev_data *adata;
+       int i, ret;
 
+       adata = pci_get_drvdata(pci);
        if (adata->acp3x_audio_mode == ACP3x_I2S_MODE) {
-               for (i = 0 ; i <  ACP3x_DEVS ; i++)
+               for (i = 0; i < ACP3x_DEVS; i++)
                        platform_device_unregister(adata->pdev[i]);
        }
+       ret = acp3x_deinit(adata->acp3x_base);
+       if (ret)
+               dev_err(&pci->dev, "ACP de-init failed\n");
+       pm_runtime_disable(&pci->dev);
+       pm_runtime_get_noresume(&pci->dev);
        pci_disable_msi(pci);
        pci_release_regions(pci);
        pci_disable_device(pci);
@@ -177,6 +335,9 @@ static struct pci_driver acp3x_driver  = {
        .id_table = snd_acp3x_ids,
        .probe = snd_acp3x_probe,
        .remove = snd_acp3x_remove,
+       .driver = {
+               .pm = &acp3x_pm,
+       }
 };
 
 module_pci_driver(acp3x_driver);