Skip to content

Conversation

@vijendarmukunda
Copy link

Add a filter to skip the RT172 VB configuration if a SmartMic Function is not found in the SDCA descriptors.

If the ACPI information is incorrect this can only be quirked further with DMI information.

Add a filter to skip the RT172 VB configuration if a SmartMic Function
is not found in the SDCA descriptors.

If the ACPI information is incorrect this can only be quirked further
with DMI information.

Signed-off-by: Vijendar Mukunda <Vijendar.Mukunda@amd.com>
bardliao
bardliao previously approved these changes Jan 2, 2026
As per design, set IPC conf structure flags to zero during acp init
sequence.

Signed-off-by: Vijendar Mukunda <Vijendar.Mukunda@amd.com>
Tested-by: Umang Jain <uajain@igalia.com>
Firmware will set dsp_ack to 1 when firmware sends response for the IPC
command issued by host. Similarly dsp_msg flag will be updated to 1.

During ACP D0 entry sequence, during firmware run sequence,
Once the firmware sends the FW_READY message in the acp irq thread,
dsp_ack_write register relative offset read from scratch memory
points to garbage value.
In this case, fw_state is SOF_FW_BOOT_READY_OK results in else
condition execution printing below dmesg log.
"IPC reply before FW_BOOT_COMPLETE".
This is not an expected sequence.

Fix the condition checks for ipc flags.

Fixes: 738a2b5 ("ASoC: SOF: amd: Add IPC support for ACP IP block")
Signed-off-by: Vijendar Mukunda <Vijendar.Mukunda@amd.com>
Tested-by: Umang Jain <uajain@igalia.com>
@vijendarmukunda
Copy link
Author

Close this PR as this patch has been pushed for upstream review

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