* Re: [lm-sensors] [PATCH] lm-sensors: update info for SMSC LPC47B357
2007-10-18 21:57 [lm-sensors] [PATCH] lm-sensors: update info for SMSC LPC47B357 Juerg Haefliger
@ 2007-10-19 12:47 ` Jean Delvare
2007-10-19 16:16 ` Juerg Haefliger
2007-10-19 21:48 ` Jean Delvare
2 siblings, 0 replies; 4+ messages in thread
From: Jean Delvare @ 2007-10-19 12:47 UTC (permalink / raw)
To: lm-sensors
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
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [lm-sensors] [PATCH] lm-sensors: update info for SMSC LPC47B357
2007-10-18 21:57 [lm-sensors] [PATCH] lm-sensors: update info for SMSC LPC47B357 Juerg Haefliger
2007-10-19 12:47 ` Jean Delvare
@ 2007-10-19 16:16 ` Juerg Haefliger
2007-10-19 21:48 ` Jean Delvare
2 siblings, 0 replies; 4+ messages in thread
From: Juerg Haefliger @ 2007-10-19 16:16 UTC (permalink / raw)
To: lm-sensors
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
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [lm-sensors] [PATCH] lm-sensors: update info for SMSC LPC47B357
2007-10-18 21:57 [lm-sensors] [PATCH] lm-sensors: update info for SMSC LPC47B357 Juerg Haefliger
2007-10-19 12:47 ` Jean Delvare
2007-10-19 16:16 ` Juerg Haefliger
@ 2007-10-19 21:48 ` Jean Delvare
2 siblings, 0 replies; 4+ messages in thread
From: Jean Delvare @ 2007-10-19 21:48 UTC (permalink / raw)
To: lm-sensors
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
^ permalink raw reply [flat|nested] 4+ messages in thread