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

Update max32670 sdk files #15399

Merged
merged 2 commits into from
Apr 19, 2023

Conversation

ozersa
Copy link
Contributor

@ozersa ozersa commented Apr 10, 2023

Update MAX32670 SDK driver files to get final updates and fixes.
There are two commits, second commit demonstrate delta, please rebase and merge (not squash merge) to we see delta in future and easily update in future incase of need.

Summary of changes

Impact of changes

Migration actions required

Documentation


Pull request type

[X] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[] No Tests required for this change (E.g docs only update)
[] Covered by existing mbed-os tests (Greentea or Unittest)
[X] Tests / results supplied as part of this PR

ARMC6_TEST_RESULT.txt
GCC_TEST_RESULT.txt


Reviewers


@0xc0170
Copy link
Contributor

0xc0170 commented Apr 12, 2023

Note we do not squash so delta like this should be in the history if there was commited any previously.

Please fix the commit msg "MAX32670 apply mbeb required changes on peripheral drivers" , I believe it should be mbed instead of mbeb.

@ozersa
Copy link
Contributor Author

ozersa commented Apr 13, 2023

Note we do not squash so delta like this should be in the history if there was commited any previously.

Please fix the commit msg "MAX32670 apply mbeb required changes on peripheral drivers" , I believe it should be mbed instead of mbeb.

Thanks, it is a typo, done.

@saheerb saheerb added release-type: patch Indentifies a PR as containing just a patch needs: CI and removed needs: work labels Apr 19, 2023
@mbed-ci
Copy link

mbed-ci commented Apr 19, 2023

Jenkins CI Test : ✔️ SUCCESS

Build Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & Artifacts

CLICK for Detailed Summary

jobs Status
jenkins-ci/mbed-os-ci_build-cloud-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-cloud-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_unittests ✔️
jenkins-ci/mbed-os-ci_build-greentea-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-greentea-ARM ✔️
jenkins-ci/mbed-os-ci_build-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_greentea-test ✔️

@0xc0170 0xc0170 merged commit 8628604 into ARMmbed:master Apr 19, 2023
@mergify mergify bot removed the ready for merge label Apr 19, 2023
@ozersa ozersa deleted the dev-update_max32670_sdk branch April 19, 2023 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-type: patch Indentifies a PR as containing just a patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants