Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ISL94202: Auto-detect number of cells #11

Open
martinjaeger opened this issue May 12, 2021 · 2 comments
Open

ISL94202: Auto-detect number of cells #11

martinjaeger opened this issue May 12, 2021 · 2 comments

Comments

@martinjaeger
Copy link
Member

The number of cells is currently still hard-coded and set in bms_init_hardware() in bms_isl94202.c.

The ISL94202 does not automatically detect the number of cells. It has to be written to a register.

Possible algorithm:

  • Start with 8 cells enabled.
  • Measure voltages and find out actual number of cells (including plausibility-check with pack voltage)
  • Set number of cells in chip.
@case06
Copy link
Contributor

case06 commented May 13, 2021

In Dacians SBMS it also has to be set manually by configs. I wonder if there is any risk for a constellation with e.g. a constellation of unusual low unloaded cells that could perhaps lead to a missinterpretation and wrong celltype-classification ?

@martinjaeger
Copy link
Member Author

Mhh, true, that could be an issue. But if a cell is below e.g. 1V the pack should not be used anyway anymore...

Today I realized that the default setting of the chip is actually 3 cells, so the algorithm should start from there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants