lkml.org 
[lkml]   [2015]   [Mar]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH v2 1/1] thermal: intel Quark SoC X1000 DTS thermal driver
Date
>-----Original Message-----
>From: Andy Shevchenko [mailto:andy.shevchenko@gmail.com]
>Sent: Friday, March 6, 2015 12:11 AM
>To: Ong, Boon Leong
>Cc: Bryan O'Donoghue; Zhang, Rui; edubezval@gmail.com; Kweh, Hock Leong;
>linux-pm@vger.kernel.org; linux-kernel@vger.kernel.org
>Subject: Re: [PATCH v2 1/1] thermal: intel Quark SoC X1000 DTS thermal driver
>
>On Tue, Mar 3, 2015 at 3:13 AM, Ong, Boon Leong
><boon.leong.ong@intel.com> wrote:
>>>> +config INTEL_QUARK_DTS_THERMAL
>>>> + tristate "Intel Quark DTS thermal driver"
>>>> + depends on X86 && IOSF_MBI
>>>
>>>Since the change is in RC-1 suggest
>>>depends on X86_INTEL_QUARK
>>>you can skip this though it's not important.
>> Ya. That is true now since the below commit has been merged:
>> 8bbc2a135b63bee6b41fa90f415521a97995a49f to mainline in 4.0-rc1
>>
>> On another thought, how about adding INTEL_QUARK_DTS_THERMAL to
>> be selected by X86_INTEL_QUARK in arch/x86/Kconfig ... and we can
>> drop the above dependencies from here.
>
>This driver as far as I understand is not critical to run Intel Quark.
>So, there is should not be a dependency in the platform definition.
Yup, Quark platform can boot up without this driver selected by default.

>
>On the other hand is the driver will work exclusively on Quark? If so,
>better to leave the mentioned dependency, .i.e.
>depends on X86_INTEL_QUARK
Yup. Currently, it is exclusive to Quark.
In v3, I make it depends on X86_INTEL_QUARK only.

\
 
 \ /
  Last update: 2015-03-05 17:41    [W:0.048 / U:0.272 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site