Takashi Iwai
> Cc: Wang, Zhenyu Z; intel-gfx@lists.freedesktop.org; Barnes, Jesse; Jeremy
> Bush; Christopher White; Bossart, Pierre-louis
> Subject: Re: [Intel-gfx] [PATCH v5] drm/i915: pass ELD to HDMI/DP audio
> driver
>
> On Mon, Nov 14, 2011 at 05:45:12PM +0800, Takashi Iw
On Mon, Nov 14, 2011 at 05:45:12PM +0800, Takashi Iwai wrote:
> At Sat, 12 Nov 2011 10:27:26 +0800,
> Wu Fengguang wrote:
> >
> > (snip)
> > > > > > > And I'm not sure whether HDMI audio is played
> > > > > > > while DPMS is off. I haven't tested it.
> > > > > >
> > > > > > It will go silence on
At Sat, 12 Nov 2011 10:27:26 +0800,
Wu Fengguang wrote:
>
> (snip)
> > > > > > And I'm not sure whether HDMI audio is played
> > > > > > while DPMS is off. I haven't tested it.
> > > > >
> > > > > It will go silence on DPMS. I noticed this while doing long term HDMI
> > > > > audio playback test
(snip)
> > > > > And I'm not sure whether HDMI audio is played
> > > > > while DPMS is off. I haven't tested it.
> > > >
> > > > It will go silence on DPMS. I noticed this while doing long term HDMI
> > > > audio playback tests. This should better be fixed in future on the
> > > > graphics side.
On Fri, Nov 11, 2011 at 07:23:05PM +0800, Takashi Iwai wrote:
> At Fri, 11 Nov 2011 19:12:57 +0800,
> Wu Fengguang wrote:
> >
> > (snip)
> > > > > One note that we don't rely on PD bit because not all (non-Intel)
> > > > > hardware report it correctly.
> > > >
> > > > Oops. Do you imply ELDV is r
At Fri, 11 Nov 2011 19:12:57 +0800,
Wu Fengguang wrote:
>
> (snip)
> > > > One note that we don't rely on PD bit because not all (non-Intel)
> > > > hardware report it correctly.
> > >
> > > Oops. Do you imply ELDV is reliable on all platforms? ;-)
> >
> > Oh hell, no :)
> > The driver tries to
(snip)
> > > One note that we don't rely on PD bit because not all (non-Intel)
> > > hardware report it correctly.
> >
> > Oops. Do you imply ELDV is reliable on all platforms? ;-)
>
> Oh hell, no :)
> The driver tries to probe explicitly via GET_PIN_SENSE HD-audio verb.
Yeah the below "HDMI sta
At Fri, 11 Nov 2011 17:24:21 +0800,
Wu Fengguang wrote:
>
> On Fri, Nov 11, 2011 at 04:49:57PM +0800, Takashi Iwai wrote:
> > At Fri, 11 Nov 2011 16:22:41 +0800,
> > Wu Fengguang wrote:
> > >
> > > On Fri, Nov 11, 2011 at 03:40:37PM +0800, Takashi Iwai wrote:
> > > > At Fri, 11 Nov 2011 10:29:25
On Fri, Nov 11, 2011 at 04:49:57PM +0800, Takashi Iwai wrote:
> At Fri, 11 Nov 2011 16:22:41 +0800,
> Wu Fengguang wrote:
> >
> > On Fri, Nov 11, 2011 at 03:40:37PM +0800, Takashi Iwai wrote:
> > > At Fri, 11 Nov 2011 10:29:25 +0800,
> > > Wu Fengguang wrote:
> > > >
> > > > On Thu, Nov 10, 2011
At Fri, 11 Nov 2011 16:22:41 +0800,
Wu Fengguang wrote:
>
> On Fri, Nov 11, 2011 at 03:40:37PM +0800, Takashi Iwai wrote:
> > At Fri, 11 Nov 2011 10:29:25 +0800,
> > Wu Fengguang wrote:
> > >
> > > On Thu, Nov 10, 2011 at 10:28:19PM +0800, Takashi Iwai wrote:
> > > > At Thu, 10 Nov 2011 21:51:50
On Fri, Nov 11, 2011 at 03:40:37PM +0800, Takashi Iwai wrote:
> At Fri, 11 Nov 2011 10:29:25 +0800,
> Wu Fengguang wrote:
> >
> > On Thu, Nov 10, 2011 at 10:28:19PM +0800, Takashi Iwai wrote:
> > > At Thu, 10 Nov 2011 21:51:50 +0800,
> > > Wu Fengguang wrote:
> > > >
> > > > > > > > So maybe the
At Fri, 11 Nov 2011 10:29:25 +0800,
Wu Fengguang wrote:
>
> On Thu, Nov 10, 2011 at 10:28:19PM +0800, Takashi Iwai wrote:
> > At Thu, 10 Nov 2011 21:51:50 +0800,
> > Wu Fengguang wrote:
> > >
> > > > > > > So maybe the hardware is in some state that is unable to provide
> > > > > > > the
> > > >
On Thu, Nov 10, 2011 at 10:28:19PM +0800, Takashi Iwai wrote:
> At Thu, 10 Nov 2011 21:51:50 +0800,
> Wu Fengguang wrote:
> >
> > > > > > So maybe the hardware is in some state that is unable to provide the
> > > > > > real ELD content?
> > > > > That's my guess as well. I think the hardware may s
At Thu, 10 Nov 2011 21:51:50 +0800,
Wu Fengguang wrote:
>
> > > > > So maybe the hardware is in some state that is unable to provide the
> > > > > real ELD content?
> > > > That's my guess as well. I think the hardware may still be doing some
> > > > form of data negotiation with the HDMI display
On Thu, Nov 10, 2011 at 09:47:46PM +0800, Wu Fengguang wrote:
> > > Got the delay - it's 72.986623-72.747632 = 239ms.
> > >
> > > [ 72.739944] HDMI hot plug event: Codec=3 Pin=6
> > > Presence_Detect=1 ELD_Valid=0
> > > [ 72.742541] HDMI status: Codec=3 Pin=6 Presence_Detect=
On Thu, Nov 10, 2011 at 09:51:50PM +0800, Wu Fengguang wrote:
> > > > > So maybe the hardware is in some state that is unable to provide the
> > > > > real ELD content?
> > > > That's my guess as well. I think the hardware may still be doing some
> > > > form of data negotiation with the HDMI disp
> > > > So maybe the hardware is in some state that is unable to provide the
> > > > real ELD content?
> > > That's my guess as well. I think the hardware may still be doing some
> > > form of data negotiation with the HDMI display device at that stage, and
> > > doesn't have the copy of the EDID
> > Got the delay - it's 72.986623-72.747632 = 239ms.
> >
> > [ 72.739944] HDMI hot plug event: Codec=3 Pin=6
> > Presence_Detect=1 ELD_Valid=0
> > [ 72.742541] HDMI status: Codec=3 Pin=6 Presence_Detect=1
> > ELD_Valid=0
> > [ 72.745082] HDMI hot plug event: Code
At Thu, 10 Nov 2011 21:17:53 +0800,
Wu Fengguang wrote:
>
> On Thu, Nov 10, 2011 at 09:01:24PM +0800, Christopher White wrote:
> > On 11/10/11 1:56 PM, Wu Fengguang wrote:
> > > On Thu, Nov 10, 2011 at 07:50:11PM +0800, Christopher White wrote:
> > >> On 11/10/11 12:22 PM, Takashi Iwai wrote:
> >
On 11/10/11 2:17 PM, Wu Fengguang wrote:
On Thu, Nov 10, 2011 at 09:01:24PM +0800, Christopher White wrote:
On 11/10/11 1:56 PM, Wu Fengguang wrote:
On Thu, Nov 10, 2011 at 07:50:11PM +0800, Christopher White wrote:
On 11/10/11 12:22 PM, Takashi Iwai wrote:
At Thu, 10 Nov 2011 12:00:53 +0100,
On Thu, Nov 10, 2011 at 09:01:24PM +0800, Christopher White wrote:
> On 11/10/11 1:56 PM, Wu Fengguang wrote:
> > On Thu, Nov 10, 2011 at 07:50:11PM +0800, Christopher White wrote:
> >> On 11/10/11 12:22 PM, Takashi Iwai wrote:
> >>> At Thu, 10 Nov 2011 12:00:53 +0100,
> >>> Christopher White wrote
At Thu, 10 Nov 2011 13:39:00 +0100,
Christopher White wrote:
>
> On 11/10/11 12:53 PM, Takashi Iwai wrote:
> > At Thu, 10 Nov 2011 12:50:11 +0100,
> > Christopher White wrote:
> >> On 11/10/11 12:22 PM, Takashi Iwai wrote:
> >>> At Thu, 10 Nov 2011 12:00:53 +0100,
> >>> Christopher White wrote:
>
On 11/10/11 1:56 PM, Wu Fengguang wrote:
On Thu, Nov 10, 2011 at 07:50:11PM +0800, Christopher White wrote:
On 11/10/11 12:22 PM, Takashi Iwai wrote:
At Thu, 10 Nov 2011 12:00:53 +0100,
Christopher White wrote:
On 11/10/11 9:55 AM, Christopher White wrote:
On 11/10/11 8:55 AM, Wu Fengguang wr
On Thu, Nov 10, 2011 at 07:50:11PM +0800, Christopher White wrote:
> On 11/10/11 12:22 PM, Takashi Iwai wrote:
> > At Thu, 10 Nov 2011 12:00:53 +0100,
> > Christopher White wrote:
> >> On 11/10/11 9:55 AM, Christopher White wrote:
> >>> On 11/10/11 8:55 AM, Wu Fengguang wrote:
> On Thu, Nov 10
On 11/10/11 12:53 PM, Takashi Iwai wrote:
At Thu, 10 Nov 2011 12:50:11 +0100,
Christopher White wrote:
On 11/10/11 12:22 PM, Takashi Iwai wrote:
At Thu, 10 Nov 2011 12:00:53 +0100,
Christopher White wrote:
On 11/10/11 9:55 AM, Christopher White wrote:
On 11/10/11 8:55 AM, Wu Fengguang wrote:
At Thu, 10 Nov 2011 12:50:11 +0100,
Christopher White wrote:
>
> On 11/10/11 12:22 PM, Takashi Iwai wrote:
> > At Thu, 10 Nov 2011 12:00:53 +0100,
> > Christopher White wrote:
> >> On 11/10/11 9:55 AM, Christopher White wrote:
> >>> On 11/10/11 8:55 AM, Wu Fengguang wrote:
> On Thu, Nov 10, 2
On 11/10/11 12:22 PM, Takashi Iwai wrote:
At Thu, 10 Nov 2011 12:00:53 +0100,
Christopher White wrote:
On 11/10/11 9:55 AM, Christopher White wrote:
On 11/10/11 8:55 AM, Wu Fengguang wrote:
On Thu, Nov 10, 2011 at 03:33:50PM +0800, Wu Fengguang wrote:
Wow I reproduced the bug and got a very i
At Thu, 10 Nov 2011 12:00:53 +0100,
Christopher White wrote:
>
> On 11/10/11 9:55 AM, Christopher White wrote:
> > On 11/10/11 8:55 AM, Wu Fengguang wrote:
> >> On Thu, Nov 10, 2011 at 03:33:50PM +0800, Wu Fengguang wrote:
> >>> Wow I reproduced the bug and got a very interesting dmesg:
> >>>
> >>
On 11/10/11 8:55 AM, Wu Fengguang wrote:
On Thu, Nov 10, 2011 at 03:33:50PM +0800, Wu Fengguang wrote:
Wow I reproduced the bug and got a very interesting dmesg:
gfx => [ 4561.287980] [drm:intel_write_eld], ELD on
[CONNECTOR:12:HDMI-A-2], [ENCODER:11:TMDS-11]
gfx => [ 4561.2917
On Thu, Nov 10, 2011 at 03:55:22PM +0800, Wu Fengguang wrote:
> On Thu, Nov 10, 2011 at 03:33:50PM +0800, Wu Fengguang wrote:
> > Wow I reproduced the bug and got a very interesting dmesg:
> >
> > gfx =>[ 4561.287980] [drm:intel_write_eld], ELD on
> > [CONNECTOR:12:HDMI-A-2], [ENCODER:11:
On Thu, Nov 10, 2011 at 03:33:50PM +0800, Wu Fengguang wrote:
> Wow I reproduced the bug and got a very interesting dmesg:
>
> gfx =>[ 4561.287980] [drm:intel_write_eld], ELD on
> [CONNECTOR:12:HDMI-A-2], [ENCODER:11:TMDS-11]
> gfx =>[ 4561.291730] [drm:ironlake_write_eld], ELD on
Wow I reproduced the bug and got a very interesting dmesg:
gfx =>[ 4561.287980] [drm:intel_write_eld], ELD on
[CONNECTOR:12:HDMI-A-2], [ENCODER:11:TMDS-11]
gfx =>[ 4561.291730] [drm:ironlake_write_eld], ELD on pipe B
gfx =>[ 4561.293804] [drm:ironlake_write_eld], Audio dir
Christopher,
On Thu, Nov 10, 2011 at 12:10:42PM +0800, Christopher White wrote:
> On 11/10/11 4:27 AM, Wu Fengguang wrote:
> > Christopher,
> >
> > Did you enabled CONFIG_SND_DEBUG and CONFIG_SND_DEBUG_VERBOSE in
> > kconfig? I've been looking for the error messages related to zeroed
> > ELD but n
Christopher,
> The dump tool did not work with that environment variable either.
> However, it occurred to me that intel_audio_dump may be too outdated in
> my distro. It was built on 2010-04-01, v1.0.2+git20100324. If I look at
> http://cgit.freedesktop.org/xorg/app/intel-gpu-tools/ I can see tha
Christopher,
Did you enabled CONFIG_SND_DEBUG and CONFIG_SND_DEBUG_VERBOSE in
kconfig? I've been looking for the error messages related to zeroed
ELD but never managed to find any in your dmesg.
Some analyzes of your dmesg:
- all drm_edid_to_eld() invocations reads valid EDID data:
[2.56000
Hi Fengguang, I am quoting both your messages below so read until the
bottom. I've even found what looks to be the cause.
On 11/9/11 2:01 PM, Wu Fengguang wrote:
Christopher,
On Wed, Nov 09, 2011 at 05:30:18PM +0800, Christopher White wrote:
Couldn't resist connecting to my machine over VNC e
Hi Christopher,
> Now, onto the intel-gpu-tools test. I ran intel_audio_dump as requested
> and it only comes back with "Couldn't map MMIO region: No such file or
> directory". I spent 10 minutes looking around on Google to no avail. It
> seems it tries to mmap() something that doesn't exist.
Good day, Fengguang! Great work! This sounds very promising!
I went through the ELD parsing code myself (drm_edid_to_eld), as my
programmer mind's curiosity killed me even though I didn't really have
time for it, and I could see that it grabs the CEA extension block,
grabs the monitor name str
Hi Christopher,
I don't find anything wrong with the ELD parsing code, however I do
find a bug that prevented ELD from being passed to the audio driver on
SandyBridge.
I just posted the fix. For your convenience, it's attached in this
email too.
Thanks,
Fengguang
On Fri, Oct 28, 2011 at 03:57:2
On 11/2/11 2:45 AM, Wu Fengguang wrote:
Hi Christopher,
The log does confirm that the drm_edid_to_eld function is running, and
that we're not far from a solution:
[ 21.061417] [drm:drm_edid_to_eld], ELD monitor TX-SR607
[ 21.061421] [drm:drm_edid_to_eld], ELD size 13, SAD count 8
It looks
Wu Fengguang intel.com> writes:
>
> Hi Christopher,
>
> > The log does confirm that the drm_edid_to_eld function is running, and
> > that we're not far from a solution:
> > [ 21.061417] [drm:drm_edid_to_eld], ELD monitor TX-SR607
> > [ 21.061421] [drm:drm_edid_to_eld], ELD size 13, SAD cou
On Wed, 2 Nov 2011 16:52:46 +0800, Wu Fengguang wrote:
> In fact Linux 3.1 does not have the ELD patch yet. It should go into
> the upcoming 3.2.
The ELD patch is on Linus' master branch.
--
keith.pack...@intel.com
pgprsJ9jvJTLR.pgp
Description: PGP signature
On Wed, Nov 2, 2011 at 6:17 AM, Sander Jansen wrote:
> On Wed, Nov 2, 2011 at 2:35 AM, Paul Menzel
> wrote:
>> Dear Sander,
>>
>>
>> Am Mittwoch, den 02.11.2011, 01:10 -0500 schrieb Sander Jansen:
>>> On Tue, Nov 1, 2011 at 8:45 PM, Wu Fengguang wrote:
>>
>>> >> The log does confirm that the drm
On Wed, Nov 2, 2011 at 2:35 AM, Paul Menzel
wrote:
> Dear Sander,
>
>
> Am Mittwoch, den 02.11.2011, 01:10 -0500 schrieb Sander Jansen:
>> On Tue, Nov 1, 2011 at 8:45 PM, Wu Fengguang wrote:
>
>> >> The log does confirm that the drm_edid_to_eld function is running, and
>> >> that we're not far fr
Hi Sander,
> On Tue, Nov 1, 2011 at 8:45 PM, Wu Fengguang wrote:
> > Hi Christopher,
> >
> >> The log does confirm that the drm_edid_to_eld function is running, and
> >> that we're not far from a solution:
> >> [ 21.061417] [drm:drm_edid_to_eld], ELD monitor TX-SR607
> >> [ 21.061421] [drm:dr
Dear Sander,
Am Mittwoch, den 02.11.2011, 01:10 -0500 schrieb Sander Jansen:
> On Tue, Nov 1, 2011 at 8:45 PM, Wu Fengguang wrote:
> >> The log does confirm that the drm_edid_to_eld function is running, and
> >> that we're not far from a solution:
> >> [ 21.061417] [drm:drm_edid_to_eld], ELD
On Tue, Nov 1, 2011 at 8:45 PM, Wu Fengguang wrote:
> Hi Christopher,
>
>> The log does confirm that the drm_edid_to_eld function is running, and
>> that we're not far from a solution:
>> [ 21.061417] [drm:drm_edid_to_eld], ELD monitor TX-SR607
>> [ 21.061421] [drm:drm_edid_to_eld], ELD size 1
Hi Christopher,
> The log does confirm that the drm_edid_to_eld function is running, and
> that we're not far from a solution:
> [ 21.061417] [drm:drm_edid_to_eld], ELD monitor TX-SR607
> [ 21.061421] [drm:drm_edid_to_eld], ELD size 13, SAD count 8
It looks all sane to this point.
> As for
On 11/1/11 12:36 PM, Wu Fengguang wrote:
Hi Christopher,
Sorry I'm just back from traveling..
No worries, I am not in any hurry, and I hope you had a great holiday! :-)
On Fri, Oct 28, 2011 at 03:54:23AM +0800, Christopher White wrote:
There appears to be some issues with the patch? I'm on S
Hi Christopher,
Sorry I'm just back from traveling..
On Fri, Oct 28, 2011 at 03:54:23AM +0800, Christopher White wrote:
> There appears to be some issues with the patch? I'm on SandyBridge and
> using the HD3000's HDMI.
>
> I've now tried manually merging the ELD patch (both files Wu Fengguang
There appears to be some issues with the patch? I'm on SandyBridge and
using the HD3000's HDMI.
I've now tried manually merging the ELD patch (both files Wu Fengguang
submitted) and compiling Kernel 3.0.4. I've also tried drm-intel-next
Kernel 3.1 pre-built from
http://kernel.ubuntu.com/~kern
On Mon, Sep 05, 2011 at 07:04:50PM +0800, Chris Wilson wrote:
> On Mon, 5 Sep 2011 09:14:00 +0800, Wu Fengguang
> wrote:
> > On Sun, Sep 04, 2011 at 08:08:37PM +0800, Chris Wilson wrote:
> > > On Sun, 4 Sep 2011 05:15:10 +0800, Wu Fengguang
> > > wrote:
> > > This patch should be split between
On Mon, 5 Sep 2011 09:14:00 +0800, Wu Fengguang wrote:
> On Sun, Sep 04, 2011 at 08:08:37PM +0800, Chris Wilson wrote:
> > On Sun, 4 Sep 2011 05:15:10 +0800, Wu Fengguang
> > wrote:
> > This patch should be split between adding the core drm functionality to
> > build the ELD and the introduction
On Sun, Sep 04, 2011 at 06:57:23PM +0800, James Cloos wrote:
> > "WF" == Wu Fengguang writes:
>
> WF> ... If only the stereo playback capability is reported, the user
> WF> won't be able to start 8-channel playback; if the 8-channel ELD is
> WF> reported, then user space applications may send
On Sun, Sep 04, 2011 at 08:08:37PM +0800, Chris Wilson wrote:
> On Sun, 4 Sep 2011 05:15:10 +0800, Wu Fengguang
> wrote:
> > Changes from v4: remove a debug call to dump_stack().
> > Thanks to Bossart for catching this!
> > ---
> >
> > Add ELD support for Intel Eaglelake, IbexPeak/Ironlake,
> >
Dear Paul,
On Sun, Sep 04, 2011 at 07:11:54PM +0800, Paul Menzel wrote:
> Dear Wu,
>
>
> I hope that is your first name.
My first name is Fengguang. "LAST NAME, FIRSTNAME" is the convention
in Intel emails. However I often forgot this and pick whatever comes
first...and happily accept both Feng
On Sun, 4 Sep 2011 05:15:10 +0800, Wu Fengguang wrote:
> Changes from v4: remove a debug call to dump_stack().
> Thanks to Bossart for catching this!
> ---
>
> Add ELD support for Intel Eaglelake, IbexPeak/Ironlake,
> SandyBridge/CougarPoint and IvyBridge/PantherPoint chips.
>
> ELD (EDID-Like D
Dear Wu,
I hope that is your first name.
Am Sonntag, den 04.09.2011, 05:15 +0800 schrieb Wu Fengguang:
> Changes from v4: remove a debug call to dump_stack().
> Thanks to Bossart for catching this!
His first name is Pierre-Louis. I do not know how you address people at
Intel though.
> ---
I t
> "WF" == Wu Fengguang writes:
WF> ... If only the stereo playback capability is reported, the user
WF> won't be able to start 8-channel playback; if the 8-channel ELD is
WF> reported, then user space applications may send 8-channel samples
WF> down, however the user may actually be listening
Changes from v4: remove a debug call to dump_stack().
Thanks to Bossart for catching this!
---
Add ELD support for Intel Eaglelake, IbexPeak/Ironlake,
SandyBridge/CougarPoint and IvyBridge/PantherPoint chips.
ELD (EDID-Like Data) describes to the HDMI/DP audio driver the audio
capabilities of the
60 matches
Mail list logo
