stable.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* FAILED: patch "[PATCH] ALSA: hda/realtek - Add Bass Speaker and fixed dac for bass" failed to apply to 5.4-stable tree
@ 2020-01-06 12:10 gregkh
  2020-01-06 20:32 ` Takashi Iwai
  0 siblings, 1 reply; 4+ messages in thread
From: gregkh @ 2020-01-06 12:10 UTC (permalink / raw)
  To: kailang, stable, tiwai; +Cc: stable


The patch below does not apply to the 5.4-stable tree.
If someone wants it applied there, or to any other stable or longterm
tree, then please email the backport, including the original git commit
id to <stable@vger.kernel.org>.

thanks,

greg k-h

------------------ original commit in Linus's tree ------------------

From e79c22695abd3b75a6aecf4ea4b9607e8d82c49c Mon Sep 17 00:00:00 2001
From: Kailang Yang <kailang@realtek.com>
Date: Thu, 19 Dec 2019 14:12:15 +0800
Subject: [PATCH] ALSA: hda/realtek - Add Bass Speaker and fixed dac for bass
 speaker

Dell has new platform which has dual speaker connecting.
They want dual speaker which use same dac for output.

Signed-off-by: Kailang Yang <kailang@realtek.com>
Cc: <stable@vger.kernel.org>
Link: https://lore.kernel.org/r/229c7efa2b474a16b7d8a916cd096b68@realtek.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>

diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index 5bc1a6d24333..2ee703c2da78 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -5908,6 +5908,8 @@ enum {
 	ALC294_FIXUP_ASUS_INTSPK_HEADSET_MIC,
 	ALC256_FIXUP_MEDION_HEADSET_NO_PRESENCE,
 	ALC294_FIXUP_ASUS_INTSPK_GPIO,
+	ALC289_FIXUP_DELL_SPK2,
+	ALC289_FIXUP_DUAL_SPK,
 };
 
 static const struct hda_fixup alc269_fixups[] = {
@@ -7009,6 +7011,21 @@ static const struct hda_fixup alc269_fixups[] = {
 		.chained = true,
 		.chain_id = ALC294_FIXUP_ASUS_INTSPK_HEADSET_MIC
 	},
+	[ALC289_FIXUP_DELL_SPK2] = {
+		.type = HDA_FIXUP_PINS,
+		.v.pins = (const struct hda_pintbl[]) {
+			{ 0x17, 0x90170130 }, /* bass spk */
+			{ }
+		},
+		.chained = true,
+		.chain_id = ALC269_FIXUP_DELL4_MIC_NO_PRESENCE
+	},
+	[ALC289_FIXUP_DUAL_SPK] = {
+		.type = HDA_FIXUP_FUNC,
+		.v.func = alc285_fixup_speaker2_to_dac1,
+		.chained = true,
+		.chain_id = ALC289_FIXUP_DELL_SPK2
+	},
 };
 
 static const struct snd_pci_quirk alc269_fixup_tbl[] = {
@@ -7081,6 +7098,8 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
 	SND_PCI_QUIRK(0x1028, 0x08ad, "Dell WYSE AIO", ALC225_FIXUP_DELL_WYSE_AIO_MIC_NO_PRESENCE),
 	SND_PCI_QUIRK(0x1028, 0x08ae, "Dell WYSE NB", ALC225_FIXUP_DELL1_MIC_NO_PRESENCE),
 	SND_PCI_QUIRK(0x1028, 0x0935, "Dell", ALC274_FIXUP_DELL_AIO_LINEOUT_VERB),
+	SND_PCI_QUIRK(0x1028, 0x097e, "Dell Precision", ALC289_FIXUP_DUAL_SPK),
+	SND_PCI_QUIRK(0x1028, 0x097d, "Dell Precision", ALC289_FIXUP_DUAL_SPK),
 	SND_PCI_QUIRK(0x1028, 0x164a, "Dell", ALC293_FIXUP_DELL1_MIC_NO_PRESENCE),
 	SND_PCI_QUIRK(0x1028, 0x164b, "Dell", ALC293_FIXUP_DELL1_MIC_NO_PRESENCE),
 	SND_PCI_QUIRK(0x103c, 0x1586, "HP", ALC269_FIXUP_HP_MUTE_LED_MIC2),


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: FAILED: patch "[PATCH] ALSA: hda/realtek - Add Bass Speaker and fixed dac for bass" failed to apply to 5.4-stable tree
  2020-01-06 12:10 FAILED: patch "[PATCH] ALSA: hda/realtek - Add Bass Speaker and fixed dac for bass" failed to apply to 5.4-stable tree gregkh
@ 2020-01-06 20:32 ` Takashi Iwai
  2020-01-07  9:39   ` Greg KH
  0 siblings, 1 reply; 4+ messages in thread
From: Takashi Iwai @ 2020-01-06 20:32 UTC (permalink / raw)
  To: gregkh; +Cc: kailang, stable

On Mon, 06 Jan 2020 13:10:30 +0100,
<gregkh@linuxfoundation.org> wrote:
> 
> 
> The patch below does not apply to the 5.4-stable tree.
> If someone wants it applied there, or to any other stable or longterm
> tree, then please email the backport, including the original git commit
> id to <stable@vger.kernel.org>.

Hm, I could cherry-pick the commit e79c22695abd (also commit
48e01504cf53 at next) cleanly on linux-5.4.y branch.

Could you try again?


thanks,

Takashi

> 
> thanks,
> 
> greg k-h
> 
> ------------------ original commit in Linus's tree ------------------
> 
> >From e79c22695abd3b75a6aecf4ea4b9607e8d82c49c Mon Sep 17 00:00:00 2001
> From: Kailang Yang <kailang@realtek.com>
> Date: Thu, 19 Dec 2019 14:12:15 +0800
> Subject: [PATCH] ALSA: hda/realtek - Add Bass Speaker and fixed dac for bass
>  speaker
> 
> Dell has new platform which has dual speaker connecting.
> They want dual speaker which use same dac for output.
> 
> Signed-off-by: Kailang Yang <kailang@realtek.com>
> Cc: <stable@vger.kernel.org>
> Link: https://lore.kernel.org/r/229c7efa2b474a16b7d8a916cd096b68@realtek.com
> Signed-off-by: Takashi Iwai <tiwai@suse.de>
> 
> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> index 5bc1a6d24333..2ee703c2da78 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -5908,6 +5908,8 @@ enum {
>  	ALC294_FIXUP_ASUS_INTSPK_HEADSET_MIC,
>  	ALC256_FIXUP_MEDION_HEADSET_NO_PRESENCE,
>  	ALC294_FIXUP_ASUS_INTSPK_GPIO,
> +	ALC289_FIXUP_DELL_SPK2,
> +	ALC289_FIXUP_DUAL_SPK,
>  };
>  
>  static const struct hda_fixup alc269_fixups[] = {
> @@ -7009,6 +7011,21 @@ static const struct hda_fixup alc269_fixups[] = {
>  		.chained = true,
>  		.chain_id = ALC294_FIXUP_ASUS_INTSPK_HEADSET_MIC
>  	},
> +	[ALC289_FIXUP_DELL_SPK2] = {
> +		.type = HDA_FIXUP_PINS,
> +		.v.pins = (const struct hda_pintbl[]) {
> +			{ 0x17, 0x90170130 }, /* bass spk */
> +			{ }
> +		},
> +		.chained = true,
> +		.chain_id = ALC269_FIXUP_DELL4_MIC_NO_PRESENCE
> +	},
> +	[ALC289_FIXUP_DUAL_SPK] = {
> +		.type = HDA_FIXUP_FUNC,
> +		.v.func = alc285_fixup_speaker2_to_dac1,
> +		.chained = true,
> +		.chain_id = ALC289_FIXUP_DELL_SPK2
> +	},
>  };
>  
>  static const struct snd_pci_quirk alc269_fixup_tbl[] = {
> @@ -7081,6 +7098,8 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
>  	SND_PCI_QUIRK(0x1028, 0x08ad, "Dell WYSE AIO", ALC225_FIXUP_DELL_WYSE_AIO_MIC_NO_PRESENCE),
>  	SND_PCI_QUIRK(0x1028, 0x08ae, "Dell WYSE NB", ALC225_FIXUP_DELL1_MIC_NO_PRESENCE),
>  	SND_PCI_QUIRK(0x1028, 0x0935, "Dell", ALC274_FIXUP_DELL_AIO_LINEOUT_VERB),
> +	SND_PCI_QUIRK(0x1028, 0x097e, "Dell Precision", ALC289_FIXUP_DUAL_SPK),
> +	SND_PCI_QUIRK(0x1028, 0x097d, "Dell Precision", ALC289_FIXUP_DUAL_SPK),
>  	SND_PCI_QUIRK(0x1028, 0x164a, "Dell", ALC293_FIXUP_DELL1_MIC_NO_PRESENCE),
>  	SND_PCI_QUIRK(0x1028, 0x164b, "Dell", ALC293_FIXUP_DELL1_MIC_NO_PRESENCE),
>  	SND_PCI_QUIRK(0x103c, 0x1586, "HP", ALC269_FIXUP_HP_MUTE_LED_MIC2),
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: FAILED: patch "[PATCH] ALSA: hda/realtek - Add Bass Speaker and fixed dac for bass" failed to apply to 5.4-stable tree
  2020-01-06 20:32 ` Takashi Iwai
@ 2020-01-07  9:39   ` Greg KH
  2020-01-07 18:59     ` Sasha Levin
  0 siblings, 1 reply; 4+ messages in thread
From: Greg KH @ 2020-01-07  9:39 UTC (permalink / raw)
  To: Takashi Iwai; +Cc: kailang, stable

On Mon, Jan 06, 2020 at 09:32:43PM +0100, Takashi Iwai wrote:
> On Mon, 06 Jan 2020 13:10:30 +0100,
> <gregkh@linuxfoundation.org> wrote:
> > 
> > 
> > The patch below does not apply to the 5.4-stable tree.
> > If someone wants it applied there, or to any other stable or longterm
> > tree, then please email the backport, including the original git commit
> > id to <stable@vger.kernel.org>.
> 
> Hm, I could cherry-pick the commit e79c22695abd (also commit
> 48e01504cf53 at next) cleanly on linux-5.4.y branch.
> 
> Could you try again?

Ugh, my fault, seems Sasha already queued these patches up in the stable
tree and I didn't notice them.  When I tried to apply them they
rightfully complained :(

sorry for the noise, all is good here.

greg k-h

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: FAILED: patch "[PATCH] ALSA: hda/realtek - Add Bass Speaker and fixed dac for bass" failed to apply to 5.4-stable tree
  2020-01-07  9:39   ` Greg KH
@ 2020-01-07 18:59     ` Sasha Levin
  0 siblings, 0 replies; 4+ messages in thread
From: Sasha Levin @ 2020-01-07 18:59 UTC (permalink / raw)
  To: Greg KH; +Cc: Takashi Iwai, kailang, stable

On Tue, Jan 07, 2020 at 10:39:12AM +0100, Greg KH wrote:
>On Mon, Jan 06, 2020 at 09:32:43PM +0100, Takashi Iwai wrote:
>> On Mon, 06 Jan 2020 13:10:30 +0100,
>> <gregkh@linuxfoundation.org> wrote:
>> >
>> >
>> > The patch below does not apply to the 5.4-stable tree.
>> > If someone wants it applied there, or to any other stable or longterm
>> > tree, then please email the backport, including the original git commit
>> > id to <stable@vger.kernel.org>.
>>
>> Hm, I could cherry-pick the commit e79c22695abd (also commit
>> 48e01504cf53 at next) cleanly on linux-5.4.y branch.
>>
>> Could you try again?
>
>Ugh, my fault, seems Sasha already queued these patches up in the stable
>tree and I didn't notice them.  When I tried to apply them they
>rightfully complained :(
>
>sorry for the noise, all is good here.

Appologies for stepping on your toes here, I picked it up so that I
could also take 48e01504cf53 ("ALSA: hda/realtek - Enable the bass
speaker of ASUS UX431FLC") which fixes a different patch we took in the
5.4 tree.

-- 
Thanks,
Sasha

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2020-01-07 18:59 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-01-06 12:10 FAILED: patch "[PATCH] ALSA: hda/realtek - Add Bass Speaker and fixed dac for bass" failed to apply to 5.4-stable tree gregkh
2020-01-06 20:32 ` Takashi Iwai
2020-01-07  9:39   ` Greg KH
2020-01-07 18:59     ` Sasha Levin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).