Received: by 2002:a25:ca44:0:0:0:0:0 with SMTP id a65csp2022043ybg; Thu, 30 Jul 2020 08:28:49 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxYKPL02PrEq9x5iYhfWbC5YZ/7ZCQv44sEo6/JL4lWkMWck7Nl+fYcWLaJ3SWnHhgykB6w X-Received: by 2002:a17:906:1986:: with SMTP id g6mr3311889ejd.404.1596122929541; Thu, 30 Jul 2020 08:28:49 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1596122929; cv=none; d=google.com; s=arc-20160816; b=kgu+Q6EeoHJ5v4Ckvt1X8YXAk1UQ6nzS2uadvg712DVPotg/T3+2lod1dv/6ZH09CM eqFzhiJ9ajCzEPBDLbebsQcmRwQRtPzLsoeHG8tpaamHMEAd3lYUCbRZO0T3pxACws8s mfHiAHS8NnZexov2wMQLn2/FI41GyU+qXrFuo5NmsPRVa6CaOg6mDC8RQlcnsEozgBNC wWOzB72136QNTdTSiFMemOJJEbpuuzDBzta4/liixlDEcySv866JturhpnnM1TrC/G7s nQzhZXV9BCkZPzJfYvJEMOO98T6BZonZAr3Jt/7H8tV4L4fq9TA89MGkcbA0VQt+tsKq gldQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:ironport-sdr:ironport-sdr; bh=bcpoeWg3gK4plohuWNImjdhN+AV89mYBY1xoH0unn/o=; b=AVGXlRUhAur6rL91jrLY8YwCJWxeVHHKO/xTy3VxT5XfOwn+1j4vUwQ/tWZ+4piz4c QhT0TGEgqs/pM+pHuC5Xubh0cw5bYfzbKy210b2Lu2z478CUAN+epLJQ7Gvn/TPckeAX T7kN5cdZScy4bvGvESKOGXU5uZeufGxhT8YvltJzXh+NwiEi6o7U+MsssdBCVyLwmn2H IbmEQKj6fq9c9HfIBm3WmdoRoBe+cDWSRNbIWqMTjTFsXd80O+CY5NkppUb3m7Qp5+w4 QYOgUIa0P59HXzdeyX0V+cQzktMf2kZ3CAl+eAs/jbOVgqkXOyQehcCfFiYvSwLkteyW b80A== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id q11si3636994edh.154.2020.07.30.08.28.26; Thu, 30 Jul 2020 08:28:49 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729786AbgG3P2D (ORCPT + 99 others); Thu, 30 Jul 2020 11:28:03 -0400 Received: from mga09.intel.com ([134.134.136.24]:56540 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726353AbgG3P2C (ORCPT ); Thu, 30 Jul 2020 11:28:02 -0400 IronPort-SDR: DFrzU6MicSBEM11B+KMjpC78HKB+ic+2mvKedKISlgeFSjfx/TiXPgBIutlYH50ogjuE7N4WaO EN6/uEXn1xxQ== X-IronPort-AV: E=McAfee;i="6000,8403,9698"; a="152842525" X-IronPort-AV: E=Sophos;i="5.75,414,1589266800"; d="scan'208";a="152842525" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Jul 2020 08:28:02 -0700 IronPort-SDR: gewmfRNTI+SQO76RR3vjSFXBpP2tQjoa0VuEZUc8PtYAj7exldhN0NOurCY5ZTcz5eot/dVMed 2aRhA6jnzNSw== X-IronPort-AV: E=Sophos;i="5.75,414,1589266800"; d="scan'208";a="465279278" Received: from jrwang2-mobl.amr.corp.intel.com (HELO [10.212.82.25]) ([10.212.82.25]) by orsmga005-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Jul 2020 08:27:59 -0700 Subject: Re: [PATCH 2/2] ASoC: Intel: Add period size constraint on strago board To: "Lu, Brent" , "alsa-devel@alsa-project.org" Cc: Guennadi Liakhovetski , "Rojewski, Cezary" , Kai Vehmanen , Kuninori Morimoto , Jie Yang , Takashi Iwai , "linux-kernel@vger.kernel.org" , Liam Girdwood , Sam McNally , Mark Brown , Ranjani Sridharan , Daniel Stuart , Andy Shevchenko , Yu-Hsuan Hsu , Damian van Soelen References: <1596020585-11517-1-git-send-email-brent.lu@intel.com> <1596020585-11517-3-git-send-email-brent.lu@intel.com> <4fe97f63-e552-3b2f-803c-53894b196bfd@linux.intel.com> From: Pierre-Louis Bossart Message-ID: Date: Thu, 30 Jul 2020 10:27:58 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org >> 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.