Merge pull request #101 from Jalle19/unit-type

Read unit type from the correct register
Jalle19 · Jan 17, 2024 · 56a6464 · 56a6464
2 parents b494729 + 217d213
commit 56a6464
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Change log
 
+## future release
+
+* Fix reading from unit type from wrong register (https://github.com/Jalle19/eda-modbus-bridge/issues/100)
+
 ## 2.7.0
 
 * Change Home Assistant auto-discovery log level to `debug` (https://github.com/Jalle19/eda-modbus-bridge/issues/93)

diff --git a/app/modbus.mjs b/app/modbus.mjs
@@ -261,7 +261,7 @@ export const getDeviceInformation = async (modbusClient) => {
         'fanType': result.data[0] ? 'EC' : 'AC',
     }
 
-    result = await mutex.runExclusive(async () => tryReadCoils(modbusClient, 72, 1))
+    result = await mutex.runExclusive(async () => tryReadCoils(modbusClient, 51, 1))
     const unitType = getUnitTypeName(result.data[0])
     deviceInformation = {
         ...deviceInformation,