On Thu, Aug 09, 2018 at 01:43:34PM +0200, Rafael J. Wysocki wrote:
> On Wednesday, August 8, 2018 11:50:37 AM CEST Mika Westerberg wrote:
> > Commit 2c4d6baf1bc4 ("ACPI / EC: Use ec_no_wakeup on more Thinkpad X1
> > Carbon 6th systems") changed the DMI table to match all systems where
> > DMI product family is "Thinkpad X1 Carbon 6th". However, the system I
> > have here has this string written differently (ThinkPad vs. Thinkpad)
> > which makes the match fail.
> >
> > In addition to that, after BIOS upgrade Robin now has the same string
> > than my system has (perhaps newer BIOS has changed the string).
> >
> > In any case add another DMI entry to acpi_ec_no_wakeup[] table hopefully
> > covering all the X1 Carbon 6th systems out there.
> >
> > Fixes: 2c4d6baf1bc4 ("ACPI / EC: Use ec_no_wakeup on more Thinkpad X1 Carbon 6th systems")
> > Cc: Robin H. Johnson <robbat2@xxxxxxxxxx>
> > Signed-off-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>
> > ---
> > drivers/acpi/ec.c | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
> > diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c
> > index 917f77f4cb55..a2f569e19610 100644
> > --- a/drivers/acpi/ec.c
> > +++ b/drivers/acpi/ec.c
> > @@ -2045,6 +2045,13 @@ static const struct dmi_system_id acpi_ec_no_wakeup[] = {
> > DMI_MATCH(DMI_PRODUCT_FAMILY, "Thinkpad X1 Carbon 6th"),
> > },
> > },
> > + {
> > + .ident = "Thinkpad X1 Carbon 6th",
> > + .matches = {
> > + DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> > + DMI_MATCH(DMI_PRODUCT_FAMILY, "ThinkPad X1 Carbon 6th"),
> > + },
> > + },
> > { },
> > };
> >
> >
>
> I've applied this but I've changed the ident field to match the
> product family string below.
Thanks!
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
