* [PATCH 0/1] ALSA: hda - add quirk for Thinkpad T430
@ 2012-10-19 14:32 joseph.salisbury
2012-10-19 14:32 ` [PATCH 1/1] " joseph.salisbury
2012-10-19 17:00 ` [PATCH 0/1] " Jonathan Nieder
0 siblings, 2 replies; 6+ messages in thread
From: joseph.salisbury @ 2012-10-19 14:32 UTC (permalink / raw)
To: linux-kernel
Cc: torvalds, perex, tiwai, david.henningsson, alsa-devel, stable
From: Joseph Salisbury <joseph.salisbury@canonical.com>
BugLink: https://bugs.launchpad.net/bugs/1060372
This patch is to enable audio on T430 Thinkpads. There is no audo from the headphone jack without this patch. This patch adds a quirk for the T430 model.
== Test Case ==
A test kernel was built with this patch and tested by a bug reporter at the following bug:
BugLink: https://bugs.launchpad.net/bugs/1060372
Joseph Salisbury (1):
ALSA: hda - add quirk for Thinkpad T430
sound/pci/hda/patch_realtek.c | 1 +
1 file changed, 1 insertion(+)
--
1.7.9.5
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/1] ALSA: hda - add quirk for Thinkpad T430
2012-10-19 14:32 [PATCH 0/1] ALSA: hda - add quirk for Thinkpad T430 joseph.salisbury
@ 2012-10-19 14:32 ` joseph.salisbury
2012-10-19 17:39 ` Greg KH
2012-10-19 19:54 ` [PATCH v2 " Jonathan Nieder
2012-10-19 17:00 ` [PATCH 0/1] " Jonathan Nieder
1 sibling, 2 replies; 6+ messages in thread
From: joseph.salisbury @ 2012-10-19 14:32 UTC (permalink / raw)
To: linux-kernel
Cc: torvalds, perex, tiwai, david.henningsson, alsa-devel, stable
From: Joseph Salisbury <joseph.salisbury@canonical.com>
Signed-off-by: Joseph Salisbury <joseph.salisbury@canonical.com>
---
sound/pci/hda/patch_realtek.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index 739f968..18fce01 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -6172,6 +6172,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
SND_PCI_QUIRK(0x17aa, 0x21e9, "Thinkpad Edge 15", ALC269_FIXUP_SKU_IGNORE),
SND_PCI_QUIRK(0x17aa, 0x21f6, "Thinkpad T530", ALC269_FIXUP_LENOVO_DOCK),
SND_PCI_QUIRK(0x17aa, 0x21fa, "Thinkpad X230", ALC269_FIXUP_LENOVO_DOCK),
+ SND_PCI_QUIRK(0x17aa, 0x21f3, "Thinkpad T430", ALC269_FIXUP_LENOVO_DOCK),
SND_PCI_QUIRK(0x17aa, 0x21fb, "Thinkpad T430s", ALC269_FIXUP_LENOVO_DOCK),
SND_PCI_QUIRK(0x17aa, 0x2203, "Thinkpad X230 Tablet", ALC269_FIXUP_LENOVO_DOCK),
SND_PCI_QUIRK(0x17aa, 0x3bf8, "Quanta FL1", ALC269_FIXUP_PCM_44K),
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 0/1] ALSA: hda - add quirk for Thinkpad T430
2012-10-19 14:32 [PATCH 0/1] ALSA: hda - add quirk for Thinkpad T430 joseph.salisbury
2012-10-19 14:32 ` [PATCH 1/1] " joseph.salisbury
@ 2012-10-19 17:00 ` Jonathan Nieder
1 sibling, 0 replies; 6+ messages in thread
From: Jonathan Nieder @ 2012-10-19 17:00 UTC (permalink / raw)
To: joseph.salisbury
Cc: linux-kernel, torvalds, perex, tiwai, david.henningsson,
alsa-devel, stable
Hi Joseph,
joseph.salisbury@canonical.com wrote:
> [Subject: [PATCH 0/1] ALSA: hda - add quirk for Thinkpad T430]]
[...]
> This patch is to enable audio on T430 Thinkpads. There is no audo
> from the headphone jack without this patch. This patch adds a quirk
> for the T430 model.
Please don't do this cover-letter thing. A cover letter can be a
good way to *summarize* a long patch series so readers are less
daunted, but any information that was needed to know in order to
evaluate a patch, briefly stated, should be in the commit message that
actually gets committed.
Thanks and hope that helps,
Jonathan
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] ALSA: hda - add quirk for Thinkpad T430
2012-10-19 14:32 ` [PATCH 1/1] " joseph.salisbury
@ 2012-10-19 17:39 ` Greg KH
2012-10-19 19:54 ` [PATCH v2 " Jonathan Nieder
1 sibling, 0 replies; 6+ messages in thread
From: Greg KH @ 2012-10-19 17:39 UTC (permalink / raw)
To: joseph.salisbury
Cc: linux-kernel, torvalds, perex, tiwai, david.henningsson,
alsa-devel, stable
On Fri, Oct 19, 2012 at 10:32:30AM -0400, joseph.salisbury@canonical.com wrote:
> From: Joseph Salisbury <joseph.salisbury@canonical.com>
>
>
> Signed-off-by: Joseph Salisbury <joseph.salisbury@canonical.com>
> ---
> sound/pci/hda/patch_realtek.c | 1 +
> 1 file changed, 1 insertion(+)
<formletter>
This is not the correct way to submit patches for inclusion in the
stable kernel tree. Please read Documentation/stable_kernel_rules.txt
for how to do this properly.
</formletter>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 1/1] ALSA: hda - add quirk for Thinkpad T430
2012-10-19 14:32 ` [PATCH 1/1] " joseph.salisbury
2012-10-19 17:39 ` Greg KH
@ 2012-10-19 19:54 ` Jonathan Nieder
2012-10-19 20:51 ` Takashi Iwai
1 sibling, 1 reply; 6+ messages in thread
From: Jonathan Nieder @ 2012-10-19 19:54 UTC (permalink / raw)
To: joseph.salisbury
Cc: linux-kernel, torvalds, perex, tiwai, david.henningsson,
alsa-devel, stable
joseph.salisbury@canonical.com wrote:
> This patch adds a quirk to enable headphone jack audio on T430 Thinkpads.
>
> Signed-off-by: Joseph Salisbury <joseph.salisbury@canonical.com>
> Cc: <stable@vger.kernel.org>
Thanks --- this is a little better.
One more nit: it looks like this was reported, tested, and based on a
patch by Stefan Freyr. Takashi, could you amend the message to
reflect that, or should the patch be resent?
Something like
-- >8 --
From: Stefán Freyr <stefan.freyr@gmail.com>
Subject: ALSA: hda - add dock support for Thinkpad T430
I have a Lenovo ThinkPad T430 and an UltraBase Series 3 docking
station.
Without this patch, if I plug my headphones into the jack on the
computer, everything works fine. The computer speakers mute and the
audio is played in the headphones. However, if I plug into the docking
station headphone jack the computer speakers are muted but there is no
audio in the headphones.
Addresses https://bugs.launchpad.net/bugs/1060372
Signed-off-by: Joseph Salisbury <joseph.salisbury@canonical.com>
Cc: stable@vger.kernel.org
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 1/1] ALSA: hda - add quirk for Thinkpad T430
2012-10-19 19:54 ` [PATCH v2 " Jonathan Nieder
@ 2012-10-19 20:51 ` Takashi Iwai
0 siblings, 0 replies; 6+ messages in thread
From: Takashi Iwai @ 2012-10-19 20:51 UTC (permalink / raw)
To: Jonathan Nieder
Cc: joseph.salisbury, linux-kernel, torvalds, perex,
david.henningsson, alsa-devel, stable
At Fri, 19 Oct 2012 12:54:21 -0700,
Jonathan Nieder wrote:
>
> joseph.salisbury@canonical.com wrote:
>
> > This patch adds a quirk to enable headphone jack audio on T430 Thinkpads.
> >
> > Signed-off-by: Joseph Salisbury <joseph.salisbury@canonical.com>
> > Cc: <stable@vger.kernel.org>
>
> Thanks --- this is a little better.
>
> One more nit: it looks like this was reported, tested, and based on a
> patch by Stefan Freyr. Takashi, could you amend the message to
> reflect that, or should the patch be resent?
Yes, applied now with the improved commit log.
thanks,
Takashi
>
> Something like
>
> -- >8 --
> From: Stefán Freyr <stefan.freyr@gmail.com>
> Subject: ALSA: hda - add dock support for Thinkpad T430
>
> I have a Lenovo ThinkPad T430 and an UltraBase Series 3 docking
> station.
>
> Without this patch, if I plug my headphones into the jack on the
> computer, everything works fine. The computer speakers mute and the
> audio is played in the headphones. However, if I plug into the docking
> station headphone jack the computer speakers are muted but there is no
> audio in the headphones.
>
> Addresses https://bugs.launchpad.net/bugs/1060372
>
> Signed-off-by: Joseph Salisbury <joseph.salisbury@canonical.com>
> Cc: stable@vger.kernel.org
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-10-19 20:51 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-10-19 14:32 [PATCH 0/1] ALSA: hda - add quirk for Thinkpad T430 joseph.salisbury
2012-10-19 14:32 ` [PATCH 1/1] " joseph.salisbury
2012-10-19 17:39 ` Greg KH
2012-10-19 19:54 ` [PATCH v2 " Jonathan Nieder
2012-10-19 20:51 ` Takashi Iwai
2012-10-19 17:00 ` [PATCH 0/1] " Jonathan Nieder
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).