Add support for ASUS ROG Crosshair X870E Hero motherboard (#1581)

* Add support for ASUS ROG Crosshair X870E Hero motherboard

* Fix a typo

* Add CPU Package temperature sensor.

* Cleanup code

* Aligning W_PUMP+ naming to others

Author: jtgorney

LibreHardwareMonitorLib/Hardware/Motherboard/Identification.cs

@@ -607,6 +607,8 @@ public static Model GetModel(string name)
                 return Model.X870E_AORUS_PRO;
             case var _ when name.Equals("X870E AORUS PRO ICE", StringComparison.OrdinalIgnoreCase):
                 return Model.X870E_AORUS_PRO_ICE;
+            case var _ when name.Equals("ROG CROSSHAIR X870E HERO", StringComparison.OrdinalIgnoreCase):
+                return Model.ROG_CROSSHAIR_X870E_HERO;
             case var _ when name.Equals("Base Board Product Name", StringComparison.OrdinalIgnoreCase):
             case var _ when name.Equals("To be filled by O.E.M.", StringComparison.OrdinalIgnoreCase):
                 return Model.Unknown;

LibreHardwareMonitorLib/Hardware/Motherboard/Lpc/Chip.cs

@@ -69,6 +69,7 @@ internal enum Chip : ushort
     NCT6687D = 0xD592,
     NCT6683D = 0xC732,
     NCT6799D = 0xD802,
+    NCT6701D = 0xD806,
 
     W83627DHG = 0xA020,
     W83627DHGP = 0xB070,
@@ -142,6 +143,7 @@ public static string GetName(Chip chip)
             case Chip.NCT6686D: return "Nuvoton NCT6686D";
             case Chip.NCT6687D: return "Nuvoton NCT6687D";
             case Chip.NCT6683D: return "Nuvoton NCT6683D";
+            case Chip.NCT6701D: return "Nuvoton NCT6701D";
 
             case Chip.W83627DHG: return "Winbond W83627DHG";
             case Chip.W83627DHGP: return "Winbond W83627DHG-P";

LibreHardwareMonitorLib/Hardware/Motherboard/Lpc/LpcIO.cs

@@ -403,6 +403,10 @@ private bool DetectWinbondFintek(LpcPort port)
                         chip = Chip.NCT6799D;
                         logicalDeviceNumber = WINBOND_NUVOTON_HARDWARE_MONITOR_LDN;
                         break;
+                    case 0x06:
+                        chip = Chip.NCT6701D;
+                        logicalDeviceNumber = WINBOND_NUVOTON_HARDWARE_MONITOR_LDN;
+                        break;
                 }
 
                 break;
@@ -493,6 +497,7 @@ private bool DetectWinbondFintek(LpcPort port)
                 case Chip.NCT6686D:
                 case Chip.NCT6687D:
                 case Chip.NCT6683D:
+                case Chip.NCT6701D:
                     _superIOs.Add(new Nct677X(chip, revision, address, port));
                     break;
 

LibreHardwareMonitorLib/Hardware/Motherboard/Lpc/Nct677X.cs

@@ -151,6 +151,7 @@ public Nct677X(Chip chip, byte revision, ushort port, LpcPort lpcPort)
             case Chip.NCT6797D:
             case Chip.NCT6798D:
             case Chip.NCT6799D:
+            case Chip.NCT6701D:
                 switch (chip)
                 {
                     case Chip.NCT6779D:
@@ -162,6 +163,7 @@ public Nct677X(Chip chip, byte revision, ushort port, LpcPort lpcPort)
                     case Chip.NCT6797D:
                     case Chip.NCT6798D:
                     case Chip.NCT6799D:
+                    case Chip.NCT6701D:
                         Fans = new float?[7];
                         Controls = new float?[7];
                         break;
@@ -192,6 +194,7 @@ public Nct677X(Chip chip, byte revision, ushort port, LpcPort lpcPort)
                     case Chip.NCT6797D:
                     case Chip.NCT6798D:
                     case Chip.NCT6799D:
+                    case Chip.NCT6701D:
                         temperaturesSources.AddRange(new TemperatureSourceData[]
                         {
                             new(SourceNct67Xxd.PECI_0, 0x073, 0x074, 7, 0x100),

LibreHardwareMonitorLib/Hardware/Motherboard/Model.cs

@@ -110,6 +110,7 @@ public enum Model
     PRIME_Z690_A,
     ROG_MAXIMUS_Z790_FORMULA,
     ROG_MAXIMUS_XII_HERO_WIFI,
+    ROG_CROSSHAIR_X870E_HERO,
 
     //BIOSTAR
     B660GTN,

LibreHardwareMonitorLib/Hardware/Motherboard/SuperIOHardware.cs

@@ -354,6 +354,7 @@ private static void GetBoardSpecificConfiguration
             case Chip.NCT6797D:
             case Chip.NCT6798D:
             case Chip.NCT6799D:
+            case Chip.NCT6701D:
             case Chip.NCT6683D:
                 GetNuvotonConfigurationD(superIO, manufacturer, model, superIOIndex, v, t, f, c);
                 break;
@@ -4362,6 +4363,36 @@ private static void GetNuvotonConfigurationD(ISuperIO superIO, Manufacturer manu
 
                         break;
 
+                    case Model.ROG_CROSSHAIR_X870E_HERO: // NCT6701D
+                        v.Add(new Voltage("Vcore", 0));
+                        v.Add(new Voltage("+5V", 1, 4, 1));
+                        v.Add(new Voltage("AVSB", 2, 34, 34));
+                        v.Add(new Voltage("+3.3V", 3, 34, 34));
+                        v.Add(new Voltage("+12V", 4, 11, 1));
+                        v.Add(new Voltage("Voltage #6", 5, true));
+                        v.Add(new Voltage("Voltage #7", 6, true));
+                        v.Add(new Voltage("+3V Standby", 7, 34, 34));
+                        v.Add(new Voltage("CMOS Battery", 8, 34, 34));
+                        v.Add(new Voltage("CPU Termination", 9, true)); // Value does not match any in hwmonitor
+                        v.Add(new Voltage("CPU VDDIO / MC", 10, 1, 1));
+
+                        t.Add(new Temperature("CPU Package", 0));
+                        t.Add(new Temperature("Motherboard", 2));
+                        t.Add(new Temperature("CPU", 22));
+
+                        fanControlNames = new[] { "Chassis Fan #1", "CPU Fan", "Chassis Fan #2", "Chassis Fan #3", "Chassis Fan #4", "Water Pump", "AIO Pump" };
+
+                        System.Diagnostics.Debug.Assert(fanControlNames.Length == superIO.Fans.Length, $"Expected {fanControlNames.Length} fan register in the SuperIO chip");
+                        System.Diagnostics.Debug.Assert(superIO.Fans.Length == superIO.Controls.Length, "Expected counts of fan controls and fan speed registers to be equal");
+
+                        for (int i = 0; i < fanControlNames.Length; i++)
+                            f.Add(new Fan(fanControlNames[i], i));
+
+                        for (int i = 0; i < fanControlNames.Length; i++)
+                            c.Add(new Control(fanControlNames[i], i));
+
+                        break;
+
                     default:
                         v.Add(new Voltage("Vcore", 0));
                         v.Add(new Voltage("Voltage #2", 1, true));