Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1620946 > unrolled thread

Re: [PATCH v2 1/3] cpufreq: Add Tegra186 cpufreq driver

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v2 1/3] cpufreq: Add Tegra186 cpufreq driver Viresh Kumar <viresh.kumar@linaro.org> - 2017-04-11 08:40 +0200
    Re: [PATCH v2 1/3] cpufreq: Add Tegra186 cpufreq driver Mikko Perttunen <cyndis@kapsi.fi> - 2017-04-11 08:40 +0200
      Re: [PATCH v2 1/3] cpufreq: Add Tegra186 cpufreq driver "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-04-20 03:00 +0200

#1620946 — Re: [PATCH v2 1/3] cpufreq: Add Tegra186 cpufreq driver

On 04-04-17, 16:43, Mikko Perttunen wrote:
> Add a new cpufreq driver for Tegra186 (and likely later).
> The CPUs are organized into two clusters, Denver and A57,
> with two and four cores respectively. CPU frequency can be
> adjusted by writing the desired rate divisor and a voltage
> hint to a special per-core register.
> 
> The frequency of each core can be set individually; however,
> this is just a hint as all CPUs in a cluster will run at
> the maximum rate of non-idle CPUs in the cluster.
> 
> Signed-off-by: Mikko Perttunen <mperttunen@nvidia.com>
> ---
> v2:
> - Many cosmetic / restructuring changes
> - Only one aperture read from DT now, with a new
>   structure containing the offset among other details
>   per cluster.
> - Don't enable the driver by default.
> 
>  drivers/cpufreq/Kconfig.arm        |   6 +
>  drivers/cpufreq/Makefile           |   1 +
>  drivers/cpufreq/tegra186-cpufreq.c | 275 +++++++++++++++++++++++++++++++++++++
>  3 files changed, 282 insertions(+)
>  create mode 100644 drivers/cpufreq/tegra186-cpufreq.c

For all three patches.

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>

-- 
viresh

[toc] | [next] | [standalone]

#1620947

On 04/11/2017 09:35 AM, Viresh Kumar wrote:
> On 04-04-17, 16:43, Mikko Perttunen wrote:
>> Add a new cpufreq driver for Tegra186 (and likely later).
>> The CPUs are organized into two clusters, Denver and A57,
>> with two and four cores respectively. CPU frequency can be
>> adjusted by writing the desired rate divisor and a voltage
>> hint to a special per-core register.
>>
>> The frequency of each core can be set individually; however,
>> this is just a hint as all CPUs in a cluster will run at
>> the maximum rate of non-idle CPUs in the cluster.
>>
>> Signed-off-by: Mikko Perttunen <mperttunen@nvidia.com>
>> ---
>> v2:
>> - Many cosmetic / restructuring changes
>> - Only one aperture read from DT now, with a new
>>   structure containing the offset among other details
>>   per cluster.
>> - Don't enable the driver by default.
>>
>>  drivers/cpufreq/Kconfig.arm        |   6 +
>>  drivers/cpufreq/Makefile           |   1 +
>>  drivers/cpufreq/tegra186-cpufreq.c | 275 +++++++++++++++++++++++++++++++++++++
>>  3 files changed, 282 insertions(+)
>>  create mode 100644 drivers/cpufreq/tegra186-cpufreq.c
>
> For all three patches.
>
> Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
>

Thanks!

Mikko

[toc] | [prev] | [next] | [standalone]

#1626905

On Tuesday, April 11, 2017 09:37:28 AM Mikko Perttunen wrote:
> On 04/11/2017 09:35 AM, Viresh Kumar wrote:
> > On 04-04-17, 16:43, Mikko Perttunen wrote:
> >> Add a new cpufreq driver for Tegra186 (and likely later).
> >> The CPUs are organized into two clusters, Denver and A57,
> >> with two and four cores respectively. CPU frequency can be
> >> adjusted by writing the desired rate divisor and a voltage
> >> hint to a special per-core register.
> >>
> >> The frequency of each core can be set individually; however,
> >> this is just a hint as all CPUs in a cluster will run at
> >> the maximum rate of non-idle CPUs in the cluster.
> >>
> >> Signed-off-by: Mikko Perttunen <mperttunen@nvidia.com>
> >> ---
> >> v2:
> >> - Many cosmetic / restructuring changes
> >> - Only one aperture read from DT now, with a new
> >>   structure containing the offset among other details
> >>   per cluster.
> >> - Don't enable the driver by default.
> >>
> >>  drivers/cpufreq/Kconfig.arm        |   6 +
> >>  drivers/cpufreq/Makefile           |   1 +
> >>  drivers/cpufreq/tegra186-cpufreq.c | 275 +++++++++++++++++++++++++++++++++++++
> >>  3 files changed, 282 insertions(+)
> >>  create mode 100644 drivers/cpufreq/tegra186-cpufreq.c
> >
> > For all three patches.
> >
> > Acked-by: Viresh Kumar <viresh.kumar@linaro.org>

I've applied the [1/3], but ACKs from the DT maintainers are needed for the
other two.

Thanks,
Rafael

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web