Skip to content

Commit

Permalink
ACPI / video: mark acpi_video_get_levels() inline
Browse files Browse the repository at this point in the history
A recent patch added a stub function for acpi_video_get_levels when
CONFIG_ACPI_VIDEO is disabled. However, this is marked as 'static'
and causes a warning about an unused function whereever the header
gets included:

In file included from ../drivers/gpu/drm/radeon/radeon_acpi.c:28:0:
include/acpi/video.h:74:12: error: 'acpi_video_get_levels' defined but not used [-Werror=unused-function]

This makes the declaration 'static inline', which gets rid of the
warning.

Signed-off-by: Arnd Bergmann <[email protected]>
Fixes: 0595009 (ACPI/video: export acpi_video_get_levels)
Signed-off-by: Rafael J. Wysocki <[email protected]>
  • Loading branch information
arndb authored and rafaeljw committed May 13, 2016
1 parent a3c8933 commit e4f35c1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion include/acpi/video.h
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ static inline bool acpi_video_handles_brightness_key_presses(void)
{
return false;
}
static int acpi_video_get_levels(struct acpi_device *device,
static inline int acpi_video_get_levels(struct acpi_device *device,
struct acpi_video_device_brightness **dev_br)
{
return -ENODEV;
Expand Down

0 comments on commit e4f35c1

Please sign in to comment.