On Fri, 03 Feb 2017 17:43:56 +0100, Takashi Iwai wrote:
Hi,
this is the final part of my cleanup / refactoring patchsets for Intel LPE audio. Now the PCM stream engine has been rewritten. It supports more flexible configuration, not only fixed 4 periods, for example, yet with a lot of code reduction.
As usual, the patches are found in topic/intel-lpe-audio-cleanup branch.
So far, from my side, now most of changes have been submitted / merged. But I still have some unclear things in the code.
- The PCM format: the code contains the handling of S16_LE, but it doesn't seem work when I enabled the info bit. What's missing? Also U24_LE format is ever supported...? How?
- The behavior in had_process_buffer_underrun() isn't clear enough. Or, more exactly, the behavior of AUD_HDMI_STATUS reg and AUD_CONFIG reg look complex, and need a bit more description for readers (including me).
Can you guys enlighten a bit?
thanks,
Takashi