lkml.org 
[lkml]   [2023]   [Feb]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH 2/2] ACPI: x86: Add more systems to quirk list for forcing StorageD3Enable
On Mon, Feb 13, 2023 at 10:35 PM Mario Limonciello
<mario.limonciello@amd.com> wrote:
>
> commit 018d6711c26e4 ("ACPI: x86: Add a quirk for Dell Inspiron 14 2-in-1
> for StorageD3Enable") introduced a quirk to allow a system with ambiguous
> use of _ADR 0 to force StorageD3Enable.
>
> It is reported that Vostro 5626 suffers same symptoms. Add this other
> system to the list as well.
>
> Suggested-by: dbilios@stdio.gr
> Link: https://bugzilla.kernel.org/show_bug.cgi?id=217003
> Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>

Can you please combine these two patches into one?

Or at least make the subjects differ?

> ---
> drivers/acpi/x86/utils.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/acpi/x86/utils.c b/drivers/acpi/x86/utils.c
> index 3409ce6513afa..b1d24718f73d7 100644
> --- a/drivers/acpi/x86/utils.c
> +++ b/drivers/acpi/x86/utils.c
> @@ -214,6 +214,7 @@ static const struct dmi_system_id force_storage_d3_dmi[] = {
> * but .NVME is needed to get StorageD3Enable node
> * https://bugzilla.kernel.org/show_bug.cgi?id=216440
> * https://bugzilla.kernel.org/show_bug.cgi?id=216773
> + * https://bugzilla.kernel.org/show_bug.cgi?id=217003
> */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> @@ -232,6 +233,12 @@ static const struct dmi_system_id force_storage_d3_dmi[] = {
> DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 16 5625"),
> }
> },
> + {
> + .matches = {
> + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> + DMI_MATCH(DMI_PRODUCT_NAME, "Vostro 5625"),
> + }
> + },
> {}
> };
>
> --
> 2.34.1
>

\
 
 \ /
  Last update: 2023-03-27 00:21    [W:0.044 / U:0.140 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site