* [PATCH v2] auxdisplay: arm-charlcd: Use SIMPLE_DEV_PM_OPS for power management
@ 2026-08-19 8:48 Li Jun
2026-08-19 13:25 ` Andy Shevchenko
0 siblings, 1 reply; 3+ messages in thread
From: Li Jun @ 2026-08-19 8:48 UTC (permalink / raw)
To: lijun01, andy, geert, fourier.thomas, bhelgaas, u.kleine-koenig,
linux-kernel
Replace the manually defined dev_pm_ops structure with the
SIMPLE_DEV_PM_OPS macro to simplify the power management code.
This macro automatically handles CONFIG_PM_SLEEP configuration
and generates freeze/thaw/poweroff/restore callbacks, reducing
boilerplate code.
Signed-off-by: Li Jun <lijun01@kylinos.cn>
---
drivers/auxdisplay/arm-charlcd.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/auxdisplay/arm-charlcd.c b/drivers/auxdisplay/arm-charlcd.c
index 70efda4f767e..8cb6e39a63b6 100644
--- a/drivers/auxdisplay/arm-charlcd.c
+++ b/drivers/auxdisplay/arm-charlcd.c
@@ -306,10 +306,7 @@ static int charlcd_resume(struct device *dev)
return 0;
}
-static const struct dev_pm_ops charlcd_pm_ops = {
- .suspend = charlcd_suspend,
- .resume = charlcd_resume,
-};
+static SIMPLE_DEV_PM_OPS(charlcd_pm_ops, charlcd_suspend, charlcd_resume);
static const struct of_device_id charlcd_match[] = {
{ .compatible = "arm,versatile-lcd", },
--
2.25.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] auxdisplay: arm-charlcd: Use SIMPLE_DEV_PM_OPS for power management
2026-08-19 8:48 [PATCH v2] auxdisplay: arm-charlcd: Use SIMPLE_DEV_PM_OPS for power management Li Jun
@ 2026-08-19 13:25 ` Andy Shevchenko
2026-08-20 1:39 ` lijun
0 siblings, 1 reply; 3+ messages in thread
From: Andy Shevchenko @ 2026-08-19 13:25 UTC (permalink / raw)
To: Li Jun
Cc: andy, geert, fourier.thomas, bhelgaas, u.kleine-koenig, linux-kernel
On Wed, Aug 19, 2026 at 04:48:38PM +0800, Li Jun wrote:
> Replace the manually defined dev_pm_ops structure with the
> SIMPLE_DEV_PM_OPS macro to simplify the power management code.
> This macro automatically handles CONFIG_PM_SLEEP configuration
> and generates freeze/thaw/poweroff/restore callbacks, reducing
> boilerplate code.
This is v2, no changelog, all v1 comments were ignored. What's going on?
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] auxdisplay: arm-charlcd: Use SIMPLE_DEV_PM_OPS for power management
2026-08-19 13:25 ` Andy Shevchenko
@ 2026-08-20 1:39 ` lijun
0 siblings, 0 replies; 3+ messages in thread
From: lijun @ 2026-08-20 1:39 UTC (permalink / raw)
To: Andy Shevchenko
Cc: andy, geert, fourier.thomas, bhelgaas, u.kleine-koenig, linux-kernel
I am so sorry for the previous email，and the subsequent mailing has
corrected these errors.
2026/8/19 21:25, Andy Shevchenko :
> On Wed, Aug 19, 2026 at 04:48:38PM +0800, Li Jun wrote:
>> Replace the manually defined dev_pm_ops structure with the
>> SIMPLE_DEV_PM_OPS macro to simplify the power management code.
>> This macro automatically handles CONFIG_PM_SLEEP configuration
>> and generates freeze/thaw/poweroff/restore callbacks, reducing
>> boilerplate code.
> This is v2, no changelog, all v1 comments were ignored. What's going on?
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-08-20 1:39 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-08-19 8:48 [PATCH v2] auxdisplay: arm-charlcd: Use SIMPLE_DEV_PM_OPS for power management Li Jun
2026-08-19 13:25 ` Andy Shevchenko
2026-08-20 1:39 ` lijun
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®