[PATCH] input: fix misc/twl6040-vibra.c warning

From: Randy Dunlap
Date: Mon Aug 29 2011 - 15:50:45 EST


From: Randy Dunlap <rdunlap@xxxxxxxxxxxx>

Fix warning from Geert's build summary emails by changing "if" to
"ifdef". Also handle the case of CONFIG_PM_SLEEP not enabled.

drivers/input/misc/twl6040-vibra.c:231:5: warning: "CONFIG_PM_SLEEP" is not defined

Builds cleanly with CONFIG_PM_SLEEP enabled or disabled.

Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
---
drivers/input/misc/twl6040-vibra.c | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)

--- linux-next-20110829.orig/drivers/input/misc/twl6040-vibra.c
+++ linux-next-20110829/drivers/input/misc/twl6040-vibra.c
@@ -228,7 +228,7 @@ static void twl6040_vibra_close(struct i
mutex_unlock(&info->mutex);
}

-#if CONFIG_PM_SLEEP
+#ifdef CONFIG_PM_SLEEP
static int twl6040_vibra_suspend(struct device *dev)
{
struct platform_device *pdev = to_platform_device(dev);
@@ -244,10 +244,13 @@ static int twl6040_vibra_suspend(struct
return 0;
}

-#endif
-
static SIMPLE_DEV_PM_OPS(twl6040_vibra_pm_ops, twl6040_vibra_suspend, NULL);

+#define TWL6040_VIBRA_PM_OPS (&twl6040_vibra_pm_ops)
+#else
+#define TWL6040_VIBRA_PM_OPS NULL
+#endif
+
static int __devinit twl6040_vibra_probe(struct platform_device *pdev)
{
struct twl4030_vibra_data *pdata = pdev->dev.platform_data;
@@ -400,7 +403,7 @@ static struct platform_driver twl6040_vi
.driver = {
.name = "twl6040-vibra",
.owner = THIS_MODULE,
- .pm = &twl6040_vibra_pm_ops,
+ .pm = TWL6040_VIBRA_PM_OPS,
},
};

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/