media: hantro: use DEFINE_RUNTIME_DEV_PM_OPS

The runtime suspend and resume callbacks are currently guarded by
CONFIG_PM. Removing the guard directly would leave the callbacks unused
when CONFIG_PM is disabled.

Define the PM operations with DEFINE_RUNTIME_DEV_PM_OPS() and use
pm_ptr() when assigning them to the platform driver. This keeps the PM
callbacks available when needed while avoiding unused-function warnings
for CONFIG_PM=n builds.

Signed-off-by: Tharit Tangkijwanichakul <tharitt97@gmail.com>
Reviewed-by: Frank Li <Frank.Li@nxp.com>
Signed-off-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>
Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org>
This commit is contained in:
Tharit Tangkijwanichakul 2026-07-29 06:04:40 +00:00 committed by Hans Verkuil
parent 556ab0b274
commit 0b02668884

View File

@ -1290,7 +1290,6 @@ static void hantro_remove(struct platform_device *pdev)
pm_runtime_disable(vpu->dev);
}
#ifdef CONFIG_PM
static int hantro_runtime_resume(struct device *dev)
{
struct hantro_dev *vpu = dev_get_drvdata(dev);
@ -1320,13 +1319,9 @@ static int hantro_runtime_suspend(struct device *dev)
clk_bulk_disable(vpu->variant->num_clocks, vpu->clocks);
return 0;
}
#endif
static const struct dev_pm_ops hantro_pm_ops = {
SET_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend,
pm_runtime_force_resume)
SET_RUNTIME_PM_OPS(hantro_runtime_suspend, hantro_runtime_resume, NULL)
};
static DEFINE_RUNTIME_DEV_PM_OPS(hantro_pm_ops, hantro_runtime_suspend,
hantro_runtime_resume, NULL);
static struct platform_driver hantro_driver = {
.probe = hantro_probe,
@ -1334,7 +1329,7 @@ static struct platform_driver hantro_driver = {
.driver = {
.name = DRIVER_NAME,
.of_match_table = of_hantro_match,
.pm = &hantro_pm_ops,
.pm = pm_ptr(&hantro_pm_ops),
},
};
module_platform_driver(hantro_driver);