Skip to content

Commit c9b6616

Browse files
hadessJiri Kosina
authored andcommitted
HID: picolcd_core: Use pm_ptr instead of #ifdef CONFIG_PM
This increases build coverage and allows to drop an #ifdef. Signed-off-by: Bastien Nocera <hadess@hadess.net> Signed-off-by: Jiri Kosina <jkosina@suse.com>
1 parent d7b7ce3 commit c9b6616

1 file changed

Lines changed: 3 additions & 7 deletions

File tree

drivers/hid/hid-picolcd_core.c

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -363,7 +363,6 @@ static int picolcd_raw_event(struct hid_device *hdev,
363363
return 1;
364364
}
365365

366-
#ifdef CONFIG_PM
367366
static int picolcd_suspend(struct hid_device *hdev, pm_message_t message)
368367
{
369368
if (PMSG_IS_AUTO(message))
@@ -401,7 +400,6 @@ static int picolcd_reset_resume(struct hid_device *hdev)
401400
picolcd_leds_set(hid_get_drvdata(hdev));
402401
return 0;
403402
}
404-
#endif
405403

406404
/* initialize keypad input device */
407405
static int picolcd_init_keys(struct picolcd_data *data,
@@ -648,11 +646,9 @@ static struct hid_driver picolcd_driver = {
648646
.probe = picolcd_probe,
649647
.remove = picolcd_remove,
650648
.raw_event = picolcd_raw_event,
651-
#ifdef CONFIG_PM
652-
.suspend = picolcd_suspend,
653-
.resume = picolcd_resume,
654-
.reset_resume = picolcd_reset_resume,
655-
#endif
649+
.suspend = pm_ptr(picolcd_suspend),
650+
.resume = pm_ptr(picolcd_resume),
651+
.reset_resume = pm_ptr(picolcd_reset_resume),
656652
};
657653
module_hid_driver(picolcd_driver);
658654

0 commit comments

Comments
 (0)