Skip to content

Fixes for find_acpi_adr_device() when some endpoints are missing#5702

Merged
bardliao merged 2 commits intothesofproject:topic/sof-devfrom
mstrozek:endpoints
Apr 1, 2026
Merged

Fixes for find_acpi_adr_device() when some endpoints are missing#5702
bardliao merged 2 commits intothesofproject:topic/sof-devfrom
mstrozek:endpoints

Conversation

@mstrozek
Copy link
Copy Markdown

Two patches to make sure find_acpi_adr_device can work well when some of the endpoints are missing and do not map 1:1 to codec_info_list

In case of missing endpoints, the sequential numbering will cause wrong
mapping. Instead, assign the original DAI index from codec_info_list.

Fixes: 5226d19 ("ASoC: SOF: Intel: use sof_sdw as default SDW machine driver")
Signed-off-by: Maciej Strozek <[email protected]>
is_endpoint_present() iterates over sdca_data.num_functions, but checks
the dai_type according to codec info list, which will cause problems if
not all endpoints from the codec info list are present. Make sure the
type of actually present functions is compared against target dai_type.

Fixes: 5226d19 ("ASoC: SOF: Intel: use sof_sdw as default SDW machine driver")
Signed-off-by: Maciej Strozek <[email protected]>
@mstrozek mstrozek requested a review from bardliao as a code owner March 18, 2026 12:52
Copy link
Copy Markdown
Collaborator

@bardliao bardliao left a comment

Choose a reason for hiding this comment

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

Good catch, thanks.

@mstrozek
Copy link
Copy Markdown
Author

Hi @bardliao is it ok to merge? Or do you prefer me to send it upstream?
Also same for this PR: #5697

@bardliao bardliao merged commit b309a4a into thesofproject:topic/sof-dev Apr 1, 2026
9 of 11 checks passed
@bardliao
Copy link
Copy Markdown
Collaborator

bardliao commented Apr 1, 2026

Hi @bardliao is it ok to merge? Or do you prefer me to send it upstream? Also same for this PR: #5697

Merged, thanks.

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.

2 participants