From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1752048AbcEJMnY (ORCPT <rfc822;w@1wt.eu>);
	Tue, 10 May 2016 08:43:24 -0400
Received: from mailout4.w1.samsung.com ([210.118.77.14]:27115 "EHLO
	mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1751498AbcEJMnW (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Tue, 10 May 2016 08:43:22 -0400
X-AuditID: cbfec7f5-f792a6d000001302-3a-5731d766c373
Subject: Re: [PATCH] ARM: dts: exynos: Only Odroid XU3-family boards use DTSI
 with CPU thermal nodes
To: Anand Moon <linux.amoon@gmail.com>
References: <1462774789-15360-1-git-send-email-k.kozlowski@samsung.com>
 <CANAwSgT9SY3kj8+beMpc1xGoC=rMBSya+2v2dyZQp6ykH+V0=g@mail.gmail.com>
 <573198BE.6020900@samsung.com>
 <CANAwSgTy6p9AVmmOWATpSO1JqVbwE0v+y_y-RLd+32RnNJTA+Q@mail.gmail.com>
Cc: Kukjin Kim <kgene@kernel.org>, Lukasz Majewski <l.majewski@samsung.com>,
        Javier Martinez Canillas <javier@osg.samsung.com>,
        devicetree <devicetree@vger.kernel.org>,
        linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
        "linux-samsung-soc@vger.kernel.org" 
	<linux-samsung-soc@vger.kernel.org>,
        Linux Kernel <linux-kernel@vger.kernel.org>,
        Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
From: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Message-id: <5731D764.8070007@samsung.com>
Date: Tue, 10 May 2016 14:43:16 +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: <CANAwSgTy6p9AVmmOWATpSO1JqVbwE0v+y_y-RLd+32RnNJTA+Q@mail.gmail.com>
Content-type: text/plain; charset=utf-8
Content-transfer-encoding: 7bit
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrHLMWRmVeSWpSXmKPExsVy+t/xq7pp1w3DDeY9ELfYOGM9q8X8I+dY
	Ld68XcNk8fqFoUX/49fMFm8ebma02PT4GqvF5V1z2CxmnN/HZLFu4y12By6PnbPusntsWtXJ
	5rF5Sb3Hln4gr2/LKkaPz5vkAtiiuGxSUnMyy1KL9O0SuDJubpzDXrCfvWL+9B62BsavrF2M
	nBwSAiYSq1cfZoOwxSQu3FsPZHNxCAksZZTo3XuEHcJ5xijx4+9KJpAqYYE0ia27FjOD2CIC
	ahJXnq5ghSj6xihx+vAMJhCHWWAGs8T1ly/BOtgEjCU2L18CtoNXQEvi+bTl7CA2i4CqROft
	TkYQW1QgQmL1umvMEDWCEj8m32MBsTkFgiXW7O0HinMADVWXmDIlFyTMLCAvsXnNW+YJjAKz
	kHTMQqiahaRqASPzKkbR1NLkguKk9FwjveLE3OLSvHS95PzcTYyQWPi6g3HpMatDjAIcjEo8
	vAK8huFCrIllxZW5hxglOJiVRHgjLwOFeFMSK6tSi/Lji0pzUosPMUpzsCiJ887c9T5ESCA9
	sSQ1OzW1ILUIJsvEwSnVwJienFnYI/bjt21i7M9b3Y55BZIuSUFvvVgbvjTd+OsT/6+UrdzJ
	+dau1abis+Ikzq86OjFngk1nzztDrRuvmj6um3vmyPo7josXlsYuUZRXWXkocGvd4cBeR7t4
	I5sta6RDoz5NXbjl058PGUuWa9e93x/TbL/D41tgasfn56+dW3l3K7iUP1FiKc5INNRiLipO
	BAD2OOoQgQIAAA==
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

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_SWITCHER.

Best regards,
Krzysztof