On 五, 2012-09-21 at 15:50 +0800, Hongbo Zhang wrote:
> On 21 September 2012 15:21, Zhang Rui <rui.zh...@intel.com> wrote:
> > On 五, 2012-09-21 at 14:57 +0800, zhanghongbo wrote:
> >> From: "hongbo.zhang" <hongbo.zh...@linaro.com>
> >>
> >> This patch set contains two patches.
> >>
> >> [PATCH 1/2]
> >> A new interface is introduced to deactive all the referenced cooling 
> >> devices
> >> when thermal zone is disabled.
> >
> > we can not deactive a cooling device directly.
> > we should deactive all the thermal_instances for this thermal zone.
> > because a cooling device may be referenced in multiple thermal zones.
> Understand.
> >
> >> Because the cooling device list is maintained
> >> in the generic thermal layer, the thermal driver cannot walk through the 
> >> list
> >> and cannot deactive its cooling devices either.
> >> This interface is needed in the .set_mode callback when the thermal zone 
> >> mode
> >> is set to "disabled".
> >>
> > Durga is introducing the cooling policy for the generic thermal layer
> > and one of them is "userspace".
> > if we set the policy to userspace, the generic thermal layer will do
> > nothing but getting input from userspace.
> > if we have an API to change the cooling policy for a thermal zone, can
> > this be used instead?
> The reason I sent out these patches is that I found out deactivation
> of cooling devices
> is necessary, and I didn't see any update of thermal framework recently.
> Since you are still working on this framework, can you consider this
> deactivation
> function in your next version? or need I resend again? I think it is
> better you do it.

yes, I'll do it.
> 
> Another propose is that let the thermal driver walk through its
> cooling device list is
> also important I think, if "userspace" mode is introduced, the thermal
> driver needs to
> manipulate its cooling device, the thermal driver cannot do this
> without knowing its
> cooling devices list. (one method to achieve this is as my last patch
> several weeks ago)
> 
thermal_instance list is enough for this, because a cooling device may
be referenced in multiple trip points for one thermal zone, the thermal
zone device should just deactive all the thermal instances when using
"userspace" governor.

> Another question, when will you update your framework, v3.7?

as this change is introduced recently, I'm not sure if we can catch up
3.7 merge window.

thanks,
rui


_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to