Skip to content

Conversation

@gmarull
Copy link
Contributor

@gmarull gmarull commented Sep 5, 2023

No description provided.

Module is using Zephyr SYS_INIT API, defined in init.h, not device.h.
Code relied on indirect includes.

Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
Because module uses SYS_INIT defined in init.h. Things likely worked
because of indirect includes.

Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
@sofci
Copy link
Collaborator

sofci commented Sep 5, 2023

Can one of the admins verify this patch?

reply test this please to run this test once

Copy link
Collaborator

@kv2019i kv2019i left a comment

Choose a reason for hiding this comment

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

Thanks @gmarull !

@marc-hb
Copy link
Collaborator

marc-hb commented Sep 5, 2023

Thanks!

Sparse error in https://github.com/thesofproject/sof/actions/runs/6082149977/job/16499306627?pr=8154 is known and should be fixed by #8147

I don't know what happened to sof-ci/jenkins, trying to re-run it.

Screenshot 2023-09-05 at 13 15 25

@marc-hb
Copy link
Collaborator

marc-hb commented Sep 5, 2023

SOFCI TEST

EDIT: sof-ci/jenkins tests are running now, no idea what happened 5 hours ago.

@marc-hb
Copy link
Collaborator

marc-hb commented Sep 6, 2023

test this please

@kv2019i
Copy link
Collaborator

kv2019i commented Sep 6, 2023

@wszypelt This seems to got stuck, can you check?

@wszypelt
Copy link

wszypelt commented Sep 6, 2023

@kv2019i Sure thing, the PR is already in the queue to be done, I think the results will be in an hour

@kv2019i kv2019i merged commit 014d262 into thesofproject:main Sep 6, 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.

8 participants