From mboxrd@z Thu Jan 1 00:00:00 1970 From: Naveen Krishna Ch Subject: Re: [PATCH 2/3] ARM: DTS: Add NTC thermistor nodes to Exynos5250 based Snow Date: Thu, 26 Jun 2014 22:23:40 +0530 Message-ID: References: <1403785148-20792-1-git-send-email-ch.naveen@samsung.com> <1403785148-20792-3-git-send-email-ch.naveen@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: Sender: linux-iio-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Doug Anderson , Kukjin Kim Cc: Naveen Krishna Chatradhi , linux-samsung-soc , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , linux-iio , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Greg Kroah-Hartman , "cpgs ." , Tushar Behera List-Id: devicetree@vger.kernel.org Hello Doug and Kukjin, On 26 June 2014 21:16, Doug Anderson wrote: > Naveen, > > On Thu, Jun 26, 2014 at 5:19 AM, Naveen Krishna Chatradhi > wrote: >> Exynos5250 based Snow board has 4 NTC thermistors to measure >> temperatures at various points on the board. >> >> IIO based ADC becomes the parent and NTC thermistors are the childs, >> via the HWMON interface. >> >> Signed-off-by: Naveen Krishna Chatradhi >> --- >> Posted earlier by Doug Anderson @ https://lkml.org/lkml/2013/3/27/453 >> >> This patch depends on (1/4 and 2/4 patches of) patchset posted >> http://www.spinics.net/lists/linux-iio/msg13486.html >> Which were applied on to Guenter Roeck's tree. >> >> cat sysfs entries exported by hwmon for 4 thermistors >> and verified the values on Snow. >> >> arch/arm/boot/dts/exynos5250-snow.dts | 34 +++++++++++++++++++++++++++++++++ >> 1 file changed, 34 insertions(+) > > NAK. > > The first chunk of exynos5250-snow devices have the thermistors > populated, but a huge chunk of devices also _don't_ have them > populated. I've realized only the rev4 and before boards have thermistors while going through the git logs of snow related dts files in an older chrome kernel. > > If I remember my history properly, rev3 and earlier all had > thermistors. Some of rev4 might have thermistors (I never got a clear > answer). ...and rev5 definitely doesn't have resistors. Aside from > thermistors there's no good reason to differentiate rev3 and rev4 > (they just have different memory). The upstream kernel may eventually > need to differentiate rev4 and rev5 since they have a different audio > codec. I've found one patch which says, "No thermistors on Rev5 and above boards" I still thought, we should support few boards out there. But, i din't knew they were removed completely. > > See for some > descriptions of the different revisions of snow and how they were > handled in the Chrome OS tree. Got it now, Thanks > > See for > thermistors talk. Patch set #1 actually split out rev3, but we then > decided that we really didn't need to use the thermistors on any of > the revisions so the later patchsets just totally take them out. So, no need of thermistor nodes on Snow. I thought https://lkml.org/lkml/2013/3/27/453 patch is missed out. > > -Doug Kukjin, Sorry for the confusion. Please drop this one. -- Thanks, (: Nav :)