Diff's for ASUS P8P67 Pro/Nuvoton NCT6776F Support

[GoogleCodeExporter]

The chipset info would probably work for all the ASUS LGA 1155 boards. We just 
need board names for the Deluxe's etc... Lemme know if I should check in/how to 
check in...

brian@lillehammar /cygdrive/d/src/open-hardware-monitor-read-only
$ svn status
?       obj
M       Hardware/Mainboard/SMBIOS.cs
M       Hardware/Mainboard/Model.cs
M       Hardware/Mainboard/SuperIOHardware.cs
M       Hardware/LPC/Chip.cs
M       Hardware/LPC/LPCIO.cs

brian@lillehammar /cygdrive/d/src/open-hardware-monitor-read-only
$ svn diff Hardware/Mainboard/SMBIOS.cs
Index: Hardware/Mainboard/SMBIOS.cs
===================================================================
--- Hardware/Mainboard/SMBIOS.cs        (revision 284)
+++ Hardware/Mainboard/SMBIOS.cs        (working copy)
@@ -322,6 +322,8 @@
             return Model.P5W_DH_Deluxe;
           case "P6X58D-E":
             return Model.P6X58D_E;
+          case "P8P67 PRO":
+            return Model.P8P67_PRO;
           case "Rampage Extreme":
             return Model.Rampage_Extreme;
           case "Rampage II GENE":

brian@lillehammar /cygdrive/d/src/open-hardware-monitor-read-only
$ svn diff Hardware/Mainboard/Model.cs
Index: Hardware/Mainboard/Model.cs
===================================================================
--- Hardware/Mainboard/Model.cs (revision 284)
+++ Hardware/Mainboard/Model.cs (working copy)
@@ -49,6 +49,7 @@
     M4A79XTD_EVO,
     P5W_DH_Deluxe,
     P6X58D_E,
+    P8P67_PRO,
     Rampage_Extreme,
     Rampage_II_GENE,

brian@lillehammar /cygdrive/d/src/open-hardware-monitor-read-only
$ svn diff Hardware/Mainboard/SuperIOHardware.cs
Index: Hardware/Mainboard/SuperIOHardware.cs
===================================================================
--- Hardware/Mainboard/SuperIOHardware.cs       (revision 284)
+++ Hardware/Mainboard/SuperIOHardware.cs       (working copy)
@@ -649,7 +649,7 @@
               break;
             case Manufacturer.ASUS:
               switch (model) {
-                case Model.P6X58D_E: // W83667HG
+                case Model.P6X58D_E: // W83667HG
                 case Model.Rampage_II_GENE: // W83667HG
                   v.Add(new Voltage("CPU VCore", 0));
                   v.Add(new Voltage("+12V", 1, 11.5f, 1.91f));
@@ -742,6 +742,7 @@
           f.Add(new Fan("Auxiliary Fan", 2));
           break;
         case Chip.NCT6771F:
+        case Chip.NCT6776F:
           v.Add(new Voltage("CPU VCore", 0));
           v.Add(new Voltage("Voltage #2", 1, true));
           v.Add(new Voltage("AVCC", 2, 34, 34));

brian@lillehammar /cygdrive/d/src/open-hardware-monitor-read-only
$ svn diff Hardware/LPC/Chip.cs
Index: Hardware/LPC/Chip.cs
===================================================================
--- Hardware/LPC/Chip.cs        (revision 284)
+++ Hardware/LPC/Chip.cs        (working copy)
@@ -54,6 +54,7 @@
     W83667HGB = 0xB350,
     W83687THF = 0x8541,
     NCT6771F = 0xB470,
+    NCT6776F = 0xC333,
     F71858 = 0x0507,
     F71862 = 0x0601,
     F71869 = 0x0814,
@@ -89,6 +90,7 @@
         case Chip.W83667HGB: return "Winbond W83667HG-B";
         case Chip.W83687THF: return "Winbond W83687THF";
         case Chip.NCT6771F: return "Nuvoton NCT6771F";
+        case Chip.NCT6776F: return "Nuvoton NCT6776F";
         case Chip.Unknown: return "Unkown";
         default: return "Unknown";
       }

brian@lillehammar /cygdrive/d/src/open-hardware-monitor-read-only
$ svn diff Hardware/LPC/LPCIO.cs
Index: Hardware/LPC/LPCIO.cs
===================================================================
--- Hardware/LPC/LPCIO.cs       (revision 284)
+++ Hardware/LPC/LPCIO.cs       (working copy)
@@ -220,6 +220,15 @@
               logicalDeviceNumber = WINBOND_NUVOTON_HARDWARE_MONITOR_LDN;
               break;
           } break;
+        case 0xC3:
+            int snot = revision & 0xF0;
+            switch (revision & 0xF0)
+            {
+                case 0x30:
+                    chip = Chip.NCT6776F;
+                    logicalDeviceNumber = WINBOND_NUVOTON_HARDWARE_MONITOR_LDN;
+                    break;
+            } break;
       }
       if (chip == Chip.Unknown) {
         if (id != 0 && id != 0xff) {
@@ -279,6 +288,7 @@
             superIOs.Add(new W836XX(chip, revision, address));
             break;
           case Chip.NCT6771F:
+          case Chip.NCT6776F:
             superIOs.Add(new NCT677X(chip, revision, address));
             break;
           case Chip.F71858:

Original issue reported on code.google.com by brian.ha...@gmail.com on 26 Jan 2011 at 6:18

[GoogleCodeExporter]

Thank you very much for the diff. I am still waiting for the NCT6776F datasheet 
to see if there are any relevant changes or additional features. If I can't get 
it within reasonable time, I will just use your patch so that the same 
implementation as for the NCT6771F is used. In any case it's good to see that 
not too much has changed in the NCT6776F. 

Original comment by moel.mich on 27 Jan 2011 at 9:43

• Changed state: Accepted
• Added labels: Type-Enhancement
• Removed labels: Type-Defect

[GoogleCodeExporter]

Perhaps Hardware/Mainboard/SuperIOHardware.cs  line 652 be:

                case Model.P6X58D_E: // W83667HG
                case Model.P8P67_PRO: // W83667HG

FYI I applied these mods to a local build and it seems to work fine. 
Temperatures and Fans read correctly.  For Voltages I get:

CPU VCore, AVCC, 3VCC, 3VSB, VBAT  which all agree w/ the latest speedfan beta.

I also get: Voltage #2, 5, 6, and 7 which are greyed out.  Matching them w/ 
speedfan values, it looks like:

Voltage #5 = VIN1
Voltage #6 = VIN2
Voltage #7 = VIN3

Not sure what Voltage #2 is - it reads identical to #5 (1.02V).  Speedfan has a 
final voltage labeled +12V which I guess might be Voltage #2 except speedfan 
shows it as 6.71V and #2 reads 1.02V.

I guess I'm don't understand the code well enough - it looks like I should 
dropping into the PRO enum but it's I think it's going to the default.  I guess 
I'll wait for an official copy...

Original comment by ted.drain@gmail.com on 12 Feb 2011 at 1:21

[GoogleCodeExporter]

This issue was closed by revision r299.

Original comment by moel.mich on 19 Mar 2011 at 10:55

• Changed state: Fixed