On 21 September 2012 16:02, Zhang Rui <rui.zh...@intel.com> wrote: > 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. OK, thanks. >> >> 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. Get it. > >> 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. I will rebase ST-Ericsson thermal driver against your new framework and try to upstream it then. > > thanks, > rui >
_______________________________________________ linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev