[PATCH] ata/pata_arasan_cf: Move arasan_cf_pm_ops out of #ifdef, #endif macros

[PATCH] ata/pata_arasan_cf: Move arasan_cf_pm_ops out of #ifdef, #endif macros

[Viresh Kumar]

#ifdef, #endif is not required in definition/usage of arasan_cf_pm_ops. So, move
this definition and its usage outside of them.

Signed-off-by: Viresh Kumar <viresh.kumar@st.com>
---
 drivers/ata/pata_arasan_cf.c |    4 +---
 1 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/drivers/ata/pata_arasan_cf.c b/drivers/ata/pata_arasan_cf.c
index fc2db2a..3239517 100644
--- a/drivers/ata/pata_arasan_cf.c
+++ b/drivers/ata/pata_arasan_cf.c
@@ -943,9 +943,9 @@ static int arasan_cf_resume(struct device *dev)
 
 	return 0;
 }
+#endif
 
 static SIMPLE_DEV_PM_OPS(arasan_cf_pm_ops, arasan_cf_suspend, arasan_cf_resume);
-#endif
 
 static struct platform_driver arasan_cf_driver = {
 	.probe		= arasan_cf_probe,
@@ -953,9 +953,7 @@ static struct platform_driver arasan_cf_driver = {
 	.driver		= {
 		.name	= DRIVER_NAME,
 		.owner	= THIS_MODULE,
-#ifdef CONFIG_PM
 		.pm	= &arasan_cf_pm_ops,
-#endif
 	},
 };
 
-- 
1.7.8.110.g4cb5d

Re: [PATCH] ata/pata_arasan_cf: Move arasan_cf_pm_ops out of #ifdef, #endif macros

[Viresh Kumar]

On 3/1/2012 1:34 PM, Viresh KUMAR wrote:
> #ifdef, #endif is not required in definition/usage of arasan_cf_pm_ops. So, move
> this definition and its usage outside of them.
> 
> Signed-off-by: Viresh Kumar <viresh.kumar@st.com>
> ---
>  drivers/ata/pata_arasan_cf.c |    4 +---
>  1 files changed, 1 insertions(+), 3 deletions(-)

Hi Jeff,

Can you please apply this patch.

-- 
viresh

Re: [PATCH] ata/pata_arasan_cf: Move arasan_cf_pm_ops out of #ifdef, #endif macros

[Viresh Kumar]

On 3/1/2012 1:34 PM, Viresh KUMAR wrote:
> #ifdef, #endif is not required in definition/usage of arasan_cf_pm_ops. So, move
> this definition and its usage outside of them.
> 
> Signed-off-by: Viresh Kumar <viresh.kumar@st.com>
> ---
>  drivers/ata/pata_arasan_cf.c |    4 +---
>  1 files changed, 1 insertions(+), 3 deletions(-)

Hi Jeff,

Can you please apply this patch?

-- 
viresh

Re: [PATCH] ata/pata_arasan_cf: Move arasan_cf_pm_ops out of #ifdef, #endif macros

[Jeff Garzik]

On 04/10/2012 05:32 AM, Viresh Kumar wrote:
> On 3/1/2012 1:34 PM, Viresh KUMAR wrote:
>> #ifdef, #endif is not required in definition/usage of arasan_cf_pm_ops. So, move
>> this definition and its usage outside of them.
>>
>> Signed-off-by: Viresh Kumar<viresh.kumar@st.com>
>> ---
>>   drivers/ata/pata_arasan_cf.c |    4 +---
>>   1 files changed, 1 insertions(+), 3 deletions(-)
>
> Hi Jeff,
>
> Can you please apply this patch?

This has been upstream for a while now.

Re: [PATCH] ata/pata_arasan_cf: Move arasan_cf_pm_ops out of #ifdef, #endif macros

[Viresh Kumar]

On 4/13/2012 1:37 AM, Jeff Garzik wrote:
> On 04/10/2012 05:32 AM, Viresh Kumar wrote:
>> On 3/1/2012 1:34 PM, Viresh KUMAR wrote:
>>> #ifdef, #endif is not required in definition/usage of arasan_cf_pm_ops. So, move
>>> this definition and its usage outside of them.
>>>
>>> Signed-off-by: Viresh Kumar<viresh.kumar@st.com>
>>> ---
>>>   drivers/ata/pata_arasan_cf.c |    4 +---
>>>   1 files changed, 1 insertions(+), 3 deletions(-)
>>
>> Hi Jeff,
>>
>> Can you please apply this patch?
> 
> This has been upstream for a while now.

I couldn't see this in linux-next/master :(

You pushed following patch earlier:

Author: Viresh Kumar <viresh.kumar@st.com>
Date:   Thu Feb 23 15:03:53 2012 +0530

    ata/pata_arasan_cf: Add Hibernation support


And this one is a fix over it.

-- 
viresh