platform/x86: pmc_atom: Match all Lex BayTrail boards with critclk_systems DMI table
authorHans de Goede <hdegoede@redhat.com>
Thu, 28 Jul 2022 18:06:35 +0000 (20:06 +0200)
committerHans de Goede <hdegoede@redhat.com>
Thu, 28 Jul 2022 18:32:00 +0000 (20:32 +0200)
The critclk_systems[] DMI match table already contains 2 Lex BayTrail
boards and patches were just submitted to add 3 more entries for the
following models: 3I380NX, 3I380A, 3I380CW.

Looking at: https://www.lex.com.tw/products/embedded-ipc-board/
we can see that Lex BayTrail makes many embedded boards with
multiple ethernet boards and none of their products are battery
powered so we don't need to worry (too much) about power consumption
when suspended.

Add a new DMI match which simply matches all Lex BayTrail boards and drop
the 2 existing board specific quirks.

Fixes: 648e921888ad ("clk: x86: Stop marking clocks as CLK_IS_CRITICAL")
Reported-by: Michael Schöne <michael.schoene@rhebo.com>
Reported-by: Paul Spooren <paul.spooren@rhebo.com>
Reported-by: Matwey V. Kornilov <matwey@sai.msu.ru>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
drivers/platform/x86/pmc_atom.c

index b8b1ed1..154317e 100644 (file)
@@ -389,21 +389,16 @@ static const struct dmi_system_id critclk_systems[] = {
                },
        },
        {
-               /* pmc_plt_clk0 - 3 are used for the 4 ethernet controllers */
-               .ident = "Lex 3I380D",
+               /*
+                * Lex System / Lex Computech Co. makes a lot of Bay Trail
+                * based embedded boards which often come with multiple
+                * ethernet controllers using multiple pmc_plt_clks. See:
+                * https://www.lex.com.tw/products/embedded-ipc-board/
+                */
+               .ident = "Lex BayTrail",
                .callback = dmi_callback,
                .matches = {
                        DMI_MATCH(DMI_SYS_VENDOR, "Lex BayTrail"),
-                       DMI_MATCH(DMI_PRODUCT_NAME, "3I380D"),
-               },
-       },
-       {
-               /* pmc_plt_clk* - are used for ethernet controllers */
-               .ident = "Lex 2I385SW",
-               .callback = dmi_callback,
-               .matches = {
-                       DMI_MATCH(DMI_SYS_VENDOR, "Lex BayTrail"),
-                       DMI_MATCH(DMI_PRODUCT_NAME, "2I385SW"),
                },
        },
        {