-
Notifications
You must be signed in to change notification settings - Fork 1
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
User/dillonsd/AuxBmsWorks #193
base: master
Are you sure you want to change the base?
Conversation
We should double check DMA, ADC and SPI settings
Only difficulty is that for CMSIS v2, CubeMx does not support memory pools so we are assuming we'll be fine with just queues. Worst case, we'll use pvMalloc and pvFree for dynamic allocation.
…ine-and-initialize-variables User/embedded/sft 167 define and initialize variables
Created functions, tasks, and unit tests relating to contactor control.
…ate-functions-and-unit-tests-relating-to-contactor-control User/embedded/sft 175 create functions and unit tests relating to contactor control
…peripheral-configs-and-initializations CAN Interrupt and Filter Config
…ate-functions-and-unit-test-relating-to-orion-work User/embedded/sft 190 create functions and unit test relating to orion work
…200-Create-functions-relating-to-CAN-Tx-work
…ate-functions-relating-to-CAN-Tx-work User/embedded/sft 200 create functions relating to can tx work
…CSolarCarTeam/Epsilon-Embedded-Software into user/embedded/SFT-202-Creating-functions-and-unit-tests-relating-to-aux-voltage-reading-work
…ating-functions-and-unit-tests-relating-to-aux-voltage-reading-work User/embedded/sft 202 creating functions and unit tests relating to aux voltage reading work
…ating-functions-and-unit-tests-relating-to-aux-voltage-reading-work Adding unit tests
Implemented stack overflow hook, and malloc failed hook, and error handler
…lement-error-handling SFT 203. Implementing error handling
…lement-memory-debug-tasks SFT 204. Implemented memory debug task
…ean-up Removed magic numbers. Added unit tests to travis.
…t-fixes OrionInterfaceTest final fixes
…he old code we take in multiple adc reads and check if they're stable before we actually use that for calculation. Wrote code for that
… queue, added function to check if orion is happy before setting the common contactor in startup task, change to only execute the orionInterface function if orion is happy before startup happens, changed orion interface to only disconnect contactor once start up is done which we set the variable in discharge contactor gatekeeper because the contactorStatusUpdate function is not running. This is very ghetto and will need to be fixed however at this point the AuxBMS should be useable
…am/Epsilon-Embedded-Software into user/Dillonsd/AuxBMSTesting
…ick not SysTick as delay func and macros are based on TaskTicks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will not be merging all 77 commits, will be picking the commits to merge.
@@ -0,0 +1,207 @@ | |||
/** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this file generated for EpsilonAuxBMS? Dillonsd Trying SPI global IRQ disabled
@@ -16,180 +16,185 @@ static const char* CMockString_isContactorError = "isContactorError"; | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like an astyle thing, check if we need this. Since it's in EpsilonAuxBMS.
uint8_t isCurrentLow(uint8_t numContactorsClosed); | ||
|
||
int isCurrentStable(uint32_t senses[]); | ||
uint32_t readCurrentThroughContactors(void); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add new line
//vTaskList(pcWriteBuffer); | ||
|
||
vPortFree(pcWriteBuffer); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New line
orionHappy |= (!orionDischargeEnableSense && !orionChargeEnableSense); | ||
|
||
return orionHappy; //if this is 1 it means that orion is NOT happy | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New line
Everything