Commit bb2b2624 authored by Anson Huang's avatar Anson Huang Committed by Jassi Brar

mailbox: imx: Add runtime PM callback to handle MU clocks

Some of i.MX8M SoCs have MU clock, they need to be managed in runtime
to make sure the MU clock can be off in runtime, add runtime PM callback
to handle MU clock.

And on i.MX8MP, the MU clock is combined with power domain and runtime
PM is enabled for the clock driver, during noirq suspend/resume phase,
runtime PM is disabled by device suspend, but the MU context save/restore
needs to enable MU clock for register access, calling clock prepare/enable
will trigger runtime resume failure and lead to system suspend failed.

Actually, the MU context save/restore is ONLY necessary for SCU IPC MU,
other MUs especially on i.MX8MP platforms which have MU clock assigned,
they need to runtime request/free mailbox channel in the consumer driver,
so no need to save/restore MU context for them, hence it can avoid this
issue, so the MU context save/restore is ONLY applied to i.MX platforms
MU instance without clock present.
Signed-off-by: default avatarAnson Huang <Anson.Huang@nxp.com>
Signed-off-by: default avatarJassi Brar <jaswinder.singh@linaro.org>
parent ba5f9fa0
...@@ -536,10 +536,13 @@ static int imx_mu_probe(struct platform_device *pdev) ...@@ -536,10 +536,13 @@ static int imx_mu_probe(struct platform_device *pdev)
if (ret < 0) if (ret < 0)
goto disable_runtime_pm; goto disable_runtime_pm;
clk_disable_unprepare(priv->clk);
return 0; return 0;
disable_runtime_pm: disable_runtime_pm:
pm_runtime_disable(dev); pm_runtime_disable(dev);
clk_disable_unprepare(priv->clk);
return ret; return ret;
} }
...@@ -547,7 +550,6 @@ static int imx_mu_remove(struct platform_device *pdev) ...@@ -547,7 +550,6 @@ static int imx_mu_remove(struct platform_device *pdev)
{ {
struct imx_mu_priv *priv = platform_get_drvdata(pdev); struct imx_mu_priv *priv = platform_get_drvdata(pdev);
clk_disable_unprepare(priv->clk);
pm_runtime_disable(priv->dev); pm_runtime_disable(priv->dev);
return 0; return 0;
...@@ -595,6 +597,7 @@ static int imx_mu_suspend_noirq(struct device *dev) ...@@ -595,6 +597,7 @@ static int imx_mu_suspend_noirq(struct device *dev)
{ {
struct imx_mu_priv *priv = dev_get_drvdata(dev); struct imx_mu_priv *priv = dev_get_drvdata(dev);
if (!priv->clk)
priv->xcr = imx_mu_read(priv, priv->dcfg->xCR); priv->xcr = imx_mu_read(priv, priv->dcfg->xCR);
return 0; return 0;
...@@ -612,15 +615,38 @@ static int imx_mu_resume_noirq(struct device *dev) ...@@ -612,15 +615,38 @@ static int imx_mu_resume_noirq(struct device *dev)
* send failed, may lead to system freeze. This issue * send failed, may lead to system freeze. This issue
* is observed by testing freeze mode suspend. * is observed by testing freeze mode suspend.
*/ */
if (!imx_mu_read(priv, priv->dcfg->xCR)) if (!imx_mu_read(priv, priv->dcfg->xCR) && !priv->clk)
imx_mu_write(priv, priv->xcr, priv->dcfg->xCR); imx_mu_write(priv, priv->xcr, priv->dcfg->xCR);
return 0; return 0;
} }
static int imx_mu_runtime_suspend(struct device *dev)
{
struct imx_mu_priv *priv = dev_get_drvdata(dev);
clk_disable_unprepare(priv->clk);
return 0;
}
static int imx_mu_runtime_resume(struct device *dev)
{
struct imx_mu_priv *priv = dev_get_drvdata(dev);
int ret;
ret = clk_prepare_enable(priv->clk);
if (ret)
dev_err(dev, "failed to enable clock\n");
return ret;
}
static const struct dev_pm_ops imx_mu_pm_ops = { static const struct dev_pm_ops imx_mu_pm_ops = {
SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(imx_mu_suspend_noirq, SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(imx_mu_suspend_noirq,
imx_mu_resume_noirq) imx_mu_resume_noirq)
SET_RUNTIME_PM_OPS(imx_mu_runtime_suspend,
imx_mu_runtime_resume, NULL)
}; };
static struct platform_driver imx_mu_driver = { static struct platform_driver imx_mu_driver = {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment