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 vot

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 > t

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 ", I can't figure ou

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

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 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 use

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

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] { >

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 ar

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 ac

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 del

[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 b/arch/