Skip to content

[stable-v2.7] [SKIP SOF-TEST] .github/zephyr: stop trying Zephyr main branch with a stable branch#8368

Merged
marc-hb merged 1 commit intothesofproject:stable-v2.7from
marc-hb:no-zmain-stable
Oct 20, 2023
Merged

[stable-v2.7] [SKIP SOF-TEST] .github/zephyr: stop trying Zephyr main branch with a stable branch#8368
marc-hb merged 1 commit intothesofproject:stable-v2.7from
marc-hb:no-zmain-stable

Conversation

@marc-hb
Copy link
Copy Markdown
Collaborator

@marc-hb marc-hb commented Oct 20, 2023

It's bold enough with the SOF main branch, it's obviously not going to work with a stable branch of SOF.

@marc-hb
.github/zephyr: stop trying Zephyr main branch with a stable branch
78c9af6 
It's bold enough with the SOF main branch, it's obviously not going to
work with a stable branch of SOF.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb
Copy link
Copy Markdown
Collaborator Author

marc-hb commented Oct 20, 2023

As reported in

@marc-hb
Copy link
Copy Markdown
Collaborator Author

marc-hb commented Oct 20, 2023

Interesting but totally unrelated sparse errors, only with MTL:

https://github.com/thesofproject/sof/actions/runs/6593183356/job/17915245012?pr=8368

/zep_workspace/zephyr/drivers/dma/dma_intel_adsp_hda.c:190:17: error: typename in expression
/zep_workspace/zephyr/drivers/dma/dma_intel_adsp_hda.c:190:26: error: Expected ; at end of statement
/zep_workspace/zephyr/drivers/dma/dma_intel_adsp_hda.c:190:26: error: got rp
/zep_workspace/zephyr/drivers/dma/dma_intel_adsp_hda.c:199:17: error: typename in expression
/zep_workspace/zephyr/drivers/dma/dma_intel_adsp_hda.c:199:26: error: Expected ; at end of statement
/zep_workspace/zephyr/drivers/dma/dma_intel_adsp_hda.c:199:26: error: got wp
/zep_workspace/zephyr/drivers/dma/dma_intel_adsp_hda.c:190:17: error: undefined identifier 'uint32_t'
/zep_workspace/zephyr/drivers/dma/dma_intel_adsp_hda.c:191:37: error: undefined identifier 'rp'
/zep_workspace/zephyr/drivers/dma/dma_intel_adsp_hda.c:199:17: error: undefined identifier 'uint32_t'
/zep_workspace/zephyr/drivers/dma/dma_intel_adsp_hda.c:200:37: error: undefined identifier 'wp'

@marc-hb marc-hb marked this pull request as ready for review October 20, 2023 22:28
@marc-hb marc-hb changed the title [SKIP SOF-TEST] .github/zephyr: stop trying Zephyr main branch with a stable branch [stable-v2.7] [SKIP SOF-TEST] .github/zephyr: stop trying Zephyr main branch with a stable branch Oct 20, 2023
@marc-hb
Copy link
Copy Markdown
Collaborator Author

marc-hb commented Oct 20, 2023 •
edited
Loading

Merging this without review because it's a successfully tested, dead-simple one-line removal in a GitHub actions config file that gets rids of a ton of noise and confusion

@marc-hb marc-hb merged commit c6af31b into thesofproject:stable-v2.7 Oct 20, 2023
@marc-hb marc-hb deleted the no-zmain-stable branch October 20, 2023 22:33
@marc-hb marc-hb mentioned this pull request Oct 20, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dbaluta dbaluta Awaiting requested review from dbaluta dbaluta is a code owner

@cujomalainey cujomalainey Awaiting requested review from cujomalainey

@lgirdwood lgirdwood Awaiting requested review from lgirdwood lgirdwood is a code owner

@jsarha jsarha Awaiting requested review from jsarha

@singalsu singalsu Awaiting requested review from singalsu

@kv2019i kv2019i Awaiting requested review from kv2019i

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@marc-hb