Hello Naveen, On Wed, Jun 25, 2014 at 7:04 AM, Naveen Krishna Ch <naveenkrishna.ch@xxxxxxxxx> wrote: > Doug, > > On 25 June 2014 03:24, Doug Anderson <dianders@xxxxxxxxxxxx> wrote: >> Naveen, >> >> On Tue, Jun 24, 2014 at 5:19 AM, Naveen Krishna Chatradhi >> <ch.naveen@xxxxxxxxxxx> wrote: >>> Exynos5420 based Peach PIT and Exynos5800 based PI boards have >>> 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 <ch.naveen@xxxxxxxxxxx> >>> Cc: Doug Anderson <dianders@xxxxxxxxxxxx> >>> --- >>> This patch needs >>> 1. MAX77802 PMIC device tree nodes (for ldo9) >>> https://www.mail-archive.com/devicetree@xxxxxxxxxxxxxxx/msg31430.html >>> 2. Uses the DT documentation for NTC thermistors updated with vendor-prefix >>> >>> arch/arm/boot/dts/exynos5420-peach-pit.dts | 32 ++++++++++++++++++++++++++++ >>> 1 file changed, 32 insertions(+) >> >> * You forgot: status="okay" for the ADC. One might question how this >> patch was tested. > > Will correct it. > >> >> * Please also include this change for pi. > > MAX77802 device node was not added for Peach_PI (exynos5800-peach-pi.dts). > I guess, Javier have a plan to send. > That's correct. I didn't include the changes to Peach Pi since I only have a Pit to test but I'll add it on the next version so you can have that patch as a dependency and use the MAX77802 LD9 regulator. > I will edit the commit message to remove Peach_PI for now. > > Also, we have few similar nodes on Peach PI and Peach PIT. > Any plans or discussion on merging the common nodes ? > >> >> * We just got through the painful process of sorting this .dts file. >> Please sort "&adc" alphabetically. Kukjin: please confirm that you >> will keep an eye on patches and make sure that they are sorted OK. >> >> >> Other than those problems this seems to work fine to me. >> >> -Doug > > Thanks for the Review. > > > -- > Shine bright, > (: Nav :) Best regards, Javier -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
