> Changes v2 -> v1: > Almost all updates here are thanks to feedback from Pierre. Can I ask for a bit of mercy on reviewers and specifically my time? I sent my comments on Friday, I haven't had time to look at the answers and there's a new version already? > - several comments and few kernel-docs have been added in areas which > felt more or less unclear > - avs_ipc_wait_busy_completion() now spins up to 'repeats_left' number > of times before giving up > - 'adsp_ba' field of struct avs_dev has been renamed to 'dsp_ba' > - 'dops' field of struct avs_spec has been renamed to 'dsp_ops' > - IPC abstraction has been simplified: SKL_ADSP_REG_HIP* regs are used > directly > - fixed allnoconfig with AVS enabled compilation issues > - fixed code loading error paths: previously requested firmware is now > released before function return to the caller > - code and function arguments tied to D0IX support have been removed > from this patchset and will be part of followup series adding that > feature instead > - enriched dev_err() messages in avs_dsp_get_core() and > avs_dsp_put_core() > - numerous wording fixes used in power/reset/stall DSP operations