All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Phil Calvin <phil@philcalvin.com>
Cc: alsa-devel@alsa-project.org, Takashi Iwai <tiwai@suse.com>
Subject: Re: [PATCH 1/1] ALSA: hda/realtek: fix mic boost on Intel NUC 8
Date: Fri, 16 Apr 2021 10:05:33 +0200	[thread overview]
Message-ID: <s5h8s5islgy.wl-tiwai@suse.de> (raw)
In-Reply-To: <80dc5663-7734-e7e5-25ef-15b5df24511a@philcalvin.com>

On Fri, 16 Apr 2021 00:01:29 +0200,
Phil Calvin wrote:
> 
> Fix two bugs with the Intel HDA Realtek ALC233 sound codec
> present in Intel NUC NUC8i7BEH and probably a few other similar
> NUC models.
> 
> These codecs advertise a 4-level microphone input boost amplifier on
> pin 0x19, but the highest two boost settings do not work correctly,
> and produce only low analog noise that does not seem to contain any
> discernible signal. There is an existing fixup for this exact problem
> but for a different PCI subsystem ID, so we re-use that logic.
> 
> Changing the boost level also triggers a DC spike in the input signal
> that bleeds off over about a second and overwhelms any input during
> that time. Thankfully, the existing fixup has the side effect of
> making the boost control show up in userspace as a mute/unmute switch,
> and this keeps (e.g.) PulseAudio from fiddling with it during normal
> input volume adjustments.
> 
> Finally, the NUC hardware has built-in inverted stereo mics. This
> patch also enables the usual fixup for this so the two channels cancel
> noise instead of the actual signal.
> 
> Signed-off-by: Phil Calvin <phil@philcalvin.com>

Thanks, applied now with a minor fix of the quirk entry position
(sorted in order).

But the patch text wasn't cleanly applicable and I had to edit it
manually.  I guess this is due to your MUA setup.  Please check it and
fix at the next time.


Takashi

      reply	other threads:[~2021-04-16  8:06 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-15 22:01 [PATCH 1/1] ALSA: hda/realtek: fix mic boost on Intel NUC 8 Phil Calvin
2021-04-16  8:05 ` 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=s5h8s5islgy.wl-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=phil@philcalvin.com \
    --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.