Re: Re: [PATCH v2 3/5] ARM: dt: tegra: add ADT7461 temperature sensor topaz00 device tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am Mittwoch, 1. Februar 2012, 00:26:04 schrieb Olof Johansson:
> On Tue, Jan 31, 2012 at 07:53:21PM +0100, Marc Dietrich wrote:
> > Add the ADT7461 temperature monitoring IC which is connected via
> > the DVC controller.
> > 
> > Signed-off-by: Marc Dietrich <marvin24@xxxxxx>
> > ---
> > 
> >  arch/arm/boot/dts/tegra-paz00.dts |    5 +++++
> >  1 files changed, 5 insertions(+), 0 deletions(-)
> > 
> > diff --git a/arch/arm/boot/dts/tegra-paz00.dts b/arch/arm/boot/dts/tegra-
> > paz00.dts
> > index f667343..81e1d30 100644
> > --- a/arch/arm/boot/dts/tegra-paz00.dts
> > +++ b/arch/arm/boot/dts/tegra-paz00.dts
> > @@ -35,6 +35,11 @@
> > 
> >         i2c@7000d000 {
> >         
> >                 clock-frequency = <400000>;
> > 
> > +
> > +               adt7461@4c {
> > +                       compatible = "adi,adt7461";
> > +                       reg = <0x4c>;
> > +               };
> 
> It's common to name the device node temp-sensor (or temperature-sensor) instead
> of the model of the actual chip. The name isn't significant for the binding,
> but it's a practice that's been used on other platforms for a long time.
> 
> I'll just fix it up when applying, if ok with you. No need to respin just for
> this.

Renaming it to something like "temp-sensor" is also ok for me. Feel free to change 
this.

Thanks!

Marc


--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux