platform/x86: pmc_atom: Match all Beckhoff Automation baytrail boards with critclk_sy...
authorSteffen Dirkwinkel <s.dirkwinkel@beckhoff.com>
Mon, 12 Apr 2021 13:30:06 +0000 (15:30 +0200)
committerHans de Goede <hdegoede@redhat.com>
Wed, 14 Apr 2021 13:52:37 +0000 (15:52 +0200)
pmc_plt_clk* clocks are used for ethernet controllers, so need to stay
turned on. This adds the affected board family to critclk_systems DMI
table, so the clocks are marked as CLK_CRITICAL and not turned off.

This replaces the previously listed boards with a match for the whole
device family CBxx63. CBxx63 matches only baytrail devices.
There are new affected boards that would otherwise need to be listed.
There are unaffected boards in the family, but having the clocks
turned on is not an issue.

Fixes: 648e921888ad ("clk: x86: Stop marking clocks as CLK_IS_CRITICAL")
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Steffen Dirkwinkel <s.dirkwinkel@beckhoff.com>
Link: https://lore.kernel.org/r/20210412133006.397679-1-linux-kernel-dev@beckhoff.com
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
drivers/platform/x86/pmc_atom.c

index ca684ed..a9d2a4b 100644 (file)
@@ -393,34 +393,10 @@ static const struct dmi_system_id critclk_systems[] = {
        },
        {
                /* pmc_plt_clk* - are used for ethernet controllers */
-               .ident = "Beckhoff CB3163",
+               .ident = "Beckhoff Baytrail",
                .matches = {
                        DMI_MATCH(DMI_SYS_VENDOR, "Beckhoff Automation"),
-                       DMI_MATCH(DMI_BOARD_NAME, "CB3163"),
-               },
-       },
-       {
-               /* pmc_plt_clk* - are used for ethernet controllers */
-               .ident = "Beckhoff CB4063",
-               .matches = {
-                       DMI_MATCH(DMI_SYS_VENDOR, "Beckhoff Automation"),
-                       DMI_MATCH(DMI_BOARD_NAME, "CB4063"),
-               },
-       },
-       {
-               /* pmc_plt_clk* - are used for ethernet controllers */
-               .ident = "Beckhoff CB6263",
-               .matches = {
-                       DMI_MATCH(DMI_SYS_VENDOR, "Beckhoff Automation"),
-                       DMI_MATCH(DMI_BOARD_NAME, "CB6263"),
-               },
-       },
-       {
-               /* pmc_plt_clk* - are used for ethernet controllers */
-               .ident = "Beckhoff CB6363",
-               .matches = {
-                       DMI_MATCH(DMI_SYS_VENDOR, "Beckhoff Automation"),
-                       DMI_MATCH(DMI_BOARD_NAME, "CB6363"),
+                       DMI_MATCH(DMI_PRODUCT_FAMILY, "CBxx63"),
                },
        },
        {