Lines of code count not available
There is still merge conflict? Please resolve merge conflict and build the code in your local before pushing.
This merge conflict is still unresolved.
leak in the comment section.
Where is alarm 191..203?
Align comment.
Keep 2 blank lines above banner.
Should this line be un-commented?
So we are going to fail this test until calibration record is ready?
In Leahi, we want to do all initialization in the init function instead of here at declaration (see Bubble.c for example).
Rename similar to ADDR definitions below. these are not commands.
Combine charger and pack data into one payload with one broadcast message.
Move these 3 override messages (should end in OVERRIDE_REQUEST) to TD Dialin section below and assign an ID in the 0x8... range.
I think we only need one data message for both charger and battery pack data.
Remove.
uncomment.
Yes Noe said to replace with new register
Updated
Sure
please add bpLeftHomeTimerCounter to inputs and outputs.
It is weird that we're responding so far from request. It is not clear to me that Dialin has requested anything at this point.
Details looks weird. Where is inputs and outputs like other function headers?
Add a blank line after banner.
Please restore before you merge to staging.
If I'm reading this right, we're saying that if bad CRC alarm is NOT enabled, initialize the alarm? That seems backward.
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.