* [PATCH] fbdev: bfin-lq035q1-fb: Use dev_pm_ops @ 2013-05-29 12:17 Lars-Peter Clausen 2013-05-29 14:24 ` Michael Hennerich 0 siblings, 1 reply; 13+ messages in thread From: Lars-Peter Clausen @ 2013-05-29 12:17 UTC (permalink / raw) To: linux-fbdev Use dev_pm_ops instead of the legacy suspend/resume callbacks. Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> --- drivers/video/bfin-lq035q1-fb.c | 22 ++++++++++++++-------- 1 file changed, 14 insertions(+), 8 deletions(-) diff --git a/drivers/video/bfin-lq035q1-fb.c b/drivers/video/bfin-lq035q1-fb.c index 29d8c04..6084c17 100644 --- a/drivers/video/bfin-lq035q1-fb.c +++ b/drivers/video/bfin-lq035q1-fb.c @@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); } -#ifdef CONFIG_PM -static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) +#ifdef CONFIG_PM_SLEEP +static int lq035q1_spidev_suspend(struct device *dev) { + struct spi_device *spi = to_spi_device(dev); + return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); } -static int lq035q1_spidev_resume(struct spi_device *spi) +static int lq035q1_spidev_resume(struct device *dev) { - int ret; + struct spi_device *spi = to_spi_device(dev); struct spi_control *ctl = spi_get_drvdata(spi); + int ret; ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); if (ret) @@ -187,9 +190,13 @@ static int lq035q1_spidev_resume(struct spi_device *spi) return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); } + +static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, + lq035q1_spidev_resume); +#define LQ035Q1_SPIDEV_PM_OPS (&lq035q1_spidev_pm_ops) + #else -# define lq035q1_spidev_suspend NULL -# define lq035q1_spidev_resume NULL +#define LQ035Q1_SPIDEV_PM_OPS NULL #endif /* Power down all displays on reboot, poweroff or halt */ @@ -708,8 +715,7 @@ static int bfin_lq035q1_probe(struct platform_device *pdev) info->spidrv.probe = lq035q1_spidev_probe; info->spidrv.remove = lq035q1_spidev_remove; info->spidrv.shutdown = lq035q1_spidev_shutdown; - info->spidrv.suspend = lq035q1_spidev_suspend; - info->spidrv.resume = lq035q1_spidev_resume; + info->spidrv.driver.pm = LQ035Q1_SPIDEV_PM_OPS; ret = spi_register_driver(&info->spidrv); if (ret < 0) { -- 1.8.0 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH] fbdev: bfin-lq035q1-fb: Use dev_pm_ops 2013-05-29 12:17 [PATCH] fbdev: bfin-lq035q1-fb: Use dev_pm_ops Lars-Peter Clausen @ 2013-05-29 14:24 ` Michael Hennerich 2013-05-29 19:20 ` Jean-Christophe PLAGNIOL-VILLARD 0 siblings, 1 reply; 13+ messages in thread From: Michael Hennerich @ 2013-05-29 14:24 UTC (permalink / raw) To: linux-fbdev On 05/29/2013 02:17 PM, Lars-Peter Clausen wrote: > Use dev_pm_ops instead of the legacy suspend/resume callbacks. > > Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> Acked-by: Michael Hennerich <michael.hennerich@analog.com> > --- > drivers/video/bfin-lq035q1-fb.c | 22 ++++++++++++++-------- > 1 file changed, 14 insertions(+), 8 deletions(-) > > diff --git a/drivers/video/bfin-lq035q1-fb.c b/drivers/video/bfin-lq035q1-fb.c > index 29d8c04..6084c17 100644 > --- a/drivers/video/bfin-lq035q1-fb.c > +++ b/drivers/video/bfin-lq035q1-fb.c > @@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > } > > -#ifdef CONFIG_PM > -static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) > +#ifdef CONFIG_PM_SLEEP > +static int lq035q1_spidev_suspend(struct device *dev) > { > + struct spi_device *spi = to_spi_device(dev); > + > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > } > > -static int lq035q1_spidev_resume(struct spi_device *spi) > +static int lq035q1_spidev_resume(struct device *dev) > { > - int ret; > + struct spi_device *spi = to_spi_device(dev); > struct spi_control *ctl = spi_get_drvdata(spi); > + int ret; > > ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); > if (ret) > @@ -187,9 +190,13 @@ static int lq035q1_spidev_resume(struct spi_device *spi) > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); > } > + > +static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, > + lq035q1_spidev_resume); > +#define LQ035Q1_SPIDEV_PM_OPS (&lq035q1_spidev_pm_ops) > + > #else > -# define lq035q1_spidev_suspend NULL > -# define lq035q1_spidev_resume NULL > +#define LQ035Q1_SPIDEV_PM_OPS NULL > #endif > > /* Power down all displays on reboot, poweroff or halt */ > @@ -708,8 +715,7 @@ static int bfin_lq035q1_probe(struct platform_device *pdev) > info->spidrv.probe = lq035q1_spidev_probe; > info->spidrv.remove = lq035q1_spidev_remove; > info->spidrv.shutdown = lq035q1_spidev_shutdown; > - info->spidrv.suspend = lq035q1_spidev_suspend; > - info->spidrv.resume = lq035q1_spidev_resume; > + info->spidrv.driver.pm = LQ035Q1_SPIDEV_PM_OPS; > > ret = spi_register_driver(&info->spidrv); > if (ret < 0) { -- Greetings, Michael -- Analog Devices GmbH Wilhelm-Wagenfeld-Str. 6 80807 Muenchen Sitz der Gesellschaft: Muenchen; Registergericht: Muenchen HRB 40368; Geschaeftsfuehrer:Dr.Carsten Suckrow, Thomas Wessel, William A. Martin, Margaret Seif ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] fbdev: bfin-lq035q1-fb: Use dev_pm_ops 2013-05-29 14:24 ` Michael Hennerich @ 2013-05-29 19:20 ` Jean-Christophe PLAGNIOL-VILLARD 2013-05-29 21:36 ` Rafael J. Wysocki 2013-05-30 0:32 ` Jingoo Han 0 siblings, 2 replies; 13+ messages in thread From: Jean-Christophe PLAGNIOL-VILLARD @ 2013-05-29 19:20 UTC (permalink / raw) To: Michael Hennerich, Rafael J. Wysocki Cc: Lars-Peter Clausen, Tomi Valkeinen, linux-fbdev, linux-pm On 16:24 Wed 29 May , Michael Hennerich wrote: > On 05/29/2013 02:17 PM, Lars-Peter Clausen wrote: > >Use dev_pm_ops instead of the legacy suspend/resume callbacks. > > > >Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> > Acked-by: Michael Hennerich <michael.hennerich@analog.com> > >--- > > drivers/video/bfin-lq035q1-fb.c | 22 ++++++++++++++-------- > > 1 file changed, 14 insertions(+), 8 deletions(-) > > > >diff --git a/drivers/video/bfin-lq035q1-fb.c b/drivers/video/bfin-lq035q1-fb.c > >index 29d8c04..6084c17 100644 > >--- a/drivers/video/bfin-lq035q1-fb.c > >+++ b/drivers/video/bfin-lq035q1-fb.c > >@@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > } > >-#ifdef CONFIG_PM > >-static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) > >+#ifdef CONFIG_PM_SLEEP > >+static int lq035q1_spidev_suspend(struct device *dev) > > { > >+ struct spi_device *spi = to_spi_device(dev); > >+ > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > } > >-static int lq035q1_spidev_resume(struct spi_device *spi) > >+static int lq035q1_spidev_resume(struct device *dev) > > { > >- int ret; > >+ struct spi_device *spi = to_spi_device(dev); > > struct spi_control *ctl = spi_get_drvdata(spi); > >+ int ret; > > ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); > > if (ret) > >@@ -187,9 +190,13 @@ static int lq035q1_spidev_resume(struct spi_device *spi) > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); > > } > >+ > >+static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, > >+ lq035q1_spidev_resume); > >+#define LQ035Q1_SPIDEV_PM_OPS (&lq035q1_spidev_pm_ops) > >+ > > #else > >-# define lq035q1_spidev_suspend NULL > >-# define lq035q1_spidev_resume NULL > >+#define LQ035Q1_SPIDEV_PM_OPS NULL > > #endif we really need to ahve a macro like for DT of_match_ptr to drop the #else I known it's related to this patch but someone should do it Rafael what do you think Best Regards, J. > > /* Power down all displays on reboot, poweroff or halt */ > >@@ -708,8 +715,7 @@ static int bfin_lq035q1_probe(struct platform_device *pdev) > > info->spidrv.probe = lq035q1_spidev_probe; > > info->spidrv.remove = lq035q1_spidev_remove; > > info->spidrv.shutdown = lq035q1_spidev_shutdown; > >- info->spidrv.suspend = lq035q1_spidev_suspend; > >- info->spidrv.resume = lq035q1_spidev_resume; > >+ info->spidrv.driver.pm = LQ035Q1_SPIDEV_PM_OPS; > > ret = spi_register_driver(&info->spidrv); > > if (ret < 0) { > > > -- > Greetings, > Michael > > -- > Analog Devices GmbH Wilhelm-Wagenfeld-Str. 6 80807 Muenchen > Sitz der Gesellschaft: Muenchen; Registergericht: Muenchen HRB 40368; > Geschaeftsfuehrer:Dr.Carsten Suckrow, Thomas Wessel, William A. Martin, > Margaret Seif > > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] fbdev: bfin-lq035q1-fb: Use dev_pm_ops 2013-05-29 19:20 ` Jean-Christophe PLAGNIOL-VILLARD @ 2013-05-29 21:36 ` Rafael J. Wysocki 2013-05-30 0:32 ` Jingoo Han 1 sibling, 0 replies; 13+ messages in thread From: Rafael J. Wysocki @ 2013-05-29 21:36 UTC (permalink / raw) To: Jean-Christophe PLAGNIOL-VILLARD Cc: Michael Hennerich, Lars-Peter Clausen, Tomi Valkeinen, linux-fbdev, linux-pm On Wednesday, May 29, 2013 09:20:13 PM Jean-Christophe PLAGNIOL-VILLARD wrote: > On 16:24 Wed 29 May , Michael Hennerich wrote: > > On 05/29/2013 02:17 PM, Lars-Peter Clausen wrote: > > >Use dev_pm_ops instead of the legacy suspend/resume callbacks. > > > > > >Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> > > Acked-by: Michael Hennerich <michael.hennerich@analog.com> > > >--- > > > drivers/video/bfin-lq035q1-fb.c | 22 ++++++++++++++-------- > > > 1 file changed, 14 insertions(+), 8 deletions(-) > > > > > >diff --git a/drivers/video/bfin-lq035q1-fb.c b/drivers/video/bfin-lq035q1-fb.c > > >index 29d8c04..6084c17 100644 > > >--- a/drivers/video/bfin-lq035q1-fb.c > > >+++ b/drivers/video/bfin-lq035q1-fb.c > > >@@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) > > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > > } > > >-#ifdef CONFIG_PM > > >-static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) > > >+#ifdef CONFIG_PM_SLEEP > > >+static int lq035q1_spidev_suspend(struct device *dev) > > > { > > >+ struct spi_device *spi = to_spi_device(dev); > > >+ > > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > > } > > >-static int lq035q1_spidev_resume(struct spi_device *spi) > > >+static int lq035q1_spidev_resume(struct device *dev) > > > { > > >- int ret; > > >+ struct spi_device *spi = to_spi_device(dev); > > > struct spi_control *ctl = spi_get_drvdata(spi); > > >+ int ret; > > > ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); > > > if (ret) > > >@@ -187,9 +190,13 @@ static int lq035q1_spidev_resume(struct spi_device *spi) > > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); > > > } > > >+ > > >+static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, > > >+ lq035q1_spidev_resume); > > >+#define LQ035Q1_SPIDEV_PM_OPS (&lq035q1_spidev_pm_ops) > > >+ > > > #else > > >-# define lq035q1_spidev_suspend NULL > > >-# define lq035q1_spidev_resume NULL > > >+#define LQ035Q1_SPIDEV_PM_OPS NULL > > > #endif > we really need to ahve a macro like for DT of_match_ptr to drop the #else > > I known it's related to this patch but someone should do it > > Rafael what do you think There's a patch adding something like that in the linux-next branch of my tree (if I understand correctly what you mean). Thanks, Rafael > > > /* Power down all displays on reboot, poweroff or halt */ > > >@@ -708,8 +715,7 @@ static int bfin_lq035q1_probe(struct platform_device *pdev) > > > info->spidrv.probe = lq035q1_spidev_probe; > > > info->spidrv.remove = lq035q1_spidev_remove; > > > info->spidrv.shutdown = lq035q1_spidev_shutdown; > > >- info->spidrv.suspend = lq035q1_spidev_suspend; > > >- info->spidrv.resume = lq035q1_spidev_resume; > > >+ info->spidrv.driver.pm = LQ035Q1_SPIDEV_PM_OPS; > > > ret = spi_register_driver(&info->spidrv); > > > if (ret < 0) { > > > > > -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] fbdev: bfin-lq035q1-fb: Use dev_pm_ops 2013-05-29 19:20 ` Jean-Christophe PLAGNIOL-VILLARD 2013-05-29 21:36 ` Rafael J. Wysocki @ 2013-05-30 0:32 ` Jingoo Han 2013-05-30 7:14 ` Jean-Christophe PLAGNIOL-VILLARD 1 sibling, 1 reply; 13+ messages in thread From: Jingoo Han @ 2013-05-30 0:32 UTC (permalink / raw) To: 'Jean-Christophe PLAGNIOL-VILLARD', 'Michael Hennerich', 'Rafael J. Wysocki' Cc: 'Lars-Peter Clausen', 'Tomi Valkeinen', linux-fbdev, linux-pm, Jingoo Han On Thursday, May 30, 2013 4:20 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > On 16:24 Wed 29 May , Michael Hennerich wrote: > > On 05/29/2013 02:17 PM, Lars-Peter Clausen wrote: > > >Use dev_pm_ops instead of the legacy suspend/resume callbacks. > > > > > >Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> > > Acked-by: Michael Hennerich <michael.hennerich@analog.com> > > >--- > > > drivers/video/bfin-lq035q1-fb.c | 22 ++++++++++++++-------- > > > 1 file changed, 14 insertions(+), 8 deletions(-) > > > > > >diff --git a/drivers/video/bfin-lq035q1-fb.c b/drivers/video/bfin-lq035q1-fb.c > > >index 29d8c04..6084c17 100644 > > >--- a/drivers/video/bfin-lq035q1-fb.c > > >+++ b/drivers/video/bfin-lq035q1-fb.c > > >@@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) > > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > > } > > >-#ifdef CONFIG_PM > > >-static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) > > >+#ifdef CONFIG_PM_SLEEP > > >+static int lq035q1_spidev_suspend(struct device *dev) > > > { > > >+ struct spi_device *spi = to_spi_device(dev); > > >+ > > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > > } > > >-static int lq035q1_spidev_resume(struct spi_device *spi) > > >+static int lq035q1_spidev_resume(struct device *dev) > > > { > > >- int ret; > > >+ struct spi_device *spi = to_spi_device(dev); > > > struct spi_control *ctl = spi_get_drvdata(spi); > > >+ int ret; > > > ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); > > > if (ret) > > >@@ -187,9 +190,13 @@ static int lq035q1_spidev_resume(struct spi_device *spi) > > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); > > > } > > >+ > > >+static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, > > >+ lq035q1_spidev_resume); > > >+#define LQ035Q1_SPIDEV_PM_OPS (&lq035q1_spidev_pm_ops) > > >+ > > > #else > > >-# define lq035q1_spidev_suspend NULL > > >-# define lq035q1_spidev_resume NULL > > >+#define LQ035Q1_SPIDEV_PM_OPS NULL > > > #endif > we really need to ahve a macro like for DT of_match_ptr to drop the #else Hi Jean-Christophe PLAGNIOL-VILLARD, I submitted the following patch. :) (https://patchwork.kernel.org/patch/2502971/) --- a/include/linux/pm.h +++ b/include/linux/pm.h @@ -55,8 +55,10 @@ struct device; #ifdef CONFIG_PM extern const char power_group_name[]; /* = "power" */ +#define pm_ops_ptr(_ptr) (_ptr) #else #define power_group_name NULL +#define pm_ops_ptr(_ptr) NULL #endif This patch was accepted by Rafael Wysocki, and will be merged to v3.11-rc1. Best regards, Jingoo Han > > I known it's related to this patch but someone should do it > > Rafael what do you think > > Best Regards, > J. > > > /* Power down all displays on reboot, poweroff or halt */ > > >@@ -708,8 +715,7 @@ static int bfin_lq035q1_probe(struct platform_device *pdev) > > > info->spidrv.probe = lq035q1_spidev_probe; > > > info->spidrv.remove = lq035q1_spidev_remove; > > > info->spidrv.shutdown = lq035q1_spidev_shutdown; > > >- info->spidrv.suspend = lq035q1_spidev_suspend; > > >- info->spidrv.resume = lq035q1_spidev_resume; > > >+ info->spidrv.driver.pm = LQ035Q1_SPIDEV_PM_OPS; > > > ret = spi_register_driver(&info->spidrv); > > > if (ret < 0) { > > > > > > -- > > Greetings, > > Michael > > > > -- > > Analog Devices GmbH Wilhelm-Wagenfeld-Str. 6 80807 Muenchen > > Sitz der Gesellschaft: Muenchen; Registergericht: Muenchen HRB 40368; > > Geschaeftsfuehrer:Dr.Carsten Suckrow, Thomas Wessel, William A. Martin, > > Margaret Seif > > > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] fbdev: bfin-lq035q1-fb: Use dev_pm_ops 2013-05-30 0:32 ` Jingoo Han @ 2013-05-30 7:14 ` Jean-Christophe PLAGNIOL-VILLARD 2013-05-30 7:52 ` Lars-Peter Clausen 0 siblings, 1 reply; 13+ messages in thread From: Jean-Christophe PLAGNIOL-VILLARD @ 2013-05-30 7:14 UTC (permalink / raw) To: Jingoo Han Cc: 'Michael Hennerich', 'Rafael J. Wysocki', 'Lars-Peter Clausen', 'Tomi Valkeinen', linux-fbdev, linux-pm On 09:32 Thu 30 May , Jingoo Han wrote: > On Thursday, May 30, 2013 4:20 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > > On 16:24 Wed 29 May , Michael Hennerich wrote: > > > On 05/29/2013 02:17 PM, Lars-Peter Clausen wrote: > > > >Use dev_pm_ops instead of the legacy suspend/resume callbacks. > > > > > > > >Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> > > > Acked-by: Michael Hennerich <michael.hennerich@analog.com> > > > >--- > > > > drivers/video/bfin-lq035q1-fb.c | 22 ++++++++++++++-------- > > > > 1 file changed, 14 insertions(+), 8 deletions(-) > > > > > > > >diff --git a/drivers/video/bfin-lq035q1-fb.c b/drivers/video/bfin-lq035q1-fb.c > > > >index 29d8c04..6084c17 100644 > > > >--- a/drivers/video/bfin-lq035q1-fb.c > > > >+++ b/drivers/video/bfin-lq035q1-fb.c > > > >@@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) > > > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > > > } > > > >-#ifdef CONFIG_PM > > > >-static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) > > > >+#ifdef CONFIG_PM_SLEEP > > > >+static int lq035q1_spidev_suspend(struct device *dev) > > > > { > > > >+ struct spi_device *spi = to_spi_device(dev); > > > >+ > > > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > > > } > > > >-static int lq035q1_spidev_resume(struct spi_device *spi) > > > >+static int lq035q1_spidev_resume(struct device *dev) > > > > { > > > >- int ret; > > > >+ struct spi_device *spi = to_spi_device(dev); > > > > struct spi_control *ctl = spi_get_drvdata(spi); > > > >+ int ret; > > > > ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); > > > > if (ret) > > > >@@ -187,9 +190,13 @@ static int lq035q1_spidev_resume(struct spi_device *spi) > > > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); > > > > } > > > >+ > > > >+static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, > > > >+ lq035q1_spidev_resume); > > > >+#define LQ035Q1_SPIDEV_PM_OPS (&lq035q1_spidev_pm_ops) > > > >+ > > > > #else > > > >-# define lq035q1_spidev_suspend NULL > > > >-# define lq035q1_spidev_resume NULL > > > >+#define LQ035Q1_SPIDEV_PM_OPS NULL > > > > #endif > > we really need to ahve a macro like for DT of_match_ptr to drop the #else > > Hi Jean-Christophe PLAGNIOL-VILLARD, > > I submitted the following patch. :) > (https://patchwork.kernel.org/patch/2502971/) > > --- a/include/linux/pm.h > +++ b/include/linux/pm.h > @@ -55,8 +55,10 @@ struct device; > > #ifdef CONFIG_PM > extern const char power_group_name[]; /* = "power" */ > +#define pm_ops_ptr(_ptr) (_ptr) > #else > #define power_group_name NULL > +#define pm_ops_ptr(_ptr) NULL > #endif > > > This patch was accepted by Rafael Wysocki, and will be merged to v3.11-rc1. > Lars-Peter please update with and Acked-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com> Rafael can you take this one via pm tree Best Regards, J. > > Best regards, > Jingoo Han > > > > > I known it's related to this patch but someone should do it > > > > Rafael what do you think > > > > Best Regards, > > J. > > > > /* Power down all displays on reboot, poweroff or halt */ > > > >@@ -708,8 +715,7 @@ static int bfin_lq035q1_probe(struct platform_device *pdev) > > > > info->spidrv.probe = lq035q1_spidev_probe; > > > > info->spidrv.remove = lq035q1_spidev_remove; > > > > info->spidrv.shutdown = lq035q1_spidev_shutdown; > > > >- info->spidrv.suspend = lq035q1_spidev_suspend; > > > >- info->spidrv.resume = lq035q1_spidev_resume; > > > >+ info->spidrv.driver.pm = LQ035Q1_SPIDEV_PM_OPS; > > > > ret = spi_register_driver(&info->spidrv); > > > > if (ret < 0) { > > > > > > > > > -- > > > Greetings, > > > Michael > > > > > > -- > > > Analog Devices GmbH Wilhelm-Wagenfeld-Str. 6 80807 Muenchen > > > Sitz der Gesellschaft: Muenchen; Registergericht: Muenchen HRB 40368; > > > Geschaeftsfuehrer:Dr.Carsten Suckrow, Thomas Wessel, William A. Martin, > > > Margaret Seif > > > > > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] fbdev: bfin-lq035q1-fb: Use dev_pm_ops 2013-05-30 7:14 ` Jean-Christophe PLAGNIOL-VILLARD @ 2013-05-30 7:52 ` Lars-Peter Clausen 2013-05-30 10:45 ` Jean-Christophe PLAGNIOL-VILLARD 0 siblings, 1 reply; 13+ messages in thread From: Lars-Peter Clausen @ 2013-05-30 7:52 UTC (permalink / raw) To: Jean-Christophe PLAGNIOL-VILLARD Cc: Jingoo Han, 'Michael Hennerich', 'Rafael J. Wysocki', 'Tomi Valkeinen', linux-fbdev, linux-pm On 05/30/2013 09:14 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > On 09:32 Thu 30 May , Jingoo Han wrote: >> On Thursday, May 30, 2013 4:20 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: >>> On 16:24 Wed 29 May , Michael Hennerich wrote: >>>> On 05/29/2013 02:17 PM, Lars-Peter Clausen wrote: >>>>> Use dev_pm_ops instead of the legacy suspend/resume callbacks. >>>>> >>>>> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> >>>> Acked-by: Michael Hennerich <michael.hennerich@analog.com> >>>>> --- >>>>> drivers/video/bfin-lq035q1-fb.c | 22 ++++++++++++++-------- >>>>> 1 file changed, 14 insertions(+), 8 deletions(-) >>>>> >>>>> diff --git a/drivers/video/bfin-lq035q1-fb.c b/drivers/video/bfin-lq035q1-fb.c >>>>> index 29d8c04..6084c17 100644 >>>>> --- a/drivers/video/bfin-lq035q1-fb.c >>>>> +++ b/drivers/video/bfin-lq035q1-fb.c >>>>> @@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); >>>>> } >>>>> -#ifdef CONFIG_PM >>>>> -static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) >>>>> +#ifdef CONFIG_PM_SLEEP >>>>> +static int lq035q1_spidev_suspend(struct device *dev) >>>>> { >>>>> + struct spi_device *spi = to_spi_device(dev); >>>>> + >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); >>>>> } >>>>> -static int lq035q1_spidev_resume(struct spi_device *spi) >>>>> +static int lq035q1_spidev_resume(struct device *dev) >>>>> { >>>>> - int ret; >>>>> + struct spi_device *spi = to_spi_device(dev); >>>>> struct spi_control *ctl = spi_get_drvdata(spi); >>>>> + int ret; >>>>> ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); >>>>> if (ret) >>>>> @@ -187,9 +190,13 @@ static int lq035q1_spidev_resume(struct spi_device *spi) >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); >>>>> } >>>>> + >>>>> +static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, >>>>> + lq035q1_spidev_resume); >>>>> +#define LQ035Q1_SPIDEV_PM_OPS (&lq035q1_spidev_pm_ops) >>>>> + >>>>> #else >>>>> -# define lq035q1_spidev_suspend NULL >>>>> -# define lq035q1_spidev_resume NULL >>>>> +#define LQ035Q1_SPIDEV_PM_OPS NULL >>>>> #endif >>> we really need to ahve a macro like for DT of_match_ptr to drop the #else >> >> Hi Jean-Christophe PLAGNIOL-VILLARD, >> >> I submitted the following patch. :) >> (https://patchwork.kernel.org/patch/2502971/) >> >> --- a/include/linux/pm.h >> +++ b/include/linux/pm.h >> @@ -55,8 +55,10 @@ struct device; >> >> #ifdef CONFIG_PM >> extern const char power_group_name[]; /* = "power" */ >> +#define pm_ops_ptr(_ptr) (_ptr) >> #else >> #define power_group_name NULL >> +#define pm_ops_ptr(_ptr) NULL >> #endif >> >> >> This patch was accepted by Rafael Wysocki, and will be merged to v3.11-rc1. >> > Lars-Peter please update with and Since the code depends on CONFIG_PM_SLEEP and not CONFIG_PM I don't think the macro will work. - Lars ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] fbdev: bfin-lq035q1-fb: Use dev_pm_ops 2013-05-30 7:52 ` Lars-Peter Clausen @ 2013-05-30 10:45 ` Jean-Christophe PLAGNIOL-VILLARD 2013-05-30 16:32 ` Lars-Peter Clausen 2013-05-31 3:32 ` Jingoo Han 0 siblings, 2 replies; 13+ messages in thread From: Jean-Christophe PLAGNIOL-VILLARD @ 2013-05-30 10:45 UTC (permalink / raw) To: Lars-Peter Clausen Cc: Jingoo Han, 'Michael Hennerich', 'Rafael J. Wysocki', 'Tomi Valkeinen', linux-fbdev, linux-pm On 09:52 Thu 30 May , Lars-Peter Clausen wrote: > On 05/30/2013 09:14 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > > On 09:32 Thu 30 May , Jingoo Han wrote: > >> On Thursday, May 30, 2013 4:20 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > >>> On 16:24 Wed 29 May , Michael Hennerich wrote: > >>>> On 05/29/2013 02:17 PM, Lars-Peter Clausen wrote: > >>>>> Use dev_pm_ops instead of the legacy suspend/resume callbacks. > >>>>> > >>>>> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> > >>>> Acked-by: Michael Hennerich <michael.hennerich@analog.com> > >>>>> --- > >>>>> drivers/video/bfin-lq035q1-fb.c | 22 ++++++++++++++-------- > >>>>> 1 file changed, 14 insertions(+), 8 deletions(-) > >>>>> > >>>>> diff --git a/drivers/video/bfin-lq035q1-fb.c b/drivers/video/bfin-lq035q1-fb.c > >>>>> index 29d8c04..6084c17 100644 > >>>>> --- a/drivers/video/bfin-lq035q1-fb.c > >>>>> +++ b/drivers/video/bfin-lq035q1-fb.c > >>>>> @@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) > >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > >>>>> } > >>>>> -#ifdef CONFIG_PM > >>>>> -static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) > >>>>> +#ifdef CONFIG_PM_SLEEP > >>>>> +static int lq035q1_spidev_suspend(struct device *dev) > >>>>> { > >>>>> + struct spi_device *spi = to_spi_device(dev); > >>>>> + > >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > >>>>> } > >>>>> -static int lq035q1_spidev_resume(struct spi_device *spi) > >>>>> +static int lq035q1_spidev_resume(struct device *dev) > >>>>> { > >>>>> - int ret; > >>>>> + struct spi_device *spi = to_spi_device(dev); > >>>>> struct spi_control *ctl = spi_get_drvdata(spi); > >>>>> + int ret; > >>>>> ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); > >>>>> if (ret) > >>>>> @@ -187,9 +190,13 @@ static int lq035q1_spidev_resume(struct spi_device *spi) > >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); > >>>>> } > >>>>> + > >>>>> +static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, > >>>>> + lq035q1_spidev_resume); > >>>>> +#define LQ035Q1_SPIDEV_PM_OPS (&lq035q1_spidev_pm_ops) > >>>>> + > >>>>> #else > >>>>> -# define lq035q1_spidev_suspend NULL > >>>>> -# define lq035q1_spidev_resume NULL > >>>>> +#define LQ035Q1_SPIDEV_PM_OPS NULL > >>>>> #endif > >>> we really need to ahve a macro like for DT of_match_ptr to drop the #else > >> > >> Hi Jean-Christophe PLAGNIOL-VILLARD, > >> > >> I submitted the following patch. :) > >> (https://patchwork.kernel.org/patch/2502971/) > >> > >> --- a/include/linux/pm.h > >> +++ b/include/linux/pm.h > >> @@ -55,8 +55,10 @@ struct device; > >> > >> #ifdef CONFIG_PM > >> extern const char power_group_name[]; /* = "power" */ > >> +#define pm_ops_ptr(_ptr) (_ptr) > >> #else > >> #define power_group_name NULL > >> +#define pm_ops_ptr(_ptr) NULL > >> #endif > >> > >> > >> This patch was accepted by Rafael Wysocki, and will be merged to v3.11-rc1. > >> > > Lars-Peter please update with and > > Since the code depends on CONFIG_PM_SLEEP and not CONFIG_PM I don't think > the macro will work. se please ad a similar macros too Best Regards, J. > > - Lars > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] fbdev: bfin-lq035q1-fb: Use dev_pm_ops 2013-05-30 10:45 ` Jean-Christophe PLAGNIOL-VILLARD @ 2013-05-30 16:32 ` Lars-Peter Clausen 2013-05-31 3:32 ` Jingoo Han 1 sibling, 0 replies; 13+ messages in thread From: Lars-Peter Clausen @ 2013-05-30 16:32 UTC (permalink / raw) To: Jean-Christophe PLAGNIOL-VILLARD Cc: Jingoo Han, 'Michael Hennerich', 'Rafael J. Wysocki', 'Tomi Valkeinen', linux-fbdev, linux-pm On 05/30/2013 12:45 PM, Jean-Christophe PLAGNIOL-VILLARD wrote: > On 09:52 Thu 30 May , Lars-Peter Clausen wrote: >> On 05/30/2013 09:14 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: >>> On 09:32 Thu 30 May , Jingoo Han wrote: >>>> On Thursday, May 30, 2013 4:20 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: >>>>> On 16:24 Wed 29 May , Michael Hennerich wrote: >>>>>> On 05/29/2013 02:17 PM, Lars-Peter Clausen wrote: >>>>>>> Use dev_pm_ops instead of the legacy suspend/resume callbacks. >>>>>>> >>>>>>> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> >>>>>> Acked-by: Michael Hennerich <michael.hennerich@analog.com> >>>>>>> --- >>>>>>> drivers/video/bfin-lq035q1-fb.c | 22 ++++++++++++++-------- >>>>>>> 1 file changed, 14 insertions(+), 8 deletions(-) >>>>>>> >>>>>>> diff --git a/drivers/video/bfin-lq035q1-fb.c b/drivers/video/bfin-lq035q1-fb.c >>>>>>> index 29d8c04..6084c17 100644 >>>>>>> --- a/drivers/video/bfin-lq035q1-fb.c >>>>>>> +++ b/drivers/video/bfin-lq035q1-fb.c >>>>>>> @@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) >>>>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); >>>>>>> } >>>>>>> -#ifdef CONFIG_PM >>>>>>> -static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) >>>>>>> +#ifdef CONFIG_PM_SLEEP >>>>>>> +static int lq035q1_spidev_suspend(struct device *dev) >>>>>>> { >>>>>>> + struct spi_device *spi = to_spi_device(dev); >>>>>>> + >>>>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); >>>>>>> } >>>>>>> -static int lq035q1_spidev_resume(struct spi_device *spi) >>>>>>> +static int lq035q1_spidev_resume(struct device *dev) >>>>>>> { >>>>>>> - int ret; >>>>>>> + struct spi_device *spi = to_spi_device(dev); >>>>>>> struct spi_control *ctl = spi_get_drvdata(spi); >>>>>>> + int ret; >>>>>>> ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); >>>>>>> if (ret) >>>>>>> @@ -187,9 +190,13 @@ static int lq035q1_spidev_resume(struct spi_device *spi) >>>>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); >>>>>>> } >>>>>>> + >>>>>>> +static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, >>>>>>> + lq035q1_spidev_resume); >>>>>>> +#define LQ035Q1_SPIDEV_PM_OPS (&lq035q1_spidev_pm_ops) >>>>>>> + >>>>>>> #else >>>>>>> -# define lq035q1_spidev_suspend NULL >>>>>>> -# define lq035q1_spidev_resume NULL >>>>>>> +#define LQ035Q1_SPIDEV_PM_OPS NULL >>>>>>> #endif >>>>> we really need to ahve a macro like for DT of_match_ptr to drop the #else >>>> >>>> Hi Jean-Christophe PLAGNIOL-VILLARD, >>>> >>>> I submitted the following patch. :) >>>> (https://patchwork.kernel.org/patch/2502971/) >>>> >>>> --- a/include/linux/pm.h >>>> +++ b/include/linux/pm.h >>>> @@ -55,8 +55,10 @@ struct device; >>>> >>>> #ifdef CONFIG_PM >>>> extern const char power_group_name[]; /* = "power" */ >>>> +#define pm_ops_ptr(_ptr) (_ptr) >>>> #else >>>> #define power_group_name NULL >>>> +#define pm_ops_ptr(_ptr) NULL >>>> #endif >>>> >>>> >>>> This patch was accepted by Rafael Wysocki, and will be merged to v3.11-rc1. >>>> >>> Lars-Peter please update with and >> >> Since the code depends on CONFIG_PM_SLEEP and not CONFIG_PM I don't think >> the macro will work. > > se please ad a similar macros too I'm not really convinced that this *_ops_ptr thing is the right approach. Besides that the scope of this patch is to convert the driver from legacy suspend callbacks to dev_pm_ops not to remove the #ifdef's. - Lars ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] fbdev: bfin-lq035q1-fb: Use dev_pm_ops 2013-05-30 10:45 ` Jean-Christophe PLAGNIOL-VILLARD 2013-05-30 16:32 ` Lars-Peter Clausen @ 2013-05-31 3:32 ` Jingoo Han 2013-05-31 5:01 ` Jean-Christophe PLAGNIOL-VILLARD 2013-06-02 19:30 ` Rafael J. Wysocki 1 sibling, 2 replies; 13+ messages in thread From: Jingoo Han @ 2013-05-31 3:32 UTC (permalink / raw) To: 'Jean-Christophe PLAGNIOL-VILLARD', 'Rafael J. Wysocki' Cc: 'Lars-Peter Clausen', 'Michael Hennerich', 'Tomi Valkeinen', linux-fbdev, linux-pm, Jingoo Han On Thursday, May 30, 2013 7:46 PM, Jean-Christophe PLAGNIOL-VILLARD wrote: > On 09:52 Thu 30 May , Lars-Peter Clausen wrote: > > On 05/30/2013 09:14 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > > > On 09:32 Thu 30 May , Jingoo Han wrote: > > >> On Thursday, May 30, 2013 4:20 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > > >>> On 16:24 Wed 29 May , Michael Hennerich wrote: > > >>>> On 05/29/2013 02:17 PM, Lars-Peter Clausen wrote: > > >>>>> Use dev_pm_ops instead of the legacy suspend/resume callbacks. > > >>>>> > > >>>>> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> > > >>>> Acked-by: Michael Hennerich <michael.hennerich@analog.com> > > >>>>> --- > > >>>>> drivers/video/bfin-lq035q1-fb.c | 22 ++++++++++++++-------- > > >>>>> 1 file changed, 14 insertions(+), 8 deletions(-) > > >>>>> > > >>>>> diff --git a/drivers/video/bfin-lq035q1-fb.c b/drivers/video/bfin-lq035q1-fb.c > > >>>>> index 29d8c04..6084c17 100644 > > >>>>> --- a/drivers/video/bfin-lq035q1-fb.c > > >>>>> +++ b/drivers/video/bfin-lq035q1-fb.c > > >>>>> @@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) > > >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > >>>>> } > > >>>>> -#ifdef CONFIG_PM > > >>>>> -static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) > > >>>>> +#ifdef CONFIG_PM_SLEEP > > >>>>> +static int lq035q1_spidev_suspend(struct device *dev) > > >>>>> { > > >>>>> + struct spi_device *spi = to_spi_device(dev); > > >>>>> + > > >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > >>>>> } > > >>>>> -static int lq035q1_spidev_resume(struct spi_device *spi) > > >>>>> +static int lq035q1_spidev_resume(struct device *dev) > > >>>>> { > > >>>>> - int ret; > > >>>>> + struct spi_device *spi = to_spi_device(dev); > > >>>>> struct spi_control *ctl = spi_get_drvdata(spi); > > >>>>> + int ret; > > >>>>> ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); > > >>>>> if (ret) > > >>>>> @@ -187,9 +190,13 @@ static int lq035q1_spidev_resume(struct spi_device *spi) > > >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); > > >>>>> } > > >>>>> + > > >>>>> +static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, > > >>>>> + lq035q1_spidev_resume); > > >>>>> +#define LQ035Q1_SPIDEV_PM_OPS (&lq035q1_spidev_pm_ops) > > >>>>> + > > >>>>> #else > > >>>>> -# define lq035q1_spidev_suspend NULL > > >>>>> -# define lq035q1_spidev_resume NULL > > >>>>> +#define LQ035Q1_SPIDEV_PM_OPS NULL > > >>>>> #endif > > >>> we really need to ahve a macro like for DT of_match_ptr to drop the #else > > >> > > >> Hi Jean-Christophe PLAGNIOL-VILLARD, > > >> > > >> I submitted the following patch. :) > > >> (https://patchwork.kernel.org/patch/2502971/) > > >> > > >> --- a/include/linux/pm.h > > >> +++ b/include/linux/pm.h > > >> @@ -55,8 +55,10 @@ struct device; > > >> > > >> #ifdef CONFIG_PM > > >> extern const char power_group_name[]; /* = "power" */ > > >> +#define pm_ops_ptr(_ptr) (_ptr) > > >> #else > > >> #define power_group_name NULL > > >> +#define pm_ops_ptr(_ptr) NULL > > >> #endif > > >> > > >> > > >> This patch was accepted by Rafael Wysocki, and will be merged to v3.11-rc1. > > >> > > > Lars-Peter please update with and > > > > Since the code depends on CONFIG_PM_SLEEP and not CONFIG_PM I don't think > > the macro will work. > > se please ad a similar macros too Hi Jean-Christophe, I added pm_sleep_ops_ptr() as below. Maybe you want it. There is no build warnings below 4 config situations. a) CONFIG_PM is enabled. b) only CONFIG_PM_SLEEP is enabled c) only CONFIG_PM_RUNTIME is enabled d) CONFIG_PM is NOT enabled. --- a/drivers/video/bfin-lq035q1-fb.c +++ b/drivers/video/bfin-lq035q1-fb.c @@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); } -#ifdef CONFIG_PM -static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) +#ifdef CONFIG_PM_SLEEP +static int lq035q1_spidev_suspend(struct device *dev) { + struct spi_device *spi = to_spi_device(dev); + return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); } -static int lq035q1_spidev_resume(struct spi_device *spi) +static int lq035q1_spidev_resume(struct device *dev) { - int ret; + struct spi_device *spi = to_spi_device(dev); struct spi_control *ctl = spi_get_drvdata(spi); + int ret; ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); if (ret) @@ -187,11 +190,11 @@ static int lq035q1_spidev_resume(struct spi_device *spi) return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); } -#else -# define lq035q1_spidev_suspend NULL -# define lq035q1_spidev_resume NULL #endif +static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, + lq035q1_spidev_resume); + /* Power down all displays on reboot, poweroff or halt */ static void lq035q1_spidev_shutdown(struct spi_device *spi) { @@ -708,8 +711,7 @@ static int bfin_lq035q1_probe(struct platform_device *pdev) info->spidrv.probe = lq035q1_spidev_probe; info->spidrv.remove = lq035q1_spidev_remove; info->spidrv.shutdown = lq035q1_spidev_shutdown; - info->spidrv.suspend = lq035q1_spidev_suspend; - info->spidrv.resume = lq035q1_spidev_resume; + info->spidrv.driver.pm = pm_sleep_ops_ptr(&lq035q1_spidev_pm_ops); ret = spi_register_driver(&info->spidrv); if (ret < 0) { diff --git a/include/linux/pm.h b/include/linux/pm.h index bd50d15..999d652 100644 --- a/include/linux/pm.h +++ b/include/linux/pm.h @@ -61,6 +61,12 @@ extern const char power_group_name[]; /* = "power" */ #define pm_ops_ptr(_ptr) NULL #endif +#ifdef CONFIG_PM_SLEEP +#define pm_sleep_ops_ptr(_ptr) (_ptr) +#else +#define pm_sleep_ops_ptr(_ptr) NULL +#endif + Hi Rafael J. Wysocki, How about adding this pm_sleep_ops_ptr() macro? My draft idea is below: However, I want other's better ideas. :) 1. The case that only CONFIG_PM_SLEEP is necessary. #ifdef CONFIG_PM_SLEEP static int *_suspend(struct device *dev) [....] static int *_resume(struct device *dev) [....] #endif static SIMPLE_DEV_PM_OPS(*_pm_ops, *_suspend, *_resume); static struct {platform/spi/i2c/etc}_driver *_driver = { .driver = { .pm = pm_sleep_ops_ptr(&*_pm_ops), 2. The case that both CONFIG_PM_SLEEP & CONFIG_PM_RUNTIME are necessary. #ifdef CONFIG_PM_SLEEP static int *_suspend(struct device *dev) [....] static int *_resume(struct device *dev) [....] #endif #ifdef CONFIG_PM_RUNTIME static int *_runtime_suspend(struct device *dev) [....] static int *_runtime_resume(struct device *dev) [....] #endif static const struct dev_pm_ops *_pm_ops = { SET_SYSTEM_SLEEP_PM_OPS(*_suspend, *_resume) SET_RUNTIME_PM_OPS(*_runtime_suspend, *_runtime_resume, NULL) }; static struct {platform/spi/i2c/etc}_driver *_driver = { .driver = { .pm = pm_ops_ptr(&*_pm_ops), Best regards, Jingoo Han ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH] fbdev: bfin-lq035q1-fb: Use dev_pm_ops 2013-05-31 3:32 ` Jingoo Han @ 2013-05-31 5:01 ` Jean-Christophe PLAGNIOL-VILLARD 2013-06-02 19:30 ` Rafael J. Wysocki 1 sibling, 0 replies; 13+ messages in thread From: Jean-Christophe PLAGNIOL-VILLARD @ 2013-05-31 5:01 UTC (permalink / raw) To: Jingoo Han Cc: 'Rafael J. Wysocki', 'Lars-Peter Clausen', 'Michael Hennerich', 'Tomi Valkeinen', linux-fbdev, linux-pm On 12:32 Fri 31 May , Jingoo Han wrote: > On Thursday, May 30, 2013 7:46 PM, Jean-Christophe PLAGNIOL-VILLARD wrote: > > On 09:52 Thu 30 May , Lars-Peter Clausen wrote: > > > On 05/30/2013 09:14 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > > > > On 09:32 Thu 30 May , Jingoo Han wrote: > > > >> On Thursday, May 30, 2013 4:20 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > > > >>> On 16:24 Wed 29 May , Michael Hennerich wrote: > > > >>>> On 05/29/2013 02:17 PM, Lars-Peter Clausen wrote: > > > >>>>> Use dev_pm_ops instead of the legacy suspend/resume callbacks. > > > >>>>> > > > >>>>> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> > > > >>>> Acked-by: Michael Hennerich <michael.hennerich@analog.com> > > > >>>>> --- > > > >>>>> drivers/video/bfin-lq035q1-fb.c | 22 ++++++++++++++-------- > > > >>>>> 1 file changed, 14 insertions(+), 8 deletions(-) > > > >>>>> > > > >>>>> diff --git a/drivers/video/bfin-lq035q1-fb.c b/drivers/video/bfin-lq035q1-fb.c > > > >>>>> index 29d8c04..6084c17 100644 > > > >>>>> --- a/drivers/video/bfin-lq035q1-fb.c > > > >>>>> +++ b/drivers/video/bfin-lq035q1-fb.c > > > >>>>> @@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) > > > >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > > >>>>> } > > > >>>>> -#ifdef CONFIG_PM > > > >>>>> -static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) > > > >>>>> +#ifdef CONFIG_PM_SLEEP > > > >>>>> +static int lq035q1_spidev_suspend(struct device *dev) > > > >>>>> { > > > >>>>> + struct spi_device *spi = to_spi_device(dev); > > > >>>>> + > > > >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > > >>>>> } > > > >>>>> -static int lq035q1_spidev_resume(struct spi_device *spi) > > > >>>>> +static int lq035q1_spidev_resume(struct device *dev) > > > >>>>> { > > > >>>>> - int ret; > > > >>>>> + struct spi_device *spi = to_spi_device(dev); > > > >>>>> struct spi_control *ctl = spi_get_drvdata(spi); > > > >>>>> + int ret; > > > >>>>> ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); > > > >>>>> if (ret) > > > >>>>> @@ -187,9 +190,13 @@ static int lq035q1_spidev_resume(struct spi_device *spi) > > > >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); > > > >>>>> } > > > >>>>> + > > > >>>>> +static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, > > > >>>>> + lq035q1_spidev_resume); > > > >>>>> +#define LQ035Q1_SPIDEV_PM_OPS (&lq035q1_spidev_pm_ops) > > > >>>>> + > > > >>>>> #else > > > >>>>> -# define lq035q1_spidev_suspend NULL > > > >>>>> -# define lq035q1_spidev_resume NULL > > > >>>>> +#define LQ035Q1_SPIDEV_PM_OPS NULL > > > >>>>> #endif > > > >>> we really need to ahve a macro like for DT of_match_ptr to drop the #else > > > >> > > > >> Hi Jean-Christophe PLAGNIOL-VILLARD, > > > >> > > > >> I submitted the following patch. :) > > > >> (https://patchwork.kernel.org/patch/2502971/) > > > >> > > > >> --- a/include/linux/pm.h > > > >> +++ b/include/linux/pm.h > > > >> @@ -55,8 +55,10 @@ struct device; > > > >> > > > >> #ifdef CONFIG_PM > > > >> extern const char power_group_name[]; /* = "power" */ > > > >> +#define pm_ops_ptr(_ptr) (_ptr) > > > >> #else > > > >> #define power_group_name NULL > > > >> +#define pm_ops_ptr(_ptr) NULL > > > >> #endif > > > >> > > > >> > > > >> This patch was accepted by Rafael Wysocki, and will be merged to v3.11-rc1. > > > >> > > > > Lars-Peter please update with and > > > > > > Since the code depends on CONFIG_PM_SLEEP and not CONFIG_PM I don't think > > > the macro will work. > > > > se please ad a similar macros too > > Hi Jean-Christophe, > > I added pm_sleep_ops_ptr() as below. > Maybe you want it. > There is no build warnings below 4 config situations. > a) CONFIG_PM is enabled. > b) only CONFIG_PM_SLEEP is enabled > c) only CONFIG_PM_RUNTIME is enabled > d) CONFIG_PM is NOT enabled. > > --- a/drivers/video/bfin-lq035q1-fb.c > +++ b/drivers/video/bfin-lq035q1-fb.c > @@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > } > > -#ifdef CONFIG_PM > -static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) > +#ifdef CONFIG_PM_SLEEP > +static int lq035q1_spidev_suspend(struct device *dev) > { > + struct spi_device *spi = to_spi_device(dev); > + > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > } > > -static int lq035q1_spidev_resume(struct spi_device *spi) > +static int lq035q1_spidev_resume(struct device *dev) > { > - int ret; > + struct spi_device *spi = to_spi_device(dev); > struct spi_control *ctl = spi_get_drvdata(spi); > + int ret; > > ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); > if (ret) > @@ -187,11 +190,11 @@ static int lq035q1_spidev_resume(struct spi_device *spi) > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); > } > -#else > -# define lq035q1_spidev_suspend NULL > -# define lq035q1_spidev_resume NULL > #endif > > +static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, > + lq035q1_spidev_resume); > + > /* Power down all displays on reboot, poweroff or halt */ > static void lq035q1_spidev_shutdown(struct spi_device *spi) > { > @@ -708,8 +711,7 @@ static int bfin_lq035q1_probe(struct platform_device *pdev) > info->spidrv.probe = lq035q1_spidev_probe; > info->spidrv.remove = lq035q1_spidev_remove; > info->spidrv.shutdown = lq035q1_spidev_shutdown; > - info->spidrv.suspend = lq035q1_spidev_suspend; > - info->spidrv.resume = lq035q1_spidev_resume; > + info->spidrv.driver.pm = pm_sleep_ops_ptr(&lq035q1_spidev_pm_ops); > > ret = spi_register_driver(&info->spidrv); > if (ret < 0) { > diff --git a/include/linux/pm.h b/include/linux/pm.h > index bd50d15..999d652 100644 > --- a/include/linux/pm.h > +++ b/include/linux/pm.h > @@ -61,6 +61,12 @@ extern const char power_group_name[]; /* = "power" */ > #define pm_ops_ptr(_ptr) NULL > #endif > > +#ifdef CONFIG_PM_SLEEP > +#define pm_sleep_ops_ptr(_ptr) (_ptr) > +#else > +#define pm_sleep_ops_ptr(_ptr) NULL > +#endif > + > > > Hi Rafael J. Wysocki, > How about adding this pm_sleep_ops_ptr() macro? > > My draft idea is below: > However, I want other's better ideas. :) > > 1. The case that only CONFIG_PM_SLEEP is necessary. > #ifdef CONFIG_PM_SLEEP > static int *_suspend(struct device *dev) > [....] > static int *_resume(struct device *dev) > [....] > #endif > > static SIMPLE_DEV_PM_OPS(*_pm_ops, *_suspend, *_resume); > > static struct {platform/spi/i2c/etc}_driver *_driver = { > .driver = { > .pm = pm_sleep_ops_ptr(&*_pm_ops), > > > 2. The case that both CONFIG_PM_SLEEP & CONFIG_PM_RUNTIME > are necessary. > > #ifdef CONFIG_PM_SLEEP > static int *_suspend(struct device *dev) > [....] > static int *_resume(struct device *dev) > [....] > #endif > > #ifdef CONFIG_PM_RUNTIME > static int *_runtime_suspend(struct device *dev) > [....] > static int *_runtime_resume(struct device *dev) > [....] > #endif > > static const struct dev_pm_ops *_pm_ops = { > SET_SYSTEM_SLEEP_PM_OPS(*_suspend, *_resume) > SET_RUNTIME_PM_OPS(*_runtime_suspend, *_runtime_resume, NULL) > }; > > static struct {platform/spi/i2c/etc}_driver *_driver = { > .driver = { > .pm = pm_ops_ptr(&*_pm_ops), > > yes for me Best Regards, J. > > Best regards, > Jingoo Han > > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] fbdev: bfin-lq035q1-fb: Use dev_pm_ops 2013-05-31 3:32 ` Jingoo Han 2013-05-31 5:01 ` Jean-Christophe PLAGNIOL-VILLARD @ 2013-06-02 19:30 ` Rafael J. Wysocki 2013-06-03 5:15 ` Jingoo Han 1 sibling, 1 reply; 13+ messages in thread From: Rafael J. Wysocki @ 2013-06-02 19:30 UTC (permalink / raw) To: Jingoo Han Cc: 'Jean-Christophe PLAGNIOL-VILLARD', 'Lars-Peter Clausen', 'Michael Hennerich', 'Tomi Valkeinen', linux-fbdev, linux-pm On Friday, May 31, 2013 12:32:57 PM Jingoo Han wrote: > On Thursday, May 30, 2013 7:46 PM, Jean-Christophe PLAGNIOL-VILLARD wrote: > > On 09:52 Thu 30 May , Lars-Peter Clausen wrote: > > > On 05/30/2013 09:14 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > > > > On 09:32 Thu 30 May , Jingoo Han wrote: > > > >> On Thursday, May 30, 2013 4:20 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > > > >>> On 16:24 Wed 29 May , Michael Hennerich wrote: > > > >>>> On 05/29/2013 02:17 PM, Lars-Peter Clausen wrote: > > > >>>>> Use dev_pm_ops instead of the legacy suspend/resume callbacks. > > > >>>>> > > > >>>>> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> > > > >>>> Acked-by: Michael Hennerich <michael.hennerich@analog.com> > > > >>>>> --- > > > >>>>> drivers/video/bfin-lq035q1-fb.c | 22 ++++++++++++++-------- > > > >>>>> 1 file changed, 14 insertions(+), 8 deletions(-) > > > >>>>> > > > >>>>> diff --git a/drivers/video/bfin-lq035q1-fb.c b/drivers/video/bfin-lq035q1-fb.c > > > >>>>> index 29d8c04..6084c17 100644 > > > >>>>> --- a/drivers/video/bfin-lq035q1-fb.c > > > >>>>> +++ b/drivers/video/bfin-lq035q1-fb.c > > > >>>>> @@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) > > > >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > > >>>>> } > > > >>>>> -#ifdef CONFIG_PM > > > >>>>> -static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) > > > >>>>> +#ifdef CONFIG_PM_SLEEP > > > >>>>> +static int lq035q1_spidev_suspend(struct device *dev) > > > >>>>> { > > > >>>>> + struct spi_device *spi = to_spi_device(dev); > > > >>>>> + > > > >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > > >>>>> } > > > >>>>> -static int lq035q1_spidev_resume(struct spi_device *spi) > > > >>>>> +static int lq035q1_spidev_resume(struct device *dev) > > > >>>>> { > > > >>>>> - int ret; > > > >>>>> + struct spi_device *spi = to_spi_device(dev); > > > >>>>> struct spi_control *ctl = spi_get_drvdata(spi); > > > >>>>> + int ret; > > > >>>>> ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); > > > >>>>> if (ret) > > > >>>>> @@ -187,9 +190,13 @@ static int lq035q1_spidev_resume(struct spi_device *spi) > > > >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); > > > >>>>> } > > > >>>>> + > > > >>>>> +static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, > > > >>>>> + lq035q1_spidev_resume); > > > >>>>> +#define LQ035Q1_SPIDEV_PM_OPS (&lq035q1_spidev_pm_ops) > > > >>>>> + > > > >>>>> #else > > > >>>>> -# define lq035q1_spidev_suspend NULL > > > >>>>> -# define lq035q1_spidev_resume NULL > > > >>>>> +#define LQ035Q1_SPIDEV_PM_OPS NULL > > > >>>>> #endif > > > >>> we really need to ahve a macro like for DT of_match_ptr to drop the #else > > > >> > > > >> Hi Jean-Christophe PLAGNIOL-VILLARD, > > > >> > > > >> I submitted the following patch. :) > > > >> (https://patchwork.kernel.org/patch/2502971/) > > > >> > > > >> --- a/include/linux/pm.h > > > >> +++ b/include/linux/pm.h > > > >> @@ -55,8 +55,10 @@ struct device; > > > >> > > > >> #ifdef CONFIG_PM > > > >> extern const char power_group_name[]; /* = "power" */ > > > >> +#define pm_ops_ptr(_ptr) (_ptr) > > > >> #else > > > >> #define power_group_name NULL > > > >> +#define pm_ops_ptr(_ptr) NULL > > > >> #endif > > > >> > > > >> > > > >> This patch was accepted by Rafael Wysocki, and will be merged to v3.11-rc1. > > > >> > > > > Lars-Peter please update with and > > > > > > Since the code depends on CONFIG_PM_SLEEP and not CONFIG_PM I don't think > > > the macro will work. > > > > se please ad a similar macros too > > Hi Jean-Christophe, > > I added pm_sleep_ops_ptr() as below. > Maybe you want it. > There is no build warnings below 4 config situations. > a) CONFIG_PM is enabled. > b) only CONFIG_PM_SLEEP is enabled > c) only CONFIG_PM_RUNTIME is enabled > d) CONFIG_PM is NOT enabled. > > --- a/drivers/video/bfin-lq035q1-fb.c > +++ b/drivers/video/bfin-lq035q1-fb.c > @@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > } > > -#ifdef CONFIG_PM > -static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) > +#ifdef CONFIG_PM_SLEEP > +static int lq035q1_spidev_suspend(struct device *dev) > { > + struct spi_device *spi = to_spi_device(dev); > + > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > } > > -static int lq035q1_spidev_resume(struct spi_device *spi) > +static int lq035q1_spidev_resume(struct device *dev) > { > - int ret; > + struct spi_device *spi = to_spi_device(dev); > struct spi_control *ctl = spi_get_drvdata(spi); > + int ret; > > ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); > if (ret) > @@ -187,11 +190,11 @@ static int lq035q1_spidev_resume(struct spi_device *spi) > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); > } > -#else > -# define lq035q1_spidev_suspend NULL > -# define lq035q1_spidev_resume NULL > #endif > > +static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, > + lq035q1_spidev_resume); > + > /* Power down all displays on reboot, poweroff or halt */ > static void lq035q1_spidev_shutdown(struct spi_device *spi) > { > @@ -708,8 +711,7 @@ static int bfin_lq035q1_probe(struct platform_device *pdev) > info->spidrv.probe = lq035q1_spidev_probe; > info->spidrv.remove = lq035q1_spidev_remove; > info->spidrv.shutdown = lq035q1_spidev_shutdown; > - info->spidrv.suspend = lq035q1_spidev_suspend; > - info->spidrv.resume = lq035q1_spidev_resume; > + info->spidrv.driver.pm = pm_sleep_ops_ptr(&lq035q1_spidev_pm_ops); > > ret = spi_register_driver(&info->spidrv); > if (ret < 0) { > diff --git a/include/linux/pm.h b/include/linux/pm.h > index bd50d15..999d652 100644 > --- a/include/linux/pm.h > +++ b/include/linux/pm.h > @@ -61,6 +61,12 @@ extern const char power_group_name[]; /* = "power" */ > #define pm_ops_ptr(_ptr) NULL > #endif > > +#ifdef CONFIG_PM_SLEEP > +#define pm_sleep_ops_ptr(_ptr) (_ptr) > +#else > +#define pm_sleep_ops_ptr(_ptr) NULL > +#endif > + > > > Hi Rafael J. Wysocki, > How about adding this pm_sleep_ops_ptr() macro? Are you submitting this instead or in addition to your patch I have in the linux-next branch? Rafael > My draft idea is below: > However, I want other's better ideas. :) > > 1. The case that only CONFIG_PM_SLEEP is necessary. > #ifdef CONFIG_PM_SLEEP > static int *_suspend(struct device *dev) > [....] > static int *_resume(struct device *dev) > [....] > #endif > > static SIMPLE_DEV_PM_OPS(*_pm_ops, *_suspend, *_resume); > > static struct {platform/spi/i2c/etc}_driver *_driver = { > .driver = { > .pm = pm_sleep_ops_ptr(&*_pm_ops), > > > 2. The case that both CONFIG_PM_SLEEP & CONFIG_PM_RUNTIME > are necessary. > > #ifdef CONFIG_PM_SLEEP > static int *_suspend(struct device *dev) > [....] > static int *_resume(struct device *dev) > [....] > #endif > > #ifdef CONFIG_PM_RUNTIME > static int *_runtime_suspend(struct device *dev) > [....] > static int *_runtime_resume(struct device *dev) > [....] > #endif > > static const struct dev_pm_ops *_pm_ops = { > SET_SYSTEM_SLEEP_PM_OPS(*_suspend, *_resume) > SET_RUNTIME_PM_OPS(*_runtime_suspend, *_runtime_resume, NULL) > }; > > static struct {platform/spi/i2c/etc}_driver *_driver = { > .driver = { > .pm = pm_ops_ptr(&*_pm_ops), > > > > Best regards, > Jingoo Han > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-pm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] fbdev: bfin-lq035q1-fb: Use dev_pm_ops 2013-06-02 19:30 ` Rafael J. Wysocki @ 2013-06-03 5:15 ` Jingoo Han 0 siblings, 0 replies; 13+ messages in thread From: Jingoo Han @ 2013-06-03 5:15 UTC (permalink / raw) To: 'Rafael J. Wysocki' Cc: 'Jean-Christophe PLAGNIOL-VILLARD', 'Lars-Peter Clausen', 'Michael Hennerich', 'Tomi Valkeinen', linux-fbdev, linux-pm, Jingoo Han On Monday, June 03, 2013 4:30 AM, Rafael J. Wysocki wrote: > On Friday, May 31, 2013 12:32:57 PM Jingoo Han wrote: > > On Thursday, May 30, 2013 7:46 PM, Jean-Christophe PLAGNIOL-VILLARD wrote: > > > On 09:52 Thu 30 May , Lars-Peter Clausen wrote: > > > > On 05/30/2013 09:14 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > > > > > On 09:32 Thu 30 May , Jingoo Han wrote: > > > > >> On Thursday, May 30, 2013 4:20 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > > > > >>> On 16:24 Wed 29 May , Michael Hennerich wrote: > > > > >>>> On 05/29/2013 02:17 PM, Lars-Peter Clausen wrote: > > > > >>>>> Use dev_pm_ops instead of the legacy suspend/resume callbacks. > > > > >>>>> > > > > >>>>> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> > > > > >>>> Acked-by: Michael Hennerich <michael.hennerich@analog.com> > > > > >>>>> --- > > > > >>>>> drivers/video/bfin-lq035q1-fb.c | 22 ++++++++++++++-------- > > > > >>>>> 1 file changed, 14 insertions(+), 8 deletions(-) > > > > >>>>> > > > > >>>>> diff --git a/drivers/video/bfin-lq035q1-fb.c b/drivers/video/bfin-lq035q1-fb.c > > > > >>>>> index 29d8c04..6084c17 100644 > > > > >>>>> --- a/drivers/video/bfin-lq035q1-fb.c > > > > >>>>> +++ b/drivers/video/bfin-lq035q1-fb.c > > > > >>>>> @@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) > > > > >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > > > >>>>> } > > > > >>>>> -#ifdef CONFIG_PM > > > > >>>>> -static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) > > > > >>>>> +#ifdef CONFIG_PM_SLEEP > > > > >>>>> +static int lq035q1_spidev_suspend(struct device *dev) > > > > >>>>> { > > > > >>>>> + struct spi_device *spi = to_spi_device(dev); > > > > >>>>> + > > > > >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > > > >>>>> } > > > > >>>>> -static int lq035q1_spidev_resume(struct spi_device *spi) > > > > >>>>> +static int lq035q1_spidev_resume(struct device *dev) > > > > >>>>> { > > > > >>>>> - int ret; > > > > >>>>> + struct spi_device *spi = to_spi_device(dev); > > > > >>>>> struct spi_control *ctl = spi_get_drvdata(spi); > > > > >>>>> + int ret; > > > > >>>>> ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); > > > > >>>>> if (ret) > > > > >>>>> @@ -187,9 +190,13 @@ static int lq035q1_spidev_resume(struct spi_device *spi) > > > > >>>>> return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); > > > > >>>>> } > > > > >>>>> + > > > > >>>>> +static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, > > > > >>>>> + lq035q1_spidev_resume); > > > > >>>>> +#define LQ035Q1_SPIDEV_PM_OPS (&lq035q1_spidev_pm_ops) > > > > >>>>> + > > > > >>>>> #else > > > > >>>>> -# define lq035q1_spidev_suspend NULL > > > > >>>>> -# define lq035q1_spidev_resume NULL > > > > >>>>> +#define LQ035Q1_SPIDEV_PM_OPS NULL > > > > >>>>> #endif > > > > >>> we really need to ahve a macro like for DT of_match_ptr to drop the #else > > > > >> > > > > >> Hi Jean-Christophe PLAGNIOL-VILLARD, > > > > >> > > > > >> I submitted the following patch. :) > > > > >> (https://patchwork.kernel.org/patch/2502971/) > > > > >> > > > > >> --- a/include/linux/pm.h > > > > >> +++ b/include/linux/pm.h > > > > >> @@ -55,8 +55,10 @@ struct device; > > > > >> > > > > >> #ifdef CONFIG_PM > > > > >> extern const char power_group_name[]; /* = "power" */ > > > > >> +#define pm_ops_ptr(_ptr) (_ptr) > > > > >> #else > > > > >> #define power_group_name NULL > > > > >> +#define pm_ops_ptr(_ptr) NULL > > > > >> #endif > > > > >> > > > > >> > > > > >> This patch was accepted by Rafael Wysocki, and will be merged to v3.11-rc1. > > > > >> > > > > > Lars-Peter please update with and > > > > > > > > Since the code depends on CONFIG_PM_SLEEP and not CONFIG_PM I don't think > > > > the macro will work. > > > > > > se please ad a similar macros too > > > > Hi Jean-Christophe, > > > > I added pm_sleep_ops_ptr() as below. > > Maybe you want it. > > There is no build warnings below 4 config situations. > > a) CONFIG_PM is enabled. > > b) only CONFIG_PM_SLEEP is enabled > > c) only CONFIG_PM_RUNTIME is enabled > > d) CONFIG_PM is NOT enabled. > > > > --- a/drivers/video/bfin-lq035q1-fb.c > > +++ b/drivers/video/bfin-lq035q1-fb.c > > @@ -170,16 +170,19 @@ static int lq035q1_spidev_remove(struct spi_device *spi) > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > } > > > > -#ifdef CONFIG_PM > > -static int lq035q1_spidev_suspend(struct spi_device *spi, pm_message_t state) > > +#ifdef CONFIG_PM_SLEEP > > +static int lq035q1_spidev_suspend(struct device *dev) > > { > > + struct spi_device *spi = to_spi_device(dev); > > + > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_SHUT); > > } > > > > -static int lq035q1_spidev_resume(struct spi_device *spi) > > +static int lq035q1_spidev_resume(struct device *dev) > > { > > - int ret; > > + struct spi_device *spi = to_spi_device(dev); > > struct spi_control *ctl = spi_get_drvdata(spi); > > + int ret; > > > > ret = lq035q1_control(spi, LQ035_DRIVER_OUTPUT_CTL, ctl->mode); > > if (ret) > > @@ -187,11 +190,11 @@ static int lq035q1_spidev_resume(struct spi_device *spi) > > > > return lq035q1_control(spi, LQ035_SHUT_CTL, LQ035_ON); > > } > > -#else > > -# define lq035q1_spidev_suspend NULL > > -# define lq035q1_spidev_resume NULL > > #endif > > > > +static SIMPLE_DEV_PM_OPS(lq035q1_spidev_pm_ops, lq035q1_spidev_suspend, > > + lq035q1_spidev_resume); > > + > > /* Power down all displays on reboot, poweroff or halt */ > > static void lq035q1_spidev_shutdown(struct spi_device *spi) > > { > > @@ -708,8 +711,7 @@ static int bfin_lq035q1_probe(struct platform_device *pdev) > > info->spidrv.probe = lq035q1_spidev_probe; > > info->spidrv.remove = lq035q1_spidev_remove; > > info->spidrv.shutdown = lq035q1_spidev_shutdown; > > - info->spidrv.suspend = lq035q1_spidev_suspend; > > - info->spidrv.resume = lq035q1_spidev_resume; > > + info->spidrv.driver.pm = pm_sleep_ops_ptr(&lq035q1_spidev_pm_ops); > > > > ret = spi_register_driver(&info->spidrv); > > if (ret < 0) { > > diff --git a/include/linux/pm.h b/include/linux/pm.h > > index bd50d15..999d652 100644 > > --- a/include/linux/pm.h > > +++ b/include/linux/pm.h > > @@ -61,6 +61,12 @@ extern const char power_group_name[]; /* = "power" */ > > #define pm_ops_ptr(_ptr) NULL > > #endif > > > > +#ifdef CONFIG_PM_SLEEP > > +#define pm_sleep_ops_ptr(_ptr) (_ptr) > > +#else > > +#define pm_sleep_ops_ptr(_ptr) NULL > > +#endif > > + > > > > > > Hi Rafael J. Wysocki, > > How about adding this pm_sleep_ops_ptr() macro? > > Are you submitting this instead or in addition to your patch I have in the > linux-next branch? Hi Rafael, I will submit this patch, soon. Thanks. :) Best regards, Jingoo Han > > Rafael > > > > My draft idea is below: > > However, I want other's better ideas. :) > > > > 1. The case that only CONFIG_PM_SLEEP is necessary. > > #ifdef CONFIG_PM_SLEEP > > static int *_suspend(struct device *dev) > > [....] > > static int *_resume(struct device *dev) > > [....] > > #endif > > > > static SIMPLE_DEV_PM_OPS(*_pm_ops, *_suspend, *_resume); > > > > static struct {platform/spi/i2c/etc}_driver *_driver = { > > .driver = { > > .pm = pm_sleep_ops_ptr(&*_pm_ops), > > > > > > 2. The case that both CONFIG_PM_SLEEP & CONFIG_PM_RUNTIME > > are necessary. > > > > #ifdef CONFIG_PM_SLEEP > > static int *_suspend(struct device *dev) > > [....] > > static int *_resume(struct device *dev) > > [....] > > #endif > > > > #ifdef CONFIG_PM_RUNTIME > > static int *_runtime_suspend(struct device *dev) > > [....] > > static int *_runtime_resume(struct device *dev) > > [....] > > #endif > > > > static const struct dev_pm_ops *_pm_ops = { > > SET_SYSTEM_SLEEP_PM_OPS(*_suspend, *_resume) > > SET_RUNTIME_PM_OPS(*_runtime_suspend, *_runtime_resume, NULL) > > }; > > > > static struct {platform/spi/i2c/etc}_driver *_driver = { > > .driver = { > > .pm = pm_ops_ptr(&*_pm_ops), > > > > > > > > Best regards, > > Jingoo Han > > > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-pm" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- > I speak only for myself. > Rafael J. Wysocki, Intel Open Source Technology Center. ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2013-06-03 5:15 UTC | newest] Thread overview: 13+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-05-29 12:17 [PATCH] fbdev: bfin-lq035q1-fb: Use dev_pm_ops Lars-Peter Clausen 2013-05-29 14:24 ` Michael Hennerich 2013-05-29 19:20 ` Jean-Christophe PLAGNIOL-VILLARD 2013-05-29 21:36 ` Rafael J. Wysocki 2013-05-30 0:32 ` Jingoo Han 2013-05-30 7:14 ` Jean-Christophe PLAGNIOL-VILLARD 2013-05-30 7:52 ` Lars-Peter Clausen 2013-05-30 10:45 ` Jean-Christophe PLAGNIOL-VILLARD 2013-05-30 16:32 ` Lars-Peter Clausen 2013-05-31 3:32 ` Jingoo Han 2013-05-31 5:01 ` Jean-Christophe PLAGNIOL-VILLARD 2013-06-02 19:30 ` Rafael J. Wysocki 2013-06-03 5:15 ` Jingoo Han
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).