From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f180.google.com (mail-pf1-f180.google.com [209.85.210.180]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 28A0737DE95 for ; Tue, 3 Mar 2026 09:10:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.180 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772529061; cv=none; b=l7q8UEQztXrrv27HL4fLpfvUU2HztGGxvLIQKC9xo2SMvWRfMEjXH83am3S4Rx7y7HDzcR3EAjR1dGQD+tiQAQpZ3/TMcO3yNpzfInU+G8e2ohIsp2gA4q3qvb8Erm3R8SV/oxVTAb+O5OyezVeaCrWabWUNf0M2trAbSd+PI3Q= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772529061; c=relaxed/simple; bh=w8Kx1qlt4D62VcByjefxrRx2gnqxk8TlOdvwp54Diog=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=gM2MnfO3rHmg4CP6McwktNeCq330wtwnNSdHX+QFldg2eKV7vNHJxrdssv3NvxLih9lJdzJYbq7ismj227kUAgFvi5YKdwnr3s3IDo04UNmmZn6qQxdeb7NQp1Vmq6U4Vhsxic+xBTG45DPSrO1GwfOuhH74JlYa6ah0e5kSvLk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=kEDM+6nt; arc=none smtp.client-ip=209.85.210.180 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="kEDM+6nt" Received: by mail-pf1-f180.google.com with SMTP id d2e1a72fcca58-827390e8a3cso3512800b3a.0 for ; Tue, 03 Mar 2026 01:10:59 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1772529059; x=1773133859; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=5ieqeIgWXtsJGqN9iYbh1K+fW5zi5u0kpmcIzoP8Jzk=; b=kEDM+6ntM9yXf82/Pl5V1bxXXUkGZskBg96YLCjFQJHINTh/T5iumYkLkzUgsnZyml aumbsRO+xRW9Wm3uMQAd07b54yfAsghyfNPRENQpDqCinVhpGUJqb+g3BsrZK40SWhoM qRwyOTUSzwpeT3lgrJT0p01xXXQzMLOslX/4vnzioPgHxB+4EIQEEo8AfKtvlNionTpB FmNV2WhKhSUdckbkYp4+lIyaE9vYYfn98Gwyp4siijvqjpVaEUCwPMHgATIBz/K8vUA+ k3QVJUh71S9PjnPfZ+My8fkZQRJFSXoXHsjjsRgzgVsnwicUZNgI/yWNIWaTs6r3KAFv HiPg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1772529059; x=1773133859; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=5ieqeIgWXtsJGqN9iYbh1K+fW5zi5u0kpmcIzoP8Jzk=; b=juTIAOJYwZ38918NCBCCdaGRfAdWT9nWfvkpzvU1YY/aFfa1sHm5RLBbZNlfUw81lG AIrH1ub+EwhS6AxkNEXehhOvdsOCLX6qV4m3raghn89JZM+vHIP0/qk+eBz1M88khmF8 HpPesGD8VcLHy9fqRNeTWcliFMOyyOh3JV4I85gPbmLt3Fny2MXKJhbCPC9dvEDqREM/ 5IP0HpZbl8DBAsuf69lWsvKpEziNT2+vtZoQTJ5cR2hJvR1NBIz8QD46IGtG/1r3vtWB l+yOf+bliplJtkmW397/d15HmtUduKqWiFlbOWatBRY2A1yyWYNOrW5Flcbnbi57YWZo lMKg== X-Forwarded-Encrypted: i=1; AJvYcCVBKqQz4LF6RSVV9knruEul940G4gszpe4WLUh8lK/HMBLGPSq+RmNBWomE8b2vhnnvRAZyHkjl3Ay+wRY=@vger.kernel.org X-Gm-Message-State: AOJu0YwvBmd9GSsrRPaQ/0C/vD7Saao3OUGeAzt2gbTH3PLwcvCrDmSd OYlabcQSxDKmivImVv0XE+5oapa24EtR799T89zojdAlMu52zKhULTDz X-Gm-Gg: ATEYQzzRYvy9OZlhFCtYE0GdDieuIjU0vCnzKHGUTj8xlcz2g9LXef9V0YLQ1G3pn9/ zIYlbSeUv5okNFpLZ7cXv3AvyAOJIoUj76Lm9kUIvdBj9GN0GD2AC+JeUvpAC3nd4rkNmXtn7mm 1Mcmy2/OUQNgFNA4ofjNbY+w9hs6zJDxr2APrN21cXxnW/T9JI5GNjcJEbzC3AAETCeFnr9XBW/ DScONLw30+fi8ZGeIkiIgei7Z2Gve9WIxC6IgVDYE09Fh6C/8t/D2kYPEX3jAqUKh3RxkrwUCB6 6HZMymyJGdktnmjAr1zkaZuyWUOVDZx272tcHLUfe7BGR8BMHm4AYZjYMiCpqt8tnaAq/TiBIv6 B9io33qGWaSkn1MbBsNSzJ00fJKEt+bRJHG/UI/mM7N1nqXm2eu8fWk0mFtiHIbco8akN2/g2cq MyfkRmUaC6zp5D0w/3EQn/8VLStoJtg8itpHywz7vbVB4X7nm9AQHVrS+j3LOAlYLTUv5WkwB5x MXyASd7HRqpROZ1bS8Cfs/obdUvxoBHHZ7mFRFqm9/o849j6lYJFNLmeBAonNnErGfCu0iO0qyb lgA= X-Received: by 2002:a05:6a00:1ad4:b0:825:2927:3aa6 with SMTP id d2e1a72fcca58-8295d82a9abmr1551612b3a.14.1772529059400; Tue, 03 Mar 2026 01:10:59 -0800 (PST) Received: from setsuna.localnet ([2403:581e:d87e:0:fc2d:ed31:e80e:412d]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-82739db4955sm16000114b3a.29.2026.03.03.01.10.52 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 03 Mar 2026 01:10:59 -0800 (PST) From: James Calligeros To: Kuninori Morimoto Cc: Liam Girdwood , Mark Brown , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Shengjiu Wang , Jaroslav Kysela , Takashi Iwai , Shenghao Ding , Kevin Lu , Baojun Xu , Frank Li , linux-sound@vger.kernel.org, devicetree@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, asahi@lists.linux.dev Subject: Re: [PATCH v3 5/7] ASoC: soc-dai: add common operation to set TDM idle mode Date: Tue, 03 Mar 2026 19:10:49 +1000 Message-ID: <4716405.LvFx2qVVIh@setsuna> In-Reply-To: <877brtacgp.wl-kuninori.morimoto.gx@renesas.com> References: <20260301-tdm-idle-slots-v3-0-c6ac5351489a@gmail.com> <20260301-tdm-idle-slots-v3-5-c6ac5351489a@gmail.com> <877brtacgp.wl-kuninori.morimoto.gx@renesas.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Hi Kuniori, On Tuesday, 3 March 2026 12:11:50=E2=80=AFpm Australian Eastern Standard Ti= me Kuninori=20 Morimoto wrote: > > +int snd_soc_dai_set_tdm_idle(struct snd_soc_dai *dai, > > + unsigned int tx_mask, unsigned int rx_mask, > > + int tx_mode, int rx_mode); >=20 > Do we need "rx_mode" ? > Only "tx_mode" is enough if my understanding was correct. =46or our (Asahi) purposes yes, we only require TX. However, TAS2770 also has a bus keeper on SDIN, and I suspect other hardware may too. Rather than break the API later on should the need to configure this arise, I decided to add the capability now. I am happy to get rid of it if we think it is superfluous at this time though. > This patch-set adds new snd_soc_dai_set_tdm_idle(), but no one is calling > it. Who use it ?? Currently no one in tree consumes this API, and it is likely that no one wi= ll until we submit our platform driver. See the linked resources in the cover letter for more details. In the interests of transparency, I don't expect to have the platform driver ready for submission any time soon; it is a bit of a mess and given that I did not write most of it I still need to wrap my head around why certain decisions were made during its design. However, we are 100% committed to getting it merged at some point. If you would prefer I can resubmit this series along with the driver once it is ready, so that there is an example consumer for the API. Regards, James Calligeros