Re: [PATCH 04/15] acer-wmi: Hide acer_{suspend,resume} for !CONFIG_PM_SLEEP

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jul 16, 2014 at 07:43:06PM +0200, Mathias Krause wrote:
> Encapsulate acer_suspend() and acer_resume with #ifdef CONFIG_PM_SLEEP
> to get rid of the following warnings:
> 
>   ../acer-wmi.c:2046:12: warning: ‘acer_suspend’ defined but not used [-Wunused-function]
>   ../acer-wmi.c:2068:12: warning: ‘acer_resume’ defined but not used [-Wunused-function]
> 
> Signed-off-by: Mathias Krause <minipli@xxxxxxxxxxxxxx>
> Cc: "Lee, Chun-Yi" <jlee@xxxxxxxx>

Signed-off-by: "Lee, Chun-Yi" <jlee@xxxxxxxx>


Thanks a lot!
Joey Lee

> ---
>  drivers/platform/x86/acer-wmi.c |    5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/drivers/platform/x86/acer-wmi.c b/drivers/platform/x86/acer-wmi.c
> index fde69ee63f..0d18c23d66 100644
> --- a/drivers/platform/x86/acer-wmi.c
> +++ b/drivers/platform/x86/acer-wmi.c
> @@ -2044,6 +2044,7 @@ static int acer_platform_remove(struct platform_device *device)
>  	return 0;
>  }
>  
> +#ifdef CONFIG_PM_SLEEP
>  static int acer_suspend(struct device *dev)
>  {
>  	u32 value;
> @@ -2084,6 +2085,10 @@ static int acer_resume(struct device *dev)
>  
>  	return 0;
>  }
> +#else
> +#define acer_suspend	NULL
> +#define acer_resume	NULL
> +#endif
>  
>  static SIMPLE_DEV_PM_OPS(acer_pm, acer_suspend, acer_resume);
>  
> -- 
> 1.7.10.4
> 
--
To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux