Commit c9ccad99 authored by Jonathan Cameron's avatar Jonathan Cameron
Browse files

iio: dac: max5821: Switch to DEFINE_SIMPLE_DEV_PM_OPS() and pm_sleep_ptr() 



Using these newer macros allows the compiler to remove the unused
structure and functions when !CONFIG_PM_SLEEP + removes the need to
mark pm functions __maybe_unused.

Signed-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
Cc: Philippe Reynes <tremyfr@yahoo.fr>
Reviewed-by: default avatarPaul Cercueil <paul@crapouillou.net>
Link: https://lore.kernel.org/r/20220621202719.13644-15-jic23@kernel.org
parent 7b9d3e85

drivers/iio/dac/max5821.c

+5 −4
Original line number Diff line number Diff line
@@ -267,7 +267,7 @@ static int max5821_write_raw(struct iio_dev *indio_dev, 
	}

}



static int __maybe_unused max5821_suspend(struct device *dev)

static int max5821_suspend(struct device *dev)

{

	u8 outbuf[2] = { MAX5821_EXTENDED_COMMAND_MODE,

			 MAX5821_EXTENDED_DAC_A |
@@ -277,7 +277,7 @@ static int __maybe_unused max5821_suspend(struct device *dev) 
	return i2c_master_send(to_i2c_client(dev), outbuf, 2);

}



static int __maybe_unused max5821_resume(struct device *dev)

static int max5821_resume(struct device *dev)

{

	u8 outbuf[2] = { MAX5821_EXTENDED_COMMAND_MODE,

			 MAX5821_EXTENDED_DAC_A |
@@ -287,7 +287,8 @@ static int __maybe_unused max5821_resume(struct device *dev) 
	return i2c_master_send(to_i2c_client(dev), outbuf, 2);

}



static SIMPLE_DEV_PM_OPS(max5821_pm_ops, max5821_suspend, max5821_resume);

static DEFINE_SIMPLE_DEV_PM_OPS(max5821_pm_ops, max5821_suspend,

				max5821_resume);



static const struct iio_info max5821_info = {

	.read_raw = max5821_read_raw,
@@ -374,7 +375,7 @@ static struct i2c_driver max5821_driver = { 
	.driver = {

		.name	= "max5821",

		.of_match_table = max5821_of_match,

		.pm     = &max5821_pm_ops,

		.pm     = pm_sleep_ptr(&max5821_pm_ops),

	},

	.probe		= max5821_probe,

	.id_table	= max5821_id,