Skip to content
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

soc/builder: add initialize_memory() #1856

Merged
merged 1 commit into from
Apr 2, 2024

Conversation

AndrewD
Copy link
Collaborator

@AndrewD AndrewD commented Dec 22, 2023

Allows the target soc to override memory initialization:
rom and/or ram.

Includes documentation for example usage to compile soc target specific software and load it to main_ram. This mechanism
can also be used to load a custom bios into ROM instead of the standard Litex bios.

Allows the target soc to override memory initialization:
  rom and/or ram.
@AndrewD AndrewD force-pushed the soc_initialize_memory branch from 9391814 to a763e82 Compare April 2, 2024 05:10
@AndrewD AndrewD merged commit 0601bf5 into enjoy-digital:master Apr 2, 2024
1 check passed
@enjoy-digital
Copy link
Owner

Sorry for the delay in the review @AndrewD. I'll review it now and will probably modify things a bit. I also have a preference to avoid that much documentation/example directly in the code and prefer having it in a wiki. I'll do the change and will provide an update here.

@enjoy-digital
Copy link
Owner

@AndrewD: I've reworked things a bit on top of your commit with:

For an example of the usage, we could add it to the wiki or to a specific design requiring it.

@AndrewD
Copy link
Collaborator Author

AndrewD commented Apr 2, 2024

I was thinking of documenting some usage in the wiki, for example this can improve the demo firmware integration.
I used this for much cleaner integration of unit testing firmware, but it can also be used to load, for example, a customised bios for a target.

@AndrewD AndrewD deleted the soc_initialize_memory branch April 22, 2024 00:53
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