Skip to content

[rom_ext] Backport #25258 and #25278 #27662

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

pamaury
Copy link
Contributor

@pamaury pamaury commented Jul 18, 2025

Backport #25258, #25278 and #25359

NOTE: first commit of #25359 is not cherry-picked because it is not necessary on master (master was changed from addresses to offset due to the use of the DT, this change triggered the fixed on earlgrey_1.0.0 when I noticed the discrepancy between offsets and addresses)

1. Initialize the ownership sealing key during the UDS stage of keymgr
   setup.
2. Call the `sku_creator_owner_init` function to initialize the
   ownership INFO page.
3. Link in the per-sku `<sku>_owner` implementations of
   `sku_creator_owner_init`.

Signed-off-by: Chris Frantz <cfrantz@google.com>
@pamaury pamaury requested review from cfrantz and a team as code owners July 18, 2025 16:14
@pamaury pamaury requested review from alees24, moidx and timothytrippel and removed request for a team July 18, 2025 16:14
The INFO page layout has changed between the ES and PROD chips.  Update
the lockdown and ownership functions relating to INFO configuration.
Update the `flash_permission_test` to verify the configuration.

Signed-off-by: Chris Frantz <cfrantz@google.com>
Add an INFO page to the various `with-flash` configurations in the
ownership transfer tests.

Signed-off-by: Chris Frantz <cfrantz@google.com>
@pamaury pamaury requested a review from jwnrt July 21, 2025 11:49
@pamaury pamaury changed the title Backport #25258 and #25278 [rom_ext] Backport #25258 and #25278 Jul 21, 2025
@moidx moidx removed the request for review from a team July 22, 2025 06:52
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