platform/x86: pmc_atom: Match all Beckhoff Automation baytrail boards with critclk_sy...

[ Upstream commit d21e5abd3a005253eb033090aab2e43bce090d89 ]

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>
Signed-off-by: Sasha Levin <sashal@kernel.org>

diff --git a/drivers/platform/x86/pmc_atom.c b/drivers/platform/x86/pmc_atom.c
index ca684ed760d147c0cc82753b28d8632b843fa889..a9d2a4b98e57022ec5deadb0ecea5f63c4c33823 100644 (file)
--- a/drivers/platform/x86/pmc_atom.c
+++ b/drivers/platform/x86/pmc_atom.c
@@ -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"),
                },
        },
        {