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.