| Message ID | 20240626230704.708234-2-festevam@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/6] drm/bridge: imx8mp-hdmi-tx: Switch to SYSTEM_SLEEP_PM_OPS() | expand |
On 27/06/2024 01:07, Fabio Estevam wrote: > From: Fabio Estevam <festevam@denx.de> > > Replace SET_RUNTIME_PM_OPS with its modern RUNTIME_PM_OPS() alternative. > > The combined usage of pm_ptr() and RUNTIME_PM_OPS() > allows the compiler to evaluate if the runtime suspend/resume() functions > are used at build time or are simply dead code. > > This allows removing the __maybe_unused notation from the runtime > suspend/resume() functions. > > Signed-off-by: Fabio Estevam <festevam@denx.de> > --- > drivers/gpu/drm/bridge/imx/imx8qm-ldb.c | 9 ++++----- > 1 file changed, 4 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/bridge/imx/imx8qm-ldb.c b/drivers/gpu/drm/bridge/imx/imx8qm-ldb.c > index 21471a9a28b2..c879e37f5811 100644 > --- a/drivers/gpu/drm/bridge/imx/imx8qm-ldb.c > +++ b/drivers/gpu/drm/bridge/imx/imx8qm-ldb.c > @@ -542,12 +542,12 @@ static void imx8qm_ldb_remove(struct platform_device *pdev) > pm_runtime_disable(&pdev->dev); > } > > -static int __maybe_unused imx8qm_ldb_runtime_suspend(struct device *dev) > +static int imx8qm_ldb_runtime_suspend(struct device *dev) > { > return 0; > } > > -static int __maybe_unused imx8qm_ldb_runtime_resume(struct device *dev) > +static int imx8qm_ldb_runtime_resume(struct device *dev) > { > struct imx8qm_ldb *imx8qm_ldb = dev_get_drvdata(dev); > struct ldb *ldb = &imx8qm_ldb->base; > @@ -559,8 +559,7 @@ static int __maybe_unused imx8qm_ldb_runtime_resume(struct device *dev) > } > > static const struct dev_pm_ops imx8qm_ldb_pm_ops = { > - SET_RUNTIME_PM_OPS(imx8qm_ldb_runtime_suspend, > - imx8qm_ldb_runtime_resume, NULL) > + RUNTIME_PM_OPS(imx8qm_ldb_runtime_suspend, imx8qm_ldb_runtime_resume, NULL) > }; > > static const struct of_device_id imx8qm_ldb_dt_ids[] = { > @@ -573,7 +572,7 @@ static struct platform_driver imx8qm_ldb_driver = { > .probe = imx8qm_ldb_probe, > .remove_new = imx8qm_ldb_remove, > .driver = { > - .pm = &imx8qm_ldb_pm_ops, > + .pm = pm_ptr(&imx8qm_ldb_pm_ops), > .name = DRIVER_NAME, > .of_match_table = imx8qm_ldb_dt_ids, > }, Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
diff --git a/drivers/gpu/drm/bridge/imx/imx8qm-ldb.c b/drivers/gpu/drm/bridge/imx/imx8qm-ldb.c index 21471a9a28b2..c879e37f5811 100644 --- a/drivers/gpu/drm/bridge/imx/imx8qm-ldb.c +++ b/drivers/gpu/drm/bridge/imx/imx8qm-ldb.c @@ -542,12 +542,12 @@ static void imx8qm_ldb_remove(struct platform_device *pdev) pm_runtime_disable(&pdev->dev); } -static int __maybe_unused imx8qm_ldb_runtime_suspend(struct device *dev) +static int imx8qm_ldb_runtime_suspend(struct device *dev) { return 0; } -static int __maybe_unused imx8qm_ldb_runtime_resume(struct device *dev) +static int imx8qm_ldb_runtime_resume(struct device *dev) { struct imx8qm_ldb *imx8qm_ldb = dev_get_drvdata(dev); struct ldb *ldb = &imx8qm_ldb->base; @@ -559,8 +559,7 @@ static int __maybe_unused imx8qm_ldb_runtime_resume(struct device *dev) } static const struct dev_pm_ops imx8qm_ldb_pm_ops = { - SET_RUNTIME_PM_OPS(imx8qm_ldb_runtime_suspend, - imx8qm_ldb_runtime_resume, NULL) + RUNTIME_PM_OPS(imx8qm_ldb_runtime_suspend, imx8qm_ldb_runtime_resume, NULL) }; static const struct of_device_id imx8qm_ldb_dt_ids[] = { @@ -573,7 +572,7 @@ static struct platform_driver imx8qm_ldb_driver = { .probe = imx8qm_ldb_probe, .remove_new = imx8qm_ldb_remove, .driver = { - .pm = &imx8qm_ldb_pm_ops, + .pm = pm_ptr(&imx8qm_ldb_pm_ops), .name = DRIVER_NAME, .of_match_table = imx8qm_ldb_dt_ids, },