diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2016-05-12 23:00:12 +0200 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2016-05-13 23:57:02 +0200 |
| commit | e4f35c1339f0cfcf38d3f63dd6fea2b070399263 (patch) | |
| tree | 12efac63f188d986d8cf5531efb9b0ff9548df57 /include/acpi | |
| parent | a3c89334f06bacf7e2f23203ad64cfd6e78dbbf4 (diff) | |
| download | lwn-e4f35c1339f0cfcf38d3f63dd6fea2b070399263.tar.gz lwn-e4f35c1339f0cfcf38d3f63dd6fea2b070399263.zip | |
ACPI / video: mark acpi_video_get_levels() inline
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 <arnd@arndb.de>
Fixes: 059500940def (ACPI/video: export acpi_video_get_levels)
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'include/acpi')
| -rw-r--r-- | include/acpi/video.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/acpi/video.h b/include/acpi/video.h index a4b96c971564..70a41f742037 100644 --- a/include/acpi/video.h +++ b/include/acpi/video.h @@ -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; |