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

WIP - BIOS: Flashboot without main ram #57

Merged
merged 1 commit into from
Feb 10, 2018

Conversation

rohitk-singh
Copy link

Modified flashboot() to skip copy to main ram if there is no main
ram, and instead execute in place out of SPI flash. (For this to
work the linker .ld will also need to redirect references to be
inside the SPI flash mapping.)

Modified flashboot() to skip copy to main ram if there is no main
ram, and instead execute in place out of SPI flash.  (For this to
work the linker .ld will also need to redirect references to be
inside the SPI flash mapping.)
@enjoy-digital
Copy link
Owner

Isn't it a duplicate of #55?

@mithro
Copy link
Collaborator

mithro commented Jan 20, 2018

@enjoy-digital - @rohitk-singh Is testing this change on the cmod-a7

@mithro mithro changed the title BIOS: Flashboot without main ram WIP - BIOS: Flashboot without main ram Jan 20, 2018
@enjoy-digital
Copy link
Owner

ok, we'll merge it once validated.

@enjoy-digital
Copy link
Owner

@rohitk-singh: Was it validated?

@rohitk-singh
Copy link
Author

@enjoy-digital No. We ran issues with Cmod A7 and were not able to program it. Most likely issues with Openocd and the 12MHz clock sourced from FTDI, which openocd controls. I'll test it with MimasV2 and then confirm. Please do not merge yet.

@enjoy-digital
Copy link
Owner

@rohitk-singh: any update?

@rohitk-singh
Copy link
Author

rohitk-singh commented Feb 10, 2018

@enjoy-digital Sorry for delay. I just tested it. Works fine. Designer just has to make sure to define flash_boot_address in the Python SoC target class with the correct memory-mapped location (in SPI Flash) of the firmware to boot. Otherwise BIOS will not support flash boot.

@enjoy-digital
Copy link
Owner

thanks! merged.

@enjoy-digital enjoy-digital merged commit 7b5bd40 into enjoy-digital:master Feb 10, 2018
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.

4 participants