Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) client-ip=2620:137:e000::3:8;
MIME-Version: 1.0
References: <20231024091336.1.Ie25ddf26b761bf185b7152c6a9884977f0ce73f9@changeid>
In-Reply-To: <20231024091336.1.Ie25ddf26b761bf185b7152c6a9884977f0ce73f9@changeid>
From:   "Rafael J. Wysocki" <rafael@kernel.org>
Date:   Tue, 24 Oct 2023 21:54:27 +0200
Message-ID: <CAJZ5v0gOmqK66NA_FAqEx59f0yd5PzdhH7jio3A0EN8=Tpp5nA@mail.gmail.com>
Subject: Re: [PATCH] ACPI: EC: Add quirk for HP 250 G7 Notebook PC
To:     Jonathan Denose <jdenose@chromium.org>
Cc:     LKML <linux-kernel@vger.kernel.org>, linux-acpi@vger.kernel.org,
        Jonathan Denose <jdenose@google.com>,
        Len Brown <lenb@kernel.org>,
        "Rafael J. Wysocki" <rafael@kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8BIT
Precedence: bulk

On Tue, Oct 24, 2023 at 4:14 PM Jonathan Denose <jdenose@chromium.org> wrote:
>
> Added GPE quirk entry for HP 250 G7 Notebook PC. This change
> allows the lid switch to be identified as the lid switch and not
> a keyboard button. With the lid switch properly identified, the
> device triggers suspend correctly on lid close.
>
> Signed-off-by: Jonathan Denose <jdenose@google.com>
> ---
>
>  drivers/acpi/ec.c | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c
> index c95d0edb0be9e..a59c11df73754 100644
> --- a/drivers/acpi/ec.c
> +++ b/drivers/acpi/ec.c
> @@ -1924,6 +1924,16 @@ static const struct dmi_system_id ec_dmi_table[] __initconst = {
>                         DMI_MATCH(DMI_PRODUCT_NAME, "HP Pavilion Gaming Laptop 15-dk1xxx"),
>                 },
>         },
> +       {
> +               /*
> +                * HP 250 G7 Notebook PC
> +                */
> +               .callback = ec_honor_dsdt_gpe,
> +               .matches = {
> +                       DMI_MATCH(DMI_SYS_VENDOR, "HP"),
> +                       DMI_MATCH(DMI_PRODUCT_NAME, "HP 250 G7 Notebook PC"),
> +               },
> +       },
>         {
>                 /*
>                  * Samsung hardware
> --

Applied as 6.7 material, thanks!