Add Support for Heltec BMS (= YYBMS YanYang) #575
Replies: 7 comments 1 reply
-
I am working on this. @Louisvdw @mr-manuel should I base that work directly on the JKBMS-BLE branch? |
Beta Was this translation helpful? Give feedback.
-
If you are not adding a BLE driver than you can merge with the master branch. The only thing is that there are a lot of changes outstanding, see #468. |
Beta Was this translation helpful? Give feedback.
-
OK thanks. The config.ini seems to be a very interesting change there... maybe the timeline to merge that into master us the main question right now |
Beta Was this translation helpful? Give feedback.
-
Unfortunately @Louisvdw has not much time right now, but he will do so after 2nd May. If you want to keep it simple merge with the |
Beta Was this translation helpful? Give feedback.
-
Out of curiosity: what is the difference between an issue and a discussion? BTW I have the driver in production already, but it will take some time to cleanup until I can file a PR. |
Beta Was this translation helpful? Give feedback.
-
The protocol is here: HeltecBMS_smrt_bms,modbus_protocol_v1.5.pdf |
Beta Was this translation helpful? Give feedback.
-
This is implemented with #508 |
Beta Was this translation helpful? Give feedback.
-
Is your feature request related to a problem? Please describe.
I read on https://github.com/Louisvdw/dbus-serialbattery/wiki/BMS-types-supported that "JKBMS / Heltec BMS" are supported, so I tried to connect my Heltec Smart BMS 9-25S 100A RS485 but it is not detected by the dbus-serialbattery driver.
Describe the solution you'd like
According the docs I have (and what I spied on the RS485 with the scope) the BMS seems to talk Modbus RTU and is different from the JKBMS. So I propose to add support for this Heltec device. I don't know about other Heltec BMSs and if there is a variant that is compatible to JKBMS, but we might also want to update the docs accordingly.
Beta Was this translation helpful? Give feedback.
All reactions