* [PATCH AUTOSEL 5.15 13/27] ACPI: video: Add backlight=native DMI quirk for Dell G15 5515 [not found] <20221119021352.1774592-1-sashal@kernel.org> @ 2022-11-19 2:13 ` Sasha Levin 2022-11-19 13:20 ` Daniel Dadap 0 siblings, 1 reply; 3+ messages in thread From: Sasha Levin @ 2022-11-19 2:13 UTC (permalink / raw) To: linux-kernel, stable Cc: Hans de Goede, Iris, Daniel Dadap, Sasha Levin, rafael, linux-acpi From: Hans de Goede <hdegoede@redhat.com> [ Upstream commit f46acc1efd4b5846de9fa05f966e504f328f34a6 ] The Dell G15 5515 has the WMI interface (and WMI call returns) expected by the nvidia-wmi-ec-backlight interface. But the backlight class device registered by the nvidia-wmi-ec-backlight driver does not actually work. The amdgpu_bl0 native GPU backlight class device does actually work, add a backlight=native DMI quirk for this. Reported-by: Iris <pawel.js@protonmail.com> Reviewed-by: Daniel Dadap <ddadap@nvidia.com> Signed-off-by: Hans de Goede <hdegoede@redhat.com> --- Changes in v2: - Add a comment that this needs to be revisited when dynamic-mux support gets added (suggested by: Daniel Dadap) Signed-off-by: Sasha Levin <sashal@kernel.org> --- drivers/acpi/video_detect.c | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/drivers/acpi/video_detect.c b/drivers/acpi/video_detect.c index b13713199ad9..8dfcb6b44936 100644 --- a/drivers/acpi/video_detect.c +++ b/drivers/acpi/video_detect.c @@ -564,6 +564,20 @@ static const struct dmi_system_id video_detect_dmi_table[] = { DMI_MATCH(DMI_BOARD_NAME, "GMxRGxx"), }, }, + /* + * Models which have nvidia-ec-wmi support, but should not use it. + * Note this indicates a likely firmware bug on these models and should + * be revisited if/when Linux gets support for dynamic mux mode. + */ + { + .callback = video_detect_force_native, + /* Dell G15 5515 */ + .matches = { + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."), + DMI_MATCH(DMI_PRODUCT_NAME, "Dell G15 5515"), + }, + }, + /* * Desktops which falsely report a backlight and which our heuristics * for this do not catch. -- 2.35.1 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH AUTOSEL 5.15 13/27] ACPI: video: Add backlight=native DMI quirk for Dell G15 5515 2022-11-19 2:13 ` [PATCH AUTOSEL 5.15 13/27] ACPI: video: Add backlight=native DMI quirk for Dell G15 5515 Sasha Levin @ 2022-11-19 13:20 ` Daniel Dadap 2022-11-24 16:53 ` Sasha Levin 0 siblings, 1 reply; 3+ messages in thread From: Daniel Dadap @ 2022-11-19 13:20 UTC (permalink / raw) To: Sasha Levin; +Cc: linux-kernel, stable, Hans de Goede, Iris, rafael, linux-acpi NACK for 5.15 (and earlier) - the nvidia-wmi-ec-backlight driver is not present in the older LTS branches. I suppose this patch could be useful for distro kernels which have backported that driver, but I'm not sure if it's then supposed to be the distro's responsibility to also backport quirks that affect other code they have backported. If the intention is to relieve distros of that responsibility, I suppose this is harmless enough. On Fri, Nov 18, 2022 at 09:13:38PM -0500, Sasha Levin wrote: > From: Hans de Goede <hdegoede@redhat.com> > > [ Upstream commit f46acc1efd4b5846de9fa05f966e504f328f34a6 ] > > The Dell G15 5515 has the WMI interface (and WMI call returns) expected > by the nvidia-wmi-ec-backlight interface. But the backlight class device > registered by the nvidia-wmi-ec-backlight driver does not actually work. > > The amdgpu_bl0 native GPU backlight class device does actually work, > add a backlight=native DMI quirk for this. > > Reported-by: Iris <pawel.js@protonmail.com> > Reviewed-by: Daniel Dadap <ddadap@nvidia.com> > Signed-off-by: Hans de Goede <hdegoede@redhat.com> > --- > Changes in v2: > - Add a comment that this needs to be revisited when dynamic-mux > support gets added (suggested by: Daniel Dadap) > Signed-off-by: Sasha Levin <sashal@kernel.org> > --- > drivers/acpi/video_detect.c | 14 ++++++++++++++ > 1 file changed, 14 insertions(+) > > diff --git a/drivers/acpi/video_detect.c b/drivers/acpi/video_detect.c > index b13713199ad9..8dfcb6b44936 100644 > --- a/drivers/acpi/video_detect.c > +++ b/drivers/acpi/video_detect.c > @@ -564,6 +564,20 @@ static const struct dmi_system_id video_detect_dmi_table[] = { > DMI_MATCH(DMI_BOARD_NAME, "GMxRGxx"), > }, > }, > + /* > + * Models which have nvidia-ec-wmi support, but should not use it. > + * Note this indicates a likely firmware bug on these models and should > + * be revisited if/when Linux gets support for dynamic mux mode. > + */ > + { > + .callback = video_detect_force_native, > + /* Dell G15 5515 */ > + .matches = { > + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."), > + DMI_MATCH(DMI_PRODUCT_NAME, "Dell G15 5515"), > + }, > + }, > + > /* > * Desktops which falsely report a backlight and which our heuristics > * for this do not catch. > -- > 2.35.1 > ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH AUTOSEL 5.15 13/27] ACPI: video: Add backlight=native DMI quirk for Dell G15 5515 2022-11-19 13:20 ` Daniel Dadap @ 2022-11-24 16:53 ` Sasha Levin 0 siblings, 0 replies; 3+ messages in thread From: Sasha Levin @ 2022-11-24 16:53 UTC (permalink / raw) To: Daniel Dadap Cc: linux-kernel, stable, Hans de Goede, Iris, rafael, linux-acpi On Sat, Nov 19, 2022 at 07:20:20AM -0600, Daniel Dadap wrote: >NACK for 5.15 (and earlier) - the nvidia-wmi-ec-backlight driver is not >present in the older LTS branches. > >I suppose this patch could be useful for distro kernels which have >backported that driver, but I'm not sure if it's then supposed to be the >distro's responsibility to also backport quirks that affect other code >they have backported. If the intention is to relieve distros of that >responsibility, I suppose this is harmless enough. I'll drop it on 5.15 and earlier, thanks! -- Thanks, Sasha ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-11-24 16:53 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20221119021352.1774592-1-sashal@kernel.org>
2022-11-19 2:13 ` [PATCH AUTOSEL 5.15 13/27] ACPI: video: Add backlight=native DMI quirk for Dell G15 5515 Sasha Levin
2022-11-19 13:20 ` Daniel Dadap
2022-11-24 16:53 ` Sasha Levin
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox