All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Elia Devito <eliadevito@gmail.com>
Cc: Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>,
	Kailang Yang <kailang@realtek.com>,
	Jian-Hong Pan <jhp@endlessos.org>,
	Hui Wang <hui.wang@canonical.com>,
	Chris Chiu <chris.chiu@canonical.com>,
	Huacai Chen <chenhuacai@kernel.org>,
	alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] ALSA: hda/realtek: Add fixup for HP Spectre x360 15-df0xxx
Date: Tue, 11 May 2021 11:10:08 +0200	[thread overview]
Message-ID: <s5ho8dhd4b3.wl-tiwai@suse.de> (raw)
In-Reply-To: <20210510171332.8313-1-eliadevito@gmail.com>

On Mon, 10 May 2021 19:13:33 +0200,
Elia Devito wrote:
> 
> Fixup to enable all 4 speaker on HP Spectre x360 15-df0xxx and probably
> on similar models.
> 
> 0x14 pin config override is required to enable all speakers and
> alc285-speaker2-to-dac1 fixup to enable volume adjustment.
> 
> BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=189331
> Signed-off-by: Elia Devito <eliadevito@gmail.com>

Could you try to put the quirk entry in the PCI SSID order?


thanks,

Takashi

> ---
>  sound/pci/hda/patch_realtek.c | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> index b4b71609dff1..be2a5cd2875d 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -6542,6 +6542,7 @@ enum {
>  	ALC285_FIXUP_HP_LIMIT_INT_MIC_BOOST,
>  	ALC295_FIXUP_ASUS_DACS,
>  	ALC295_FIXUP_HP_OMEN,
> +	ALC285_FIXUP_HP_SPECTRE_X360,
>  };
>  
>  static const struct hda_fixup alc269_fixups[] = {
> @@ -8099,6 +8100,15 @@ static const struct hda_fixup alc269_fixups[] = {
>  		.chained = true,
>  		.chain_id = ALC269_FIXUP_HP_LINE1_MIC1_LED,
>  	},
> +	[ALC285_FIXUP_HP_SPECTRE_X360] = {
> +		.type = HDA_FIXUP_PINS,
> +		.v.pins = (const struct hda_pintbl[]) {
> +			{ 0x14, 0x90170110 }, /* enable top speaker */
> +			{}
> +		},
> +		.chained = true,
> +		.chain_id = ALC285_FIXUP_SPEAKER2_TO_DAC1,
> +	},
>  };
>  
>  static const struct snd_pci_quirk alc269_fixup_tbl[] = {
> @@ -8278,6 +8288,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
>  	SND_PCI_QUIRK(0x103c, 0x87f4, "HP", ALC287_FIXUP_HP_GPIO_LED),
>  	SND_PCI_QUIRK(0x103c, 0x87f5, "HP", ALC287_FIXUP_HP_GPIO_LED),
>  	SND_PCI_QUIRK(0x103c, 0x87f7, "HP Spectre x360 14", ALC245_FIXUP_HP_X360_AMP),
> +	SND_PCI_QUIRK(0x103c, 0x8519, "HP Spectre x360 15-df0xxx", ALC285_FIXUP_HP_SPECTRE_X360),
>  	SND_PCI_QUIRK(0x103c, 0x8846, "HP EliteBook 850 G8 Notebook PC", ALC285_FIXUP_HP_GPIO_LED),
>  	SND_PCI_QUIRK(0x103c, 0x884c, "HP EliteBook 840 G8 Notebook PC", ALC285_FIXUP_HP_GPIO_LED),
>  	SND_PCI_QUIRK(0x103c, 0x8898, "HP EliteBook 845 G8 Notebook PC", ALC285_FIXUP_HP_LIMIT_INT_MIC_BOOST),
> @@ -8665,6 +8676,7 @@ static const struct hda_model_fixup alc269_fixup_models[] = {
>  	{.id = ALC274_FIXUP_HP_MIC, .name = "alc274-hp-mic-detect"},
>  	{.id = ALC245_FIXUP_HP_X360_AMP, .name = "alc245-hp-x360-amp"},
>  	{.id = ALC295_FIXUP_HP_OMEN, .name = "alc295-hp-omen"},
> +	{.id = ALC285_FIXUP_HP_SPECTRE_X360, .name = "alc285-hp-spectre-x360"},
>  	{}
>  };
>  #define ALC225_STANDARD_PINS \
> -- 
> 2.31.1
> 

WARNING: multiple messages have this Message-ID (diff)
From: Takashi Iwai <tiwai@suse.de>
To: Elia Devito <eliadevito@gmail.com>
Cc: Chris Chiu <chris.chiu@canonical.com>,
	alsa-devel@alsa-project.org, Kailang Yang <kailang@realtek.com>,
	linux-kernel@vger.kernel.org, Huacai Chen <chenhuacai@kernel.org>,
	Jian-Hong Pan <jhp@endlessos.org>, Takashi Iwai <tiwai@suse.com>,
	Hui Wang <hui.wang@canonical.com>
Subject: Re: [PATCH] ALSA: hda/realtek: Add fixup for HP Spectre x360 15-df0xxx
Date: Tue, 11 May 2021 11:10:08 +0200	[thread overview]
Message-ID: <s5ho8dhd4b3.wl-tiwai@suse.de> (raw)
In-Reply-To: <20210510171332.8313-1-eliadevito@gmail.com>

On Mon, 10 May 2021 19:13:33 +0200,
Elia Devito wrote:
> 
> Fixup to enable all 4 speaker on HP Spectre x360 15-df0xxx and probably
> on similar models.
> 
> 0x14 pin config override is required to enable all speakers and
> alc285-speaker2-to-dac1 fixup to enable volume adjustment.
> 
> BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=189331
> Signed-off-by: Elia Devito <eliadevito@gmail.com>

Could you try to put the quirk entry in the PCI SSID order?


thanks,

Takashi

> ---
>  sound/pci/hda/patch_realtek.c | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> index b4b71609dff1..be2a5cd2875d 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -6542,6 +6542,7 @@ enum {
>  	ALC285_FIXUP_HP_LIMIT_INT_MIC_BOOST,
>  	ALC295_FIXUP_ASUS_DACS,
>  	ALC295_FIXUP_HP_OMEN,
> +	ALC285_FIXUP_HP_SPECTRE_X360,
>  };
>  
>  static const struct hda_fixup alc269_fixups[] = {
> @@ -8099,6 +8100,15 @@ static const struct hda_fixup alc269_fixups[] = {
>  		.chained = true,
>  		.chain_id = ALC269_FIXUP_HP_LINE1_MIC1_LED,
>  	},
> +	[ALC285_FIXUP_HP_SPECTRE_X360] = {
> +		.type = HDA_FIXUP_PINS,
> +		.v.pins = (const struct hda_pintbl[]) {
> +			{ 0x14, 0x90170110 }, /* enable top speaker */
> +			{}
> +		},
> +		.chained = true,
> +		.chain_id = ALC285_FIXUP_SPEAKER2_TO_DAC1,
> +	},
>  };
>  
>  static const struct snd_pci_quirk alc269_fixup_tbl[] = {
> @@ -8278,6 +8288,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
>  	SND_PCI_QUIRK(0x103c, 0x87f4, "HP", ALC287_FIXUP_HP_GPIO_LED),
>  	SND_PCI_QUIRK(0x103c, 0x87f5, "HP", ALC287_FIXUP_HP_GPIO_LED),
>  	SND_PCI_QUIRK(0x103c, 0x87f7, "HP Spectre x360 14", ALC245_FIXUP_HP_X360_AMP),
> +	SND_PCI_QUIRK(0x103c, 0x8519, "HP Spectre x360 15-df0xxx", ALC285_FIXUP_HP_SPECTRE_X360),
>  	SND_PCI_QUIRK(0x103c, 0x8846, "HP EliteBook 850 G8 Notebook PC", ALC285_FIXUP_HP_GPIO_LED),
>  	SND_PCI_QUIRK(0x103c, 0x884c, "HP EliteBook 840 G8 Notebook PC", ALC285_FIXUP_HP_GPIO_LED),
>  	SND_PCI_QUIRK(0x103c, 0x8898, "HP EliteBook 845 G8 Notebook PC", ALC285_FIXUP_HP_LIMIT_INT_MIC_BOOST),
> @@ -8665,6 +8676,7 @@ static const struct hda_model_fixup alc269_fixup_models[] = {
>  	{.id = ALC274_FIXUP_HP_MIC, .name = "alc274-hp-mic-detect"},
>  	{.id = ALC245_FIXUP_HP_X360_AMP, .name = "alc245-hp-x360-amp"},
>  	{.id = ALC295_FIXUP_HP_OMEN, .name = "alc295-hp-omen"},
> +	{.id = ALC285_FIXUP_HP_SPECTRE_X360, .name = "alc285-hp-spectre-x360"},
>  	{}
>  };
>  #define ALC225_STANDARD_PINS \
> -- 
> 2.31.1
> 

  reply	other threads:[~2021-05-11  9:10 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-10 17:13 [PATCH] ALSA: hda/realtek: Add fixup for HP Spectre x360 15-df0xxx Elia Devito
2021-05-10 17:13 ` Elia Devito
2021-05-11  9:10 ` Takashi Iwai [this message]
2021-05-11  9:10   ` Takashi Iwai
2021-05-11 12:46   ` [PATCH v2] " Elia Devito
2021-05-11 12:46     ` Elia Devito
2021-05-11 13:45     ` Takashi Iwai
2021-05-11 13:45       ` Takashi Iwai
2021-05-11 14:53       ` Elia Devito
2021-05-11 14:53         ` Elia Devito

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=s5ho8dhd4b3.wl-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=chenhuacai@kernel.org \
    --cc=chris.chiu@canonical.com \
    --cc=eliadevito@gmail.com \
    --cc=hui.wang@canonical.com \
    --cc=jhp@endlessos.org \
    --cc=kailang@realtek.com \
    --cc=linux-kernel@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.