Re: [LinuxBIOS] [PATCH] Re: superiotool: conflicting SID for PC97317?

[Uwe Hermann]

On Fri, Dec 07, 2007 at 07:59:21PM +0100, Ulf Jordan wrote:
> > There was never a PC97371 made by National.
> >
> > -Bari
>
> Attched is a patch that fixes the typos in superiotool/nsc.c.
>
> /ulf
> Fix typo. According to National's datasheet PC87317 has SID = 0xd0 and
> PC97317 has SID = 0xdf. PC87371/PC97371 do not seem to exist.
>
> Signed-off-by: Ulf Jordan
> Index: nsc.c
>
> --- nsc.c	(revision 2998)
> +++ nsc.c	(working copy)
> @@ -25,9 +25,9 @@
>  #define CHIP_REV_REG	0x27	/* Super I/O revision ID (SRID) */
>
> static const struct superio_registers reg_table[] = {
> {0xd0, "PC87371", {	/* From sensors-detect */
> {0xd0, "PC87317", {
> {EOT}}},
> {0xdf, "PC97371", {	/* From sensors-detect */
> {0xdf, "PC97317", {
> {EOT}}},
> {0xe1, "PC87360", {
> {NOLDN, NULL,

Thanks, r2999.

I agree with both of you that this seems to be a typo.

Please also send a patch to the lm-sensors list to fix sensors-detect.

Uwe.