From mboxrd@z Thu Jan 1 00:00:00 1970 From: Krzysztof Kozlowski Date: Tue, 26 Jan 2016 00:46:26 +0000 Subject: Re: [PATCH] fbdev: s6e8ax0: avoid unused function warnings Message-Id: <56A6C1E2.7060804@samsung.com> List-Id: References: <1453734300-1958304-1-git-send-email-arnd@arndb.de> In-Reply-To: <1453734300-1958304-1-git-send-email-arnd@arndb.de> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org On 26.01.2016 00:04, Arnd Bergmann wrote: > The s6e8ax0 suspend/resume functions are hidden inside of an #ifdef > when CONFIG_PM is set to avoid unused function warnings, but they > call some other functions that nothing else calls, and we get warnings > about those: > > drivers/video/fbdev/exynos/s6e8ax0.c:449:13: error: 's6e8ax0_sleep_in' defined but not used [-Werror=unused-function] > drivers/video/fbdev/exynos/s6e8ax0.c:485:13: error: 's6e8ax0_display_off' defined but not used [-Werror=unused-function] > > This marks the PM functions as __maybe_unused so the compiler can > silently drop them when they are not referenced. > > Signed-off-by: Arnd Bergmann > --- > drivers/video/fbdev/exynos/s6e8ax0.c | 13 ++++--------- > 1 file changed, 4 insertions(+), 9 deletions(-) > Reviewed-by: Krzysztof Kozlowski Best regards, Krzysztof