# HG changeset patch # User moel.mich # Date 1282593606 0 # Node ID c9d8de4725467b8958a3fd0690511c63e5675dc4 # Parent 81ab5e53122ec44f16699b982f23fca5523fc425 Fixed a few problems with the Heatmaster. diff -r 81ab5e53122e -r c9d8de472546 Hardware/Heatmaster/Heatmaster.cs --- a/Hardware/Heatmaster/Heatmaster.cs Sun Aug 22 21:53:11 2010 +0000 +++ b/Hardware/Heatmaster/Heatmaster.cs Mon Aug 23 20:00:06 2010 +0000 @@ -61,6 +61,8 @@ private bool available = false; + private StringBuilder buffer = new StringBuilder(); + private string ReadLine(int timeout) { int i = 0; StringBuilder builder = new StringBuilder(); @@ -82,7 +84,7 @@ private string ReadField(int device, char field) { serialPort.WriteLine("[0:" + device + "]R" + field); for (int i = 0; i < 5; i++) { - string s = ReadLine(100); + string s = ReadLine(200); Match match = Regex.Match(s, @"-\[0:" + device.ToString() + @"\]R" + Regex.Escape(field.ToString()) + ":(.*)"); if (match.Success) @@ -111,7 +113,7 @@ private bool WriteField(int device, char field, string value) { serialPort.WriteLine("[0:" + device + "]W" + field + ":" + value); for (int i = 0; i < 5; i++) { - string s = ReadLine(100); + string s = ReadLine(200); Match match = Regex.Match(s, @"-\[0:" + device.ToString() + @"\]W" + Regex.Escape(field.ToString()) + ":" + value); if (match.Success) @@ -222,42 +224,52 @@ get { return "Heatmaster"; } } + private void ProcessUpdateLine(string line) { + Match match = Regex.Match(line, @">\[0:(\d+)\]([0-9:\|-]+)"); + if (match.Success) { + int device; + if (int.TryParse(match.Groups[1].Value, out device)) { + foreach (string s in match.Groups[2].Value.Split('|')) { + string[] strings = s.Split(':'); + int[] ints = new int[strings.Length]; + for (int i = 0; i < ints.Length; i++) + ints[i] = int.Parse(strings[i]); + switch (device) { + case 32: + if (ints.Length == 3 && ints[0] <= fans.Length) { + fans[ints[0] - 1].Value = ints[1]; + controls[ints[0] - 1].Value = (100 / 255.0f) * ints[2]; + } + break; + case 48: + if (ints.Length == 2 && ints[0] <= temperatures.Length) + temperatures[ints[0] - 1].Value = 0.1f * ints[1]; + break; + case 64: + if (ints.Length == 3 && ints[0] <= flows.Length) + flows[ints[0] - 1].Value = 0.1f * ints[1]; + break; + case 80: + if (ints.Length == 2 && ints[0] <= relays.Length) + relays[ints[0] - 1].Value = 100 * ints[1]; + break; + } + } + } + } + } + public override void Update() { if (!available) return; while (serialPort.BytesToRead > 0) { - Match match = Regex.Match(ReadLine(0), @">\[0:(\d+)\]([0-9:\|-]+)"); - if (match.Success) { - int device; - if (int.TryParse(match.Groups[1].Value, out device)) { - foreach (string s in match.Groups[2].Value.Split('|')) { - string[] strings = s.Split(':'); - int[] ints = new int[strings.Length]; - for (int i = 0; i < ints.Length; i++) - ints[i] = int.Parse(strings[i]); - switch (device) { - case 32: - if (ints.Length == 3 && ints[0] <= fans.Length) { - fans[ints[0] - 1].Value = ints[1]; - controls[ints[0] - 1].Value = (100 / 255.0f) * ints[2]; - } - break; - case 48: - if (ints.Length == 2 && ints[0] <= temperatures.Length) - temperatures[ints[0] - 1].Value = 0.1f * ints[1]; - break; - case 64: - if (ints.Length == 3 && ints[0] <= flows.Length) - flows[ints[0] - 1].Value = 0.1f * ints[1]; - break; - case 80: - if (ints.Length == 2 && ints[0] <= relays.Length) - relays[ints[0] - 1].Value = 100 * ints[1]; - break; - } - } - } + byte b = (byte)serialPort.ReadByte(); + if (b == 0x0D) { + ProcessUpdateLine(buffer.ToString()); + buffer.Length = 0; + } else { + buffer.Append((char)b); } } } diff -r 81ab5e53122e -r c9d8de472546 Hardware/Heatmaster/HeatmasterGroup.cs --- a/Hardware/Heatmaster/HeatmasterGroup.cs Sun Aug 22 21:53:11 2010 +0000 +++ b/Hardware/Heatmaster/HeatmasterGroup.cs Mon Aug 23 20:00:06 2010 +0000 @@ -38,8 +38,10 @@ using System; using System.Collections.Generic; using System.IO.Ports; +using System.Security; using System.Text; using System.Threading; +using Microsoft.Win32; namespace OpenHardwareMonitor.Hardware.Heatmaster { internal class HeatmasterGroup : IGroup { @@ -63,11 +65,36 @@ Thread.Sleep(1); } throw new TimeoutException(); - } + } + + private static string[] GetRegistryPortNames() { + List result = new List(); + try { + RegistryKey key = Registry.LocalMachine.OpenSubKey( + @"SYSTEM\CurrentControlSet\Enum\USB\Vid_10c4&Pid_ea60&Mi_00"); + if (key != null) { + foreach (string subKeyName in key.GetSubKeyNames()) { + RegistryKey subKey = + key.OpenSubKey(subKeyName + "\\" + "Device Parameters"); + if (subKey != null) { + string name = subKey.GetValue("PortName") as string; + if (name != null) + result.Add((string)name); + } + } + } + } catch (SecurityException) { } + return result.ToArray(); + } public HeatmasterGroup(ISettings settings) { + + // No implementation for Heatmaster on Unix systems + int p = (int)System.Environment.OSVersion.Platform; + if ((p == 4) || (p == 128)) + return; - string[] portNames = SerialPort.GetPortNames(); + string[] portNames = GetRegistryPortNames(); for (int i = portNames.Length - 1; i >= 0; i--) { try { @@ -85,50 +112,46 @@ } if (serialPort.IsOpen) { - if (serialPort.CtsHolding) { - serialPort.DiscardInBuffer(); - serialPort.DiscardOutBuffer(); - serialPort.Write(new byte[] { 0xAA }, 0, 1); + serialPort.DiscardInBuffer(); + serialPort.DiscardOutBuffer(); + serialPort.Write(new byte[] { 0xAA }, 0, 1); - int j = 0; - while (serialPort.BytesToRead == 0 && j < 10) { - Thread.Sleep(20); - j++; + int j = 0; + while (serialPort.BytesToRead == 0 && j < 10) { + Thread.Sleep(20); + j++; + } + if (serialPort.BytesToRead > 0) { + bool flag = false; + while (serialPort.BytesToRead > 0 && !flag) { + flag |= (serialPort.ReadByte() == 0xAA); } - if (serialPort.BytesToRead > 0) { - bool flag = false; - while (serialPort.BytesToRead > 0 && !flag) { - flag |= (serialPort.ReadByte() == 0xAA); - } - if (flag) { - serialPort.WriteLine("[0:0]RH"); - try { - int k = 0; - int revision = 0; - while (k < 5) { - string line = ReadLine(serialPort, 100); - if (line.StartsWith("-[0:0]RH:")) { - int.TryParse(line.Substring(9), out revision); - break; - } - k++; + if (flag) { + serialPort.WriteLine("[0:0]RH"); + try { + int k = 0; + int revision = 0; + while (k < 5) { + string line = ReadLine(serialPort, 100); + if (line.StartsWith("-[0:0]RH:")) { + int.TryParse(line.Substring(9), out revision); + break; } - isValid = (revision == 770); - if (!isValid) { - report.Append("Status: Wrong Hardware Revision " + - revision.ToString()); - } - } catch (TimeoutException) { - report.AppendLine("Status: Timeout Reading Revision"); + k++; } - } else { - report.AppendLine("Status: Wrong Startflag"); + isValid = (revision == 770); + if (!isValid) { + report.Append("Status: Wrong Hardware Revision " + + revision.ToString()); + } + } catch (TimeoutException) { + report.AppendLine("Status: Timeout Reading Revision"); } } else { - report.AppendLine("Status: No Response"); + report.AppendLine("Status: Wrong Startflag"); } } else { - report.AppendLine("Status: Not Clear to Send"); + report.AppendLine("Status: No Response"); } serialPort.DiscardInBuffer(); serialPort.Close(); diff -r 81ab5e53122e -r c9d8de472546 Properties/AssemblyVersion.cs --- a/Properties/AssemblyVersion.cs Sun Aug 22 21:53:11 2010 +0000 +++ b/Properties/AssemblyVersion.cs Mon Aug 23 20:00:06 2010 +0000 @@ -38,5 +38,5 @@ using System; using System.Reflection; -[assembly: AssemblyVersion("0.1.37.4")] -[assembly: AssemblyFileVersion("0.1.37.4")] +[assembly: AssemblyVersion("0.1.37.7")] +[assembly: AssemblyFileVersion("0.1.37.7")]