MIME-Version: 1.0
In-Reply-To: <CAOh2x==HB5Jn0iy3dbaL=G0jD5UcP=TwQcxhzQXz9-kbQBdA8A@mail.gmail.com>
References: <1351079900-32236-1-git-send-email-hongbo.zhang@linaro.com>
	<1351615741-24134-1-git-send-email-hongbo.zhang@linaro.com>
	<1351615741-24134-6-git-send-email-hongbo.zhang@linaro.com>
	<CAOh2x==HB5Jn0iy3dbaL=G0jD5UcP=TwQcxhzQXz9-kbQBdA8A@mail.gmail.com>
Date: Tue, 6 Nov 2012 15:25:48 +0800
Message-ID: <CAJLyvQz8_81b9HFU4ffyJrD8b3W_DbgFVGrbkhO=di2cmSeL7A@mail.gmail.com>
Subject: Re: [PATCH V3 5/5] Thermal: Add ST-Ericsson DB8500 thermal properties
 and platform data.
From: Hongbo Zhang <hongbo.zhang@linaro.org>
To: viresh kumar <viresh.kumar@linaro.org>
Cc: linaro-dev@lists.linaro.org, linux-kernel@vger.kernel.org,
        linux-pm@vger.kernel.org, rui.zhang@intel.com, amit.kachhap@linaro.org,
        patches@linaro.org, linaro-kernel@lists.linaro.org,
        STEricsson_nomadik_linux@list.st.com, kernel@igloocommunity.org,
        "hongbo.zhang" <hongbo.zhang@linaro.com>
Content-Type: text/plain; charset=ISO-8859-1
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1952
Lines: 46

On 31 October 2012 10:18, viresh kumar <viresh.kumar@linaro.org> wrote:
> On Tue, Oct 30, 2012 at 10:19 PM, hongbo.zhang <hongbo.zhang@linaro.org> wrote:
>> From: "hongbo.zhang" <hongbo.zhang@linaro.com>
>
> Just a minor comment below.
>
>> This patch adds device tree properties for ST-Ericsson DB8500 thermal driver,
>> also adds the platform data to support the old fashion.
>>
>> Signed-off-by: hongbo.zhang <hongbo.zhang@linaro.com>
>
> Reviewed-by: Viresh Kumar <viresh.kumar@linaro.org>
>
>> ---
>>  arch/arm/boot/dts/dbx5x0.dtsi      | 14 +++++++++
>>  arch/arm/boot/dts/snowball.dts     | 31 ++++++++++++++++++
>>  arch/arm/configs/u8500_defconfig   |  4 +++
>>  arch/arm/mach-ux500/board-mop500.c | 64 ++++++++++++++++++++++++++++++++++++++
>>  4 files changed, 113 insertions(+)
>
>> diff --git a/arch/arm/configs/u8500_defconfig b/arch/arm/configs/u8500_defconfig
>> index cc5e7a8..34918c4 100644
>> --- a/arch/arm/configs/u8500_defconfig
>> +++ b/arch/arm/configs/u8500_defconfig
>> @@ -118,3 +118,7 @@ CONFIG_DEBUG_KERNEL=y
>>  CONFIG_DEBUG_INFO=y
>>  # CONFIG_FTRACE is not set
>>  CONFIG_DEBUG_USER=y
>> +CONFIG_THERMAL=y
>> +CONFIG_CPU_THERMAL=y
>> +CONFIG_DB8500_THERMAL=y
>> +CONFIG_DB8500_CPUFREQ_COOLING=y
>
> Have you entered these manually?? Or used make savedefconfig?
Yes these are added manually, make savedefconfig should be better, but
it seems the original defconfig file are not well generated by
savedefconfig, this file will be changed too much if I use make
savedefconfig. If this is an issue, I will inform the maintainer to
update this.
After consideration, for this time, I will use savedefconfig to find
right places for my configs, discarding the other dis-order ones.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/