All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Hernan Ganzo <gp@sneemgp.com>
Cc: linux-sound@vger.kernel.org, tiwai@suse.com, perex@perex.cz
Subject: Re: [PATCH] ALSA: hda/realtek: Limit internal mic boost on HP Pavilion 15 (103c:2164)
Date: Mon, 28 Sep 2026 13:33:46 +0200	[thread overview]
Message-ID: <871pad7gpx.wl-tiwai@suse.de> (raw)
In-Reply-To: <20260921205538.41192-1-gp@sneemgp.com>

On Mon, 21 Sep 2026 22:55:38 +0200,
Hernan Ganzo wrote:
> 
> The HP Pavilion 15 with the ALC3227 codec (SSID 103c:2164) exposes the full
> four-step internal mic boost on pin 0x12: nsteps=3 with stepsize=0x2f, that is
> 0, 12, 24 and 36 dB. Combined with the ADC capture gain of +30 dB the analogue
> cascade reaches 66 dB, and the internal microphone clips on the ambient noise of
> a quiet room; the upper two steps are unusable at any useful input level. This
> is the defect class ALC269_FIXUP_LIMIT_INT_MIC_BOOST exists for.
> 
> The machine matches no SSID quirk and falls through to the pin table, which
> selects ALC269_FIXUP_HP_MUTE_LED_MIC1 through a pin signature shared with
> several other HP models. Add an SSID entry instead, using
> ALC269_FIXUP_LIMIT_INT_MIC_BOOST_MUTE_LED: the existing fixup that applies
> alc269_fixup_limit_int_mic_boost() and chains to the same mute LED fixup, as
> already done for 103c:218b. The boost becomes 0 or 12 dB and the LED handling is
> unchanged.
> 
> Tested on the affected machine by selecting the same fixup through the model
> string (snd-hda-intel.model), which bypasses the SSID and pin tables, with the
> driver confirming the selection:
> 
>   snd_hda_codec_alc269 hdaudioC1D0: ALC3227: picked fixup limit-mic-boost (model specified)
> 
> With that, Internal Mic Boost reports Limits 0 - 1 (0 or 12 dB) instead of 0 - 3
> (0 to 36 dB), so alc269_fixup_limit_int_mic_boost() clamps this codec's internal
> mic pin as intended. Note that /proc/asound/card1/codec#0 keeps printing the
> unclamped hardware caps (nsteps=0x03): that file reads codec parameters uncached
> by design, while the fixup overrides the driver's cached parameter that the
> control is built from.
> 
> This does not by itself make the default capture volume sane -- the remaining
> 12 dB on top of the +30 dB ADC gain is still hotter than this machine's usable
> operating point. It removes the two unusable steps from the hardware range, which
> is what a quirk of this class can do.
> 
> Signed-off-by: Hernan Ganzo <gp@sneemgp.com>

Applied now.  Thanks.


Takashi

      reply	other threads:[~2026-09-28 11:33 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-21 20:55 [PATCH] ALSA: hda/realtek: Limit internal mic boost on HP Pavilion 15 (103c:2164) Hernan Ganzo
2026-09-28 11:33 ` Takashi Iwai [this message]

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=871pad7gpx.wl-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=gp@sneemgp.com \
    --cc=linux-sound@vger.kernel.org \
    --cc=perex@perex.cz \
    --cc=tiwai@suse.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.