Input: goodix - 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: Hans de Goede <hdegoede@redhat.com> Cc: Bastien Nocera <hadess@hadess.net> Reviewed-by: Hans de Goede <hdegoede@redhat.com> Reviewed-by: Linus Walleij <linus.walleij@linaro.org> Link: https://lore.kernel.org/r/20230102181842.718010-47-jic23@kernel.org Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
This commit is contained in:
parent
df0f673485
commit
05ebb78bf8
|
@ -1390,7 +1390,7 @@ static void goodix_ts_remove(struct i2c_client *client)
|
|||
wait_for_completion(&ts->firmware_loading_complete);
|
||||
}
|
||||
|
||||
static int __maybe_unused goodix_suspend(struct device *dev)
|
||||
static int goodix_suspend(struct device *dev)
|
||||
{
|
||||
struct i2c_client *client = to_i2c_client(dev);
|
||||
struct goodix_ts_data *ts = i2c_get_clientdata(client);
|
||||
|
@ -1437,7 +1437,7 @@ static int __maybe_unused goodix_suspend(struct device *dev)
|
|||
return 0;
|
||||
}
|
||||
|
||||
static int __maybe_unused goodix_resume(struct device *dev)
|
||||
static int goodix_resume(struct device *dev)
|
||||
{
|
||||
struct i2c_client *client = to_i2c_client(dev);
|
||||
struct goodix_ts_data *ts = i2c_get_clientdata(client);
|
||||
|
@ -1486,7 +1486,7 @@ static int __maybe_unused goodix_resume(struct device *dev)
|
|||
return 0;
|
||||
}
|
||||
|
||||
static SIMPLE_DEV_PM_OPS(goodix_pm_ops, goodix_suspend, goodix_resume);
|
||||
static DEFINE_SIMPLE_DEV_PM_OPS(goodix_pm_ops, goodix_suspend, goodix_resume);
|
||||
|
||||
static const struct i2c_device_id goodix_ts_id[] = {
|
||||
{ "GDIX1001:00", 0 },
|
||||
|
@ -1532,7 +1532,7 @@ static struct i2c_driver goodix_ts_driver = {
|
|||
.name = "Goodix-TS",
|
||||
.acpi_match_table = ACPI_PTR(goodix_acpi_match),
|
||||
.of_match_table = of_match_ptr(goodix_of_match),
|
||||
.pm = &goodix_pm_ops,
|
||||
.pm = pm_sleep_ptr(&goodix_pm_ops),
|
||||
},
|
||||
};
|
||||
module_i2c_driver(goodix_ts_driver);
|
||||
|
|
Loading…
Reference in New Issue