Received: by 2002:ab2:6816:0:b0:1f9:5764:f03e with SMTP id t22csp2994134lqo; Tue, 21 May 2024 03:50:04 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCWss44ACEe0BaExASHHcbzsdzNggtwSkRYtKqTmRf8QLW8R30kyfOyftMRNX3bMB10hhJNnETogjYOsfmXoZ5akZ7LI2xOwRJrAqBb0BQ== X-Google-Smtp-Source: AGHT+IFbc3EVbpCFSp8yUa5lZ13r+XHw0nQiAAfBuIck0W3bEC7ufLWEfUjr1Fq9EjKADu3akI+1 X-Received: by 2002:a50:f616:0:b0:56c:d35:1775 with SMTP id 4fb4d7f45d1cf-5734d6dea64mr28513641a12.35.1716288603976; Tue, 21 May 2024 03:50:03 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1716288603; cv=pass; d=google.com; s=arc-20160816; b=HOB9HzJvTCg4PPmLAOVD/LJKV3Lvzu2D8zhYSJWY6ooruSXQmn/m00IVSAdcrwivUg MLvrjLW8jnst6D7YM0sEk8D1KpwDqWcNtrbLLSnL/XgRWPfTDvXBpjbJy9FqptgKNXqW 1EiMGeVkVw1kkjaUgu+0Qjii5iPIKLgvxfrCR8DtfjGjLH6WcNbYI/UjZFWWgBJ8mVvl mWQoYXvU6hXREBdLmqiP0qN3N608eczGw0fs5I8uxxPc75ub9mU/BXxADcgieyrdrTSm ZxHqi8kuhDi9cs6LSd+UyBYIDCGZf/zqQ4ZKODnPQ1+uUSL3Ua7c/fATc4mTrW/bGnQB Q40w== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:message-id:subject:cc :to:from:date:dkim-signature; bh=NlQEmdCM5CKvJMc5j0LtU4m60PQ9JvKqt9LeE9hpIas=; fh=z929IZvYO10nnyO4x+bvRMcqh7lK/NkGrcn8KD9GnuU=; b=pSDcTVonYw77ow0dAYbqt1OwBFz0Z3xglKyMbGpYkJix7pId+eTkgP7bcvIsCtusSL ZNzWaWd6z6HpKL+8Zvmhc+Ap/k4gzOtiEc5WxbebfmPqSD2HpF+4ZJ8b8+9/ZBonWDrQ GyyYPTwn+nePpPslhX49dyr1mp1dUZpy44qlv5l2A4g23vqqe7SirdvbBo8fyM0uGRTX dhmAscV4t3RuWMmB6NG4nWUQxbiW54678BjuOR5QpLxN0Wv92Fuy/+yBVQHVldeyUm+z hMqO9YFvMD6omV3AThxkrtGkhobzlGR52GH7UIMQWWAZEFToop7ssOgHG6qco9C26Nrn vKcw==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=QkCGxQgO; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-184824-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-184824-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [147.75.80.249]) by mx.google.com with ESMTPS id 4fb4d7f45d1cf-5733c36e842si14051082a12.569.2024.05.21.03.50.03 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 21 May 2024 03:50:03 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-184824-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=QkCGxQgO; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-184824-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-184824-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id AE07D1F2293F for ; Tue, 21 May 2024 10:50:03 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 3DC2075815; Tue, 21 May 2024 10:49:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="QkCGxQgO" Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4682471B3D; Tue, 21 May 2024 10:49:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716288570; cv=none; b=na+uFEtBXvCGz3qLzHQFwkAdXLG2/i7Lrus5DITJQ2vCtKuxMYsfCz1V6hQSbgsxR1JWVw/nqI2RN1B+mTWQfrgP+Q5y8jXv7OVW91ndsx+Tk7CDzAOpm4D+QQG6nKoefWofRLvCGGzyH49Rn2v6NYPe3nTzPsUjSYPFSF9YcQU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716288570; c=relaxed/simple; bh=6KPrfDvxePiYqQ+U86QGOtRIQhJqkZNmdPViVsnR34A=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=J+qe6KnFRARArbjdPJq6cwsPu4NKgl3CAjQWdcMIFU9CGY+8XHaqZNO8pgt/gXlkYq15nQeZhwpCe/TSsA8GurPiUHaNTpUI9y6sxvgnTjj6ku3g5QW6l+I0yHS6qBjeYoJf3HPXRT1cOOS4NkPiZaG7UBt06s6JdqiUOhI4J8Y= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=QkCGxQgO; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9034EC32789; Tue, 21 May 2024 10:49:25 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1716288570; bh=6KPrfDvxePiYqQ+U86QGOtRIQhJqkZNmdPViVsnR34A=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=QkCGxQgOHgBFCqFIlV4VgKAH9/B6+1F+Do1GmLFKqPl1Igib7uV61Ae3Xkqy7JuLn 3cbFhuqfwf58ScdoX0J9/J8vTCOeD5Kip4AIehMr9L2lnevQxOZmoxk2fSjrTxU0Sa tZI+VSdqYEFh9l2Sj7pD5AjLcDfoMHzG+2qKUMaW/PT39BqCr39gGKwjCcqVt49d8n SC/7q0VHuRPRf3/c19Sbj1hHOrIZcEscJEN5EHv30PXm/YrePix9wM+ykthsAxBJcx RF/IsckJu+5LM+j2eELP6rptO7nE223xSnwXI0IxvSPS7oVyMucBQHmH7pks1gSWcC tCtE0o/rXxVtA== Date: Tue, 21 May 2024 11:49:21 +0100 From: Mark Brown To: Alvin =?utf-8?Q?=C5=A0ipraga?= Cc: Alvin =?utf-8?Q?=C5=A0ipraga?= , Greg Kroah-Hartman , "Rafael J. Wysocki" , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Linus Walleij , Bartosz Golaszewski , Liam Girdwood , Jaroslav Kysela , Takashi Iwai , Michael Turquette , Stephen Boyd , Andi Shyti , Saravana Kannan , Emil Abildgaard Svendsen , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-gpio@vger.kernel.org" , "linux-sound@vger.kernel.org" , "linux-clk@vger.kernel.org" , "linux-i2c@vger.kernel.org" Subject: Re: [PATCH 07/13] ASoC: codecs: add AD24xx codec driver Message-ID: References: <20240517-a2b-v1-0-b8647554c67b@bang-olufsen.dk> <20240517-a2b-v1-7-b8647554c67b@bang-olufsen.dk> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="IfNqVr8A6eo57Tqb" Content-Disposition: inline In-Reply-To: X-Cookie: Eloquence is logic on fire. --IfNqVr8A6eo57Tqb Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, May 21, 2024 at 06:46:21AM +0000, Alvin =C5=A0ipraga wrote: > On Fri, May 17, 2024 at 04:03:50PM GMT, Mark Brown wrote: > > On Fri, May 17, 2024 at 02:58:05PM +0200, Alvin =C5=A0ipraga wrote: > > > +static const char *const ad24xx_codec_slot_size_text[] =3D { > > > + "8 bits", "12 bits", "16 bits", "20 bits", > > > + "24 bits", "28 bits", "32 bits", > > > +}; > >=20 > > Why is this configured by the user rather than via set_tdm_slot(), and > > how would one usefully use this at runtime? >=20 > This configures the slot size of A2B data slots, not the slot size on > the TDM interface. Typically one would expect it to be the same, so your > question is valid. But it is not a strict requirement as far as the A2B > bus and hardware is concerned. >=20 > To give a concrete example, the TDM interface might run with a TDM slot > size of 32 bits, but the PCM data is in reality 24 bits padded to 32 > bits. In this case, A2B bus bandwidth can be saved by configuring the > "{Up,Down}stream Slot Size" kcontrol to "24 bits". >=20 > More detailed information can be found in the manual in [1] section 3-22 > "I2S/TDM Port Programming Concepts", where an analogous example is > given. That still doesn't sound like something that should be configured dynamically by the user. Based on that description it sounds like it's just the sample size so should cope from hw_params. --IfNqVr8A6eo57Tqb Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmZMfDAACgkQJNaLcl1U h9ACLgf9F/4Hlcaz6ms2pQ1jUz65htj1EgnNuZtgs/G5QOQR/dSlJToXp1Fsk0+B LX+uL2Bi+Va82Rv/PoN/GXymodv97hPrbFvwmyNHUaDncUXdlZSBVzo0h/3uk9Hf P2yvFxZLGkFVMrCxAaA+/C75nwRoftdmd/DbxGCtmcfy9VCO95LVf9jP6pHmAWk6 JGJ+VUBmwqZZ3NDyFqRvujjpE1uFgWrvgracm1PAujirj0hcYbhXwVpOTxh8HWBL 1WhXTof/JrHLwVWAWFEE2nuh2ZziHMiq1sWYA6tCxhdtQZSc4z2EJnI+qXJfrd2b ZHe0aWB3mYxQ+jN2cq3QGaG2KR7PUA== =Woy1 -----END PGP SIGNATURE----- --IfNqVr8A6eo57Tqb--