Commit 100f84be authored by Nicolas Toromanoff's avatar Nicolas Toromanoff Committed by Herbert Xu

crypto: stm32/crc32 - don't sleep in runtime pm

Ensure stm32_crc_update() and stm32_crc_init() can be called
in atomic context and can't sleep.

Add pm_runtime_irq_safe() to make pm_runtime_get_sync() atomic.
Change runtime pm to call clk_enable()/clk_disable() and change
system pm to unprepare/prepare the clock and force runtime pm
suspend/resume.
Signed-off-by: default avatarNicolas Toromanoff <nicolas.toromanoff@st.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent 10b89c43
...@@ -297,6 +297,7 @@ static int stm32_crc_probe(struct platform_device *pdev) ...@@ -297,6 +297,7 @@ static int stm32_crc_probe(struct platform_device *pdev)
pm_runtime_get_noresume(dev); pm_runtime_get_noresume(dev);
pm_runtime_set_active(dev); pm_runtime_set_active(dev);
pm_runtime_irq_safe(dev);
pm_runtime_enable(dev); pm_runtime_enable(dev);
platform_set_drvdata(pdev, crc); platform_set_drvdata(pdev, crc);
...@@ -350,34 +351,60 @@ static int stm32_crc_remove(struct platform_device *pdev) ...@@ -350,34 +351,60 @@ static int stm32_crc_remove(struct platform_device *pdev)
return 0; return 0;
} }
#ifdef CONFIG_PM static int __maybe_unused stm32_crc_suspend(struct device *dev)
static int stm32_crc_runtime_suspend(struct device *dev)
{ {
struct stm32_crc *crc = dev_get_drvdata(dev); struct stm32_crc *crc = dev_get_drvdata(dev);
int ret;
clk_disable_unprepare(crc->clk); ret = pm_runtime_force_suspend(dev);
if (ret)
return ret;
clk_unprepare(crc->clk);
return 0; return 0;
} }
static int stm32_crc_runtime_resume(struct device *dev) static int __maybe_unused stm32_crc_resume(struct device *dev)
{ {
struct stm32_crc *crc = dev_get_drvdata(dev); struct stm32_crc *crc = dev_get_drvdata(dev);
int ret; int ret;
ret = clk_prepare_enable(crc->clk); ret = clk_prepare(crc->clk);
if (ret) { if (ret) {
dev_err(crc->dev, "Failed to prepare_enable clock\n"); dev_err(crc->dev, "Failed to prepare clock\n");
return ret;
}
return pm_runtime_force_resume(dev);
}
static int __maybe_unused stm32_crc_runtime_suspend(struct device *dev)
{
struct stm32_crc *crc = dev_get_drvdata(dev);
clk_disable(crc->clk);
return 0;
}
static int __maybe_unused stm32_crc_runtime_resume(struct device *dev)
{
struct stm32_crc *crc = dev_get_drvdata(dev);
int ret;
ret = clk_enable(crc->clk);
if (ret) {
dev_err(crc->dev, "Failed to enable clock\n");
return ret; return ret;
} }
return 0; return 0;
} }
#endif
static const struct dev_pm_ops stm32_crc_pm_ops = { static const struct dev_pm_ops stm32_crc_pm_ops = {
SET_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend, SET_SYSTEM_SLEEP_PM_OPS(stm32_crc_suspend,
pm_runtime_force_resume) stm32_crc_resume)
SET_RUNTIME_PM_OPS(stm32_crc_runtime_suspend, SET_RUNTIME_PM_OPS(stm32_crc_runtime_suspend,
stm32_crc_runtime_resume, NULL) stm32_crc_runtime_resume, NULL)
}; };
......
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