Re: [PATCH] cpufreq: ti-cpufreq: Handle deferred probe with dev_err_probe()

From: Viresh Kumar
Date: Thu Jun 06 2024 - 05:05:11 EST


On 06-06-24, 08:58, Primoz Fiser wrote:
> Handle deferred probing gracefully by using dev_err_probe() to not
> spam console with unnecessary error messages.
>
> Fixes: f88d152dc739 ("cpufreq: ti: Migrate to dev_pm_opp_set_config()")
> Signed-off-by: Primoz Fiser <primoz.fiser@xxxxxxxxx>
> ---
> drivers/cpufreq/ti-cpufreq.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/cpufreq/ti-cpufreq.c b/drivers/cpufreq/ti-cpufreq.c
> index 714ed53753fa..5af85c4cbad0 100644
> --- a/drivers/cpufreq/ti-cpufreq.c
> +++ b/drivers/cpufreq/ti-cpufreq.c
> @@ -417,7 +417,7 @@ static int ti_cpufreq_probe(struct platform_device *pdev)
>
> ret = dev_pm_opp_set_config(opp_data->cpu_dev, &config);
> if (ret < 0) {
> - dev_err(opp_data->cpu_dev, "Failed to set OPP config\n");
> + dev_err_probe(opp_data->cpu_dev, ret, "Failed to set OPP config\n");
> goto fail_put_node;
> }

Applied. Thanks.

--
viresh