| Message ID | 20170609103328.3130682-1-arnd@arndb.de |
|---|---|
| State | Accepted |
| Commit | d4c9354b4015aa8d1b032c368682dd20a00f4e1f |
| Headers | show |
On Fri, Jun 09, 2017 at 12:32:48PM +0200, Arnd Bergmann wrote: > We get a harmless warning when CONFIG_RUNTIME_PM is disabled: > > drivers/input/touchscreen/stmfts.c:760:12: error: 'stmfts_runtime_resume' defined but not used [-Werror=unused-function] > static int stmfts_runtime_resume(struct device *dev) > drivers/input/touchscreen/stmfts.c:748:12: error: 'stmfts_runtime_suspend' defined but not used [-Werror=unused-function] > static int stmfts_runtime_suspend(struct device *dev) > > The regular PM functions are already marked as __maybe_unused, so let's > do the same for the runtime-PM as well. > > Fixes: 78bcac7b2ae1 ("Input: add support for the STMicroelectronics FingerTip touchscreen") > Signed-off-by: Arnd Bergmann <arnd@arndb.de> Applied, thank you. > --- > drivers/input/touchscreen/stmfts.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/input/touchscreen/stmfts.c b/drivers/input/touchscreen/stmfts.c > index 802d0e82d034..157fdb4bb2e8 100644 > --- a/drivers/input/touchscreen/stmfts.c > +++ b/drivers/input/touchscreen/stmfts.c > @@ -745,7 +745,7 @@ static int stmfts_remove(struct i2c_client *client) > return 0; > } > > -static int stmfts_runtime_suspend(struct device *dev) > +static int __maybe_unused stmfts_runtime_suspend(struct device *dev) > { > struct stmfts_data *sdata = dev_get_drvdata(dev); > int ret; > @@ -757,7 +757,7 @@ static int stmfts_runtime_suspend(struct device *dev) > return ret; > } > > -static int stmfts_runtime_resume(struct device *dev) > +static int __maybe_unused stmfts_runtime_resume(struct device *dev) > { > struct stmfts_data *sdata = dev_get_drvdata(dev); > int ret; > -- > 2.9.0 > -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/input/touchscreen/stmfts.c b/drivers/input/touchscreen/stmfts.c index 802d0e82d034..157fdb4bb2e8 100644 --- a/drivers/input/touchscreen/stmfts.c +++ b/drivers/input/touchscreen/stmfts.c @@ -745,7 +745,7 @@ static int stmfts_remove(struct i2c_client *client) return 0; } -static int stmfts_runtime_suspend(struct device *dev) +static int __maybe_unused stmfts_runtime_suspend(struct device *dev) { struct stmfts_data *sdata = dev_get_drvdata(dev); int ret; @@ -757,7 +757,7 @@ static int stmfts_runtime_suspend(struct device *dev) return ret; } -static int stmfts_runtime_resume(struct device *dev) +static int __maybe_unused stmfts_runtime_resume(struct device *dev) { struct stmfts_data *sdata = dev_get_drvdata(dev); int ret;
We get a harmless warning when CONFIG_RUNTIME_PM is disabled: drivers/input/touchscreen/stmfts.c:760:12: error: 'stmfts_runtime_resume' defined but not used [-Werror=unused-function] static int stmfts_runtime_resume(struct device *dev) drivers/input/touchscreen/stmfts.c:748:12: error: 'stmfts_runtime_suspend' defined but not used [-Werror=unused-function] static int stmfts_runtime_suspend(struct device *dev) The regular PM functions are already marked as __maybe_unused, so let's do the same for the runtime-PM as well. Fixes: 78bcac7b2ae1 ("Input: add support for the STMicroelectronics FingerTip touchscreen") Signed-off-by: Arnd Bergmann <arnd@arndb.de> --- drivers/input/touchscreen/stmfts.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- 2.9.0 -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html