At Thu, 28 Feb 2013 09:49:23 +0100,
David Henningsson wrote:
>
> On 02/28/2013 08:21 AM, Takashi Iwai wrote:
> > At Thu, 28 Feb 2013 08:18:30 +0100,
> > David Henningsson wrote:
> >>
> >> The user reports a regression (choppy audio), and that
> >> position_fix=1 (or position_fix=4) fixes the issue.
> >>
> >> This is a Poulsbo controller (8086:811b (rev 07)), so might be
> >> worth checking if other Poulsbo machines suffer from the same bug.
> >>
> >> Cc: <email address hidden> [v3.7+]
> >> BugLink: https://bugs.launchpad.net/bugs/1132562
> >> Signed-off-by: David Henningsson <email address hidden>
> >
> > Please check whether the latest sound git tree works as is.
> > We enabled the LPIB delay counting for Poulsbo there, and it may fix
> > most cases.
>
> The test was done on a 3.8 based kernel. Isn't LPIB delay counting
> enabled there as well?
No, it was added fairly recently.
I didn't put Cc to stable for the commit
f748abcc5bf62de007019d841f7caba81cc3d673. If this is confirmed to
work in general for Poulsbo, we can merge to stable.
But, overall I wonder why POSFIX_LPIB is needed for this device.
In the recent kernels up to 3.8, LPIB is used as default for Poulsbo.
At Thu, 28 Feb 2013 09:49:23 +0100, /bugs.launchpad .net/bugs/ 1132562
David Henningsson wrote:
>
> On 02/28/2013 08:21 AM, Takashi Iwai wrote:
> > At Thu, 28 Feb 2013 08:18:30 +0100,
> > David Henningsson wrote:
> >>
> >> The user reports a regression (choppy audio), and that
> >> position_fix=1 (or position_fix=4) fixes the issue.
> >>
> >> This is a Poulsbo controller (8086:811b (rev 07)), so might be
> >> worth checking if other Poulsbo machines suffer from the same bug.
> >>
> >> Cc: <email address hidden> [v3.7+]
> >> BugLink: https:/
> >> Signed-off-by: David Henningsson <email address hidden>
> >
> > Please check whether the latest sound git tree works as is.
> > We enabled the LPIB delay counting for Poulsbo there, and it may fix
> > most cases.
>
> The test was done on a 3.8 based kernel. Isn't LPIB delay counting
> enabled there as well?
No, it was added fairly recently. 007019d841f7cab a81cc3d673. If this is confirmed to
I didn't put Cc to stable for the commit
f748abcc5bf62de
work in general for Poulsbo, we can merge to stable.
But, overall I wonder why POSFIX_LPIB is needed for this device.
In the recent kernels up to 3.8, LPIB is used as default for Poulsbo.
Takashi
Takashi
> /launchpadlibra rian.net/ 132247367/ AlsaInfo. txt ) hda/hda_ intel.c | 1 + pci/hda/ hda_intel. c b/sound/ pci/hda/ hda_intel. c pci/hda/ hda_intel. c pci/hda/ hda_intel. c disabled( struct pci_dev *pci) QUIRK(0x1025, 0x0244, "Acer AO751h", POS_FIX_LPIB), QUIRK(0x1028, 0x01cc, "Dell D820", POS_FIX_LPIB), QUIRK(0x1028, 0x01de, "Dell Precision 390", POS_FIX_LPIB), QUIRK(0x103c, 0x306d, "HP dv3", POS_FIX_LPIB), /launchpad. net/~diwic
> (Alsa-info is here: https:/
>
> >
> >
> > Takashi
> >
> >
> >> ---
> >> sound/pci/
> >> 1 file changed, 1 insertion(+)
> >>
> >> diff --git a/sound/
> >> index 4cea6bb6..3505f1e 100644
> >> --- a/sound/
> >> +++ b/sound/
> >> @@ -3114,6 +3114,7 @@ static bool check_hdmi_
> >> * white/black-listing for position_fix
> >> */
> >> static struct snd_pci_quirk position_fix_list[] = {
> >> + SND_PCI_
> >> SND_PCI_
> >> SND_PCI_
> >> SND_PCI_
> >> --
> >> 1.7.9.5
> >>
> >
>
>
>
> --
> David Henningsson, Canonical Ltd.
> https:/
>