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

Re-arrange OLED-display output to support max. cell-number #2

Closed
case06 opened this issue Feb 5, 2017 · 3 comments
Closed

Re-arrange OLED-display output to support max. cell-number #2

case06 opened this issue Feb 5, 2017 · 3 comments

Comments

@case06
Copy link
Contributor

case06 commented Feb 5, 2017

The current version supports up to 12 cells so far. For showing the output of the max possible number of 15 cells, its necessary to re-arrange the layout, to gain one more row, which is needed to display all 15 cells in 3 columns and 5 rows. This will be some fizzy work (i'll do it later, when i deploy my BQ40 board) but should be in general possible on a 128x64 display: we need 7 rows in total, which gives us 9 points per row; that is 8 points font-size/height plus 1 point for a separation line between the rows.

@martinjaeger
Copy link
Member

Maybe not the right place for the 24V version of the board (with 10s bq76930)... either we should rename this repository or open a separate one for the 15s version.

@martinjaeger martinjaeger changed the title ToDo: Re-arrange OLED-display output to support max. cell-number Re-arrange OLED-display output to support max. cell-number Feb 5, 2017
@case06
Copy link
Contributor Author

case06 commented Feb 5, 2017

I would strongly recommend/opt for having just only one repository for an universal firmware, which supports any possible number of cells just by changing/configuring some settings (otherwise we would generate to much overhead/redundancy/noise). Even if that means to rename it and i have to do a new fork ;)

But oth i could also live with a "wrong" or misleading name of the software which could be justiifed by "historical growth" ... and btw., its not too much misleading, because of its subtitle "Firmware for 24/48V BMS board using bq76930 (max. 10s cells) " ... so maybe its ok, to simply change this subtitle towards something like "Firmware for LibreSolar BMS boards using bq76930 (max. 10 cells / 36V) or bq76940 (max 15 cells / 54V)".

@martinjaeger
Copy link
Member

Fixed with Zephyr driver added in 5438273.

The OLED uses two separate screens for BMSs with >8 cells.

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