-
Notifications
You must be signed in to change notification settings - Fork 100
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
Conversation
…ons as the template README
…s the README template
…e as the README template
…event builds for FTHR kit
… and warning messages in the README
…nd add warning messages in the README
@@ -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. |
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.
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.
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.
I made a note to make that change. Since it affects multiple parts, I'll do that in a separate PR.
Also, I recommend merging the latest changes from main so you have the auto-labeler updates. |
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.
bueno
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.