From mboxrd@z Thu Jan 1 00:00:00 1970 From: Naveen Krishna Ch Subject: Re: [PATCH 4/4 v2] ARM: DTS: Add NTC thermistor nodes to Exynos5420 based Peach_PIT Date: Thu, 26 Jun 2014 11:46:59 +0530 Message-ID: References: <1403677773-21141-1-git-send-email-ch.naveen@samsung.com> <1403677773-21141-5-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 Cc: Naveen Krishna Chatradhi , linux-samsung-soc , linux-iio , Kukjin Kim , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , "cpgs ." , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Lars-Peter Clausen List-Id: devicetree@vger.kernel.org Doug, On 26 June 2014 02:10, Doug Anderson wrote: > Naveen, > > On Tue, Jun 24, 2014 at 11:29 PM, Naveen Krishna Chatradhi > wrote: >> Exynos5420 based Peach PIT 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 >> Cc: Doug Anderson >> --- >> Changes since v1: >> 1. Arranged the ADC node alphabetical >> 2. Added "status = "okay"" to the ADC node >> >> This patch needs >> 1. MAX77802 PMIC device tree nodes (for ldo9) >> https://www.mail-archive.com/devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org/msg31430.html >> 2. Uses the DT documentation for NTC thermistors updated with vendor-prefix >> >> This patch set is tested >> 1. On Peach PIT board via sysfs entry exposed by hwmon >> localhost ~ # cat /sys/class/hwmon/hwmon*/device/temp1_input >> 37436 >> 36449 >> 38560 >> 38059 >> >> 2. Backward compatibility is tested by replacing "murata" with "ntc" >> in the dts nodes and running the above test. >> >> arch/arm/boot/dts/exynos5420-peach-pit.dts | 34 ++++++++++++++++++++++++++++ >> 1 file changed, 34 insertions(+) > > Now that Javier has spun his patch, perhaps you can fold the "pi" > version in here too? Sure, thanks for the information. Snow also needs the adc nodes. Will try to implement an adc fragment for PIT, PI and Snow. > > Reviewed-by: Doug Anderson > Tested-by: Doug Anderson -- Shine bright, (: Nav :)