Skip to content

Cherry pick adi/max32657 platform support #128

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
Contributor

@ozersa ozersa commented Apr 14, 2025

This PR fetch commit that pushed after TF-Mv2.1.2 tags, gerrit/release-2.1.x, commits here

Final commit ("platform: ext: adi: pull in update from TF-M main") comes from gerrit/main branch to it build with zephyr/tf-m.

@ozersa
Copy link
Contributor Author

ozersa commented Apr 29, 2025

Hi @tomi-font this update requires for zephyrproject-rtos/zephyr#88706
Please review when you get a chance.

@ozersa
Copy link
Contributor Author

ozersa commented May 5, 2025

@tomi-font it is a kind reminder.

Copy link
Collaborator

@tomi-font tomi-font left a comment

Choose a reason for hiding this comment

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

Please cherry pick the commits with -x so that we know the commits are cherry picks and we have the hash of the upstream commits in the commit messages.

ozersa and others added 10 commits May 5, 2025 18:36
Add configuration flags for ITS, PS, and CODE_RAM section
To they be configurable over CFS GUI incase of need.

Change-Id: I80f5c2604f06e1f442dbf5c295d906b5dc96c2cf
Signed-off-by: Sadik Ozer <[email protected]>
(cherry picked from commit a1b21622e35b78c53e2afe2a0e62d5cd5e1142a9)
Incase of CONFIG_NO_OPTIMIZATION flag been set or
CONFIG_DEBUG flag been set 1KB RAM_CODE section not
enought to store flags, info... the section overflow
it is increased to 2KB to meet size

Change-Id: Ia7059c2fb66220094beaef1e88a3148e91f1f073
Signed-off-by: Sadik Ozer <[email protected]>
(cherry picked from commit 3362fd85e60e198c66bd22de9175b9ba87cb612f)
Protected Storage partition is enabled in the configuration file for the
MAX32657 platform. Sufficient flash space is allocated for the services.
Note that PS is not enabled by default in case of small profile of TF-M.

Change-Id: I8a046227b36cfbb3f6c1aaf10a9d25ac48cd2d31
Signed-off-by: Jayashree Srinivasan <[email protected]>
(cherry picked from commit 96174fefb11627fe64f5df92a8d528d22e0db5c4)
Default small profile configurations does not support Protected
Storage encryption. This commit enables single part functions in
crypto library needed for PS Encryption. Algorithms needs for
PS Encryption and Key derivation are done in zephyr configuration.
Furthermore, 16KB flash is provided for PS.

Change-Id: Idf4893caa75eb50bd6042c2a097ad73ccce5fb68
Signed-off-by: Jayashree Srinivasan <[email protected]>
(cherry picked from commit 97882fd395325f6b616c430b717fb3ff12c54dbb)
Fix a broken link in the maintainers file.

Change-Id: Ia3865e36edec3c00893785e4729d4f7937eeb57b
Signed-off-by: Tomi Fontanilles <[email protected]>
(cherry picked from commit 9c27b32bcebb37fe887b2e89584c9fc16bd4bd1c)
Fix a document reference.

Change-Id: I4be1e2aab7f81db0e2b953cbd9261fc5e730fffc
Signed-off-by: Tomi Fontanilles <[email protected]>
(cherry picked from commit 771317e45ffc0a2c8213a81164ffde975417aa7d)
Similarly to what has been done for psa_can_do_hash() and for
the same root cause, exposing the same behaviour.

Signed-off-by: Antonio de Angelis <[email protected]>
Change-Id: I09e6adfb718951b17c6d3bf77e6b7f5cd1135e0d
(cherry picked from commit df80fc236f57050fd47a257beb16ad1f5a54b6cb)
The readme file is all uppercase letters, so it must be referenced
that way.

Change-Id: Ia54e6234c2aa19f076dc72f9602adcfacd77627f
Signed-off-by: Tomi Fontanilles <[email protected]>
(cherry picked from commit 5d9f340feb9d8fb7ff7e582a7087a8b20f0cea0c)
Remove unneeded section from tf-m doc

Change-Id: I8dec60ab39d0cb21279ffaf556b79b9bb9f04dd4
Signed-off-by: Sadik Ozer <[email protected]>
(cherry picked from commit 4a399134838e157a131ba1a93d8b16b9240fef2e)
Pull in TF-M main commit 152f786

Change-Id: I7d5acb11ab1e8f7a8677b65404133d15630536a9
Signed-off-by: Hao Zhang <[email protected]>
(cherry picked from commit dd897de)
@ozersa ozersa force-pushed the gerrit-to-zephyr-fork branch from 89b24c8 to 5cd08a4 Compare May 5, 2025 15:42
@ozersa
Copy link
Contributor Author

ozersa commented May 5, 2025

Please cherry pick the commits with -x so that we know the commits are cherry picks and we have the hash of the upstream commits in the commit messages.

Sure, Done.

@ozersa
Copy link
Contributor Author

ozersa commented May 8, 2025

@tomi-font please revisit.

Copy link
Collaborator

@tomi-font tomi-font left a comment

Choose a reason for hiding this comment

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

Looks good. Only complaint I have left is the PR title could be more clear (e.g. cherry pick adi/max32657 platform support).

@ozersa ozersa changed the title MAX32657: Gerrit to Zephyr Fork Cherry pick adi/max32657 platform support May 9, 2025
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.

5 participants