Hi Rafael, Matthew, Here is a second attempt at always registering only a single /sys/class/backlight device per panel. This first round of testing has shown that native works well even on systems so old that the don't have acpi_video backlight control support. This patch series makes native be preferred over vendor, which should avoid the problems seen with the 6.1 changes before the fixes. ATM there is one known model where this will cause a regression, the Sony Vaio PCG-FRV3 from 2003. I plan to add a DMI quirk for that in the next version of this series, but I'm waiting for some more testing (to check that the vendor interface does actually work) first. I will also do another blogpost, focussing on asking users to see if they have a laptop which provides a combination of vendor + native backlight interfaces, which may be impacted by this series. This is the main reason why this is a RFC for now. Note this applies on top of my series with fixes for 6.1: https://lore.kernel.org/linux-acpi/20221104212108.73537-1-hdegoede@xxxxxxxxxx/ So we need to be a bit careful with merging this to avoid non trivial conflicts. Assuming that series gets send to Linus for 6.1 through the pdx86 tree, it might be best for me to rebase pdx86/for-next on top of the fixes and then this can (also) be merged through pdx86/for-next without conflicts. Regards, Hans p.s. Matthew, this should keep your custom coreboot laptop working since acpi_video_backlight_use_native() will return true there now. Hans de Goede (2): ACPI: video: Simplify __acpi_video_get_backlight_type() ACPI: video: prefer native over vendor drivers/acpi/video_detect.c | 52 ++++++++++++++----------------------- 1 file changed, 19 insertions(+), 33 deletions(-) -- 2.37.3
