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

west.yml: Update rimage revision to 4ce79b15 #7630

Merged
merged 1 commit into from
Jun 2, 2023

Conversation

fredoh9
Copy link
Contributor

@fredoh9 fredoh9 commented May 16, 2023

SRC fixes were merged. Changed rimage submodule revision to 8f250a9

@marc-hb
Copy link
Collaborator

marc-hb commented May 17, 2023

@kv2019i
Copy link
Collaborator

kv2019i commented May 25, 2023

@fredoh9 ping? @singalsu Do you depend on this?

@fredoh9
Copy link
Contributor Author

fredoh9 commented May 25, 2023

yes, we need this for SRC. @singalsu can you confirm too?

thesofproject/sof-test#1035

@marc-hb
Copy link
Collaborator

marc-hb commented May 30, 2023

If we uprev rimage then we should really get this fix too:

@marc-hb marc-hb marked this pull request as draft May 30, 2023 19:40
@fredoh9
Copy link
Contributor Author

fredoh9 commented Jun 1, 2023

Trying to fast forward to the latest, will update soon after some tests

Pull in following rimage changes:

4ce79b1 (HEAD, manifest-rev) kpb: update kpb uuid for lnl
649b0a6 file_simple: Allocate correct buffer size for writing sections out
1ea1327 README.md: remove "install" target
8f250a9 Config: Change indent for SRCINTC
acd8a2b Config: Change SRCINTC for tgl-cavs.toml and tgl-h-cavs.toml to LL
77d4a2a cmake: remove "install" target

Signed-off-by: Fred Oh <[email protected]>
@fredoh9 fredoh9 changed the title west.yml: Update rimage revision to 8f250a99 west.yml: Update rimage revision to 4ce79b15 Jun 1, 2023
@fredoh9 fredoh9 marked this pull request as ready for review June 1, 2023 21:26
@fredoh9
Copy link
Contributor Author

fredoh9 commented Jun 1, 2023

@kv2019i @marc-hb @singalsu @ranj063
I updated with latest rimage. I was able to test MTLP_RVP_HDA locally and PR test results also looks good.

@kv2019i
Copy link
Collaborator

kv2019i commented Jun 2, 2023

On PM fail in https://sof-ci.01.org/sofpr/PR7630/build8846/devicetest/index.html , mandatory checks pass, merging.

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.

5 participants