Skip to content

Conversation

CyrilKoe
Copy link
Contributor

@CyrilKoe CyrilKoe commented Jun 3, 2025

This is useful to build crt0 in external SDKs, or external bootroms in SoCs.

@CyrilKoe CyrilKoe force-pushed the ck/bootdata branch 3 times, most recently from 4933f49 to 972a736 Compare June 3, 2025 17:08
@colluca
Copy link
Collaborator

colluca commented Jun 4, 2025

What is exactly the use for this? Because it has no use in the snitch_cluster repo, so it's arguable whether it should have a place here or just be defined by whatever external system needs it.

The only benefit I see of hosting it here would be code reuse, is if it's reused by a lot of systems. But the clustergen tool already provides most desired reuse. Not sure we really need to reuse what seems to me as a pretty specific C/C++ file.

Also maybe you want to look at the generated HAL C/C++ headers in the runtime https://github.com/pulp-platform/snitch_cluster/blob/main/target/snitch_cluster/sw/runtime/common/snitch_cluster_cfg.h.tpl. Perhaps this simple interface could also serve your use case. I want to anyways extend it and move it to a more reasonable location.

This is useful to build crt0 in external SDKs, or external bootroms in
SoCs.
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.

2 participants