[PATCH 1/5] ASoC: soc-dai: update snd_soc_dai_delay() to snd_soc_pcm_dai_delay()
Pierre-Louis Bossart
pierre-louis.bossart at linux.intel.com
Mon Nov 29 16:42:34 CET 2021
On 11/26/21 8:09 AM, Mark Brown wrote:
> On Thu, Nov 25, 2021 at 11:41:38PM +0000, Kuninori Morimoto wrote:
>
>> Sorry for my late response.
>> (It was PC replace week)
>
> No worries, hope the new PC is working well!
>
>>>> I would think it's more accurate to update the delay information while
>>>> dealing with the hw_ptr update, no?
>
>>> Morimoto-san?
>
>> I think your opinion is very correct.
>> But this patch-set is for "cleanup/refactoring",
>> and your opinion is for "add new feature", I think.
>
> Hrm, right - this isn't making anything better or worse in terms of the
> accuracy, it's just moving things around so Pierre's suggestion is a
> separate thing.
Indeed, I misunderstood the series are removing the update of the
runtime->delay field while updating the hw_ptr, but this is only
shuffling code around. This is still updating both avail/delay in the
same manner so no objections. Sorry for the noise.
More information about the Alsa-devel
mailing list