Update CMSIS

[ETSells]

Description

Update CMSIS for our new HAL version.

Issue Link

Parent: SunDevilRocketry/Engine-Controller-Firmware#44

Testing

• Passes existing unit tests
• Unit tests modified (link the test changes as a child PR)
• Integration test performed

Attach any test artifacts here, if relevant.

Other

Leave any additional notes here

Reviewer Checklist

Standards

• Follows FCF Architectural Standards
• Follows SDR Coding Standards
• Code complexity/function Size is minimized
• Code is testable
• Code is readable and commented properly
• License terms are respected

Error Handling

• Potentially unsafe functions return a status code
• Error returns properly handled

Memory

• Stack allocated memory is scoped correctly
• Heap allocated memory is avoided
• Globally allocated memory is minimized except when necessary
• Pointers are used correctly
• Concurrency has been considered

Performance

• Rate limiters are respected
• Busy waiting is avoided
• "Delay" calls are not used in performance sensitive code

[niekky]

The diff is ridiculously huge so can you list out some noticeable changes from old engine lib to the one with CMSIS?

[ETSells]

literally just copied in a newer cmsis to replace the old. the newer HAL had an incompatibility in the ADCs which is resolved with this newer version.

[niekky]

Okay, so we will have to verify ADC works with the newer HAL but we need the wheatstone bridge pcb for this and I'm currently working on it.

[ETSells]

okay, heard

[ETSells]

bruh i forgot this existed. closing unmerged, rev 3 just pulled up CMSIS