Re: [PATCH] Re: ALSA HDA Intel stoped to work in 2.6.16-*

From: Rafael J. Wysocki
Date: Sat Mar 07 2009 - 13:00:48 EST


(adding CCs).

Usually, sending such things to the LKML alone doesn't really work. Please
always send a CC to the relevant subsystem maintainer.

On Saturday 07 March 2009, Daniel Gimpelevich wrote:
> At Mon, 6 Mar 2006 15:16:15 +0100,
> Takashi Iwai wrote:
>
> > At Sat, 4 Mar 2006 15:51:14 +0100,
> > Adrian Bunk wrote:
> >>
> >> On Sat, Mar 04, 2006 at 02:29:02AM -0300, Otavio Salvador wrote:
> >> > Takashi Iwai <tiwai@xxxxxxx> writes:
> >> >
> >> > > Are you sure that your device has PCI SUB-system id 8086:2668 ?
> >> >
> >> > oh no! Sorry!
> >> >
> >> > 0000:00:1b.0 0403: 8086:2668 (rev 04)
> >> > Subsystem: 152d:0729
> >> > ^^^^^^^^^
> >>
> >> Can you make a patch with the correct id test whether it fixes your
> >> problem (without model=basic)?
> >
> > This one should work for his device.
> >
> >
> > Takashi
> > ===
> >
> > [PATCH] Add default entry for CTL Travel Master U553W
> >
> > Added the default entry of ALC880 configuration table for
> > CTL Travel Master U553W.
> >
> > Signed-off-by: Takashi Iwai <tiwai@xxxxxxx>
> >
> > ---
> > diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> > index b767552..d5cd3a1 100644
> > --- a/sound/pci/hda/patch_realtek.c
> > +++ b/sound/pci/hda/patch_realtek.c
> > @@ -2948,6 +2948,8 @@ static struct hda_board_config alc260_cf
> > { .modelname = "basic", .config = ALC260_BASIC },
> > { .pci_subvendor = 0x104d, .pci_subdevice = 0x81bb,
> > .config = ALC260_BASIC }, /* Sony VAIO */
> > + { .pci_subvendor = 0x152d, .pci_subdevice = 0x0729,
> > + .config = ALC260_BASIC }, /* CTL Travel Master U553W */
> > { .modelname = "hp", .config = ALC260_HP },
> > { .pci_subvendor = 0x103c, .pci_subdevice = 0x3010, .config = ALC260_HP },
> > { .pci_subvendor = 0x103c, .pci_subdevice = 0x3011, .config = ALC260_HP },
>
> It has been three years since the above patch was posted with a request for
> testing. No testing reply ever appeared, and the patch was committed as
> submitted. On the relevant hardware, I determined the patch to be incorrect.
> For the git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git
> head, the following patch appears to fix it correctly:
>
>
> Signed-off-by: Daniel Gimpelevich <daniel@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
>
> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> index 6c26afc..87ec806 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -5692,7 +5692,7 @@ static struct snd_pci_quirk alc260_cfg_tbl[] = {
> SND_PCI_QUIRK(0x104d, 0x81cc, "Sony VAIO", ALC260_BASIC),
> SND_PCI_QUIRK(0x104d, 0x81cd, "Sony VAIO", ALC260_BASIC),
> SND_PCI_QUIRK(0x10cf, 0x1326, "Fujitsu S702X", ALC260_FUJITSU_S702X),
> - SND_PCI_QUIRK(0x152d, 0x0729, "CTL U553W", ALC260_BASIC),
> + SND_PCI_QUIRK(0x152d, 0x0729, "CTL U553W", ALC260_WILL),
> SND_PCI_QUIRK(0x161f, 0x2057, "Replacer 672V", ALC260_REPLACER_672V),
> SND_PCI_QUIRK(0x1631, 0xc017, "PB V7900", ALC260_WILL),
> {}
>
> --
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/