On 05/10/2016 04:00 PM, Anand Moon wrote: > Hi Krzysztof, > > On 10 May 2016 at 18:13, Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx> wrote: >> On 05/10/2016 11:29 AM, Anand Moon wrote: >>>> >>>> I am sorry but I do not understand. >>>> 1. Are you saying that my patch causes the issue? >>>> 2. Are you saying that I should not send some patches because of >>>> conflicting work? (that would be peculiar...) >>>> 3. Are you saying that I should not apply this patch to me tree because >>>> of conflicting work? >>>> 4. Other? >>>> >>>> Best regards. >>>> Krzysztof >>>> >>>> >>> >>> Sorry for the confusion it's not about you patch. >>> The feature is not working correctly as per my testing with the >>> original source code. >>> It need to be more tuned or some other approach to the issue need to found. >> >> Known issue. You don't have cpufreq cooling enabled. I sent patches for >> that and they are in the mainline. However cpufreq cooling requires >> cpufreq which cannot work with BL_SWITCHER and on Exynos542x on >> exynos_defconfig BL_SWITCHER is enabled. >> >> Just disable the BL_Krzysztof. >> >> Best regards, >> Krzysztof > > I have already disabled BL_SWITCHER in my exynos_defconfig. to support > CONFIG_CPUFREQ_DT. > > But how about adjust the the cputemp so that all the core will have > same tip points. I don't understand you. All of the cores have same trip points. Best regards, Krzysztof > > xcpu0-t 0:[ 44][>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> A50 > A60 P70 P85 C120 > ] > xcpu1-t 1:[ 46][>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> > A85 A103 A110 C120 > ] > xcpu2-t 2:[ 51][>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> > A85 A103 A110 C120 ] > xcpu3-t 3:[ 47][>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> > A85 A103 A110 C120 ] > xgpu-th 4:[ 48][>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> > A85 A103 A110 C120 ] -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |