NXP folder architecture improvement #33987
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
PR #33987: Size comparison from e1d746a to ff1392b Full report (4 builds for cc32xx, mbed, stm32)
|
|
PR #33987: Size comparison from e1d746a to 7ed0bc7 Full report (96 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
|
PR #33987: Size comparison from e1d746a to 6b82df0 Increases above 0.2%:
Full report (96 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
andy31415
reviewed
Jun 20, 2024
andy31415
reviewed
Jun 20, 2024
andy31415
reviewed
Jun 25, 2024
andy31415
approved these changes
Jun 25, 2024
andy31415
reviewed
Jun 28, 2024
andy31415
reviewed
Jun 28, 2024
andy31415
reviewed
Jun 28, 2024
andy31415
reviewed
Jun 28, 2024
Adding a reference to the nxp_matter_support repo that would be used to get NXP SDK support files. Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com>
…upport repo Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com>
…all nxp SDK with west for all platform Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com>
…_sdk.py script + updating environement setup documentation Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com>
Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com>
…ocessing Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com>
- dataclass is now used - subprocess dependencies on cwd removed with possible (for west update and west forall cwd is still required) - python logging module is now used Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com>
9f94e32 to
d4dba59
Compare
|
PR #33987: Size comparison from 35eba86 to d4dba59 Full report (96 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
austina-csa
pushed a commit
to austina-csa/connectedhomeip
that referenced
this pull request
Jul 10, 2024
* [NXP] Adding nxp_matter_support submodule Adding a reference to the nxp_matter_support repo that would be used to get NXP SDK support files. Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> * [NXP] Moving NXP Matter SDK support files to a dedicated nxp_matter_support repo Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> * [NXP][script] Adding an update nxp sdk script allowing to initialize all nxp SDK with west for all platform Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> * [NXP][doc][k32w0] Updating nxp sdk update steps to use new update_nxp_sdk.py script + updating environement setup documentation Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> * [NXP][k32w0] Updating default sdk path to nxp_matter_support Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> * Restyled by gn * Restyled by prettier-markdown * [NXP][script] Updating update_nxp_sdk script to clean force update processing Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> * Updating nxp_sdk update script after code review - dataclass is now used - subprocess dependencies on cwd removed with possible (for west update and west forall cwd is still required) - python logging module is now used Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> --------- Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> Co-authored-by: Restyled.io <commits@restyled.io>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
Improving NXP matter folder architecture by moving NXP SDK support files in a dedicated submodule.
The aim of this folder architecture rework is to reduce the number of NXP SDK files that are currently stored in the connectedhome repo by moving them in a dedicated “nxp_matter_support” repo that would be located in “third_party/nxp/nxp_matter_support”.
Main changes summary: