Devicetree
 help / color / mirror / Atom feed
From: Stefano Radaelli <stefano.radaelli21@gmail.com>
To: sashiko-reviews@lists.linux.dev
Cc: devicetree@vger.kernel.org, imx@lists.linux.dev, robh@kernel.org,
	conor+dt@kernel.org, Frank.Li@kernel.org
Subject: Re: [PATCH v1 1/2] arm64: dts: imx8mm-var-dart: Enable WM8904 DMIC input
Date: Fri, 16 Oct 2026 10:57:58 +0200	[thread overview]
Message-ID: <atHnFpMVpoumQ4UX@Lord-Beerus.station> (raw)
In-Reply-To: <sashiko-outbox-174034@kernel.org>

On Fri, Oct 16, 2026 at 08:49:52AM +0000, sashiko-bot@kernel.org wrote:
> 
> [Severity: Medium]
> Does this patch also need to update the simple-audio-card,routing array in
> the sound-wm8904 node?
> 
> Currently, the routing in arch/arm64/boot/dts/freescale/imx8mm-var-dart.dtsi
> still maps IN1L to "Microphone Jack":
> 
> 	sound-wm8904 {
> 		...
> 		simple-audio-card,routing =
> 			...
> 			"IN1L", "Microphone Jack",
> 			"IN1R", "Microphone Jack";
> 		...
> 	}
> 
> Exposing an onboard digital microphone (DMIC) as a "Microphone Jack" causes
> semantic confusion in ALSA userspace. Audio routing daemons like PulseAudio
> and PipeWire use widget names to distinguish between built-in microphones and
> external pluggable headsets.
> 
> Leaving the newly configured digital IN1L input tied to an external analog
> jack breaks userspace jack detection logic and presents incorrect routing
> options.
> 

“Microphone Jack” is the existing DAPM widget name; it does not select
analog operation or enable jack detection.
This board does not configure microphone jack detection through
simple-audio-card, so I do not see how retaining this name would cause
the reported userspace jack detection failure.

This patch adds the missing IN1L DMIC configuration.
The same configuration, including the existing widget name and routing,
is also used on the Variscite DART-MX91, DART-MX93 and DART-MX95.

A more descriptive widget name could be considered separately.
I would prefer to keep this patch focused on the missing DMIC property.

Stefano

  reply	other threads:[~2026-10-16  8:58 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-16  8:40 [PATCH v1 0/2] arm64: dts: imx8mm: Enable DMIC input on Variscite boards Stefano Radaelli
2026-10-16  8:40 ` [PATCH v1 1/2] arm64: dts: imx8mm-var-dart: Enable WM8904 DMIC input Stefano Radaelli
2026-10-16  8:49   ` sashiko-bot
2026-10-16  8:57     ` Stefano Radaelli [this message]
2026-10-16  8:40 ` [PATCH v1 2/2] arm64: dts: imx8mm-var-som: " Stefano Radaelli

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=atHnFpMVpoumQ4UX@Lord-Beerus.station \
    --to=stefano.radaelli21@gmail.com \
    --cc=Frank.Li@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=imx@lists.linux.dev \
    --cc=robh@kernel.org \
    --cc=sashiko-reviews@lists.linux.dev \
    /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