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

chore(Examples): Test examples on Rev. B production silicon for MAX32655 #649

Merged
merged 35 commits into from
Jul 6, 2023

Conversation

Jacob-Scheiffler
Copy link
Contributor

This PR includes the changes related to testing, debugging, and updating each of the non-BLE examples for the MAX32655 Rev. B production parts. The Flash and I2C_Playback examples each have minor bug fixes and the changes for the rest of the examples are mainly just README updates.

@@ -14,17 +14,19 @@ Universal instructions on building, flashing, and debugging this project can be

### Project-Specific Build Notes

To comply with the CoreMark rules, the only source files which are included in this example directory are the core_portme.c/.h files, the rest (including main) are located in the [Coremark](../../../Libraries/Coremark/) library.
* To comply with the CoreMark rules, the only source files which are included in this example directory are the core_portme.c/.h files, the rest (including main) are located in the [Coremark](../../../Libraries/Coremark/) library.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very minor, but I think it should be CoreMark (it's a registered trademark). We goofed when we added the library and named it Coremark. We can't change the directory name, but we can start naming it properly going forward.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made a note to make that change. Since it affects multiple parts, I'll do that in a separate PR.

@sihyung-maxim
Copy link
Contributor

Also, I recommend merging the latest changes from main so you have the auto-labeler updates.

@sihyung-maxim sihyung-maxim changed the title chore(Examples, PeriphDrivers): MAX32655: test examples on Rev. B production silicon chore(Examples, PeriphDrivers): Test examples on Rev. B production silicon for MAX32655 Jul 5, 2023
@Jacob-Scheiffler Jacob-Scheiffler changed the title chore(Examples, PeriphDrivers): Test examples on Rev. B production silicon for MAX32655 chore(Examples): Test examples on Rev. B production silicon for MAX32655 Jul 5, 2023
Copy link
Contributor

@Jake-Carter Jake-Carter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bueno

@Jacob-Scheiffler Jacob-Scheiffler merged commit 0116a6f into main Jul 6, 2023
@Jacob-Scheiffler Jacob-Scheiffler deleted the dev/me17b_examples_test branch July 6, 2023 14:36
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.

3 participants