Skip to content

west.yml: update rimage - #7069

Closed
ranj063 wants to merge 1 commit into
thesofproject:mainfrom
ranj063:fix/rimage_update
Closed

west.yml: update rimage#7069
ranj063 wants to merge 1 commit into
thesofproject:mainfrom
ranj063:fix/rimage_update

Conversation

@ranj063

@ranj063 ranj063 commented Feb 10, 2023

Copy link
Copy Markdown
Collaborator

Update rimage to d48ae7a to include the changes needed for adding the module init config in the manifest.

Signed-off-by: Ranjani Sridharan ranjani.sridharan@linux.intel.com

@ranj063

ranj063 commented Feb 10, 2023

Copy link
Copy Markdown
Collaborator Author

@lyakh Can you please take a look at this. You recent raimge patch seems to be breaking builds

@lyakh

lyakh commented Feb 10, 2023

Copy link
Copy Markdown
Collaborator

@lyakh Can you please take a look at this. You recent raimge patch seems to be breaking builds

@ranj063 yes, I know thesofproject/rimage#131 . Sorry about the breakage, I actually knew there was a problem before it was merged and I was certain I posted a comment in the PR to ask not to merge it yet, but now I see that there isn't such a comment there from me. No idea what happened. So please wait updating rimage hash before merging above PR. And maybe we need to add at least a couple of simple SOF build tests to rimage CI...

@ranj063
ranj063 force-pushed the fix/rimage_update branch 2 times, most recently from 2dce8be to 9cc530b Compare February 10, 2023 16:10
Update rimage to 1e0a85b to include the
changes needed for adding the module init config in the manifest.

Signed-off-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>

@kv2019i kv2019i left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

CI is mostly good, but quickbuild has failures.

@kv2019i

kv2019i commented Feb 13, 2023

Copy link
Copy Markdown
Collaborator

@wszypelt This seems to have hit an rsync error, can you check?

@ranj063

ranj063 commented Feb 13, 2023

Copy link
Copy Markdown
Collaborator Author

@lyakh im closing this. Can you pelase submit a new one afteryour rimage fixes are merged?

@ranj063 ranj063 closed this Feb 13, 2023
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