Re: [alsa-devel] Linux 3.2-rc1
Hmm, I don't know what caused this to trigger, but I'm adding both the i915 people and the HDA people to the cc, and they can fight to the death about this in the HDMI Thunderdome.
Guys: One.. Two.. Three.. FIGHT!
Linus
On Tue, Nov 8, 2011 at 6:55 AM, Nick Bowler nbowler@elliptictech.com wrote:
Mode switches are very noisy on an Intel G45 in 3.2-rc1:
HDMI hot plug event: Codec=3 Pin=3 Presence_Detect=1 ELD_Valid=1 HDMI status: Codec=3 Pin=3 Presence_Detect=1 ELD_Valid=1 HDMI: detected monitor W2253 at connection type HDMI HDMI: available speakers: FL/FR HDMI: supports coding type LPCM: channels = 2, rates = 32000 44100 48000, bits = 16 20 24
These lines get printed every single switch; previously only a single line was printed once at boot (the "HDMI status" line).
Cheers,
Nick Bowler, Elliptic Technologies (http://www.elliptictech.com/)
At Tue, 8 Nov 2011 12:23:30 -0800, Linus Torvalds wrote:
Hmm, I don't know what caused this to trigger, but I'm adding both the i915 people and the HDA people to the cc, and they can fight to the death about this in the HDMI Thunderdome.
It must be the new addition of ELD-passing code.
Fengguang, can the drm or i915 driver check whether ELD is changed or not? Writing ELD at each time even when unchanged confuses the audio side, as if the monitor is hotplugged.
Guys: One.. Two.. Three.. FIGHT!
Round two!
Takashi
LinusOn Tue, Nov 8, 2011 at 6:55 AM, Nick Bowler nbowler@elliptictech.com wrote:
Mode switches are very noisy on an Intel G45 in 3.2-rc1:
HDMI hot plug event: Codec=3 Pin=3 Presence_Detect=1 ELD_Valid=1 HDMI status: Codec=3 Pin=3 Presence_Detect=1 ELD_Valid=1 HDMI: detected monitor W2253 at connection type HDMI HDMI: available speakers: FL/FR HDMI: supports coding type LPCM: channels = 2, rates = 32000 44100 48000, bits = 16 20 24
These lines get printed every single switch; previously only a single line was printed once at boot (the "HDMI status" line).
Cheers,
Nick Bowler, Elliptic Technologies (http://www.elliptictech.com/)
On Wed, Nov 09, 2011 at 03:40:19PM +0800, Takashi Iwai wrote:
At Tue, 8 Nov 2011 12:23:30 -0800, Linus Torvalds wrote:
Hmm, I don't know what caused this to trigger, but I'm adding both the i915 people and the HDA people to the cc, and they can fight to the death about this in the HDMI Thunderdome.
It must be the new addition of ELD-passing code.
Yes, it's my fault.
Fengguang, can the drm or i915 driver check whether ELD is changed or not? Writing ELD at each time even when unchanged confuses the audio side, as if the monitor is hotplugged.
It's sure possible to mask out the extra events. I'll work out a patch tomorrow.
Guys: One.. Two.. Three.. FIGHT!
Round two!
Three to fight!
Thanks, Fengguang
On Tue, Nov 8, 2011 at 6:55 AM, Nick Bowler nbowler@elliptictech.com wrote:
Mode switches are very noisy on an Intel G45 in 3.2-rc1:
HDMI hot plug event: Codec=3 Pin=3 Presence_Detect=1 ELD_Valid=1 HDMI status: Codec=3 Pin=3 Presence_Detect=1 ELD_Valid=1 HDMI: detected monitor W2253 at connection type HDMI HDMI: available speakers: FL/FR HDMI: supports coding type LPCM: channels = 2, rates = 32000 44100 48000, bits = 16 20 24
These lines get printed every single switch; previously only a single line was printed once at boot (the "HDMI status" line).
Cheers,
Nick Bowler, Elliptic Technologies (http://www.elliptictech.com/)
Hi Nick,
On Wed, Nov 09, 2011 at 03:40:19PM +0800, Takashi Iwai wrote:
At Tue, 8 Nov 2011 12:23:30 -0800, Linus Torvalds wrote:
Hmm, I don't know what caused this to trigger, but I'm adding both the i915 people and the HDA people to the cc, and they can fight to the death about this in the HDMI Thunderdome.
It must be the new addition of ELD-passing code.
Fengguang, can the drm or i915 driver check whether ELD is changed or not? Writing ELD at each time even when unchanged confuses the audio side, as if the monitor is hotplugged.
The attached patch is tested OK to prevent extra hot plug events.
However it has one side effect: when HDMI monitor is hot removed, the ELD keeps remain valid. I need to find a way to test for the presence of the monitor and handle that case as well. When all done, I'll submit the patches together for review.
Thanks, Fengguang
participants (3)
-
Linus Torvalds -
Takashi Iwai -
Wu Fengguang