[LinuxBIOS] [PATCH] Re: superiotool: conflicting SID for PC97317?
Uwe Hermann
uwe at hermann-uwe.de
Fri Dec 7 22:56:28 CET 2007
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
More information about the coreboot
mailing list