| Message ID | 20240909095529.2325103-1-marcin.juszkiewicz@linaro.org |
|---|---|
| State | Accepted |
| Commit | 8d5459f1a7f0831562d6d3f7917606708e79ccd5 |
| Headers | show |
| Series | cpufreq: use proper units for frequency | expand |
On Mon, 9 Sept 2024 at 15:25, Marcin Juszkiewicz <marcin.juszkiewicz@linaro.org> wrote: > > When I booted my RK3588 based system I noticed that cpufreq complained > about system clock: > > [ +0.007211] cpufreq: cpufreq_online: CPU0: Running at unlisted initial frequency: 816000 KHz, changing to: 1008000 KHz > > Then I realized that unit is displayed wrong: "KHz" instead of "kHz". > > Signed-off-by: Marcin Juszkiewicz <marcin.juszkiewicz@linaro.org> > --- > drivers/cpufreq/cpufreq.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c > index 04fc786dd2c0..76da29c2bd3f 100644 > --- a/drivers/cpufreq/cpufreq.c > +++ b/drivers/cpufreq/cpufreq.c > @@ -1539,7 +1539,7 @@ static int cpufreq_online(unsigned int cpu) > * frequency for longer duration. Hence, a BUG_ON(). > */ > BUG_ON(ret); > - pr_info("%s: CPU%d: Running at unlisted initial frequency: %u KHz, changing to: %u KHz\n", > + pr_info("%s: CPU%d: Running at unlisted initial frequency: %u kHz, changing to: %u kHz\n", > __func__, policy->cpu, old_freq, policy->cur); > } > } Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
On Wed, Sep 18, 2024 at 5:04 PM Viresh Kumar <viresh.kumar@linaro.org> wrote: > > On Mon, 9 Sept 2024 at 15:25, Marcin Juszkiewicz > <marcin.juszkiewicz@linaro.org> wrote: > > > > When I booted my RK3588 based system I noticed that cpufreq complained > > about system clock: > > > > [ +0.007211] cpufreq: cpufreq_online: CPU0: Running at unlisted initial frequency: 816000 KHz, changing to: 1008000 KHz > > > > Then I realized that unit is displayed wrong: "KHz" instead of "kHz". > > > > Signed-off-by: Marcin Juszkiewicz <marcin.juszkiewicz@linaro.org> > > --- > > drivers/cpufreq/cpufreq.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c > > index 04fc786dd2c0..76da29c2bd3f 100644 > > --- a/drivers/cpufreq/cpufreq.c > > +++ b/drivers/cpufreq/cpufreq.c > > @@ -1539,7 +1539,7 @@ static int cpufreq_online(unsigned int cpu) > > * frequency for longer duration. Hence, a BUG_ON(). > > */ > > BUG_ON(ret); > > - pr_info("%s: CPU%d: Running at unlisted initial frequency: %u KHz, changing to: %u KHz\n", > > + pr_info("%s: CPU%d: Running at unlisted initial frequency: %u kHz, changing to: %u kHz\n", > > __func__, policy->cpu, old_freq, policy->cur); > > } > > } > > Acked-by: Viresh Kumar <viresh.kumar@linaro.org> Applied as 6.13 material, thanks!
diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index 04fc786dd2c0..76da29c2bd3f 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c @@ -1539,7 +1539,7 @@ static int cpufreq_online(unsigned int cpu) * frequency for longer duration. Hence, a BUG_ON(). */ BUG_ON(ret); - pr_info("%s: CPU%d: Running at unlisted initial frequency: %u KHz, changing to: %u KHz\n", + pr_info("%s: CPU%d: Running at unlisted initial frequency: %u kHz, changing to: %u kHz\n", __func__, policy->cpu, old_freq, policy->cur); } }
When I booted my RK3588 based system I noticed that cpufreq complained about system clock: [ +0.007211] cpufreq: cpufreq_online: CPU0: Running at unlisted initial frequency: 816000 KHz, changing to: 1008000 KHz Then I realized that unit is displayed wrong: "KHz" instead of "kHz". Signed-off-by: Marcin Juszkiewicz <marcin.juszkiewicz@linaro.org> --- drivers/cpufreq/cpufreq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)