Quite some difference between storeCellVoltage:
My version:
void storeCellVoltage(uint8_t dataRegister, uint8_t readRegisterData[(2*numberOfSlaves + 5)])
{
for(int i=0; i<numberOfSlaves; i++)
{
uint16_t measVoltage = ((readRegisterData[(2*numberOfSlaves + 1) - i*2] << 8) + readRegisterData ...

That sounds great. Please, share what the error was so I and others can learn from it.


I haven't solved the problem yet, but I noticed that the lowest cell in the code you sent me is in 14 modules, and in the code I'm using, the lowest cell is in the 5th module and I think that the problem lies ...

No need to use CAN but you then need to setup the SPI-bus in another way.


Hello, I was able to run the test code that you sent me and I compared the series output of the two codes and I think I understood what the main error is.

Output from testCode:


Entering Configuration Mode Failure ...

bexander wrote: ↑Sat Oct 26, 2024 12:31 pm The CAN init fails. The SPI-bus setup is done by CAN init, hence the MAX17841 init fails as well.
I suggest you start by looking at why the CAN-init fails.

Can I not use CAN to run this program?

Have you tried swapping position of the different modules? Do you still have the same effect when having module 7 as module 1 or as module 12?

Test with this SW and all modules. Do you still have the same behaviour?
BMS_test_OI.ino


Hello, after I set my pin out to the code they sent me, I got ...

if one cell always discharging. I would look for a hardware problem.

first check the cell voltage with a multimeter. and compare them with the bms readings.

if without the bms connected the battery still discharges.
maybe the cell is bad or the bms chip is damaged and the balance is always on ...

When cell balancing is active you can not trust the cell voltage readings. When a cell is balanced it will have effect on adjecent cell voltage readings. To measure the cell voltages correctly, no balancing can be active while taking measurement.

An example from an old log.
Voltage reading at ...

Thanks for the answer @bexander , but if you see here


0 3947 1 3952 2 3947 3 3950 4 3949 5 3952
Module: 0
All Cells to Balance
111111
Cell to Balance Filtered :101010
1 3 5 Voltage Diff Min-Max: 119
0 3952 1 3951 2 3947 3 3942 4 3943 5 3954
Module: 1
All Cells to Balance
111111
Cell to Balance ...

I think DEVCFG1 should fix this, unless it's already fixed: ADDRUNLOCK Disables write-protection of device address DA[4:0]. Cleared only by HELLOALL
command (write-protected)

Hello @bexander , I finally tried to balance the battery today with the attached code and it also has log files from the series motoring in which it can be seen that when the balance is turned on, it discharges even the most discharged cell, which should not be the case. (I use 18 modules)

Best ...

I'm try to use @manny code for my peugeot 208-e battery(2p6s)(18 modules) (I've attached the code) and recently started researching how the balance works and I've been testing the balancing logic with the MAX17823B chips recently and found a problem with the balancing. Because in my battery the most ...

Hi,

Yep code can be found in this thread https://openinverter.org/forum/viewtopic.php?p=20335#p20335

I've rewritten this code for my project (using the Peugeot Battery), it needs a bit of tidying before I publish it here, but I hope to do this soon.


Hi VWJoe, I have a battery from an Opel ...