Skip to content

Conversation

sreeramIfx
Copy link
Contributor

  • Sample required to enable CM7 cores on XMC7200

@sreeramIfx sreeramIfx marked this pull request as ready for review May 30, 2025 18:44
@github-actions github-actions bot added the area: Samples Samples label May 30, 2025
@github-actions github-actions bot requested review from kartben and nashif May 30, 2025 18:45
 - Sample required to enable CM7 cores on XMC7200

Signed-off-by: Sreeram Tatapudi <[email protected]>
Copy link

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't see how this sample is demonstrating anything Zephyr related. It's just a demonstration of Cypress HAL.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, But this application is required to be able to run zephyr apps on the cm7 cores

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

@kartben kartben Jun 3, 2025

Choose a reason for hiding this comment

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

Basically what @JarmouniA said :) Please consider closing this PR if this sounds like an acceptable solution

Copy link
Contributor Author

@sreeramIfx sreeramIfx Jun 4, 2025

Choose a reason for hiding this comment

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

@kartben @JarmouniA Please note that this application is expected to run on the M0 core to enable the M7 cores. Doing the M7 enabling from the soc_m0p.c would mean that the M7 core would be enabled before or during the driver initializations on the M0 core. It does not give the M0 applications the control/flexibility to enable M7 at a later point in time (for additional configuration or application logic related reasons). Providing a standalone application would provide that flexibility. Please let me know your thoughts.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kartben @JarmouniA Any thoughts on this?

Copy link
Contributor

Choose a reason for hiding this comment

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

@kartben @JarmouniA Please note that this application is expected to run on the M0 core to enable the M7 cores.

That sounds exactly like a SoC Hook to me.

Doing the M7 enabling from the soc_m0p.c would mean that the M7 core would be enabled before or during the driver initializations on the M0 core.

You can choose when you want the hook to run
see include/zephyr/platform/hooks.h

It does not give the M0 applications the control/flexibility to enable M7 at a later point in time (for additional configuration or application logic related reasons).

Zephyr does not aim to cover all the usecases of a given platform, it only gives a stable base to build on. The rest is up to the user depending on how far they want to go.

Providing a standalone application would provide that flexibility.

What is implemented here is not a Zephyr app logic, it is not using any Zephyr API or features.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants