Commit 0fda2c65 authored by Jonathan Cameron's avatar Jonathan Cameron
Browse files

iio: adc: ti-am335x: 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: Miquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: default avatarMiquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: default avatarPaul Cercueil <paul@crapouillou.net>
Link: https://lore.kernel.org/r/20220621202719.13644-9-jic23@kernel.org
parent 0b1e58e9

drivers/iio/adc/ti_am335x_adc.c

+4 −4
Original line number Diff line number Diff line
@@ -702,7 +702,7 @@ static int tiadc_remove(struct platform_device *pdev) 
	return 0;

}



static int __maybe_unused tiadc_suspend(struct device *dev)

static int tiadc_suspend(struct device *dev)

{

	struct iio_dev *indio_dev = dev_get_drvdata(dev);

	struct tiadc_device *adc_dev = iio_priv(indio_dev);
@@ -715,7 +715,7 @@ static int __maybe_unused tiadc_suspend(struct device *dev) 
	return 0;

}



static int __maybe_unused tiadc_resume(struct device *dev)

static int tiadc_resume(struct device *dev)

{

	struct iio_dev *indio_dev = dev_get_drvdata(dev);

	struct tiadc_device *adc_dev = iio_priv(indio_dev);
@@ -732,7 +732,7 @@ static int __maybe_unused tiadc_resume(struct device *dev) 
	return 0;

}



static SIMPLE_DEV_PM_OPS(tiadc_pm_ops, tiadc_suspend, tiadc_resume);

static DEFINE_SIMPLE_DEV_PM_OPS(tiadc_pm_ops, tiadc_suspend, tiadc_resume);



static const struct of_device_id ti_adc_dt_ids[] = {

	{ .compatible = "ti,am3359-adc", },
@@ -744,7 +744,7 @@ MODULE_DEVICE_TABLE(of, ti_adc_dt_ids); 
static struct platform_driver tiadc_driver = {

	.driver = {

		.name   = "TI-am335x-adc",

		.pm	= &tiadc_pm_ops,

		.pm	= pm_sleep_ptr(&tiadc_pm_ops),

		.of_match_table = ti_adc_dt_ids,

	},

	.probe	= tiadc_probe,