diff options
| author | Liao Chang <liaochang1@huawei.com> | 2023-08-15 09:40:02 +0800 | 
|---|---|---|
| committer | Viresh Kumar <viresh.kumar@linaro.org> | 2023-08-17 14:12:11 +0530 | 
| commit | ba6ea77d0e75113b52fdae6e01473476cb48a83c (patch) | |
| tree | dc72ae1ba53d38be20641ef036f5c49ca0cfcb7f /drivers/cpufreq/cpufreq.c | |
| parent | d3dec5bb61ce113cc84146bf4a72f752256d3767 (diff) | |
cpufreq: Prefer to print cpuid in MIN/MAX QoS register error message
When a cpufreq_policy is allocated, the cpus, related_cpus and real_cpus
of policy are still unset. Therefore, it is preferable to print the
passed 'cpu' parameter instead of a empty 'cpus' cpumask in error
message when registering MIN/MAX QoS notifier fails.
Signed-off-by: Liao Chang <liaochang1@huawei.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Diffstat (limited to 'drivers/cpufreq/cpufreq.c')
| -rw-r--r-- | drivers/cpufreq/cpufreq.c | 8 | 
1 files changed, 4 insertions, 4 deletions
| diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index 50bbc969ffe5..a757f90aa9d6 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c @@ -1234,16 +1234,16 @@ static struct cpufreq_policy *cpufreq_policy_alloc(unsigned int cpu)  	ret = freq_qos_add_notifier(&policy->constraints, FREQ_QOS_MIN,  				    &policy->nb_min);  	if (ret) { -		dev_err(dev, "Failed to register MIN QoS notifier: %d (%*pbl)\n", -			ret, cpumask_pr_args(policy->cpus)); +		dev_err(dev, "Failed to register MIN QoS notifier: %d (CPU%u)\n", +			ret, cpu);  		goto err_kobj_remove;  	}  	ret = freq_qos_add_notifier(&policy->constraints, FREQ_QOS_MAX,  				    &policy->nb_max);  	if (ret) { -		dev_err(dev, "Failed to register MAX QoS notifier: %d (%*pbl)\n", -			ret, cpumask_pr_args(policy->cpus)); +		dev_err(dev, "Failed to register MAX QoS notifier: %d (CPU%u)\n", +			ret, cpu);  		goto err_min_qos_notifier;  	} | 
