[PATCH 15/17] ASoC: Intel: avs: Implement CLDMA transfer
Cezary Rojewski
cezary.rojewski at intel.com
Fri Feb 25 20:38:07 CET 2022
On 2022-02-25 3:18 AM, Pierre-Louis Bossart wrote:
>> +int hda_cldma_stop(struct hda_cldma *cl)
>> +{
>> + unsigned int reg;
>> + int ret;
>> +
>> + /* disable interrupts */
>> + snd_hdac_adsp_updatel(cl, AVS_ADSP_REG_ADSPIC,
>> + AVS_ADSP_ADSPIC_CLDMA, 0);
>
> single line?
>
>> + snd_hdac_stream_updateb(cl, SD_CTL, SD_INT_MASK | SD_CTL_DMA_START, 0);
>> +
>> + /* await DMA engine stop */
>> + ret = snd_hdac_stream_readb_poll(cl, SD_CTL, reg,
>> + !(reg & SD_CTL_DMA_START),
>
> move to previous line? we can use 100 chars these days, and that's what
> you do below.
>
>> + AVS_CL_OP_INTERVAL_US, AVS_CL_OP_TIMEOUT_US);
>> + cancel_delayed_work_sync(&cl->memcpy_work);
>> +
>> + return ret;
>> +}
Ack on both.
More information about the Alsa-devel
mailing list