Commit 31b34d62 authored by Jonathan Cameron's avatar Jonathan Cameron Committed by Guenter Roeck
Browse files

hwmon: (it87) Switch to DEFINE_SIMPLE_DEV_PM_OPS() and pm_sleep_ptr() 



These newer PM macros allow the compiler to see what code it can remove
if !CONFIG_PM_SLEEP. This allows the removal of __maybe_unused markings
whilst achieving the same result.

Signed-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
Link: https://lore.kernel.org/r/20220925172759.3573439-7-jic23@kernel.org


Signed-off-by: default avatarGuenter Roeck <linux@roeck-us.net>
parent 5e866400

drivers/hwmon/it87.c

+4 −4
Original line number Diff line number Diff line
@@ -3179,7 +3179,7 @@ static int it87_probe(struct platform_device *pdev) 
	return PTR_ERR_OR_ZERO(hwmon_dev);

}



static void __maybe_unused it87_resume_sio(struct platform_device *pdev)

static void it87_resume_sio(struct platform_device *pdev)

{

	struct it87_data *data = dev_get_drvdata(&pdev->dev);

	int err;
@@ -3211,7 +3211,7 @@ static void __maybe_unused it87_resume_sio(struct platform_device *pdev) 
	superio_exit(data->sioaddr);

}



static int __maybe_unused it87_resume(struct device *dev)

static int it87_resume(struct device *dev)

{

	struct platform_device *pdev = to_platform_device(dev);

	struct it87_data *data = dev_get_drvdata(dev);
@@ -3238,12 +3238,12 @@ static int __maybe_unused it87_resume(struct device *dev) 
	return 0;

}



static SIMPLE_DEV_PM_OPS(it87_dev_pm_ops, NULL, it87_resume);

static DEFINE_SIMPLE_DEV_PM_OPS(it87_dev_pm_ops, NULL, it87_resume);



static struct platform_driver it87_driver = {

	.driver = {

		.name	= DRVNAME,

		.pm     = &it87_dev_pm_ops,

		.pm     = pm_sleep_ptr(&it87_dev_pm_ops),

	},

	.probe	= it87_probe,

};