-
Notifications
You must be signed in to change notification settings - Fork 0
Add support for Implied ApobNvCopy BHD directory entry #222
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
base: main
Are you sure you want to change the base?
Conversation
That flash_location seems to be about 291 MiB ? Uhh. There's a limitation of my flash allocator as it is currently used: It can't do custom reserved ranges (but it can do reserved ranges in general and there is one used). Therefore, specifying Are you sure that specifying It might be that the allocator allocates anyway if you don't specify We should definitely add a test for that case now that we need it, though. Or, the hacky solution would be since we only use 16 MiB of the flash anyway, set |
For eMCR our plan was to add an The value in this draft PR is just a placeholder.
for testing. However, the ABL (specifically the PMU eMCR firmware) ends up trying to read from address 0x0 so it is either unhappy with this, or not finding it because I did not place it in the second level directory, and I think I may need to. |
I see! There is pretty complete second level directory support in amd-host-image-builder by now--so we can try it with that. |
Yes, this is compiling, but the PSP doesn't seem to boot. I suspect I need to put more items in there like one of the stock ROMs. + {
+ source: {
+ SecondLevelDirectory: {
+ entries: [
+ {
+ "source": "Implied",
+ "target": {
+ "type": "Apob",
+ "ram_destination_address": 0x4000000
+ }
+ },
+ {
+ "source": "Implied",
+ target: {
+ type: "ApobNvCopy",
+ flash_location: 0x7000000,
+ size: 0xd0000
+ }
+ }
+ ]
+ }
+ },
+ target: {
+ type: "SecondLevelDirectory"
+ }
+ }, |
In order to support eMCR - https://github.com/oxidecomputer/stlouis/issues/707
The flash address here is a placeholder, but it looks like the right thing was done: