ASoC: amd: acp: remove acp poweroff function

BIOS invokes ACP Power off sequence based on ACP device state.
Remove redundant code from ACP PCI driver for ACP Power off sequence.

Signed-off-by: Syed Saba Kareem <Syed.SabaKareem@amd.com>
Link: https://lore.kernel.org/r/Message-Id:
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/amd/acp/acp-rembrandt.c b/sound/soc/amd/acp/acp-rembrandt.c
index 5c455cc041137a65acc47ec0875454d6143df58d..1b997837c7d81def5fc3ba3cfaedffa2a980d890 100644 (file)
--- a/sound/soc/amd/acp/acp-rembrandt.c
+++ b/sound/soc/amd/acp/acp-rembrandt.c
@@ -204,23 +204,6 @@ static int acp6x_power_on(void __iomem *base)
        return -ETIMEDOUT;
 }
 
-static int acp6x_power_off(void __iomem *base)
-{
-       u32 val;
-       int timeout;
-
-       writel(ACP_PGFSM_CNTL_POWER_OFF_MASK,
-              base + ACP6X_PGFSM_CONTROL);
-       timeout = 0;
-       while (++timeout < 500) {
-               val = readl(base + ACP6X_PGFSM_STATUS);
-               if ((val & ACP_PGFSM_STATUS_MASK) == ACP_POWERED_OFF)
-                       return 0;
-               udelay(1);
-       }
-       return -ETIMEDOUT;
-}
-
 static int acp6x_reset(void __iomem *base)
 {
        u32 val;
@@ -299,14 +282,6 @@ static int rmb_acp_deinit(void __iomem *base)
        }
 
        writel(0x00, base + ACP_CONTROL);
-
-       /* power off */
-       ret = acp6x_power_off(base);
-       if (ret) {
-               pr_err("ACP power off failed\n");
-               return ret;
-       }
-
        return 0;
 }
 

diff --git a/sound/soc/amd/acp/acp-renoir.c b/sound/soc/amd/acp/acp-renoir.c
index b3cbc7f19ec544b329534c74057aa0f4044bfc67..f188365fe214b7d23c82780e1989e59f3bc6f52a 100644 (file)
--- a/sound/soc/amd/acp/acp-renoir.c
+++ b/sound/soc/amd/acp/acp-renoir.c
@@ -169,17 +169,6 @@ static int acp3x_power_on(void __iomem *base)
        return readl_poll_timeout(base + ACP_PGFSM_STATUS, val, !val, DELAY_US, ACP_TIMEOUT);
 }
 
-static int acp3x_power_off(void __iomem *base)
-{
-       u32 val;
-
-       writel(ACP_PWR_OFF_MASK, base + ACP_PGFSM_CONTROL);
-
-       return readl_poll_timeout(base + ACP_PGFSM_STATUS, val,
-                                 (val & ACP_PGFSM_STAT_MASK) == ACP_POWERED_OFF,
-                                 DELAY_US, ACP_TIMEOUT);
-}
-
 static int acp3x_reset(void __iomem *base)
 {
        u32 val;
@@ -246,12 +235,6 @@ static int rn_acp_deinit(void __iomem *base)
                return ret;
 
        writel(0x00, base + ACP_CONTROL);
-
-       /* power off */
-       ret = acp3x_power_off(base);
-       if (ret)
-               return ret;
-
        return 0;
 }
 static int renoir_audio_probe(struct platform_device *pdev)