Commit 2ca1d2a0 authored by Geert Uytterhoeven's avatar Geert Uytterhoeven Committed by Hans Verkuil
Browse files

media: renesas: fdp1: Convert to RUNTIME_PM_OPS() 



Convert the Renesas Fine Display Processor driver from
SET_RUNTIME_PM_OPS() to RUNTIME_PM_OPS() and pm_ptr().  This lets us
drop the __maybe_unused annotations from its runtime suspend and resume
callbacks, and reduces kernel size in case CONFIG_PM is disabled.

Signed-off-by: default avatarGeert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: default avatarFabrizio Castro <fabrizio.castro.jz@renesas.com>
Reviewed-by: default avatarLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Signed-off-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: default avatarHans Verkuil <hverkuil+cisco@kernel.org>
parent cde682d1

drivers/media/platform/renesas/rcar_fdp1.c

+4 −6
Original line number Diff line number Diff line
@@ -2408,7 +2408,7 @@ static void fdp1_remove(struct platform_device *pdev) 
	rcar_fcp_put(fdp1->fcp);

}



static int __maybe_unused fdp1_pm_runtime_suspend(struct device *dev)

static int fdp1_pm_runtime_suspend(struct device *dev)

{

	struct fdp1_dev *fdp1 = dev_get_drvdata(dev);
@@ -2417,7 +2417,7 @@ static int __maybe_unused fdp1_pm_runtime_suspend(struct device *dev) 
	return 0;

}



static int __maybe_unused fdp1_pm_runtime_resume(struct device *dev)

static int fdp1_pm_runtime_resume(struct device *dev)

{

	struct fdp1_dev *fdp1 = dev_get_drvdata(dev);
@@ -2428,9 +2428,7 @@ static int __maybe_unused fdp1_pm_runtime_resume(struct device *dev) 
}



static const struct dev_pm_ops fdp1_pm_ops = {

	SET_RUNTIME_PM_OPS(fdp1_pm_runtime_suspend,

			   fdp1_pm_runtime_resume,

			   NULL)

	RUNTIME_PM_OPS(fdp1_pm_runtime_suspend, fdp1_pm_runtime_resume, NULL)

};



static const struct of_device_id fdp1_dt_ids[] = {
@@ -2445,7 +2443,7 @@ static struct platform_driver fdp1_pdrv = { 
	.driver		= {

		.name	= DRIVER_NAME,

		.of_match_table = fdp1_dt_ids,

		.pm	= &fdp1_pm_ops,

		.pm	= pm_ptr(&fdp1_pm_ops),

	},

};