Re: [PATCH] platform/chrome: cros_ec_lightbar - hide unused PM functions

2017-06-27 Thread Benson Leung
Hi Arnd, On Tue, Jun 27, 2017 at 05:36:36PM +0200, Arnd Bergmann wrote: > The only reference to the new functions is inside of an #ifdef, > which now causes a harmless warning when CONFIG_PM_SLEEP is not set: > > chrome/cros_ec_dev.c:478:12: error: 'ec_device_resume' defined but not used > [-Wer

Re: [PATCH] platform/chrome: cros_ec_lightbar - hide unused PM functions

2017-06-27 Thread Guenter Roeck
On Tue, Jun 27, 2017 at 8:36 AM, Arnd Bergmann wrote: > The only reference to the new functions is inside of an #ifdef, > which now causes a harmless warning when CONFIG_PM_SLEEP is not set: > > chrome/cros_ec_dev.c:478:12: error: 'ec_device_resume' defined but not used > [-Werror=unused-function

[PATCH] platform/chrome: cros_ec_lightbar - hide unused PM functions

2017-06-27 Thread Arnd Bergmann
The only reference to the new functions is inside of an #ifdef, which now causes a harmless warning when CONFIG_PM_SLEEP is not set: chrome/cros_ec_dev.c:478:12: error: 'ec_device_resume' defined but not used [-Werror=unused-function] chrome/cros_ec_dev.c:469:12: error: 'ec_device_suspend' define