iio: dac: max517: 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: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Cc: Marcus Folkesson <marcus.folkesson@gmail.com>
Reviewed-by: Paul Cercueil <paul@crapouillou.net>
Link: https://lore.kernel.org/r/20220621202719.13644-14-jic23@kernel.org
This commit is contained in:
Jonathan Cameron 2022-06-21 21:26:56 +01:00
parent 83a2aa2646
commit 7b9d3e85d7

View File

@ -100,21 +100,21 @@ static int max517_write_raw(struct iio_dev *indio_dev,
return ret; return ret;
} }
static int __maybe_unused max517_suspend(struct device *dev) static int max517_suspend(struct device *dev)
{ {
u8 outbuf = COMMAND_PD; u8 outbuf = COMMAND_PD;
return i2c_master_send(to_i2c_client(dev), &outbuf, 1); return i2c_master_send(to_i2c_client(dev), &outbuf, 1);
} }
static int __maybe_unused max517_resume(struct device *dev) static int max517_resume(struct device *dev)
{ {
u8 outbuf = 0; u8 outbuf = 0;
return i2c_master_send(to_i2c_client(dev), &outbuf, 1); return i2c_master_send(to_i2c_client(dev), &outbuf, 1);
} }
static SIMPLE_DEV_PM_OPS(max517_pm_ops, max517_suspend, max517_resume); static DEFINE_SIMPLE_DEV_PM_OPS(max517_pm_ops, max517_suspend, max517_resume);
static const struct iio_info max517_info = { static const struct iio_info max517_info = {
.read_raw = max517_read_raw, .read_raw = max517_read_raw,
@ -201,7 +201,7 @@ MODULE_DEVICE_TABLE(i2c, max517_id);
static struct i2c_driver max517_driver = { static struct i2c_driver max517_driver = {
.driver = { .driver = {
.name = MAX517_DRV_NAME, .name = MAX517_DRV_NAME,
.pm = &max517_pm_ops, .pm = pm_sleep_ptr(&max517_pm_ops),
}, },
.probe = max517_probe, .probe = max517_probe,
.id_table = max517_id, .id_table = max517_id,