Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2014-01-02 Thread Viresh Kumar
On 2 January 2014 16:38, bilhuang wrote: > Actually, I don't have plan or resource on doing this, would it be better > that you help to do that instead? Thanks. Point taken. I am there to help if required. So, initially you can just make Tegra work according to the new file we were talking

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2014-01-02 Thread bilhuang
On 12/23/2013 01:06 PM, Viresh Kumar wrote: Ccc'ing Grant and Rob as well. On 20 December 2013 21:59, Stephen Warren wrote: No, I definitely don't agree here. The rules for arch/arm64 are: no platform-specific code. We should immediately start planning for that. If this means renaming the

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2014-01-02 Thread bilhuang
On 12/23/2013 01:06 PM, Viresh Kumar wrote: Ccc'ing Grant and Rob as well. On 20 December 2013 21:59, Stephen Warren swar...@wwwdotorg.org wrote: No, I definitely don't agree here. The rules for arch/arm64 are: no platform-specific code. We should immediately start planning for that. If this

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2014-01-02 Thread Viresh Kumar
On 2 January 2014 16:38, bilhuang bilhu...@nvidia.com wrote: Actually, I don't have plan or resource on doing this, would it be better that you help to do that instead? Thanks. Point taken. I am there to help if required. So, initially you can just make Tegra work according to the new file we

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-22 Thread Viresh Kumar
Ccc'ing Grant and Rob as well. On 20 December 2013 21:59, Stephen Warren wrote: > No, I definitely don't agree here. The rules for arch/arm64 are: no > platform-specific code. We should immediately start planning for that. > If this means renaming the file that creates the virtual device from >

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-22 Thread Viresh Kumar
Ccc'ing Grant and Rob as well. On 20 December 2013 21:59, Stephen Warren swar...@wwwdotorg.org wrote: No, I definitely don't agree here. The rules for arch/arm64 are: no platform-specific code. We should immediately start planning for that. If this means renaming the file that creates the

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-20 Thread Stephen Warren
On 12/20/2013 03:42 AM, bilhuang wrote: > On 12/20/2013 06:33 PM, Viresh Kumar wrote: >> On 20 December 2013 15:55, bilhuang wrote: >>> Don't you think it worth creating a file here so this can be shared to >>> arm64? >> >> We will see how to handle virtual devices when we will start getting >>

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-20 Thread bilhuang
On 12/20/2013 06:33 PM, Viresh Kumar wrote: On 20 December 2013 15:55, bilhuang wrote: Don't you think it worth creating a file here so this can be shared to arm64? We will see how to handle virtual devices when we will start getting arm64 SoCs. Probably we might end up writing a single file

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-20 Thread Viresh Kumar
On 20 December 2013 15:55, bilhuang wrote: > Don't you think it worth creating a file here so this can be shared to > arm64? We will see how to handle virtual devices when we will start getting arm64 SoCs. Probably we might end up writing a single file in cpufreq, if required, that will create

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-20 Thread bilhuang
On 12/20/2013 05:31 PM, Viresh Kumar wrote: On 19 December 2013 16:48, Bill Huang wrote: diff --git a/drivers/cpufreq/Kconfig.arm b/drivers/cpufreq/Kconfig.arm index ce52ed9..22dfc43 100644 --- a/drivers/cpufreq/Kconfig.arm +++ b/drivers/cpufreq/Kconfig.arm @@ -225,6 +225,18 @@ config

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-20 Thread Viresh Kumar
On 19 December 2013 16:48, Bill Huang wrote: > diff --git a/drivers/cpufreq/Kconfig.arm b/drivers/cpufreq/Kconfig.arm > index ce52ed9..22dfc43 100644 > --- a/drivers/cpufreq/Kconfig.arm > +++ b/drivers/cpufreq/Kconfig.arm > @@ -225,6 +225,18 @@ config ARM_TEGRA_CPUFREQ > help >

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-20 Thread Viresh Kumar
On 19 December 2013 16:48, Bill Huang bilhu...@nvidia.com wrote: diff --git a/drivers/cpufreq/Kconfig.arm b/drivers/cpufreq/Kconfig.arm index ce52ed9..22dfc43 100644 --- a/drivers/cpufreq/Kconfig.arm +++ b/drivers/cpufreq/Kconfig.arm @@ -225,6 +225,18 @@ config ARM_TEGRA_CPUFREQ help

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-20 Thread bilhuang
On 12/20/2013 05:31 PM, Viresh Kumar wrote: On 19 December 2013 16:48, Bill Huang bilhu...@nvidia.com wrote: diff --git a/drivers/cpufreq/Kconfig.arm b/drivers/cpufreq/Kconfig.arm index ce52ed9..22dfc43 100644 --- a/drivers/cpufreq/Kconfig.arm +++ b/drivers/cpufreq/Kconfig.arm @@ -225,6 +225,18

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-20 Thread Viresh Kumar
On 20 December 2013 15:55, bilhuang bilhu...@nvidia.com wrote: Don't you think it worth creating a file here so this can be shared to arm64? We will see how to handle virtual devices when we will start getting arm64 SoCs. Probably we might end up writing a single file in cpufreq, if required,

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-20 Thread bilhuang
On 12/20/2013 06:33 PM, Viresh Kumar wrote: On 20 December 2013 15:55, bilhuang bilhu...@nvidia.com wrote: Don't you think it worth creating a file here so this can be shared to arm64? We will see how to handle virtual devices when we will start getting arm64 SoCs. Probably we might end up

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-20 Thread Stephen Warren
On 12/20/2013 03:42 AM, bilhuang wrote: On 12/20/2013 06:33 PM, Viresh Kumar wrote: On 20 December 2013 15:55, bilhuang bilhu...@nvidia.com wrote: Don't you think it worth creating a file here so this can be shared to arm64? We will see how to handle virtual devices when we will start

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-19 Thread Stephen Warren
On 12/19/2013 04:18 AM, Bill Huang wrote: > Re-model Tegra20 cpufreq driver as below. > > * Rename tegra-cpufreq.c to tegra20-cpufreq.c since this file supports > only Tegra20. > * Add probe function so defer probe can be used when we're going to > support DVFS. > * Create a fake cpufreq

[PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-19 Thread Bill Huang
Re-model Tegra20 cpufreq driver as below. * Rename tegra-cpufreq.c to tegra20-cpufreq.c since this file supports only Tegra20. * Add probe function so defer probe can be used when we're going to support DVFS. * Create a fake cpufreq platform device with its name being

[PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-19 Thread Bill Huang
Re-model Tegra20 cpufreq driver as below. * Rename tegra-cpufreq.c to tegra20-cpufreq.c since this file supports only Tegra20. * Add probe function so defer probe can be used when we're going to support DVFS. * Create a fake cpufreq platform device with its name being

Re: [PATCH v5 1/1] cpufreq: tegra: Re-model Tegra20 cpufreq driver

2013-12-19 Thread Stephen Warren
On 12/19/2013 04:18 AM, Bill Huang wrote: Re-model Tegra20 cpufreq driver as below. * Rename tegra-cpufreq.c to tegra20-cpufreq.c since this file supports only Tegra20. * Add probe function so defer probe can be used when we're going to support DVFS. * Create a fake cpufreq platform