Hi,
On 7/13/22 19:21, Limonciello, Mario wrote:
> [Public]
>
>
>
>> -----Original Message-----
>> From: Werner Sembach <w...@tuxedocomputers.com>
>> Sent: Wednesday, July 13, 2022 12:08
>> To: Hans de Goede <hdego...@redhat.com>; Ben Skeggs
>> <bske...@redhat.com>; Karol Herbst <kher...@redhat.com>; Lyude
>> <ly...@redhat.com>; Daniel Dadap <dda...@nvidia.com>; Maarten
>> Lankhorst <maarten.lankho...@linux.intel.com>; Maxime Ripard
>> <mrip...@kernel.org>; Thomas Zimmermann <tzimmerm...@suse.de>;
>> Jani Nikula <jani.nik...@linux.intel.com>; Joonas Lahtinen
>> <joonas.lahti...@linux.intel.com>; Rodrigo Vivi <rodrigo.v...@intel.com>;
>> Tvrtko Ursulin <tvrtko.ursu...@linux.intel.com>; Deucher, Alexander
>> <alexander.deuc...@amd.com>; Koenig, Christian
>> <christian.koe...@amd.com>; p...@vger.kernel.org; Pan, Xinhui
>> <xinhui....@amd.com>; Rafael J . Wysocki <raf...@kernel.org>; Mika
>> Westerberg <mika.westerb...@linux.intel.com>; Lukas Wunner
>> <lu...@wunner.de>; Mark Gross <markgr...@kernel.org>; Andy
>> Shevchenko <a...@kernel.org>
>> Cc: nouv...@lists.freedesktop.org; Daniel Vetter <dan...@ffwll.ch>; David
>> Airlie <airl...@linux.ie>; intel-gfx <intel-...@lists.freedesktop.org>; dri-
>> de...@lists.freedesktop.org; amd-...@lists.freedesktop.org; Len Brown
>> <l...@kernel.org>; linux-a...@vger.kernel.org; platform-driver-
>> x...@vger.kernel.org
>> Subject: Re: [PATCH v2 27/29] ACPI: video: Drop Clevo/TUXEDO NL5xRU and
>> NL5xNU acpi_backlight=native quirks
>>
>> Hi,
>>
>> On 7/12/22 21:39, Hans de Goede wrote:
>>> acpi_backlight=native is the default for these, but as the comment
>>> explains the quirk was still necessary because even briefly registering
>>> the acpi_video0 backlight; and then unregistering it once the native
>>> driver showed up, was leading to issues.
>>>
>>> After the "ACPI: video: Make backlight class device registration
>>> a separate step" patch from earlier in this patch-series, we no
>>> longer briefly register the acpi_video0 backlight on systems where
>>> the native driver should be used.
>>>
>>> So this is no longer an issue an the quirks are no longer needed.
>>>
>>> Cc: Werner Sembach <w...@tuxedocomputers.com>
>>> Signed-off-by: Hans de Goede <hdego...@redhat.com>
>>
>> Tested and can confirm: The quirks are no longer needed with this Patchset.
>>
>> Tested-by: Werner Sembach <w...@tuxedocomputers.com>
>
> Probably should include this link tag in this commit too then as it fixes
> the Tong Fang systems too.
>
> Link: https://bugzilla.kernel.org/show_bug.cgi?id=215683
Good point, I've added this to the version in my personal tree.
Regards,
Hans
>
>>
>> Kind Regards,
>>
>> Werner Sembach
>>
>>> ---
>>> drivers/acpi/video_detect.c | 75 -------------------------------------
>>> 1 file changed, 75 deletions(-)
>>>
>>> diff --git a/drivers/acpi/video_detect.c b/drivers/acpi/video_detect.c
>>> index 2a4d376a703e..4b9395d1bda7 100644
>>> --- a/drivers/acpi/video_detect.c
>>> +++ b/drivers/acpi/video_detect.c
>>> @@ -599,81 +599,6 @@ static const struct dmi_system_id
>> video_detect_dmi_table[] = {
>>> DMI_MATCH(DMI_BOARD_NAME, "N250P"),
>>> },
>>> },
>>> - /*
>>> - * Clevo NL5xRU and NL5xNU/TUXEDO Aura 15 Gen1 and Gen2 have
>> both a
>>> - * working native and video interface. However the default detection
>>> - * mechanism first registers the video interface before unregistering
>>> - * it again and switching to the native interface during boot. This
>>> - * results in a dangling SBIOS request for backlight change for some
>>> - * reason, causing the backlight to switch to ~2% once per boot on
>> the
>>> - * first power cord connect or disconnect event. Setting the native
>>> - * interface explicitly circumvents this buggy behaviour, by avoiding
>>> - * the unregistering process.
>>> - */
>>> - {
>>> - .callback = video_detect_force_native,
>>> - .ident = "Clevo NL5xRU",
>>> - .matches = {
>>> - DMI_MATCH(DMI_SYS_VENDOR, "TUXEDO"),
>>> - DMI_MATCH(DMI_BOARD_NAME, "NL5xRU"),
>>> - },
>>> - },
>>> - {
>>> - .callback = video_detect_force_native,
>>> - .ident = "Clevo NL5xRU",
>>> - .matches = {
>>> - DMI_MATCH(DMI_SYS_VENDOR,
>> "SchenkerTechnologiesGmbH"),
>>> - DMI_MATCH(DMI_BOARD_NAME, "NL5xRU"),
>>> - },
>>> - },
>>> - {
>>> - .callback = video_detect_force_native,
>>> - .ident = "Clevo NL5xRU",
>>> - .matches = {
>>> - DMI_MATCH(DMI_SYS_VENDOR, "Notebook"),
>>> - DMI_MATCH(DMI_BOARD_NAME, "NL5xRU"),
>>> - },
>>> - },
>>> - {
>>> - .callback = video_detect_force_native,
>>> - .ident = "Clevo NL5xRU",
>>> - .matches = {
>>> - DMI_MATCH(DMI_SYS_VENDOR, "TUXEDO"),
>>> - DMI_MATCH(DMI_BOARD_NAME, "AURA1501"),
>>> - },
>>> - },
>>> - {
>>> - .callback = video_detect_force_native,
>>> - .ident = "Clevo NL5xRU",
>>> - .matches = {
>>> - DMI_MATCH(DMI_SYS_VENDOR, "TUXEDO"),
>>> - DMI_MATCH(DMI_BOARD_NAME, "EDUBOOK1502"),
>>> - },
>>> - },
>>> - {
>>> - .callback = video_detect_force_native,
>>> - .ident = "Clevo NL5xNU",
>>> - .matches = {
>>> - DMI_MATCH(DMI_SYS_VENDOR, "TUXEDO"),
>>> - DMI_MATCH(DMI_BOARD_NAME, "NL5xNU"),
>>> - },
>>> - },
>>> - {
>>> - .callback = video_detect_force_native,
>>> - .ident = "Clevo NL5xNU",
>>> - .matches = {
>>> - DMI_MATCH(DMI_SYS_VENDOR,
>> "SchenkerTechnologiesGmbH"),
>>> - DMI_MATCH(DMI_BOARD_NAME, "NL5xNU"),
>>> - },
>>> - },
>>> - {
>>> - .callback = video_detect_force_native,
>>> - .ident = "Clevo NL5xNU",
>>> - .matches = {
>>> - DMI_MATCH(DMI_SYS_VENDOR, "Notebook"),
>>> - DMI_MATCH(DMI_BOARD_NAME, "NL5xNU"),
>>> - },
>>> - },
>>>
>>> /*
>>> * Desktops which falsely report a backlight and which our heuristics
