Northeastern-Electric-Racing / Shepherd-BMS

Our from-scratch Battery Management Application
3 stars 0 forks source link

Port and Make changes to EEPROM Directory #10

Closed dyldonahue closed 11 months ago

dyldonahue commented 1 year ago

Port over eepromdirectory files, and make some changes because i coded this lazily apparently.

Basically, I made no safety guards here. Way too much trust in the user to manually handle data size and stuff for eeprom read and writes. So, on top of porting it over, add some safety. This will probably be just checking the data pointers for NULL and error handling if NULL, and passing in an additional parameter to explicitly state the size of read/write

Lastly if @HamzaIqbal69 has finished eeprom driver for Cerberus, replace the Arduino EEPROM.h library refences with his API's. (can ask me to check whether this step is doable yet or not)

Branch is called eeprom_direct on Shep1.0 repo

dyldonahue commented 1 year ago

eeprom driver is currently on a branch for Cerberus. When it gets merged it will appear with the rest of the drivers in Embedded-Base, but for now, view it in Cerberus. The APIs should stay the same even if tweaks still need to be made, so you can use them in this ticket. Obviosuly, because that driver is not yet present in shepherd2, youll get errors, but dont worry about that for now