ACPI / battery: Cleanup Lenovo Ideapad Miix 320 DMI table entry
authorHans de Goede <hdegoede@redhat.com>
Sun, 23 Feb 2020 14:29:41 +0000 (15:29 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 4 Mar 2020 09:32:50 +0000 (10:32 +0100)
The Lenovo Ideapad Miix 320 bat_dmi_table entry is using weird indentation
and is the only entry which (unnecessarily) uses DMI_EXACT_MATCH instead
of DMI_MATCH, fixup both to make the entry consistent with the others.

While at it also update the comments for battery_do_not_check_pmic_quirk
entries, adding a bit of text explaining why the quirk is necessary.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/battery.c

index 111a407..366c389 100644 (file)
@@ -1365,19 +1365,19 @@ static const struct dmi_system_id bat_dmi_table[] __initconst = {
                },
        },
        {
-               /* ECS EF20EA */
+               /* ECS EF20EA, AXP288 PMIC but uses separate fuel-gauge */
                .callback = battery_do_not_check_pmic_quirk,
                .matches = {
                        DMI_MATCH(DMI_PRODUCT_NAME, "EF20EA"),
                },
        },
        {
-               /* Lenovo Ideapad Miix 320 */
+               /* Lenovo Ideapad Miix 320, AXP288 PMIC, separate fuel-gauge */
                .callback = battery_do_not_check_pmic_quirk,
                .matches = {
-                 DMI_EXACT_MATCH(DMI_SYS_VENDOR, "LENOVO"),
-                 DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "80XF"),
-                 DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "Lenovo MIIX 320-10ICR"),
+                       DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
+                       DMI_MATCH(DMI_PRODUCT_NAME, "80XF"),
+                       DMI_MATCH(DMI_PRODUCT_VERSION, "Lenovo MIIX 320-10ICR"),
                },
        },
        {},