[lm-sensors] [PATCH] lm-sensors: update info for SMSC LPC47B357

[lm-sensors] [PATCH] lm-sensors: update info for SMSC LPC47B357

[Juerg Haefliger]

[-- Attachment #1: Type: text/plain, Size: 112 bytes --]

Update the info for the SMSC LPC47B357 in sensors-detect.

Signed-off-by: Juerg Haefliger <juergh at gmail.com>

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: sensors-detect-update-for-lpc47b357.patch --]
[-- Type: text/x-patch; name=sensors-detect-update-for-lpc47b357.patch, Size: 492 bytes --]

Index: lm-sensors/prog/detect/sensors-detect
===================================================================
--- lm-sensors.orig/prog/detect/sensors-detect	2007-10-18 14:54:16.417255000 -0700
+++ lm-sensors/prog/detect/sensors-detect	2007-10-18 14:55:16.105166000 -0700
@@ -1752,8 +1752,8 @@
 	devid => 0x56,
       },
       {
-	name => "SMSC LPC47B357 Super IO",
-	# no datasheet
+	name => "SMSC LPC47B357/M967 Super IO",
+	driver => "not-a-sensor",
 	devid => 0x5d,
       },
       {

[-- Attachment #3: Type: text/plain, Size: 153 bytes --]

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: [lm-sensors] [PATCH] lm-sensors: update info for SMSC LPC47B357

[Jean Delvare]

Hi Juerg,

On Thu, 18 Oct 2007 14:57:55 -0700, Juerg Haefliger wrote:
> Update the info for the SMSC LPC47B357 in sensors-detect.
> 
> Signed-off-by: Juerg Haefliger <juergh at gmail.com>

> Index: lm-sensors/prog/detect/sensors-detect
> =================================> --- lm-sensors.orig/prog/detect/sensors-detect	2007-10-18 14:54:16.417255000 -0700
> +++ lm-sensors/prog/detect/sensors-detect	2007-10-18 14:55:16.105166000 -0700
> @@ -1752,8 +1752,8 @@
>  	devid => 0x56,
>        },
>        {
> -	name => "SMSC LPC47B357 Super IO",
> -	# no datasheet
> +	name => "SMSC LPC47B357/M967 Super IO",
> +	driver => "not-a-sensor",
>  	devid => 0x5d,
>        },
>        {

Are you sure of yourself? I've found this article:
http://www.digit-life.com/articles2/roundupmobo/fs-d1627-a-i865pe.html

"System monitoring is supported by the SMSC LPC47M967-NC chip (this
chip (Hardware Monitor + Super I/O) is rarely used, we happened to see
it only on Intel and FIC models). It controls: 

    * processor, AGP bus voltages, +3.3, +5 and +12 V;
    * rotation speed of 2 ordinary fans and one in the power supply
      unit (it requires a special cable connected to the board);
    * temperatures of the processor (built-in sensor), the board
      (built-in sensor) and an external temperature sensor (supplied)."

Could it be that LPC47M967 = LPC47B357 + hardware monitoring?

Of course it's also possible that the article is completely wrong...

-- 
Jean Delvare

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: [lm-sensors] [PATCH] lm-sensors: update info for SMSC LPC47B357

[Juerg Haefliger]

Hi Jean,


On 10/19/07, Jean Delvare <khali@linux-fr.org> wrote:
> Hi Juerg,
>
> On Thu, 18 Oct 2007 14:57:55 -0700, Juerg Haefliger wrote:
> > Update the info for the SMSC LPC47B357 in sensors-detect.
> >
> > Signed-off-by: Juerg Haefliger <juergh at gmail.com>
>
> > Index: lm-sensors/prog/detect/sensors-detect
> > =================================> > --- lm-sensors.orig/prog/detect/sensors-detect        2007-10-18 14:54:16.417255000 -0700
> > +++ lm-sensors/prog/detect/sensors-detect     2007-10-18 14:55:16.105166000 -0700
> > @@ -1752,8 +1752,8 @@
> >       devid => 0x56,
> >        },
> >        {
> > -     name => "SMSC LPC47B357 Super IO",
> > -     # no datasheet
> > +     name => "SMSC LPC47B357/M967 Super IO",
> > +     driver => "not-a-sensor",
> >       devid => 0x5d,
> >        },
> >        {
>
> Are you sure of yourself? I've found this article:
> http://www.digit-life.com/articles2/roundupmobo/fs-d1627-a-i865pe.html
>
> "System monitoring is supported by the SMSC LPC47M967-NC chip (this
> chip (Hardware Monitor + Super I/O) is rarely used, we happened to see
> it only on Intel and FIC models). It controls:
>
>     * processor, AGP bus voltages, +3.3, +5 and +12 V;
>     * rotation speed of 2 ordinary fans and one in the power supply
>       unit (it requires a special cable connected to the board);
>     * temperatures of the processor (built-in sensor), the board
>       (built-in sensor) and an external temperature sensor (supplied)."
>
> Could it be that LPC47M967 = LPC47B357 + hardware monitoring?

I received the datasheets for both chips. Both of them advertise
themselves as 0x5d and neither has any HW monitoring capabilities.
Sorry, should have mentioned that in the patch email.

...juerg


> Of course it's also possible that the article is completely wrong...
>
> --
> Jean Delvare
>

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: [lm-sensors] [PATCH] lm-sensors: update info for SMSC LPC47B357

[Jean Delvare]

On Fri, 19 Oct 2007 09:16:51 -0700, Juerg Haefliger wrote:
> On 10/19/07, Jean Delvare <khali@linux-fr.org> wrote:
> > On Thu, 18 Oct 2007 14:57:55 -0700, Juerg Haefliger wrote:
> > > Update the info for the SMSC LPC47B357 in sensors-detect.
> > >
> > > Signed-off-by: Juerg Haefliger <juergh at gmail.com>
> >
> > > Index: lm-sensors/prog/detect/sensors-detect
> > > =================================> > > --- lm-sensors.orig/prog/detect/sensors-detect        2007-10-18 14:54:16.417255000 -0700
> > > +++ lm-sensors/prog/detect/sensors-detect     2007-10-18 14:55:16.105166000 -0700
> > > @@ -1752,8 +1752,8 @@
> > >       devid => 0x56,
> > >        },
> > >        {
> > > -     name => "SMSC LPC47B357 Super IO",
> > > -     # no datasheet
> > > +     name => "SMSC LPC47B357/M967 Super IO",
> > > +     driver => "not-a-sensor",
> > >       devid => 0x5d,
> > >        },
> > >        {
> >
> > Are you sure of yourself? I've found this article:
> > http://www.digit-life.com/articles2/roundupmobo/fs-d1627-a-i865pe.html
> >
> > "System monitoring is supported by the SMSC LPC47M967-NC chip (this
> > chip (Hardware Monitor + Super I/O) is rarely used, we happened to see
> > it only on Intel and FIC models). It controls:
> >
> >     * processor, AGP bus voltages, +3.3, +5 and +12 V;
> >     * rotation speed of 2 ordinary fans and one in the power supply
> >       unit (it requires a special cable connected to the board);
> >     * temperatures of the processor (built-in sensor), the board
> >       (built-in sensor) and an external temperature sensor (supplied)."
> >
> > Could it be that LPC47M967 = LPC47B357 + hardware monitoring?
> 
> I received the datasheets for both chips. Both of them advertise
> themselves as 0x5d and neither has any HW monitoring capabilities.
> Sorry, should have mentioned that in the patch email.

OK, so the article was crap. Thanks for the clarification, I've applied
your patch now.

-- 
Jean Delvare

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors