[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 <jordan at chalmers.se>
> 
> 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.
-- 
http://www.hermann-uwe.de  | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org