Commit 3679ac1
committed
fix: scope EVC cumulative-energy big word order to GEN1 only
The order32="big" override on charge_added_cum (from wills106#2178) was
verified on an X3-EVC-11kW Gen1 (serial C311*, ARM v1.18): at the same
instant the input copy 0x10-0x11 read [0, 3] (high-word-first) while
its holding twin 0x619-0x61A read [3, 0] and decoded to the correct
0.3 kWh little-endian.
Issue wills106#2230 shows the opposite population: an X3-HAC 22kW Gen2
(ARM v2.03) follows the documented little order, so the global
override inflated its lifetime total by a factor of 65536
(322.6 kWh -> 21,141,913.6).
The definition is now split per generation: GEN1 keeps order32="big"
(hardware-verified), GEN2 returns to the documented default order.
Fixes wills106#2230 (word-order part).1 parent d35604f commit 3679ac1
1 file changed
Lines changed: 13 additions & 0 deletions
Lines changed: 13 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1118 | 1118 | | |
1119 | 1119 | | |
1120 | 1120 | | |
| 1121 | + | |
| 1122 | + | |
| 1123 | + | |
| 1124 | + | |
| 1125 | + | |
| 1126 | + | |
| 1127 | + | |
| 1128 | + | |
| 1129 | + | |
| 1130 | + | |
| 1131 | + | |
| 1132 | + | |
| 1133 | + | |
1121 | 1134 | | |
1122 | 1135 | | |
1123 | 1136 | | |
| |||
0 commit comments