All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] thermal: hisilicon: Add dependency on the clock driver to allow frequency scaling
       [not found] <32d418f24a4da44ad4ca5b2b1276ffcea2aa7d6c.1466424352.git.amit.kucheria@linaro.org>
@ 2016-06-20 12:16 ` Amit Kucheria
  2016-06-20 13:16   ` Leo Yan
  0 siblings, 1 reply; 8+ messages in thread
From: Amit Kucheria @ 2016-06-20 12:16 UTC (permalink / raw)
  To: linux-kernel; +Cc: linux-pm, leo.yan, Eduardo Valentin, kongxinwei, Zhang Rui

The Hisilicon clock stub driver is needed to allow the thermal drivers to
actually scale the frequency. Make it an automatic dependency.

Signed-off-by: Amit Kucheria <amit.kucheria@linaro.org>
---
 drivers/thermal/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 22ae1f7..4e843f7 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -178,6 +178,7 @@ config THERMAL_EMULATION
 config HISI_THERMAL
 	tristate "Hisilicon thermal driver"
 	depends on (ARCH_HISI && CPU_THERMAL && OF) || COMPILE_TEST
+	select STUB_CLK_HI6220
 	help
 	  Enable this to plug hisilicon's thermal sensor driver into the Linux
 	  thermal framework. cpufreq is used as the cooling device to throttle
-- 
2.5.0

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH] thermal: hisilicon: Add dependency on the clock driver to allow frequency scaling
  2016-06-20 12:16 ` [PATCH] thermal: hisilicon: Add dependency on the clock driver to allow frequency scaling Amit Kucheria
@ 2016-06-20 13:16   ` Leo Yan
  2016-06-26 16:32     ` Amit Kucheria
  0 siblings, 1 reply; 8+ messages in thread
From: Leo Yan @ 2016-06-20 13:16 UTC (permalink / raw)
  To: Amit Kucheria
  Cc: linux-kernel, linux-pm, Eduardo Valentin, kongxinwei, Zhang Rui

Hi Amit,

On Mon, Jun 20, 2016 at 05:46:36PM +0530, Amit Kucheria wrote:
> The Hisilicon clock stub driver is needed to allow the thermal drivers to
> actually scale the frequency. Make it an automatic dependency.
> 
> Signed-off-by: Amit Kucheria <amit.kucheria@linaro.org>
> ---
>  drivers/thermal/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
> index 22ae1f7..4e843f7 100644
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -178,6 +178,7 @@ config THERMAL_EMULATION
>  config HISI_THERMAL
>  	tristate "Hisilicon thermal driver"
>  	depends on (ARCH_HISI && CPU_THERMAL && OF) || COMPILE_TEST
> +	select STUB_CLK_HI6220

Acked-by: Leo Yan <leo.yan@linaro.org>

>  	help
>  	  Enable this to plug hisilicon's thermal sensor driver into the Linux
>  	  thermal framework. cpufreq is used as the cooling device to throttle
> -- 
> 2.5.0
> 

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] thermal: hisilicon: Add dependency on the clock driver to allow frequency scaling
  2016-06-20 13:16   ` Leo Yan
@ 2016-06-26 16:32     ` Amit Kucheria
  2016-07-07  6:43       ` Amit Kucheria
  0 siblings, 1 reply; 8+ messages in thread
From: Amit Kucheria @ 2016-06-26 16:32 UTC (permalink / raw)
  To: Leo Yan, xuwei5
  Cc: LKML, Linux PM list, Eduardo Valentin, kongxinwei, Zhang Rui

On Mon, Jun 20, 2016 at 6:46 PM, Leo Yan <leo.yan@linaro.org> wrote:
> Hi Amit,
>
> On Mon, Jun 20, 2016 at 05:46:36PM +0530, Amit Kucheria wrote:
>> The Hisilicon clock stub driver is needed to allow the thermal drivers to
>> actually scale the frequency. Make it an automatic dependency.
>>
>> Signed-off-by: Amit Kucheria <amit.kucheria@linaro.org>
>> ---
>>  drivers/thermal/Kconfig | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
>> index 22ae1f7..4e843f7 100644
>> --- a/drivers/thermal/Kconfig
>> +++ b/drivers/thermal/Kconfig
>> @@ -178,6 +178,7 @@ config THERMAL_EMULATION
>>  config HISI_THERMAL
>>       tristate "Hisilicon thermal driver"
>>       depends on (ARCH_HISI && CPU_THERMAL && OF) || COMPILE_TEST
>> +     select STUB_CLK_HI6220
>
> Acked-by: Leo Yan <leo.yan@linaro.org>

Thanks Leo. Eduardo, will you take this through your tree or should
Wei include this in his tree?

Regards,
Amit

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] thermal: hisilicon: Add dependency on the clock driver to allow frequency scaling
  2016-06-26 16:32     ` Amit Kucheria
@ 2016-07-07  6:43       ` Amit Kucheria
  2016-07-25 12:29         ` Amit Kucheria
  0 siblings, 1 reply; 8+ messages in thread
From: Amit Kucheria @ 2016-07-07  6:43 UTC (permalink / raw)
  To: Leo Yan, xuwei5, guodong.xu
  Cc: LKML, Linux PM list, Eduardo Valentin, kongxinwei, Zhang Rui

On Sun, Jun 26, 2016 at 10:02 PM, Amit Kucheria
<amit.kucheria@linaro.org> wrote:
> On Mon, Jun 20, 2016 at 6:46 PM, Leo Yan <leo.yan@linaro.org> wrote:
>> Hi Amit,
>>
>> On Mon, Jun 20, 2016 at 05:46:36PM +0530, Amit Kucheria wrote:
>>> The Hisilicon clock stub driver is needed to allow the thermal drivers to
>>> actually scale the frequency. Make it an automatic dependency.
>>>
>>> Signed-off-by: Amit Kucheria <amit.kucheria@linaro.org>
>>> ---
>>>  drivers/thermal/Kconfig | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
>>> index 22ae1f7..4e843f7 100644
>>> --- a/drivers/thermal/Kconfig
>>> +++ b/drivers/thermal/Kconfig
>>> @@ -178,6 +178,7 @@ config THERMAL_EMULATION
>>>  config HISI_THERMAL
>>>       tristate "Hisilicon thermal driver"
>>>       depends on (ARCH_HISI && CPU_THERMAL && OF) || COMPILE_TEST
>>> +     select STUB_CLK_HI6220
>>
>> Acked-by: Leo Yan <leo.yan@linaro.org>
>
> Thanks Leo. Eduardo, will you take this through your tree or should
> Wei include this in his tree?

Ping!

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] thermal: hisilicon: Add dependency on the clock driver to allow frequency scaling
  2016-07-07  6:43       ` Amit Kucheria
@ 2016-07-25 12:29         ` Amit Kucheria
  2016-07-25 13:19           ` Wei Xu
  0 siblings, 1 reply; 8+ messages in thread
From: Amit Kucheria @ 2016-07-25 12:29 UTC (permalink / raw)
  To: Leo Yan, xuwei5, Guodong Xu
  Cc: LKML, Linux PM list, Eduardo Valentin, kongxinwei, Zhang Rui

On Thu, Jul 7, 2016 at 12:13 PM, Amit Kucheria <amit.kucheria@linaro.org> wrote:
> On Sun, Jun 26, 2016 at 10:02 PM, Amit Kucheria
> <amit.kucheria@linaro.org> wrote:
>> On Mon, Jun 20, 2016 at 6:46 PM, Leo Yan <leo.yan@linaro.org> wrote:
>>> Hi Amit,
>>>
>>> On Mon, Jun 20, 2016 at 05:46:36PM +0530, Amit Kucheria wrote:
>>>> The Hisilicon clock stub driver is needed to allow the thermal drivers to
>>>> actually scale the frequency. Make it an automatic dependency.
>>>>
>>>> Signed-off-by: Amit Kucheria <amit.kucheria@linaro.org>
>>>> ---
>>>>  drivers/thermal/Kconfig | 1 +
>>>>  1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
>>>> index 22ae1f7..4e843f7 100644
>>>> --- a/drivers/thermal/Kconfig
>>>> +++ b/drivers/thermal/Kconfig
>>>> @@ -178,6 +178,7 @@ config THERMAL_EMULATION
>>>>  config HISI_THERMAL
>>>>       tristate "Hisilicon thermal driver"
>>>>       depends on (ARCH_HISI && CPU_THERMAL && OF) || COMPILE_TEST
>>>> +     select STUB_CLK_HI6220
>>>
>>> Acked-by: Leo Yan <leo.yan@linaro.org>
>>
>> Thanks Leo. Eduardo, will you take this through your tree or should
>> Wei include this in his tree?
>
> Ping!

Wei, could you pull this into your tree for the next release?

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] thermal: hisilicon: Add dependency on the clock driver to allow frequency scaling
  2016-07-25 12:29         ` Amit Kucheria
@ 2016-07-25 13:19           ` Wei Xu
  2016-08-01  6:13             ` Amit Kucheria
  0 siblings, 1 reply; 8+ messages in thread
From: Wei Xu @ 2016-07-25 13:19 UTC (permalink / raw)
  To: Amit Kucheria, Leo Yan, Guodong Xu, Eduardo Valentin
  Cc: LKML, Linux PM list, kongxinwei, Zhang Rui



On 25/07/2016 13:29, Amit Kucheria wrote:
> On Thu, Jul 7, 2016 at 12:13 PM, Amit Kucheria <amit.kucheria@linaro.org> wrote:
>> On Sun, Jun 26, 2016 at 10:02 PM, Amit Kucheria
>> <amit.kucheria@linaro.org> wrote:
>>> On Mon, Jun 20, 2016 at 6:46 PM, Leo Yan <leo.yan@linaro.org> wrote:
>>>> Hi Amit,
>>>>
>>>> On Mon, Jun 20, 2016 at 05:46:36PM +0530, Amit Kucheria wrote:
>>>>> The Hisilicon clock stub driver is needed to allow the thermal drivers to
>>>>> actually scale the frequency. Make it an automatic dependency.
>>>>>
>>>>> Signed-off-by: Amit Kucheria <amit.kucheria@linaro.org>
>>>>> ---
>>>>>  drivers/thermal/Kconfig | 1 +
>>>>>  1 file changed, 1 insertion(+)
>>>>>
>>>>> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
>>>>> index 22ae1f7..4e843f7 100644
>>>>> --- a/drivers/thermal/Kconfig
>>>>> +++ b/drivers/thermal/Kconfig
>>>>> @@ -178,6 +178,7 @@ config THERMAL_EMULATION
>>>>>  config HISI_THERMAL
>>>>>       tristate "Hisilicon thermal driver"
>>>>>       depends on (ARCH_HISI && CPU_THERMAL && OF) || COMPILE_TEST
>>>>> +     select STUB_CLK_HI6220
>>>>
>>>> Acked-by: Leo Yan <leo.yan@linaro.org>
>>>
>>> Thanks Leo. Eduardo, will you take this through your tree or should
>>> Wei include this in his tree?
>>
>> Ping!

Hi Amit

> 
> Wei, could you pull this into your tree for the next release?

I am OK If Eduardo is OK to pick up my pull request.

Hi Eduardo,

Is it OK for me to send a pull request to you about this?
Or could you let us know whether you could directly pick it up?
Thanks!

Best Regards,
Wei

> 
> 

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] thermal: hisilicon: Add dependency on the clock driver to allow frequency scaling
  2016-07-25 13:19           ` Wei Xu
@ 2016-08-01  6:13             ` Amit Kucheria
  2016-08-19  8:02               ` Zhang Rui
  0 siblings, 1 reply; 8+ messages in thread
From: Amit Kucheria @ 2016-08-01  6:13 UTC (permalink / raw)
  To: Wei Xu
  Cc: Leo Yan, Guodong Xu, Eduardo Valentin, LKML, Linux PM list,
	kongxinwei, Zhang Rui

On Mon, Jul 25, 2016 at 6:49 PM, Wei Xu <xuwei5@hisilicon.com> wrote:
>
>
> On 25/07/2016 13:29, Amit Kucheria wrote:
>> On Thu, Jul 7, 2016 at 12:13 PM, Amit Kucheria <amit.kucheria@linaro.org> wrote:
>>> On Sun, Jun 26, 2016 at 10:02 PM, Amit Kucheria
>>> <amit.kucheria@linaro.org> wrote:
>>>> On Mon, Jun 20, 2016 at 6:46 PM, Leo Yan <leo.yan@linaro.org> wrote:
>>>>> Hi Amit,
>>>>>
>>>>> On Mon, Jun 20, 2016 at 05:46:36PM +0530, Amit Kucheria wrote:
>>>>>> The Hisilicon clock stub driver is needed to allow the thermal drivers to
>>>>>> actually scale the frequency. Make it an automatic dependency.
>>>>>>
>>>>>> Signed-off-by: Amit Kucheria <amit.kucheria@linaro.org>
>>>>>> ---
>>>>>>  drivers/thermal/Kconfig | 1 +
>>>>>>  1 file changed, 1 insertion(+)
>>>>>>
>>>>>> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
>>>>>> index 22ae1f7..4e843f7 100644
>>>>>> --- a/drivers/thermal/Kconfig
>>>>>> +++ b/drivers/thermal/Kconfig
>>>>>> @@ -178,6 +178,7 @@ config THERMAL_EMULATION
>>>>>>  config HISI_THERMAL
>>>>>>       tristate "Hisilicon thermal driver"
>>>>>>       depends on (ARCH_HISI && CPU_THERMAL && OF) || COMPILE_TEST
>>>>>> +     select STUB_CLK_HI6220
>>>>>
>>>>> Acked-by: Leo Yan <leo.yan@linaro.org>
>>>>
>>>> Thanks Leo. Eduardo, will you take this through your tree or should
>>>> Wei include this in his tree?
>>>
>>> Ping!
>
> Hi Amit
>
>>
>> Wei, could you pull this into your tree for the next release?
>
> I am OK If Eduardo is OK to pick up my pull request.
>
> Hi Eduardo,
>
> Is it OK for me to send a pull request to you about this?
> Or could you let us know whether you could directly pick it up?
> Thanks!

Wei, if you can sign-off on this, perhaps Zhang can pick it up? I
haven't heard from Eduardo for over a month.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] thermal: hisilicon: Add dependency on the clock driver to allow frequency scaling
  2016-08-01  6:13             ` Amit Kucheria
@ 2016-08-19  8:02               ` Zhang Rui
  0 siblings, 0 replies; 8+ messages in thread
From: Zhang Rui @ 2016-08-19  8:02 UTC (permalink / raw)
  To: Amit Kucheria, Wei Xu
  Cc: Leo Yan, Guodong Xu, Eduardo Valentin, LKML, Linux PM list, kongxinwei

On 一, 2016-08-01 at 11:43 +0530, Amit Kucheria wrote:
> On Mon, Jul 25, 2016 at 6:49 PM, Wei Xu <xuwei5@hisilicon.com> wrote:
> > 
> > 
> > 
> > On 25/07/2016 13:29, Amit Kucheria wrote:
> > > 
> > > On Thu, Jul 7, 2016 at 12:13 PM, Amit Kucheria <amit.kucheria@lin
> > > aro.org> wrote:
> > > > 
> > > > On Sun, Jun 26, 2016 at 10:02 PM, Amit Kucheria
> > > > <amit.kucheria@linaro.org> wrote:
> > > > > 
> > > > > On Mon, Jun 20, 2016 at 6:46 PM, Leo Yan <leo.yan@linaro.org>
> > > > > wrote:
> > > > > > 
> > > > > > Hi Amit,
> > > > > > 
> > > > > > On Mon, Jun 20, 2016 at 05:46:36PM +0530, Amit Kucheria
> > > > > > wrote:
> > > > > > > 
> > > > > > > The Hisilicon clock stub driver is needed to allow the
> > > > > > > thermal drivers to
> > > > > > > actually scale the frequency. Make it an automatic
> > > > > > > dependency.
> > > > > > > 
> > > > > > > Signed-off-by: Amit Kucheria <amit.kucheria@linaro.org>
> > > > > > > ---
> > > > > > >  drivers/thermal/Kconfig | 1 +
> > > > > > >  1 file changed, 1 insertion(+)
> > > > > > > 
> > > > > > > diff --git a/drivers/thermal/Kconfig
> > > > > > > b/drivers/thermal/Kconfig
> > > > > > > index 22ae1f7..4e843f7 100644
> > > > > > > --- a/drivers/thermal/Kconfig
> > > > > > > +++ b/drivers/thermal/Kconfig
> > > > > > > @@ -178,6 +178,7 @@ config THERMAL_EMULATION
> > > > > > >  config HISI_THERMAL
> > > > > > >       tristate "Hisilicon thermal driver"
> > > > > > >       depends on (ARCH_HISI && CPU_THERMAL && OF) ||
> > > > > > > COMPILE_TEST
> > > > > > > +     select STUB_CLK_HI6220
> > > > > > Acked-by: Leo Yan <leo.yan@linaro.org>
> > > > > Thanks Leo. Eduardo, will you take this through your tree or
> > > > > should
> > > > > Wei include this in his tree?
> > > > Ping!
> > Hi Amit
> > 
> > > 
> > > 
> > > Wei, could you pull this into your tree for the next release?
> > I am OK If Eduardo is OK to pick up my pull request.
> > 
> > Hi Eduardo,
> > 
> > Is it OK for me to send a pull request to you about this?
> > Or could you let us know whether you could directly pick it up?
> > Thanks!
> Wei, if you can sign-off on this, perhaps Zhang can pick it up? I
> haven't heard from Eduardo for over a month.

Patch queued for 4.9.

thanks,
rui

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2016-08-19  8:03 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <32d418f24a4da44ad4ca5b2b1276ffcea2aa7d6c.1466424352.git.amit.kucheria@linaro.org>
2016-06-20 12:16 ` [PATCH] thermal: hisilicon: Add dependency on the clock driver to allow frequency scaling Amit Kucheria
2016-06-20 13:16   ` Leo Yan
2016-06-26 16:32     ` Amit Kucheria
2016-07-07  6:43       ` Amit Kucheria
2016-07-25 12:29         ` Amit Kucheria
2016-07-25 13:19           ` Wei Xu
2016-08-01  6:13             ` Amit Kucheria
2016-08-19  8:02               ` Zhang Rui

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.