Re: [REGRESSION][BISECTED] Dell Precision 7780 wakes up on its own from suspend

Andy Shevchenko
2025-12-05T13:52:05Z

On Thu, Dec 04, 2025 at 11:21:32PM -0600, Mario Limonciello (AMD) (kernel.org) wrote:
> On 12/4/2025 10:01 PM, Askar Safin wrote:
> > "Mario Limonciello (AMD) (kernel.org)" <superm1@kernel.org>:

...

> > > But yes; failing all that it's viable to make a quirk. You can follow
> > > any of the ones I've submitted for modeling how to do it. Here's the
> > > most recent one I've done.
> > >
> > > https://github.com/torvalds/linux/commit/23800ad1265f10c2bc6f42154ce4d20e59f2900e
> >
> > Thank you! I will write something like this:
> >
> > + .matches = {
> > + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > + DMI_MATCH(DMI_PRODUCT_FAMILY, "Precision"),
> > + },
> > + .driver_data = &(struct acpi_gpiolib_dmi_quirk) {
> > + .ignore_wake = "VEN_0488:00@355",
> > + },
> >
> > I will test it and then properly send (hopefully today or tomorrow).
>
> This match it too liberal. We don't have any evidence it applies to
> anything more than your single system. I'd pick some different strings.
> For example Product SKU is probably a good string to use because that's very
> unique.

+1. If we ever see another one, we can then think of a more generic quirk.

--
With Best Regards,
Andy Shevchenko