Skip to content
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

Cmsis update #37

Closed
wants to merge 65 commits into from
Closed

Cmsis update #37

wants to merge 65 commits into from

Conversation

wallarug
Copy link
Contributor

@wallarug wallarug commented Apr 2, 2020

Updates CMSIS from version 4.0 to 5.6.0.

I am not a fan of this PR. The better way would be to scrap the CMSIS folder structure and dump the new structure that came in with V5.0 in SeeSaw. I am going to try this now.

I ran a quick test and it builds. I have not tried with a board. If someone could test some boards before pushing it would be good.

As discussed in #36

If someone could help me fix my Git History as well for SeeSaw that would be very helpful. I have been on a bit of an adventure.

…unction in the config. This was an issue identified before but is more serious for the SAMD51 since it uses different P.Funcs for timers and TC. Default action is to use P.Func E (4) since that is what the script did before.
…lising clocks etc. On build many errors through. Other errors:

- bsp/bsp_nvmctrl.h:  NVMCTRLB_RWS_Msk not declared
- bsp/bsp_nvmctrl.h:  NVMCTRL_CTRLB_MANW was not declared
- bsp/bsp_nvmctrl.h:  no member READY (should be READY1 or READY0)
- bsp/bsp_gpio.h:  no member CTRL, STATUS, ADC and more...
- bsp/bsp_adc.h:  see above
- source/bsp.cpp:  see above
- source/bsp.cpp:  SERCOMx_IRQn, none defined.
No clue what 'work' I have done.  But we have a merge conflict.
…as changed greatly. RTX is now included in the same package so this should be moved
@wallarug
Copy link
Contributor Author

wallarug commented Apr 2, 2020

Closing PR as #38 is better.

@wallarug wallarug closed this Apr 2, 2020
@wallarug wallarug deleted the cmsis-update branch April 4, 2020 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant