RE: [PATCH v2 1/1] thermal: intel Quark SoC X1000 DTS thermal driver

[Ong, Boon Leong]

>-----Original Message-----
>From: Bryan O'Donoghue [mailto:pure.logic@xxxxxxxxxxxxxxxxx]
>Sent: Tuesday, March 3, 2015 12:12 AM
>To: Ong, Boon Leong; Zhang, Rui; edubezval@xxxxxxxxx; Kweh, Hock Leong
>Cc: linux-pm@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
>Subject: Re: [PATCH v2 1/1] thermal: intel Quark SoC X1000 DTS thermal driver
>
>On 25/02/15 16:12, Ong Boon Leong 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.

I can send version-3 which above change if that is agreeable by community & maintainer.

>
>Tested works for me:
>
>root@clanton:~# cat /sys/class/thermal/thermal_zone0/temp
>62
>
>Apply finger to die directly - doesn't burn but it's a bit uncomfortable
Yup... this is how I check also =-) 

Thanks Bryan. 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/