diff mbox series

mfd: rk808: mark pm functions __maybe_unused

Message ID 20190708125308.3778575-1-arnd@arndb.de
State Accepted
Commit 5752bc4373b21c3fb1dd6db4dcdd569fae391a1d
Headers show
Series mfd: rk808: mark pm functions __maybe_unused | expand

Commit Message

Arnd Bergmann July 8, 2019, 12:53 p.m. UTC
The newly added suspend/resume functions are only used if CONFIG_PM
is enabled:

drivers/mfd/rk808.c:752:12: error: 'rk8xx_resume' defined but not used [-Werror=unused-function]
drivers/mfd/rk808.c:732:12: error: 'rk8xx_suspend' defined but not used [-Werror=unused-function]

Mark them as __maybe_unused so the compiler can silently drop them
when they are not needed.

Fixes: 586c1b4125b3 ("mfd: rk808: Add RK817 and RK809 support")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>

---
 drivers/mfd/rk808.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

-- 
2.20.0

Comments

Lee Jones July 25, 2019, 12:02 p.m. UTC | #1
On Mon, 08 Jul 2019, Arnd Bergmann wrote:

> The newly added suspend/resume functions are only used if CONFIG_PM

> is enabled:

> 

> drivers/mfd/rk808.c:752:12: error: 'rk8xx_resume' defined but not used [-Werror=unused-function]

> drivers/mfd/rk808.c:732:12: error: 'rk8xx_suspend' defined but not used [-Werror=unused-function]

> 

> Mark them as __maybe_unused so the compiler can silently drop them

> when they are not needed.

> 

> Fixes: 586c1b4125b3 ("mfd: rk808: Add RK817 and RK809 support")

> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

> ---

>  drivers/mfd/rk808.c | 4 ++--

>  1 file changed, 2 insertions(+), 2 deletions(-)


Applied, thanks.

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
diff mbox series

Patch

diff --git a/drivers/mfd/rk808.c b/drivers/mfd/rk808.c
index 601cefb5c9d8..9a9e6315ba46 100644
--- a/drivers/mfd/rk808.c
+++ b/drivers/mfd/rk808.c
@@ -729,7 +729,7 @@  static int rk808_remove(struct i2c_client *client)
 	return 0;
 }
 
-static int rk8xx_suspend(struct device *dev)
+static int __maybe_unused rk8xx_suspend(struct device *dev)
 {
 	struct rk808 *rk808 = i2c_get_clientdata(rk808_i2c_client);
 	int ret = 0;
@@ -749,7 +749,7 @@  static int rk8xx_suspend(struct device *dev)
 	return ret;
 }
 
-static int rk8xx_resume(struct device *dev)
+static int __maybe_unused rk8xx_resume(struct device *dev)
 {
 	struct rk808 *rk808 = i2c_get_clientdata(rk808_i2c_client);
 	int ret = 0;