lkml.org 
[lkml]   [2020]   [Jul]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 2/2] ASoC: Intel: Add period size constraint on strago board
From
Date


>> Is this patch required if you've already constrained the period sizes for the
>> platform driver in patch1?
>
> Yes or alsa will select 320 as default period size for it.

ok, then that's a miss in your patch1. 320 samples is a multiple of 1ms
for 48kHz rates. I think it was valid only for the 16kHz VoIP paths used
in some versions of Android, but that we don't support in the upstream code.

To build on Takashi's answer, the real ask here is to require that the
period be a multiple of 1ms, because that's the fundamental
design/limitation of firmware. It doesn't matter if it's 48, 96, 192,
240, 480, 960 samples.

\
 
 \ /
  Last update: 2020-07-30 17:28    [W:0.134 / U:0.484 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site