Commit cd0bb67f authored by Martin Kaiser's avatar Martin Kaiser Committed by Herbert Xu

hwrng: imx-rngc - simplify the power management definitions

Use the SIMPLE_DEV_PM_OPS() macro instead of populating a struct
dev_pm_ops directly. The suspend and resume functions will now be used
for both hibernation and suspend to ram.

If power management is disabled, SIMPLE_DEV_PM_OPS() evaluates to
nothing, The two functions won't be used and won't be included in the
kernel. Mark them as __maybe_unused to clarify that this is intended
behaviour.

With these modifications in place, we don't need the #ifdefs for power
management any more.
Signed-off-by: default avatarMartin Kaiser <martin@kaiser.cx>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent e1a018e6
...@@ -282,8 +282,7 @@ static int __exit imx_rngc_remove(struct platform_device *pdev) ...@@ -282,8 +282,7 @@ static int __exit imx_rngc_remove(struct platform_device *pdev)
return 0; return 0;
} }
#ifdef CONFIG_PM static int __maybe_unused imx_rngc_suspend(struct device *dev)
static int imx_rngc_suspend(struct device *dev)
{ {
struct imx_rngc *rngc = dev_get_drvdata(dev); struct imx_rngc *rngc = dev_get_drvdata(dev);
...@@ -292,7 +291,7 @@ static int imx_rngc_suspend(struct device *dev) ...@@ -292,7 +291,7 @@ static int imx_rngc_suspend(struct device *dev)
return 0; return 0;
} }
static int imx_rngc_resume(struct device *dev) static int __maybe_unused imx_rngc_resume(struct device *dev)
{ {
struct imx_rngc *rngc = dev_get_drvdata(dev); struct imx_rngc *rngc = dev_get_drvdata(dev);
...@@ -301,11 +300,7 @@ static int imx_rngc_resume(struct device *dev) ...@@ -301,11 +300,7 @@ static int imx_rngc_resume(struct device *dev)
return 0; return 0;
} }
static const struct dev_pm_ops imx_rngc_pm_ops = { SIMPLE_DEV_PM_OPS(imx_rngc_pm_ops, imx_rngc_suspend, imx_rngc_resume);
.suspend = imx_rngc_suspend,
.resume = imx_rngc_resume,
};
#endif
static const struct of_device_id imx_rngc_dt_ids[] = { static const struct of_device_id imx_rngc_dt_ids[] = {
{ .compatible = "fsl,imx25-rngb", .data = NULL, }, { .compatible = "fsl,imx25-rngb", .data = NULL, },
...@@ -316,9 +311,7 @@ MODULE_DEVICE_TABLE(of, imx_rngc_dt_ids); ...@@ -316,9 +311,7 @@ MODULE_DEVICE_TABLE(of, imx_rngc_dt_ids);
static struct platform_driver imx_rngc_driver = { static struct platform_driver imx_rngc_driver = {
.driver = { .driver = {
.name = "imx_rngc", .name = "imx_rngc",
#ifdef CONFIG_PM
.pm = &imx_rngc_pm_ops, .pm = &imx_rngc_pm_ops,
#endif
.of_match_table = imx_rngc_dt_ids, .of_match_table = imx_rngc_dt_ids,
}, },
.remove = __exit_p(imx_rngc_remove), .remove = __exit_p(imx_rngc_remove),
......
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