-
Notifications
You must be signed in to change notification settings - Fork 12
Update sdk to MCUX_2.16.100 #9
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
Conversation
693979e
to
749463f
Compare
Hi, |
Drats. I've only been starting to test with rt1176 so far myself. I'll have to take another look at the new SDK files, see if there's another way those particular drivers are referenced / sourced from common. The weird discrepancy between chip sdks downloaded from SDK builder vs the official GitHub structure is horribly confusing and difficult to manage. I'm not too sure how the existing chip folders were sourced / updated to be honest? I'm not sure if they were just manually downloaded from SDK builder nor which drivers / middleware would have been selected when they were. This is my effort to bring some consistency and traceability to this SDK repo, hopefully I can achieve that without breaking existing projects using it too much... |
I do not have the answers, I am too straggling with understanding what NXP is doing. |
I guess the structure of the git repo is their attempt to reduce duplication! From what I can see It's generally structured with only chip specify drivers in the individual folders, then cmake include files to reference the relevant common drivers. This structure makes far more sense to me and is hopefully indicative of their future path, but it's a shame downloads from other sources mix all the drivers into one folder rather than maintaining common vs specific folders structures. I generally work on medical products in a regulated environment where traceability of code is crucial. From this I strongly dislike and kind of manual download / unpack / reorganise process for updating libraries, it's so hard to document where and how they've been sourced. Even worse when the download comes from a builder website that custom packs things from user settings - this prevents automation and recording of download links etc. |
I had an interest in this for micropython, but now I see people are working to remove this dependency in favor of direct upstream use see: micropython/micropython#11516 What is your use case? |
Thanks for that link, I too am coming at this from the micropython direction. I hadn't seen that PR where the rework effort to migrate to the official submodule has already been done! I would prefer that really, even though it's a large submodule it's the official source so should be the correct way forward. |
This script grabs the latest release (or tag passed as argument) from NXP's github and updates the contents of existing target chip drivers from the download. It also updates the common drivers from the downloaded SDK and symlinks them as appropriate into the target chip drivers dirs.
749463f
to
a298e8a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for the PR, like others, I also prefer to use official sdk repo. This repo is created before NXP introduce the sdk repo. In fact, tinyusb already switch to use official sdk.
From: https://api.github.com/repos/nxp-mcuxpresso/mcux-sdk/tarball/MCUX_2.16.100
For details see #10