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

[prodc] Add the prodc owner configuration #24799

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

cfrantz
Copy link
Contributor

@cfrantz cfrantz commented Oct 16, 2024

  1. Add the json representation of the prodc owner configuration.
  2. Use the binary form in the sku_creator_owner_init function to set the owner configuration on as-yet un-owned chips.

Copy link
Contributor

@jettr jettr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know how much my review is worth, but LGTM

@cfrantz cfrantz marked this pull request as ready for review October 17, 2024 22:36
@cfrantz cfrantz requested a review from a team as a code owner October 17, 2024 22:36
@cfrantz cfrantz requested review from engdoreis, moidx and timothytrippel and removed request for a team and engdoreis October 17, 2024 22:36
1. Add the json representation of the `prodc` owner configuration.
2. Use the binary form in the `sku_creator_owner_init` function to set
   the owner configuration on as-yet un-owned chips.

Signed-off-by: Chris Frantz <[email protected]>
@cfrantz
Copy link
Contributor Author

cfrantz commented Oct 21, 2024

The CI failure is not related to this PR.

@cfrantz cfrantz merged commit 572bdfe into lowRISC:earlgrey_es_sival Oct 21, 2024
23 of 25 checks passed
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.

3 participants