Input: bu21013_ts - switch to DEFINE_SIMPLE_DEV_PM_OPS() and pm_sleep_ptr()
SIMPLE_DEV_PM_OPS() is deprecated as it requires explicit protection against unused function warnings. The new combination of pm_sleep_ptr() and DEFINE_SIMPLE_DEV_PM_OPS() allows the compiler to see the functions, thus suppressing the warning, but still allowing the unused code to be removed. Thus also drop the __maybe_unused markings. Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> Cc: Linus Walleij <linus.walleij@linaro.org> Reviewed-by: Linus Walleij <linus.walleij@linaro.org> Link: https://lore.kernel.org/r/20230102181842.718010-36-jic23@kernel.org Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
This commit is contained in:
parent
48877f8e7a
commit
bed5cf8f00
@ -560,7 +560,7 @@ static void bu21013_remove(struct i2c_client *client)
|
|||||||
/* The resources will be freed by devm */
|
/* The resources will be freed by devm */
|
||||||
}
|
}
|
||||||
|
|
||||||
static int __maybe_unused bu21013_suspend(struct device *dev)
|
static int bu21013_suspend(struct device *dev)
|
||||||
{
|
{
|
||||||
struct i2c_client *client = to_i2c_client(dev);
|
struct i2c_client *client = to_i2c_client(dev);
|
||||||
struct bu21013_ts *ts = i2c_get_clientdata(client);
|
struct bu21013_ts *ts = i2c_get_clientdata(client);
|
||||||
@ -575,7 +575,7 @@ static int __maybe_unused bu21013_suspend(struct device *dev)
|
|||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int __maybe_unused bu21013_resume(struct device *dev)
|
static int bu21013_resume(struct device *dev)
|
||||||
{
|
{
|
||||||
struct i2c_client *client = to_i2c_client(dev);
|
struct i2c_client *client = to_i2c_client(dev);
|
||||||
struct bu21013_ts *ts = i2c_get_clientdata(client);
|
struct bu21013_ts *ts = i2c_get_clientdata(client);
|
||||||
@ -604,7 +604,7 @@ static int __maybe_unused bu21013_resume(struct device *dev)
|
|||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static SIMPLE_DEV_PM_OPS(bu21013_dev_pm_ops, bu21013_suspend, bu21013_resume);
|
static DEFINE_SIMPLE_DEV_PM_OPS(bu21013_dev_pm_ops, bu21013_suspend, bu21013_resume);
|
||||||
|
|
||||||
static const struct i2c_device_id bu21013_id[] = {
|
static const struct i2c_device_id bu21013_id[] = {
|
||||||
{ DRIVER_TP, 0 },
|
{ DRIVER_TP, 0 },
|
||||||
@ -615,7 +615,7 @@ MODULE_DEVICE_TABLE(i2c, bu21013_id);
|
|||||||
static struct i2c_driver bu21013_driver = {
|
static struct i2c_driver bu21013_driver = {
|
||||||
.driver = {
|
.driver = {
|
||||||
.name = DRIVER_TP,
|
.name = DRIVER_TP,
|
||||||
.pm = &bu21013_dev_pm_ops,
|
.pm = pm_sleep_ptr(&bu21013_dev_pm_ops),
|
||||||
},
|
},
|
||||||
.probe_new = bu21013_probe,
|
.probe_new = bu21013_probe,
|
||||||
.remove = bu21013_remove,
|
.remove = bu21013_remove,
|
||||||
|
Loading…
Reference in New Issue
Block a user