From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752543AbcEJOD3 (ORCPT ); Tue, 10 May 2016 10:03:29 -0400 Received: from mailout3.w1.samsung.com ([210.118.77.13]:36251 "EHLO mailout3.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752125AbcEJODN (ORCPT ); Tue, 10 May 2016 10:03:13 -0400 X-AuditID: cbfec7f4-f796c6d000001486-49-5731ea1d7da7 Subject: Re: [PATCH] ARM: dts: exynos: Only Odroid XU3-family boards use DTSI with CPU thermal nodes To: Anand Moon References: <1462774789-15360-1-git-send-email-k.kozlowski@samsung.com> <573198BE.6020900@samsung.com> <5731D764.8070007@samsung.com> Cc: Kukjin Kim , Lukasz Majewski , Javier Martinez Canillas , devicetree , linux-arm-kernel , "linux-samsung-soc@vger.kernel.org" , Linux Kernel , Bartlomiej Zolnierkiewicz From: Krzysztof Kozlowski Message-id: <5731EA1B.4070201@samsung.com> Date: Tue, 10 May 2016 16:03:07 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.7.2 MIME-version: 1.0 In-reply-to: Content-type: text/plain; charset=utf-8 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrHLMWRmVeSWpSXmKPExsVy+t/xq7qyrwzDDb5ulLLYOGM9q8X8I+dY Ld68XcNk8fqFoUX/49fMFm8ebma02PT4GqvF5V1z2CxmnN/HZLFu4y12By6PnbPusntsWtXJ 5rF5Sb3Hln4gr2/LKkaPz5vkAtiiuGxSUnMyy1KL9O0SuDKenD7FWDBZsGLTtIVMDYxzeboY OTkkBEwkdrWvZ4KwxSQu3FvP1sXIxSEksJRRYsr6d4wQzjNGib/9ncwgVcICaRJbdy0Gs0UE 1CSuPF3BClF0j0miq38VM4jDLDCDWeL6y5dgc9kEjCU2L1/CBmLzCmhJ7Jj9lBXEZhFQlbi7 6DhYjahAhMTqddeYIWoEJX5MvscCYnMKBEs8/LYNKM4BNFRdYsqUXJAws4C8xOY1b5knMArM QtIxC6FqFpKqBYzMqxhFU0uTC4qT0nMN9YoTc4tL89L1kvNzNzFCYuHLDsbFx6wOMQpwMCrx 8O7gMgwXYk0sK67MPcQowcGsJMJb/BAoxJuSWFmVWpQfX1Sak1p8iFGag0VJnHfurvchQgLp iSWp2ampBalFMFkmDk6pBsZVbq1/PK5bxMQ1/PsYmpiXsGFl8TqF9ZvmXCjckyNcxmfAd1hC cirrdYflrxpTJ6SLvrIV+cdr+KPw89UAJ/8kl7DlkvVsgc+Llli+e6735MaWxuiaVb/M1iTb T1HJ0dwq83Riv9/n0CK5cz8/2fI02pn+5OB2PX5l8qxlTMrml+1v5tZrLVRiKc5INNRiLipO BACWl5fWgQIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 05/10/2016 04:00 PM, Anand Moon wrote: > Hi Krzysztof, > > On 10 May 2016 at 18:13, Krzysztof Kozlowski 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 ]