Skip to content

west.yml: Update Zephyr revision - #7642

Merged
kv2019i merged 1 commit into
thesofproject:mainfrom
LaurentiuM1234:west_update
May 24, 2023
Merged

kv2019i merged 1 commit into
thesofproject:mainfrom
LaurentiuM1234:west_update

Conversation

@LaurentiuM1234

Copy link
Copy Markdown
Contributor

Update Zephyr revision to contain CONFIG_DCACHE_LINE_SIZE additions.

@LaurentiuM1234

This comment was marked as resolved.

@marc-hb

marc-hb commented May 18, 2023

Copy link
Copy Markdown
Collaborator

@LaurentiuM1234

This comment was marked as resolved.

@marc-hb

This comment was marked as resolved.

@aborisovich

This comment was marked as resolved.

Update Zephyr revision to contain CONFIG_DCACHE_LINE_SIZE
additions.

Signed-off-by: Laurentiu Mihalcea <laurentiu.mihalcea@nxp.com>
@kv2019i

kv2019i commented May 22, 2023

Copy link
Copy Markdown
Collaborator

@kv2019i

kv2019i commented May 23, 2023

Copy link
Copy Markdown
Collaborator

Ready to go, would need another review from someone...

@marc-hb

marc-hb commented May 23, 2023 •

Copy link
Copy Markdown
Collaborator

Ready to go, would need another review from someone...

This is just a one-line Zephyr revision bump so there is not much to review... unless you want someone (not me!) to (re-)review the corresponding Zephyr changes?

@kv2019i

kv2019i commented May 24, 2023

Copy link
Copy Markdown
Collaborator

@marc-hb wrote:

This is just a one-line Zephyr revision bump so there is not much to review... unless you want someone (not me!) to (re-)review the corresponding Zephyr changes?

Right, this is not your normal review, but I think it's good to stick to normal review process and have more than one person review the PR and indicate the Zephyr update is good to go. I don't expect people to find issues in the code, but +1 is an indication that they are aware of the update and give their blessing for it. In ideal world, our CI would be perfect and maintainers could just look at the CI and press merge, but alas we are not quite there.

@kv2019i
kv2019i merged commit d735c91 into thesofproject:main May 24, 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.

6 participants