Skip to content

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

Merged
merged 3 commits into from
Dec 25, 2024

Conversation

andrewleech
Copy link

@andrewleech andrewleech commented Oct 31, 2024

@alonbl
Copy link

alonbl commented Nov 1, 2024

Hi,
In #9 there are missing files for example for MIMXRT1064 fsl_cache.* is missing

@andrewleech
Copy link
Author

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...

@alonbl
Copy link

alonbl commented Nov 2, 2024

I do not have the answers, I am too straggling with understanding what NXP is doing.
They have so many common files between products, instead of duplicate them over and over they could have created smarter structure.
I gave up using their git repo and always download their SDK once they build it.

@andrewleech
Copy link
Author

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.
I'd much prefer something like a json file being available too to reference common drivers, I really don't like cmake personally.

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.
It's handy for hobbyists who just want to get something going ( and helped me have an impressive demo running on a new dev kit within minutes ) but terrible for ongoing product support.

@alonbl
Copy link

alonbl commented Nov 2, 2024

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?

@andrewleech
Copy link
Author

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.
@andrewleech andrewleech force-pushed the sdk_update_MCUX_2.16.100 branch from 749463f to a298e8a Compare November 14, 2024 12:04
Copy link
Owner

@hathach hathach left a 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.

@hathach hathach merged commit 72fa606 into hathach:master Dec 25, 2024
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.

4 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy