Re: [PATCH 1/2] cpufreq : cppc: Break out if we match the HiSilicon cppc workaround

["Rafael J. Wysocki" <rjw@rjwysocki.net>]

On Tuesday, December 24, 2019 2:56:29 AM CET Hanjun Guo wrote:
> Bail out if we match the OEM information, to save some possible
> extra iteration. And update the code to fix minor coding style issue.
> 
> Signed-off-by: Hanjun Guo <guohanjun@huawei.com>
> ---
>  drivers/cpufreq/cppc_cpufreq.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/cpufreq/cppc_cpufreq.c b/drivers/cpufreq/cppc_cpufreq.c
> index 8d8da76..d0ca300 100644
> --- a/drivers/cpufreq/cppc_cpufreq.c
> +++ b/drivers/cpufreq/cppc_cpufreq.c
> @@ -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;
> +		}
>  	}
>  }
>  
> 

Both this and the [2/2] applies as 5.6 material with reworked
subjects and changelog changes.  Thanks!