Re: [PATCH] powerpc/44x: Add hwmon support to Sequoia device tree

2008-09-09 Thread Matthias Fuchs
On Monday 08 September 2008 18:03, Scott Wood wrote: On Sat, Sep 06, 2008 at 05:07:15PM +0200, Wolfgang Grandegger wrote: How about dtt? That's what we use in U-Boot for digital temperature sensors as well. I vote against any acronyms that are not very well known. Even knowing that it's

Re: [PATCH] powerpc/44x: Add hwmon support to Sequoia device tree

2008-09-09 Thread Josh Boyer
On Tue, Sep 09, 2008 at 01:52:29PM +0200, Matthias Fuchs wrote: On Monday 08 September 2008 18:03, Scott Wood wrote: On Sat, Sep 06, 2008 at 05:07:15PM +0200, Wolfgang Grandegger wrote: How about dtt? That's what we use in U-Boot for digital temperature sensors as well. I vote against any

Re: [PATCH] powerpc/44x: Add hwmon support to Sequoia device tree

2008-09-08 Thread Scott Wood
On Sat, Sep 06, 2008 at 05:07:15PM +0200, Wolfgang Grandegger wrote: How about dtt? That's what we use in U-Boot for digital temperature sensors as well. I vote against any acronyms that are not very well known. Even knowing that it's supposed to be digital temperature something, I can't

Re: [PATCH] powerpc/44x: Add hwmon support to Sequoia device tree

2008-09-06 Thread Josh Boyer
On Fri, Sep 05, 2008 at 08:17:03PM -0400, Sean MacLennan wrote: On Fri, 5 Sep 2008 11:00:18 -0500 Scott Wood [EMAIL PROTECTED] wrote: On Fri, Sep 05, 2008 at 12:19:43PM +0200, Stefan Roese wrote: + +[EMAIL PROTECTED] { Not sure if we shouldn't use

Re: [PATCH] powerpc/44x: Add hwmon support to Sequoia device tree

2008-09-06 Thread Matthias Fuchs
Hi, I have chosen hwmon because those drivers are placed in the kernel's drivers/hwmon subdirectory. temp sounds to much like temporary for me :-) We can name it sensor - inspired by the lm-sensors projects that takes care of such sensors. Because these type of devices are not always used to

Re: [PATCH] powerpc/44x: Add hwmon support to Sequoia device tree

2008-09-06 Thread Sean MacLennan
On Sat, 6 Sep 2008 14:40:41 +0200 Matthias Fuchs [EMAIL PROTECTED] wrote: Sean, I can take your adi instead of analog for this special sensor. I think I was told to use adi because it is the stock market symbol for analog devices. Cheers, Sean

Re: [PATCH] powerpc/44x: Add hwmon support to Sequoia device tree

2008-09-06 Thread Wolfgang Grandegger
Matthias Fuchs wrote: Hi, I have chosen hwmon because those drivers are placed in the kernel's drivers/hwmon subdirectory. temp sounds to much like temporary for me :-) How about dtt? That's what we use in U-Boot for digital temperature sensors as well. We can name it sensor - inspired by

Re: [PATCH] powerpc/44x: Add hwmon support to Sequoia device tree

2008-09-05 Thread Stefan Roese
On Thursday 04 September 2008, Matthias Fuchs wrote: This patch adds support for the AD7414 temperature sensor on Sequoia PPC440EPx board. Signed-off-by: Matthias Fuchs [EMAIL PROTECTED] --- arch/powerpc/boot/dts/sequoia.dts |9 + 1 files changed, 9 insertions(+), 0

Re: [PATCH] powerpc/44x: Add hwmon support to Sequoia device tree

2008-09-05 Thread Matthias Fuchs
Hi, I was inspired by some I2C RTC node (tqm5200.dts). Those nodes are named [EMAIL PROTECTED] But to be in common I can change it. Also the vendor is differnet in warp's dts file. adi (analog device inc.) against analog. So I will update my patch to be compatible with warp.dts. I keep your

Re: [PATCH] powerpc/44x: Add hwmon support to Sequoia device tree

2008-09-05 Thread Scott Wood
On Fri, Sep 05, 2008 at 12:19:43PM +0200, Stefan Roese wrote: + + [EMAIL PROTECTED] { Not sure if we shouldn't use [EMAIL PROTECTED] { here. This is the way it is already done in warp.dts. We shouldn't. Node names are

Re: [PATCH] powerpc/44x: Add hwmon support to Sequoia device tree

2008-09-05 Thread Sean MacLennan
On Fri, 5 Sep 2008 11:00:18 -0500 Scott Wood [EMAIL PROTECTED] wrote: On Fri, Sep 05, 2008 at 12:19:43PM +0200, Stefan Roese wrote: + + [EMAIL PROTECTED] { Not sure if we shouldn't use [EMAIL PROTECTED] { here. This is

[PATCH] powerpc/44x: Add hwmon support to Sequoia device tree

2008-09-04 Thread Matthias Fuchs
This patch adds support for the AD7414 temperature sensor on Sequoia PPC440EPx board. Signed-off-by: Matthias Fuchs [EMAIL PROTECTED] --- arch/powerpc/boot/dts/sequoia.dts |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/arch/powerpc/boot/dts/sequoia.dts