Skip to content

Add MAX32657 NS Board #88706

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

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

ozersa
Copy link
Collaborator

@ozersa ozersa commented Apr 16, 2025

Add MAX32657 NS board support.

This PR will be enabled after: #88651

Blinky Example Output:
image

@ozersa ozersa mentioned this pull request Apr 16, 2025
Copy link

github-actions bot commented Apr 16, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
trusted-firmware-m zephyrproject-rtos/trusted-firmware-m@e2288c1 (main) zephyrproject-rtos/trusted-firmware-m#128 zephyrproject-rtos/trusted-firmware-m#128/files

DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@@ -2,4 +2,5 @@
# SPDX-License-Identifier: Apache-2.0

config BOARD_MAX32657EVKIT
select SOC_MAX32657 if BOARD_MAX32657EVKIT_MAX32657
select SOC_MAX32657 if BOARD_MAX32657EVKIT_MAX32657 || \
BOARD_MAX32657EVKIT_MAX32657_NS
Copy link
Collaborator

Choose a reason for hiding this comment

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

fix alignment

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

My local tab size was 4, so missed this one, updated it as 8 and fixed. thanks.

BOARD: max32657evkit/max32657/ns
================================

The ``max32657evkit/max32657/ns`` board configuration is used to build the secure firmware
Copy link
Collaborator

Choose a reason for hiding this comment

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

board target, no such term as board configuration in zephyr

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Added to classify them to simplify readability. I can update it if require?

  • BOARD: max32657evkit/max32657
  • BOARD: max32657evkit/max32657/ns

@ozersa ozersa force-pushed the add-MAX32657-NS branch 3 times, most recently from 03b0cb7 to 88f3b8d Compare April 24, 2025 07:11
ozersa and others added 10 commits April 29, 2025 10:53
Update tf-m node to get MAX32657 support

Signed-off-by: Sadik Ozer <[email protected]>
This commit adds MAX32657 Non-Secure peripheral support

Signed-off-by: Sadik Ozer <[email protected]>
This commit enables max32657 NS board support

To build:
- west build -b max32657evkit/max32657/ns -p

Signed-off-by: Sadik Ozer <[email protected]>
MAX32657 has one UART interface,
It can be used either on TF-M or Zephyr
Enabling debug (TFM_SPM_LOG_LEVEL || TFM_PARTITION_LOG_LEVEL)
will transfer it to the TF-M side
Disabling TF-M debug will transfer it to the Zephyr side.

This commit disable TFM debugs

Signed-off-by: Sadik Ozer <[email protected]>
This commit provides flashing tfm_merged.hex (includes
mcuboot+tfm+ns_image) on default.

Signed-off-by: Mert Ekren <[email protected]>
JWT test fails on max32657evkit/max32657/ns board, until it's issue been
fixed it is excluded from automated test to workflow been succeeded

Signed-off-by: Sadik Ozer <[email protected]>
This commit enables secure_partion sample for max32657evkit NS board

Signed-off-by: Sadik Ozer <[email protected]>
This commit enable psa_protected_storage test for max32657evkit/ns board

Signed-off-by: Sadik Ozer <[email protected]>
Update psa_crypto test not work on MAX32657
This commit enable/disable flags for MAX32657 to
make it works for MAX32657.

Signed-off-by: Sadik Ozer <[email protected]>
Add configuration flag to manage tf-m hal_adi repository during build
hal_adi can be used either over zephyr or download during build

Signed-off-by: Sadik Ozer <[email protected]>
@ozersa ozersa marked this pull request as ready for review April 29, 2025 08:29
@github-actions github-actions bot added platform: ADI Analog Devices, Inc. area: TF-M ARM Trusted Firmware-M (TF-M) area: Samples Samples area: JSON labels Apr 29, 2025
@github-actions github-actions bot requested review from d3zd3z and galak April 29, 2025 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: JSON area: Samples Samples area: TF-M ARM Trusted Firmware-M (TF-M) DNM (manifest) This PR should not be merged (controlled by action-manifest) manifest manifest-hal_adi manifest-trusted-firmware-m platform: ADI Analog Devices, Inc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants