Messages in this thread | | | Subject | Re: [alsa-devel] [PATCH v2] staging: bcm2835-audio: interpolate audio delay | From | Kirill Marinushkin <> | Date | Thu, 25 Oct 2018 19:20:40 +0200 |
| |
Hello Takashi, Mike,
@Takashi
On 10/25/18 09:37, Takashi Iwai wrote: > Well, in the API POV, it's nothing wrong to keep hwptr sticking while > updating only delay value. It implies that the hardware chip doesn't > provide the hwptr update.
Thank you for the clarification. Modifying `runtime->delay` from the `pointer` function looked wrong for me. Now I understand the motivation and the use-case. I will be more careful when analyzing the code which doesn't fit my expectations.
@Mike
I was wrong. You can ignore my comments. Please don't take them personal: it's all about having high-quality code in kernel.
Best Regards, Kirill
| |