Line 620 BMS.c does something I'm not too sure what exactly. I'm about 99% sure it's supposed to be blinking an LED for debugging but it would be great to know if we need this or not.

        case RTCIV_RTCTEVIFG:    P1OUT |= BIT0; break; // I think this is blinking an LED or some shit

BMS_collectData() is due for a refactor as well. Could pobably seperate into static inline functions for better readability. It's a bit of a wall of code at the moment

extern void BMS_collectData()