[PATCH 1/3] spi: bcm63xx-hsspi: Use SIMPLE_DEV_PM_OPS macro

[PATCH 1/3] spi: bcm63xx-hsspi: Use SIMPLE_DEV_PM_OPS macro

[Jingoo Han]

Use SIMPLE_DEV_PM_OPS macro in order to make the code simpler.

Signed-off-by: Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
---
 drivers/spi/spi-bcm63xx-hsspi.c |    5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/spi/spi-bcm63xx-hsspi.c b/drivers/spi/spi-bcm63xx-hsspi.c
index 3ad3e0c..5a211e9 100644
--- a/drivers/spi/spi-bcm63xx-hsspi.c
+++ b/drivers/spi/spi-bcm63xx-hsspi.c
@@ -454,9 +454,8 @@ static int bcm63xx_hsspi_resume(struct device *dev)
 }
 #endif
 
-static const struct dev_pm_ops bcm63xx_hsspi_pm_ops = {
-	SET_SYSTEM_SLEEP_PM_OPS(bcm63xx_hsspi_suspend, bcm63xx_hsspi_resume)
-};
+static SIMPLE_DEV_PM_OPS(bcm63xx_hsspi_pm_ops, bcm63xx_hsspi_suspend,
+			 bcm63xx_hsspi_resume);
 
 static struct platform_driver bcm63xx_hsspi_driver = {
 	.driver = {
-- 
1.7.10.4


--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 2/3] spi: fsl-dspi: Use SIMPLE_DEV_PM_OPS macro

[Jingoo Han]

Use SIMPLE_DEV_PM_OPS macro in order to make the code simpler.

Signed-off-by: Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
---
 drivers/spi/spi-fsl-dspi.c |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/spi/spi-fsl-dspi.c b/drivers/spi/spi-fsl-dspi.c
index 42ede0d..7f0dddb 100644
--- a/drivers/spi/spi-fsl-dspi.c
+++ b/drivers/spi/spi-fsl-dspi.c
@@ -437,9 +437,7 @@ static int dspi_resume(struct device *dev)
 }
 #endif /* CONFIG_PM_SLEEP */
 
-static const struct dev_pm_ops dspi_pm = {
-	SET_SYSTEM_SLEEP_PM_OPS(dspi_suspend, dspi_resume)
-};
+static SIMPLE_DEV_PM_OPS(dspi_pm, dspi_suspend, dspi_resume);
 
 static struct regmap_config dspi_regmap_config = {
 	.reg_bits = 32,
-- 
1.7.10.4


--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/3] spi: fsl-dspi: Use SIMPLE_DEV_PM_OPS macro

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 143 bytes --]

On Wed, Feb 26, 2014 at 10:30:14AM +0900, Jingoo Han wrote:
> Use SIMPLE_DEV_PM_OPS macro in order to make the code simpler.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[PATCH 3/3] spi: sirf: Use SIMPLE_DEV_PM_OPS macro

[Jingoo Han]

Use SIMPLE_DEV_PM_OPS macro in order to make the code simpler.

Signed-off-by: Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
---
 drivers/spi/spi-sirf.c |    5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/spi/spi-sirf.c b/drivers/spi/spi-sirf.c
index 5210b94..dc962a1 100644
--- a/drivers/spi/spi-sirf.c
+++ b/drivers/spi/spi-sirf.c
@@ -718,9 +718,8 @@ static int spi_sirfsoc_resume(struct device *dev)
 }
 #endif
 
-static const struct dev_pm_ops spi_sirfsoc_pm_ops = {
-	SET_SYSTEM_SLEEP_PM_OPS(spi_sirfsoc_suspend, spi_sirfsoc_resume)
-};
+static SIMPLE_DEV_PM_OPS(spi_sirfsoc_pm_ops, spi_sirfsoc_suspend,
+			 spi_sirfsoc_resume);
 
 static const struct of_device_id spi_sirfsoc_of_match[] = {
 	{ .compatible = "sirf,prima2-spi", },
-- 
1.7.10.4


--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 3/3] spi: sirf: Use SIMPLE_DEV_PM_OPS macro

[Barry Song]

2014-02-26 9:32 GMT+08:00 Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>:
> Use SIMPLE_DEV_PM_OPS macro in order to make the code simpler.
>
> Signed-off-by: Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>

thanks, but this has been done and merged.

> ---
>  drivers/spi/spi-sirf.c |    5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/spi/spi-sirf.c b/drivers/spi/spi-sirf.c
> index 5210b94..dc962a1 100644
> --- a/drivers/spi/spi-sirf.c
> +++ b/drivers/spi/spi-sirf.c
> @@ -718,9 +718,8 @@ static int spi_sirfsoc_resume(struct device *dev)
>  }
>  #endif
>
> -static const struct dev_pm_ops spi_sirfsoc_pm_ops = {
> -       SET_SYSTEM_SLEEP_PM_OPS(spi_sirfsoc_suspend, spi_sirfsoc_resume)
> -};
> +static SIMPLE_DEV_PM_OPS(spi_sirfsoc_pm_ops, spi_sirfsoc_suspend,
> +                        spi_sirfsoc_resume);
>
>  static const struct of_device_id spi_sirfsoc_of_match[] = {
>         { .compatible = "sirf,prima2-spi", },
> --
> 1.7.10.4
>
>
-barry
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 3/3] spi: sirf: Use SIMPLE_DEV_PM_OPS macro

[Jingoo Han]

On Wednesday, February 26, 2014 5:32 PM, Barry Song wrote:
> 2014-02-26 9:32 GMT+08:00 Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>:
> > Use SIMPLE_DEV_PM_OPS macro in order to make the code simpler.
> >
> > Signed-off-by: Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
> 
> thanks, but this has been done and merged.

Thank you for your comment. However I have a question. :-)

My patch replaces 'SET_SYSTEM_SLEEP_PM_OPS' with 'SIMPLE_DEV_PM_OPS'.
However, I cannot find the same patch in SPI mailing-list.
Would you let me know how to find the same patch that was submitted
earlier?

Best regards,
Jingoo Han

> 
> > ---
> >  drivers/spi/spi-sirf.c |    5 ++---
> >  1 file changed, 2 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/spi/spi-sirf.c b/drivers/spi/spi-sirf.c
> > index 5210b94..dc962a1 100644
> > --- a/drivers/spi/spi-sirf.c
> > +++ b/drivers/spi/spi-sirf.c
> > @@ -718,9 +718,8 @@ static int spi_sirfsoc_resume(struct device *dev)
> >  }
> >  #endif
> >
> > -static const struct dev_pm_ops spi_sirfsoc_pm_ops = {
> > -       SET_SYSTEM_SLEEP_PM_OPS(spi_sirfsoc_suspend, spi_sirfsoc_resume)
> > -};
> > +static SIMPLE_DEV_PM_OPS(spi_sirfsoc_pm_ops, spi_sirfsoc_suspend,
> > +                        spi_sirfsoc_resume);
> >
> >  static const struct of_device_id spi_sirfsoc_of_match[] = {
> >         { .compatible = "sirf,prima2-spi", },
> > --
> > 1.7.10.4
> >
> >
> -barry

--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 3/3] spi: sirf: Use SIMPLE_DEV_PM_OPS macro

[Barry Song]

2014-02-26 16:55 GMT+08:00 Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>:
> On Wednesday, February 26, 2014 5:32 PM, Barry Song wrote:
>> 2014-02-26 9:32 GMT+08:00 Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>:
>> > Use SIMPLE_DEV_PM_OPS macro in order to make the code simpler.
>> >
>> > Signed-off-by: Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
>>
>> thanks, but this has been done and merged.
>
> Thank you for your comment. However I have a question. :-)
>
> My patch replaces 'SET_SYSTEM_SLEEP_PM_OPS' with 'SIMPLE_DEV_PM_OPS'.
> However, I cannot find the same patch in SPI mailing-list.
> Would you let me know how to find the same patch that was submitted
> earlier?

sorry, i misunderstood your patch.

Acked-by: Barry Song <Baohua.Song-kQvG35nSl+M@public.gmane.org>

thanks a lot!

>
> Best regards,
> Jingoo Han
>
>>
>> > ---
>> >  drivers/spi/spi-sirf.c |    5 ++---
>> >  1 file changed, 2 insertions(+), 3 deletions(-)
>> >
>> > diff --git a/drivers/spi/spi-sirf.c b/drivers/spi/spi-sirf.c
>> > index 5210b94..dc962a1 100644
>> > --- a/drivers/spi/spi-sirf.c
>> > +++ b/drivers/spi/spi-sirf.c
>> > @@ -718,9 +718,8 @@ static int spi_sirfsoc_resume(struct device *dev)
>> >  }
>> >  #endif
>> >
>> > -static const struct dev_pm_ops spi_sirfsoc_pm_ops = {
>> > -       SET_SYSTEM_SLEEP_PM_OPS(spi_sirfsoc_suspend, spi_sirfsoc_resume)
>> > -};
>> > +static SIMPLE_DEV_PM_OPS(spi_sirfsoc_pm_ops, spi_sirfsoc_suspend,
>> > +                        spi_sirfsoc_resume);
>> >
>> >  static const struct of_device_id spi_sirfsoc_of_match[] = {
>> >         { .compatible = "sirf,prima2-spi", },
>> > --
>> > 1.7.10.4
>> >
>> >
>> -barry


-barry
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 3/3] spi: sirf: Use SIMPLE_DEV_PM_OPS macro

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 143 bytes --]

On Wed, Feb 26, 2014 at 10:32:48AM +0900, Jingoo Han wrote:
> Use SIMPLE_DEV_PM_OPS macro in order to make the code simpler.

applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH 1/3] spi: bcm63xx-hsspi: Use SIMPLE_DEV_PM_OPS macro

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 143 bytes --]

On Wed, Feb 26, 2014 at 10:28:24AM +0900, Jingoo Han wrote:
> Use SIMPLE_DEV_PM_OPS macro in order to make the code simpler.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]