On Wed, Aug 14, 2024 at 10:10 PM Andy Shevchenko <andy@xxxxxxxxxx> wrote: > > On Wed, Aug 14, 2024 at 09:01:56PM +0200, Hans de Goede wrote: > > Hi Rafael, > > > > 6.10 has a new backlight driver for UART attached backlight controller > > boards found in some Dell All in One models. > > > > Now the first AIO has turned up which has not only the DSDT bits for this, > > but also an actual controller attached to the UART, yet it is not using > > this controller for backlight control (it needs GPU native control). > > > > I did not tie the dell-uart-backlight into acpi_video_get_backlight_type() > > yet, so the first 2 patches in this series deal with that and the third > > patch adds a DMI quirk to select native backlight control on top. > > > > Backlight control used to work on the Dell OptiPlex 7760 AIO with kernel > > 6.9 and older, so this is a regression and I would like to see this > > series merged as fixes for 6.11. > > > > Rafael, the drivers/platform/x86/dell/dell-uart-backlight.c are small > > and isolated. So I believe it is best if you take the entire series, > > to avoid conflicts if any other drivers/apci/video_detect.c DMI quirks > > show up this cycle. > > FWIW, > Reviewed-by: Andy Shevchenko <andy@xxxxxxxxxx> > > I find this quite small and I agree this would be nice to have as fixes. Applied as fixes for 6.11-rc, thanks!
