Linux Sound subsystem development
 help / color / mirror / Atom feed
From: Charles Keepax <ckeepax@opensource.cirrus.com>
To: Krzysztof Kozlowski <krzk@kernel.org>
Cc: Srinivas Kandagatla <srinivas.kandagatla@oss.qualcomm.com>,
	Mark Brown <broonie@kernel.org>,
	Liam Girdwood <lgirdwood@gmail.com>,
	Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>,
	Maciej Strozek <mstrozek@opensource.cirrus.com>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Srinivas Kandagatla <srini@kernel.org>,
	Bard Liao <yung-chuan.liao@linux.intel.com>,
	Pierre-Louis Bossart <pierre-louis.bossart@linux.dev>,
	Richard Fitzgerald <rf@opensource.cirrus.com>,
	Jorijn van der Graaf <jorijnvdgraaf@catcrafts.net>,
	linux-sound@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	devicetree@vger.kernel.org, patches@opensource.cirrus.com,
	linux-kernel@vger.kernel.org
Subject: Re: [RFC PATCH 0/8] ASoC: SDCA: enable on DT platforms and add Qualcomm WCD9378 (Tambora) codec
Date: Thu, 30 Jul 2026 11:20:36 +0100	[thread overview]
Message-ID: <amsldHHER2lRaoqf@opensource.cirrus.com> (raw)
In-Reply-To: <ec991e49-c23a-4c11-b0bb-32c12c427eee@kernel.org>

On Thu, Jul 30, 2026 at 11:11:41AM +0200, Krzysztof Kozlowski wrote:
> On 30/07/2026 11:08, Charles Keepax wrote:
> > On Wed, Jul 29, 2026 at 01:41:57PM +0200, Krzysztof Kozlowski wrote:
> >> On 23/07/2026 12:17, Charles Keepax wrote:
> >>> On Thu, Jul 23, 2026 at 12:42:10AM +0100, Srinivas Kandagatla wrote:
> > Do we really want to take this line? It basically forces us to
> > do "board files" again, which is the problem DT was solving in
> 
> Again? We since long, long, long do board files and they are approved.
> Although now they are called software nodes, but same stuff.

I have mostly seen those used for work arounds where the device
firmware is wrong or missing information. I am not sure I have
seen the sort of extensive use you are suggesting here?

> > the first place. I can't speak for Qualcomm, but at least for
> > our devices the SDCA properties can change a fair amount
> > between different laptops.
> 
> Sure, all properties which are not implied by compatible make sense for DT.

This is likely to be a very fluid set of things, at absolute
best maybe a super set is implied by the compatible but most of
properties can and will be tweaked per SKU.

There are also still plenty of advantages to the things that
are implied by the compatible being in the firmware. One of the
useful features of SDCA is that new hardware could be supported
without needing kernel updates (admittedly we are not quite
there yet on the Linux side but that is still the direction/goal).

> > Also there is the fact SDCA is already a thing and is gaining a
> > fair amount of traction pushing against it does really feel like
> > it is just making life much harder for ourselves.
> 
> I do not understand this comment. I am not pushing against SDCA. I am
> giving you guideline what can go to DT. Are you saying that SDCA somehow
> received review/approval of DT maintainers, so that we are bound by it now?

Obviously not, you guys are the maintainers and thus free to
merge/reject whatever you want. I am just highlighting the
reality that SDCA is now a widely adopted specification and
trying to steer the DT implementation in a direction that bears
no resemblence to that specification is going to be much harder
for everyone involved, likely including DT maintainers.

Thanks,
Charles

  reply	other threads:[~2026-07-30 10:21 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-22 23:42 [RFC PATCH 0/8] ASoC: SDCA: enable on DT platforms and add Qualcomm WCD9378 (Tambora) codec Srinivas Kandagatla
2026-07-22 23:42 ` [RFC PATCH 1/8] ASoC: SDCA: hw_params: program upstream Input Terminals for OT DAI Srinivas Kandagatla
2026-07-24 13:14   ` Charles Keepax
2026-07-24 13:53     ` Charles Keepax
2026-07-24 16:35     ` Srinivas Kandagatla
2026-07-27  8:33       ` Charles Keepax
2026-07-27 12:42         ` Srinivas Kandagatla
2026-07-22 23:42 ` [RFC PATCH 2/8] ASoC: SDCA: allow building without ACPI Srinivas Kandagatla
2026-07-22 23:42 ` [RFC PATCH 3/8] ASoC: SDCA: expose class helpers with hw_ops for non-DisCo platforms Srinivas Kandagatla
2026-07-22 23:42 ` [RFC PATCH 4/8] ASoC: SDCA: add PDE pre/post-pmu hooks to hw_ops Srinivas Kandagatla
2026-07-22 23:42 ` [RFC PATCH 5/8] ASoC: SDCA: class_function: xlate sound-dai cell by entity index Srinivas Kandagatla
2026-07-22 23:42 ` [RFC PATCH 6/8] ASoC: SDCA: register SDCA_FUNCTION_TYPE_SIMPLE_JACK in class function driver Srinivas Kandagatla
2026-07-23 10:55   ` Charles Keepax
2026-07-22 23:42 ` [RFC PATCH 7/8] dt-bindings: sound: qcom: add Tambora WCD9378 SDCA codec Srinivas Kandagatla
2026-07-23  9:14   ` Konrad Dybcio
2026-07-23 13:28     ` Srinivas Kandagatla
2026-07-23 13:31       ` Konrad Dybcio
2026-07-29 11:40   ` Krzysztof Kozlowski
2026-07-29 12:17     ` Srinivas Kandagatla
2026-07-29 12:30       ` Krzysztof Kozlowski
2026-07-29 12:36         ` Srinivas Kandagatla
2026-07-29 12:43           ` Krzysztof Kozlowski
2026-07-29 13:02             ` Srinivas Kandagatla
2026-07-29 13:23               ` Krzysztof Kozlowski
2026-07-29 13:34                 ` Srinivas Kandagatla
2026-07-29 14:04                   ` Krzysztof Kozlowski
2026-07-29 17:23                     ` Jorijn van der Graaf
2026-07-22 23:42 ` [RFC PATCH 8/8] ASoC: codecs: add Qualcomm Tambora (WCD9378) " Srinivas Kandagatla
2026-07-23 10:17 ` [RFC PATCH 0/8] ASoC: SDCA: enable on DT platforms and add Qualcomm WCD9378 (Tambora) codec Charles Keepax
2026-07-23 13:24   ` Srinivas Kandagatla
2026-07-24 12:36     ` Charles Keepax
2026-07-24 14:31       ` Srinivas Kandagatla
2026-07-24 15:40         ` Charles Keepax
2026-07-24 16:29           ` Srinivas Kandagatla
2026-07-29 11:14         ` Srinivas Kandagatla
2026-07-29 11:41   ` Krzysztof Kozlowski
2026-07-30  9:08     ` Charles Keepax
2026-07-30  9:11       ` Krzysztof Kozlowski
2026-07-30 10:20         ` Charles Keepax [this message]
2026-07-23 19:40 ` Jorijn van der Graaf
2026-07-24 12:17   ` Srinivas Kandagatla
2026-07-24 19:14     ` Jorijn van der Graaf

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=amsldHHER2lRaoqf@opensource.cirrus.com \
    --to=ckeepax@opensource.cirrus.com \
    --cc=broonie@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=jorijnvdgraaf@catcrafts.net \
    --cc=krzk+dt@kernel.org \
    --cc=krzk@kernel.org \
    --cc=lgirdwood@gmail.com \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-sound@vger.kernel.org \
    --cc=mstrozek@opensource.cirrus.com \
    --cc=patches@opensource.cirrus.com \
    --cc=perex@perex.cz \
    --cc=pierre-louis.bossart@linux.dev \
    --cc=rf@opensource.cirrus.com \
    --cc=robh@kernel.org \
    --cc=srini@kernel.org \
    --cc=srinivas.kandagatla@oss.qualcomm.com \
    --cc=tiwai@suse.com \
    --cc=yung-chuan.liao@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox