[coreboot-gerrit] Change in coreboot[master]: lenovo/x230: introduce FHD variant

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/28950 )

Change subject: lenovo/x230: introduce FHD variant
......................................................................

Patch Set 12:
> Patch Set 12:
>
> Sorry about omitting OS info -- tested on Linux 5.7.11 (Arch, same behavior on KDE Plasma, Gnome, and LXDE)
> > Patch Set 12:
> > > Patch Set 12:
> > >
> > > I'm seeing an issue where zero-brightness (Backlight off) is being treated as full brightness. So instead of turning off the backlight when the lid is closed, it's set to max.
> > >
> > > Tested with patchset 12 on master @ e2f5fb25
> >
> > Which OS did you test this on?

Intel backlight handling on Linux is rather messed up. There's multiple ways to do it, and there's quirk handling (which is itself a quirk).