]> git.itanic.dy.fi Git - linux-stable/commitdiff
ACPI: video: Add comment about acpi_video_backlight_use_native() usage
authorHans de Goede <hdegoede@redhat.com>
Thu, 26 Oct 2023 13:53:03 +0000 (15:53 +0200)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 20 Nov 2023 16:47:07 +0000 (17:47 +0100)
Add a comment explaining that acpi_video_backlight_use_native() MUST
only be used by GPU drivers and that it must NOT be used on other places.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
include/acpi/video.h

index 4230392b5b0b4f209636e7bbbdb3c8d04b6530ff..3d538d4178abb1a324adb8def3d5132abd71925c 100644 (file)
@@ -75,6 +75,15 @@ static inline enum acpi_backlight_type acpi_video_get_backlight_type(void)
        return __acpi_video_get_backlight_type(false, NULL);
 }
 
+/*
+ * This function MUST only be called by GPU drivers to check if the driver
+ * should register a backlight class device. This function not only checks
+ * if a GPU native backlight device should be registered it *also* tells
+ * the ACPI video-detect code that native GPU backlight control is available.
+ * Therefor calling this from any place other then the GPU driver is wrong!
+ * To check if GPU native backlight control is used in other places instead use:
+ *   if (acpi_video_get_backlight_type() == acpi_backlight_native) { ... }
+ */
 static inline bool acpi_video_backlight_use_native(void)
 {
        return __acpi_video_get_backlight_type(true, NULL) == acpi_backlight_native;