All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] tc1100-wmi: fix build warning when CONFIG_PM not enabled
@ 2016-01-06 18:02 Colin King
  2016-01-11 18:31 ` Darren Hart
  0 siblings, 1 reply; 2+ messages in thread
From: Colin King @ 2016-01-06 18:02 UTC (permalink / raw)
  To: Darren Hart, platform-driver-x86; +Cc: linux-kernel

From: Colin Ian King <colin.king@canonical.com>

Conditionally declare suspend_data on CONFIG_PM to avoid
the following warning when CONFIG_OM is not enabled:

drivers/platform/x86/tc1100-wmi.c:55:27: warning:
  'suspend_data' defined but not used [-Wunused-variable]

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 drivers/platform/x86/tc1100-wmi.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/platform/x86/tc1100-wmi.c b/drivers/platform/x86/tc1100-wmi.c
index 89aa976..65b0a48 100644
--- a/drivers/platform/x86/tc1100-wmi.c
+++ b/drivers/platform/x86/tc1100-wmi.c
@@ -52,7 +52,9 @@ struct tc1100_data {
 	u32 jogdial;
 };
 
+#ifdef CONFIG_PM
 static struct tc1100_data suspend_data;
+#endif
 
 /* --------------------------------------------------------------------------
 				Device Management
-- 
2.7.0.rc3


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] tc1100-wmi: fix build warning when CONFIG_PM not enabled
  2016-01-06 18:02 [PATCH] tc1100-wmi: fix build warning when CONFIG_PM not enabled Colin King
@ 2016-01-11 18:31 ` Darren Hart
  0 siblings, 0 replies; 2+ messages in thread
From: Darren Hart @ 2016-01-11 18:31 UTC (permalink / raw)
  To: Colin King; +Cc: platform-driver-x86, linux-kernel

On Wed, Jan 06, 2016 at 06:02:59PM +0000, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
> 
> Conditionally declare suspend_data on CONFIG_PM to avoid
> the following warning when CONFIG_OM is not enabled:
> 
> drivers/platform/x86/tc1100-wmi.c:55:27: warning:
>   'suspend_data' defined but not used [-Wunused-variable]
> 
> Signed-off-by: Colin Ian King <colin.king@canonical.com>

Queued, thank you.

-- 
Darren Hart
Intel Open Source Technology Center

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-01-11 18:31 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-06 18:02 [PATCH] tc1100-wmi: fix build warning when CONFIG_PM not enabled Colin King
2016-01-11 18:31 ` Darren Hart

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.