apollo-lhc / cm_mcu

Microcontroller source code for the APOLLO blade for the CMS tracker HL-LHC upgrade.
MIT License
2 stars 2 forks source link

Turn FreeRTOS use into a submodule #185

Closed pwittich closed 1 year ago

pwittich commented 1 year ago

To make more uniform builds we now turn FreeRTOS into a submodule. This also makes for one less thing to download.

pwittich commented 1 year ago

@pkotamnives can you try this out in a freshly cloned repo and see if it works?

pkotamnives commented 1 year ago

@pkotamnives can you try this out in a freshly cloned repo and see if it works?

hi @pwittich, I have tested this branch with REV1 and REV2 and they both can be built just fine. Also, for REV2 which I have a board to test with, a few commands such as i2crr, semaphore, and clkmon requiring semaphore functionalities gave expected results. Is there anything else I should test before we merge this change to master?

pwittich commented 1 year ago

@pkotamnives can you try this out in a freshly cloned repo and see if it works?

hi @pwittich, I have tested this branch with REV1 and REV2 and they both can be built just fine. Also, for REV2 which I have a board to test with, a few commands such as i2crr, semaphore, and clkmon requiring semaphore functionalities gave expected results. Is there anything else I should test before we merge this change to master?

Thanks; the main issue would have been in the build. So if that worked for you I think I can merge this.