All of lore.kernel.org
 help / color / mirror / Atom feed
* Regression in 2.6.19 and 2.6.20 for snd_hda_intel
@ 2006-12-31  1:20 Larry Finger
  2006-12-31  2:23 ` Daniel Drake
  0 siblings, 1 reply; 13+ messages in thread
From: Larry Finger @ 2006-12-31  1:20 UTC (permalink / raw)
  To: LKML; +Cc: Stephen Hemminger

I have a new laptop - an HP Pavilion dv2125nr with an AMD Turion 64 X2 Mobile CPU. With any kernel
later than 2.6.18, the sound does not work. The audio interface is reported by lspci as "Audio
device: nVidia Corporation MCP51 High Definition Audio (rev a2)", and uses the snd_hda_intel module.

Using git bisect, I determined that there are two problems. If commit a7da6ce564a8092..., which is
entitled "hda-codec-Add independent headphone volume control" is present, I get no sound at all.
When this commit is reverted, I get sound, but playing a sound file results in about an 0.5 sec
fragment being replayed over and over forever. If commit 7376d013fc6d3a45..., which is entitled
"Simple patch to enable Message Signalled Interrupts for the HDA Intel audio controller" is
reverted, the sound works fine.

Please tell me what other information is required.

Larry



^ permalink raw reply	[flat|nested] 13+ messages in thread
* Re: Regression in 2.6.19 and 2.6.20 for snd_hda_intel
  2006-12-31  1:20 Regression in 2.6.19 and 2.6.20 for snd_hda_intel Larry Finger
@ 2006-12-31  2:23 ` Daniel Drake
  2006-12-31  3:31   ` Larry Finger
  0 siblings, 1 reply; 13+ messages in thread
From: Daniel Drake @ 2006-12-31  2:23 UTC (permalink / raw)
  To: Larry Finger; +Cc: LKML, Stephen Hemminger

Larry Finger wrote:
> When this commit is reverted, I get sound, but playing a sound file results in about an 0.5 sec
> fragment being replayed over and over forever. If commit 7376d013fc6d3a45..., which is entitled
> "Simple patch to enable Message Signalled Interrupts for the HDA Intel audio controller" is
> reverted, the sound works fine.

MSI is now disabled by default on this driver, so I'm pretty sure you 
only have the first bug talking in terms of the most recent kernels.

Daniel

^ permalink raw reply	[flat|nested] 13+ messages in thread
* Re: Regression in 2.6.19 and 2.6.20 for snd_hda_intel
  2006-12-31  2:23 ` Daniel Drake
@ 2006-12-31  3:31   ` Larry Finger
  2006-12-31  3:46       ` Daniel Drake
  0 siblings, 1 reply; 13+ messages in thread
From: Larry Finger @ 2006-12-31  3:31 UTC (permalink / raw)
  To: Daniel Drake; +Cc: LKML, Stephen Hemminger

Daniel Drake wrote:
> Larry Finger wrote:
>> When this commit is reverted, I get sound, but playing a sound file
>> results in about an 0.5 sec
>> fragment being replayed over and over forever. If commit
>> 7376d013fc6d3a45..., which is entitled
>> "Simple patch to enable Message Signalled Interrupts for the HDA Intel
>> audio controller" is
>> reverted, the sound works fine.
> 
> MSI is now disabled by default on this driver, so I'm pretty sure you
> only have the first bug talking in terms of the most recent kernels.

You are correct. Only the "hda_codec-Add independent headphone volume control" needs to be reverted.

Thanks,

Larry

^ permalink raw reply	[flat|nested] 13+ messages in thread
* Re: Regression in 2.6.19 and 2.6.20 for snd_hda_intel
  2006-12-31  3:31   ` Larry Finger
@ 2006-12-31  3:46       ` Daniel Drake
  0 siblings, 0 replies; 13+ messages in thread
From: Daniel Drake @ 2006-12-31  3:46 UTC (permalink / raw)
  To: Larry Finger; +Cc: LKML, Stephen Hemminger, alsa-devel

Larry Finger wrote:
> You are correct. Only the "hda_codec-Add independent headphone volume control" needs to be reverted.

The best course of action is probably to file a report here:
https://bugtrack.alsa-project.org/alsa-bug

Daniel

^ permalink raw reply	[flat|nested] 13+ messages in thread
* Re: Regression in 2.6.19 and 2.6.20 for snd_hda_intel
@ 2006-12-31  3:46       ` Daniel Drake
  0 siblings, 0 replies; 13+ messages in thread
From: Daniel Drake @ 2006-12-31  3:46 UTC (permalink / raw)
  To: Larry Finger; +Cc: alsa-devel, LKML, Stephen Hemminger

Larry Finger wrote:
> You are correct. Only the "hda_codec-Add independent headphone volume control" needs to be reverted.

The best course of action is probably to file a report here:
https://bugtrack.alsa-project.org/alsa-bug

Daniel

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

^ permalink raw reply	[flat|nested] 13+ messages in thread
* Re: Regression in 2.6.19 and 2.6.20 for snd_hda_intel
  2006-12-31  3:46       ` Daniel Drake
  (?)
@ 2006-12-31  4:08       ` Tobin Davis
  2006-12-31  4:50           ` Larry Finger
  -1 siblings, 1 reply; 13+ messages in thread
From: Tobin Davis @ 2006-12-31  4:08 UTC (permalink / raw)
  To: Daniel Drake; +Cc: alsa-devel, LKML, Stephen Hemminger, Larry Finger


[-- Attachment #1.1: Type: text/plain, Size: 1004 bytes --]

Which alsa patch was this?  I'm not seeing anything in the hg logs for
this.  Or is this something from the kernel side?

Tobin

On Sat, 2006-12-30 at 22:46 -0500, Daniel Drake wrote:

> Larry Finger wrote:
> > You are correct. Only the "hda_codec-Add independent headphone volume control" needs to be reverted.
> 
> The best course of action is probably to file a report here:
> https://bugtrack.alsa-project.org/alsa-bug
> 
> Daniel
> 
> -------------------------------------------------------------------------
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the chance to share your
> opinions on IT & business topics through brief surveys - and earn cash
> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/alsa-devel

-- 
Tobin Davis <tdavis@dsl-only.net>

[-- Attachment #1.2: Type: text/html, Size: 2108 bytes --]

[-- Attachment #2: Type: text/plain, Size: 347 bytes --]

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

[-- Attachment #3: Type: text/plain, Size: 161 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 13+ messages in thread
* Re: [Alsa-devel] Regression in 2.6.19 and 2.6.20 for snd_hda_intel
  2006-12-31  4:08       ` Tobin Davis
@ 2006-12-31  4:50           ` Larry Finger
  0 siblings, 0 replies; 13+ messages in thread
From: Larry Finger @ 2006-12-31  4:50 UTC (permalink / raw)
  To: Tobin Davis; +Cc: Daniel Drake, alsa-devel, LKML, Stephen Hemminger

Tobin Davis wrote:
> Which alsa patch was this?  I'm not seeing anything in the hg logs for
> this.  Or is this something from the kernel side?

It seems to have come from suse. The full commit message is:

commit a7da6ce564a80952d9c0b210deca5a8cd3474a31
Author: Takashi Iwai <tiwai@suse.de>
Date:   Wed Sep 6 14:03:14 2006 +0200

    [ALSA] hda-codec - Add independent headphone volume control

    This patch addes the support of the independent 'Headphone' volume
    control to the generic codec parser.  Some codecs (e.g. Conexant)
    have separate connections to the headphone and the independent amp
    adjustment is needed.

    Signed-off-by: Takashi Iwai <tiwai@suse.de>
    Signed-off-by: Jaroslav Kysela <perex@suse.cz>

:100644 100644 dedfc5b... 97e9af1... M  sound/pci/hda/hda_generic.c

Larry



^ permalink raw reply	[flat|nested] 13+ messages in thread
* Re: Regression in 2.6.19 and 2.6.20 for snd_hda_intel
@ 2006-12-31  4:50           ` Larry Finger
  0 siblings, 0 replies; 13+ messages in thread
From: Larry Finger @ 2006-12-31  4:50 UTC (permalink / raw)
  To: Tobin Davis; +Cc: Daniel Drake, alsa-devel, LKML, Stephen Hemminger

Tobin Davis wrote:
> Which alsa patch was this?  I'm not seeing anything in the hg logs for
> this.  Or is this something from the kernel side?

It seems to have come from suse. The full commit message is:

commit a7da6ce564a80952d9c0b210deca5a8cd3474a31
Author: Takashi Iwai <tiwai@suse.de>
Date:   Wed Sep 6 14:03:14 2006 +0200

    [ALSA] hda-codec - Add independent headphone volume control

    This patch addes the support of the independent 'Headphone' volume
    control to the generic codec parser.  Some codecs (e.g. Conexant)
    have separate connections to the headphone and the independent amp
    adjustment is needed.

    Signed-off-by: Takashi Iwai <tiwai@suse.de>
    Signed-off-by: Jaroslav Kysela <perex@suse.cz>

:100644 100644 dedfc5b... 97e9af1... M  sound/pci/hda/hda_generic.c

Larry



-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

^ permalink raw reply	[flat|nested] 13+ messages in thread
* Re: [Alsa-devel] Regression in 2.6.19 and 2.6.20 for snd_hda_intel
  2006-12-31  4:50           ` Larry Finger
@ 2007-01-01 17:32             ` Stephen Hemminger
  -1 siblings, 0 replies; 13+ messages in thread
From: Stephen Hemminger @ 2007-01-01 17:32 UTC (permalink / raw)
  To: Larry Finger; +Cc: Tobin Davis, Daniel Drake, alsa-devel, LKML

On Sat, 30 Dec 2006 22:50:41 -0600
Larry Finger <larry.finger@lwfinger.net> wrote:

> Tobin Davis wrote:
> > Which alsa patch was this?  I'm not seeing anything in the hg logs for
> > this.  Or is this something from the kernel side?
> 
> It seems to have come from suse. The full commit message is:
> 
> commit a7da6ce564a80952d9c0b210deca5a8cd3474a31
> Author: Takashi Iwai <tiwai@suse.de>
> Date:   Wed Sep 6 14:03:14 2006 +0200
> 
>     [ALSA] hda-codec - Add independent headphone volume control
> 
>     This patch addes the support of the independent 'Headphone' volume
>     control to the generic codec parser.  Some codecs (e.g. Conexant)
>     have separate connections to the headphone and the independent amp
>     adjustment is needed.
> 
>     Signed-off-by: Takashi Iwai <tiwai@suse.de>
>     Signed-off-by: Jaroslav Kysela <perex@suse.cz>
> 
> :100644 100644 dedfc5b... 97e9af1... M  sound/pci/hda/hda_generic.c
> 
> Larry
> 
> 

Just make sure you don't break the headphone jack on some systems.
The headphone jack on the Panasonic T5 didn't work until recent kernels
(probably this was the thing that made it work).

-- 
Stephen Hemminger <shemminger@osdl.org>

^ permalink raw reply	[flat|nested] 13+ messages in thread
* Re: Regression in 2.6.19 and 2.6.20 for snd_hda_intel
@ 2007-01-01 17:32             ` Stephen Hemminger
  0 siblings, 0 replies; 13+ messages in thread
From: Stephen Hemminger @ 2007-01-01 17:32 UTC (permalink / raw)
  To: Larry Finger; +Cc: Daniel Drake, alsa-devel, LKML

On Sat, 30 Dec 2006 22:50:41 -0600
Larry Finger <larry.finger@lwfinger.net> wrote:

> Tobin Davis wrote:
> > Which alsa patch was this?  I'm not seeing anything in the hg logs for
> > this.  Or is this something from the kernel side?
> 
> It seems to have come from suse. The full commit message is:
> 
> commit a7da6ce564a80952d9c0b210deca5a8cd3474a31
> Author: Takashi Iwai <tiwai@suse.de>
> Date:   Wed Sep 6 14:03:14 2006 +0200
> 
>     [ALSA] hda-codec - Add independent headphone volume control
> 
>     This patch addes the support of the independent 'Headphone' volume
>     control to the generic codec parser.  Some codecs (e.g. Conexant)
>     have separate connections to the headphone and the independent amp
>     adjustment is needed.
> 
>     Signed-off-by: Takashi Iwai <tiwai@suse.de>
>     Signed-off-by: Jaroslav Kysela <perex@suse.cz>
> 
> :100644 100644 dedfc5b... 97e9af1... M  sound/pci/hda/hda_generic.c
> 
> Larry
> 
> 

Just make sure you don't break the headphone jack on some systems.
The headphone jack on the Panasonic T5 didn't work until recent kernels
(probably this was the thing that made it work).

-- 
Stephen Hemminger <shemminger@osdl.org>

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

^ permalink raw reply	[flat|nested] 13+ messages in thread
* Re: Regression in 2.6.19 and 2.6.20 for snd_hda_intel
  2007-01-01 17:32             ` Stephen Hemminger
  (?)
@ 2007-01-02 17:51             ` Tobin Davis
  -1 siblings, 0 replies; 13+ messages in thread
From: Tobin Davis @ 2007-01-02 17:51 UTC (permalink / raw)
  To: Stephen Hemminger; +Cc: Daniel Drake, alsa-devel, LKML, Larry Finger


[-- Attachment #1.1: Type: text/plain, Size: 1747 bytes --]

This is part of the generic code.  It's main purpose is to provide basic
support until full featured support is created for a system.  If there
are systems that need this patch to work, we should look into adding
them to the main patch_* codec drivers.

For example, the Conexant codecs are now getting supported directly, as
are via and atihdmi.

If you have some information on these Panasonic models, please forward
it to me, thanks.

Tobin

On Mon, 2007-01-01 at 09:32 -0800, Stephen Hemminger wrote:

> On Sat, 30 Dec 2006 22:50:41 -0600
> Larry Finger <larry.finger@lwfinger.net> wrote:
> 
> > Tobin Davis wrote:
> > > Which alsa patch was this?  I'm not seeing anything in the hg logs for
> > > this.  Or is this something from the kernel side?
> > 
> > It seems to have come from suse. The full commit message is:
> > 
> > commit a7da6ce564a80952d9c0b210deca5a8cd3474a31
> > Author: Takashi Iwai <tiwai@suse.de>
> > Date:   Wed Sep 6 14:03:14 2006 +0200
> > 
> >     [ALSA] hda-codec - Add independent headphone volume control
> > 
> >     This patch addes the support of the independent 'Headphone' volume
> >     control to the generic codec parser.  Some codecs (e.g. Conexant)
> >     have separate connections to the headphone and the independent amp
> >     adjustment is needed.
> > 
> >     Signed-off-by: Takashi Iwai <tiwai@suse.de>
> >     Signed-off-by: Jaroslav Kysela <perex@suse.cz>
> > 
> > :100644 100644 dedfc5b... 97e9af1... M  sound/pci/hda/hda_generic.c
> > 
> > Larry
> > 
> > 
> 
> Just make sure you don't break the headphone jack on some systems.
> The headphone jack on the Panasonic T5 didn't work until recent kernels
> (probably this was the thing that made it work).
> 

-- 
Tobin Davis <tdavis@dsl-only.net>

[-- Attachment #1.2: Type: text/html, Size: 3262 bytes --]

[-- Attachment #2: Type: text/plain, Size: 347 bytes --]

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

[-- Attachment #3: Type: text/plain, Size: 161 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 13+ messages in thread
* Re: [Alsa-devel] Regression in 2.6.19 and 2.6.20 for snd_hda_intel
  2006-12-31  4:50           ` Larry Finger
@ 2007-01-08 12:06             ` Takashi Iwai
  -1 siblings, 0 replies; 13+ messages in thread
From: Takashi Iwai @ 2007-01-08 12:06 UTC (permalink / raw)
  To: Larry Finger
  Cc: Tobin Davis, Daniel Drake, alsa-devel, LKML, Stephen Hemminger

At Sat, 30 Dec 2006 22:50:41 -0600,
Larry Finger wrote:
> 
> Tobin Davis wrote:
> > Which alsa patch was this?  I'm not seeing anything in the hg logs for
> > this.  Or is this something from the kernel side?
> 
> It seems to have come from suse. The full commit message is:
> 
> commit a7da6ce564a80952d9c0b210deca5a8cd3474a31
> Author: Takashi Iwai <tiwai@suse.de>
> Date:   Wed Sep 6 14:03:14 2006 +0200
> 
>     [ALSA] hda-codec - Add independent headphone volume control
> 
>     This patch addes the support of the independent 'Headphone' volume
>     control to the generic codec parser.  Some codecs (e.g. Conexant)
>     have separate connections to the headphone and the independent amp
>     adjustment is needed.
> 
>     Signed-off-by: Takashi Iwai <tiwai@suse.de>
>     Signed-off-by: Jaroslav Kysela <perex@suse.cz>
> 
> :100644 100644 dedfc5b... 97e9af1... M  sound/pci/hda/hda_generic.c

Make sure that you unmuted both Master and Headphone volumes (if you
have).
With this patch, you may have now two individual controls while the
older version has only one.


Takashi

^ permalink raw reply	[flat|nested] 13+ messages in thread
* Re: Regression in 2.6.19 and 2.6.20 for snd_hda_intel
@ 2007-01-08 12:06             ` Takashi Iwai
  0 siblings, 0 replies; 13+ messages in thread
From: Takashi Iwai @ 2007-01-08 12:06 UTC (permalink / raw)
  To: Larry Finger; +Cc: Daniel Drake, alsa-devel, LKML, Stephen Hemminger

At Sat, 30 Dec 2006 22:50:41 -0600,
Larry Finger wrote:
> 
> Tobin Davis wrote:
> > Which alsa patch was this?  I'm not seeing anything in the hg logs for
> > this.  Or is this something from the kernel side?
> 
> It seems to have come from suse. The full commit message is:
> 
> commit a7da6ce564a80952d9c0b210deca5a8cd3474a31
> Author: Takashi Iwai <tiwai@suse.de>
> Date:   Wed Sep 6 14:03:14 2006 +0200
> 
>     [ALSA] hda-codec - Add independent headphone volume control
> 
>     This patch addes the support of the independent 'Headphone' volume
>     control to the generic codec parser.  Some codecs (e.g. Conexant)
>     have separate connections to the headphone and the independent amp
>     adjustment is needed.
> 
>     Signed-off-by: Takashi Iwai <tiwai@suse.de>
>     Signed-off-by: Jaroslav Kysela <perex@suse.cz>
> 
> :100644 100644 dedfc5b... 97e9af1... M  sound/pci/hda/hda_generic.c

Make sure that you unmuted both Master and Headphone volumes (if you
have).
With this patch, you may have now two individual controls while the
older version has only one.


Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

^ permalink raw reply	[flat|nested] 13+ messages in thread
end of thread, other threads:[~2007-01-08 12:06 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-12-31  1:20 Regression in 2.6.19 and 2.6.20 for snd_hda_intel Larry Finger
2006-12-31  2:23 ` Daniel Drake
2006-12-31  3:31   ` Larry Finger
2006-12-31  3:46     ` Daniel Drake
2006-12-31  3:46       ` Daniel Drake
2006-12-31  4:08       ` Tobin Davis
2006-12-31  4:50         ` [Alsa-devel] " Larry Finger
2006-12-31  4:50           ` Larry Finger
2007-01-01 17:32           ` [Alsa-devel] " Stephen Hemminger
2007-01-01 17:32             ` Stephen Hemminger
2007-01-02 17:51             ` Tobin Davis
2007-01-08 12:06           ` [Alsa-devel] " Takashi Iwai
2007-01-08 12:06             ` Takashi Iwai

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.