Re: [PATCH] cpufreq: freq_table: Initialize cpuinfo.max_freq to correct max frequency.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Thara Gopinath <thara.gopinath@xxxxxxxxxx>
Subject: Re: [PATCH] cpufreq: freq_table: Initialize cpuinfo.max_freq to correct max frequency.
From: Viresh Kumar <viresh.kumar@xxxxxxxxxx>
Date: Wed, 17 Nov 2021 12:54:05 +0530
Cc: Steev Klimaszewski <steev@xxxxxxxx>, rafael@xxxxxxxxxx, bjorn.andersson@xxxxxxxxxx, linux-pm@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-arm-msm@xxxxxxxxxxxxxxx
In-reply-to: <fd153d84-411a-c843-eab9-2dc66940a3d3@linaro.org>
References: <20211115195011.52999-1-thara.gopinath@linaro.org> <5ae2c644-4743-c62c-b17c-96945a0e6a01@kali.org> <20211116035935.wmazontuznhys6qu@vireshk-i7> <fd153d84-411a-c843-eab9-2dc66940a3d3@linaro.org>
User-agent: NeoMutt/20180716-391-311a52

On 16-11-21, 10:27, Thara Gopinath wrote:
> 	policy->max is unconditionally set to max_freq in the line before "if
> (policy->cpuinfo.max_freq < max_freq)".

So the current code is not correct then :)

-- 
viresh

References:
- [PATCH] cpufreq: freq_table: Initialize cpuinfo.max_freq to correct max frequency.
  - From: Thara Gopinath
- Re: [PATCH] cpufreq: freq_table: Initialize cpuinfo.max_freq to correct max frequency.
  - From: Steev Klimaszewski
- Re: [PATCH] cpufreq: freq_table: Initialize cpuinfo.max_freq to correct max frequency.
  - From: Viresh Kumar
- Re: [PATCH] cpufreq: freq_table: Initialize cpuinfo.max_freq to correct max frequency.
  - From: Thara Gopinath

Prev by Date: Re: [PATCH 1/2] dt-bindings: soc: qcom: llcc-qcom: Add SM8350 compatible and defines
Next by Date: Re: [PATCH] drm/scheduler: fix drm_sched_job_add_implicit_dependencies harder
Previous by thread: Re: [PATCH] cpufreq: freq_table: Initialize cpuinfo.max_freq to correct max frequency.
Next by thread: Re: [PATCH] cpufreq: freq_table: Initialize cpuinfo.max_freq to correct max frequency.
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [Linux for Sparc] [IETF Annouce] [Security] [Bugtraq] [Linux MIPS] [ECOS] [Asterisk Internet PBX] [Linux API]