Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Mikko Perttunen
On 11/17/2014 03:08 PM, Thierry Reding wrote: On Mon, Nov 17, 2014 at 02:51:24PM +0200, Mikko Perttunen wrote: On 11/17/2014 01:43 PM, Thierry Reding wrote: On Fri, Nov 14, 2014 at 12:47:33PM +0200, Mikko Perttunen wrote: Tested-by: Mikko Perttunen One potential issue I can see is that if

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Thierry Reding
On Mon, Nov 17, 2014 at 01:51:42PM +0100, Lukasz Majewski wrote: > Hi Thierry, > > > On Fri, Nov 14, 2014 at 12:24:37PM +0100, Lukasz Majewski wrote: > > > Hi Mikko, > > > > > > > Tested-by: Mikko Perttunen > > > > > > Thanks for testing. > > > > > > > > > > > One potential issue I can see

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Thierry Reding
On Mon, Nov 17, 2014 at 02:51:24PM +0200, Mikko Perttunen wrote: > On 11/17/2014 01:43 PM, Thierry Reding wrote: > >On Fri, Nov 14, 2014 at 12:47:33PM +0200, Mikko Perttunen wrote: > >>Tested-by: Mikko Perttunen > >> > >>One potential issue I can see is that if the cpufreq driver fails to probe >

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Lukasz Majewski
Hi Thierry, > On Mon, Nov 17, 2014 at 12:50:13PM +0100, Lukasz Majewski wrote: > > Hi Thierry, > > > > > On Fri, Nov 14, 2014 at 12:47:33PM +0200, Mikko Perttunen wrote: > > > > Tested-by: Mikko Perttunen > > > > > > > > One potential issue I can see is that if the cpufreq driver > > > > fails

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Mikko Perttunen
On 11/17/2014 01:43 PM, Thierry Reding wrote: On Fri, Nov 14, 2014 at 12:47:33PM +0200, Mikko Perttunen wrote: Tested-by: Mikko Perttunen One potential issue I can see is that if the cpufreq driver fails to probe then you'll never get the thermal driver either. For example, Tegra124 currently

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Lukasz Majewski
Hi Thierry, > On Fri, Nov 14, 2014 at 12:24:37PM +0100, Lukasz Majewski wrote: > > Hi Mikko, > > > > > Tested-by: Mikko Perttunen > > > > Thanks for testing. > > > > > > > > One potential issue I can see is that if the cpufreq driver fails > > > to probe then you'll never get the thermal

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Thierry Reding
On Mon, Nov 17, 2014 at 12:50:13PM +0100, Lukasz Majewski wrote: > Hi Thierry, > > > On Fri, Nov 14, 2014 at 12:47:33PM +0200, Mikko Perttunen wrote: > > > Tested-by: Mikko Perttunen > > > > > > One potential issue I can see is that if the cpufreq driver fails > > > to probe then you'll never

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Thierry Reding
On Fri, Nov 14, 2014 at 12:24:37PM +0100, Lukasz Majewski wrote: > Hi Mikko, > > > Tested-by: Mikko Perttunen > > Thanks for testing. > > > > > One potential issue I can see is that if the cpufreq driver fails to > > probe then you'll never get the thermal driver either. For example, > >

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Lukasz Majewski
Hi Thierry, > On Fri, Nov 14, 2014 at 12:47:33PM +0200, Mikko Perttunen wrote: > > Tested-by: Mikko Perttunen > > > > One potential issue I can see is that if the cpufreq driver fails > > to probe then you'll never get the thermal driver either. For > > example, Tegra124 currently has no

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Thierry Reding
On Fri, Nov 14, 2014 at 12:47:33PM +0200, Mikko Perttunen wrote: > Tested-by: Mikko Perttunen > > One potential issue I can see is that if the cpufreq driver fails to probe > then you'll never get the thermal driver either. For example, Tegra124 > currently has no cpufreq driver, so if

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Thierry Reding
On Thu, Nov 13, 2014 at 06:02:42PM +0100, Lukasz Majewski wrote: > When CPU freq is used as a thermal zone cooling device, one needs to wait > until cpufreq subsystem is properly initialized. > > This code is similar to the one already available in imx_thermal.c file. > > Signed-off-by: Lukasz

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Thierry Reding
On Thu, Nov 13, 2014 at 06:02:42PM +0100, Lukasz Majewski wrote: When CPU freq is used as a thermal zone cooling device, one needs to wait until cpufreq subsystem is properly initialized. This code is similar to the one already available in imx_thermal.c file. Signed-off-by: Lukasz

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Thierry Reding
On Fri, Nov 14, 2014 at 12:47:33PM +0200, Mikko Perttunen wrote: Tested-by: Mikko Perttunen mikko.perttu...@kapsi.fi One potential issue I can see is that if the cpufreq driver fails to probe then you'll never get the thermal driver either. For example, Tegra124 currently has no cpufreq

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Lukasz Majewski
Hi Thierry, On Fri, Nov 14, 2014 at 12:47:33PM +0200, Mikko Perttunen wrote: Tested-by: Mikko Perttunen mikko.perttu...@kapsi.fi One potential issue I can see is that if the cpufreq driver fails to probe then you'll never get the thermal driver either. For example, Tegra124 currently

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Thierry Reding
On Fri, Nov 14, 2014 at 12:24:37PM +0100, Lukasz Majewski wrote: Hi Mikko, Tested-by: Mikko Perttunen mikko.perttu...@kapsi.fi Thanks for testing. One potential issue I can see is that if the cpufreq driver fails to probe then you'll never get the thermal driver either. For

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Thierry Reding
On Mon, Nov 17, 2014 at 12:50:13PM +0100, Lukasz Majewski wrote: Hi Thierry, On Fri, Nov 14, 2014 at 12:47:33PM +0200, Mikko Perttunen wrote: Tested-by: Mikko Perttunen mikko.perttu...@kapsi.fi One potential issue I can see is that if the cpufreq driver fails to probe then you'll

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Lukasz Majewski
Hi Thierry, On Fri, Nov 14, 2014 at 12:24:37PM +0100, Lukasz Majewski wrote: Hi Mikko, Tested-by: Mikko Perttunen mikko.perttu...@kapsi.fi Thanks for testing. One potential issue I can see is that if the cpufreq driver fails to probe then you'll never get the thermal

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Mikko Perttunen
On 11/17/2014 01:43 PM, Thierry Reding wrote: On Fri, Nov 14, 2014 at 12:47:33PM +0200, Mikko Perttunen wrote: Tested-by: Mikko Perttunen mikko.perttu...@kapsi.fi One potential issue I can see is that if the cpufreq driver fails to probe then you'll never get the thermal driver either. For

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Lukasz Majewski
Hi Thierry, On Mon, Nov 17, 2014 at 12:50:13PM +0100, Lukasz Majewski wrote: Hi Thierry, On Fri, Nov 14, 2014 at 12:47:33PM +0200, Mikko Perttunen wrote: Tested-by: Mikko Perttunen mikko.perttu...@kapsi.fi One potential issue I can see is that if the cpufreq driver fails

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Thierry Reding
On Mon, Nov 17, 2014 at 02:51:24PM +0200, Mikko Perttunen wrote: On 11/17/2014 01:43 PM, Thierry Reding wrote: On Fri, Nov 14, 2014 at 12:47:33PM +0200, Mikko Perttunen wrote: Tested-by: Mikko Perttunen mikko.perttu...@kapsi.fi One potential issue I can see is that if the cpufreq driver

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Thierry Reding
On Mon, Nov 17, 2014 at 01:51:42PM +0100, Lukasz Majewski wrote: Hi Thierry, On Fri, Nov 14, 2014 at 12:24:37PM +0100, Lukasz Majewski wrote: Hi Mikko, Tested-by: Mikko Perttunen mikko.perttu...@kapsi.fi Thanks for testing. One potential issue I can see is that

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-17 Thread Mikko Perttunen
On 11/17/2014 03:08 PM, Thierry Reding wrote: On Mon, Nov 17, 2014 at 02:51:24PM +0200, Mikko Perttunen wrote: On 11/17/2014 01:43 PM, Thierry Reding wrote: On Fri, Nov 14, 2014 at 12:47:33PM +0200, Mikko Perttunen wrote: Tested-by: Mikko Perttunen mikko.perttu...@kapsi.fi One potential

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-14 Thread Lukasz Majewski
Hi Mikko, > Tested-by: Mikko Perttunen Thanks for testing. > > One potential issue I can see is that if the cpufreq driver fails to > probe then you'll never get the thermal driver either. For example, > Tegra124 currently has no cpufreq driver, so if CONFIG_CPU_THERMAL > was enabled, then

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-14 Thread Mikko Perttunen
Tested-by: Mikko Perttunen One potential issue I can see is that if the cpufreq driver fails to probe then you'll never get the thermal driver either. For example, Tegra124 currently has no cpufreq driver, so if CONFIG_CPU_THERMAL was enabled, then the soctherm driver would never be able to

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-14 Thread Mikko Perttunen
Tested-by: Mikko Perttunen mikko.perttu...@kapsi.fi One potential issue I can see is that if the cpufreq driver fails to probe then you'll never get the thermal driver either. For example, Tegra124 currently has no cpufreq driver, so if CONFIG_CPU_THERMAL was enabled, then the soctherm driver

Re: [PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-14 Thread Lukasz Majewski
Hi Mikko, Tested-by: Mikko Perttunen mikko.perttu...@kapsi.fi Thanks for testing. One potential issue I can see is that if the cpufreq driver fails to probe then you'll never get the thermal driver either. For example, Tegra124 currently has no cpufreq driver, so if CONFIG_CPU_THERMAL

[PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-13 Thread Lukasz Majewski
When CPU freq is used as a thermal zone cooling device, one needs to wait until cpufreq subsystem is properly initialized. This code is similar to the one already available in imx_thermal.c file. Signed-off-by: Lukasz Majewski --- drivers/thermal/tegra_soctherm.c | 7 +++ 1 file changed, 7

[PATCH 5/8] thermal:cpu cooling:tegra: Provide deferred probing for tegra driver

2014-11-13 Thread Lukasz Majewski
When CPU freq is used as a thermal zone cooling device, one needs to wait until cpufreq subsystem is properly initialized. This code is similar to the one already available in imx_thermal.c file. Signed-off-by: Lukasz Majewski l.majew...@samsung.com --- drivers/thermal/tegra_soctherm.c | 7