cpufreq : CPPC: Break out if HiSilicon CPPC workaround is matched
authorHanjun Guo <guohanjun@huawei.com>
Tue, 24 Dec 2019 01:56:29 +0000 (09:56 +0800)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Sun, 29 Dec 2019 21:17:26 +0000 (22:17 +0100)
Bail out if we match the OEM information, to save some possible
extra iteration.

Also update the code to fix minor coding style issue.

Signed-off-by: Hanjun Guo <guohanjun@huawei.com>
[ rjw: Subject ]
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/cpufreq/cppc_cpufreq.c

index 8d8da76..d0ca300 100644 (file)
@@ -39,7 +39,7 @@
 static struct cppc_cpudata **all_cpu_data;
 
 struct cppc_workaround_oem_info {
-       char oem_id[ACPI_OEM_ID_SIZE +1];
+       char oem_id[ACPI_OEM_ID_SIZE + 1];
        char oem_table_id[ACPI_OEM_TABLE_ID_SIZE + 1];
        u32 oem_revision;
 };
@@ -93,8 +93,10 @@ static void cppc_check_hisi_workaround(void)
        for (i = 0; i < ARRAY_SIZE(wa_info); i++) {
                if (!memcmp(wa_info[i].oem_id, tbl->oem_id, ACPI_OEM_ID_SIZE) &&
                    !memcmp(wa_info[i].oem_table_id, tbl->oem_table_id, ACPI_OEM_TABLE_ID_SIZE) &&
-                   wa_info[i].oem_revision == tbl->oem_revision)
+                   wa_info[i].oem_revision == tbl->oem_revision) {
                        apply_hisi_workaround = true;
+                       break;
+               }
        }
 }