From: Takashi Iwai <tiwai@suse.de>
To: Stephen Warren <swarren@nvidia.com>
Cc: "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
"Wang, Zhenyu Z" <zhenyu.z.wang@intel.com>,
"intel-gfx@lists.freedesktop.org"
<intel-gfx@lists.freedesktop.org>,
"Barnes, Jesse" <jesse.barnes@intel.com>,
Christopher White <c.white@pulseforce.com>,
Jeremy Bush <contractfrombelow@gmail.com>,
Wu Fengguang <fengguang.wu@intel.com>,
"Bossart, Pierre-louis" <pierre-louis.bossart@intel.com>
Subject: Re: [PATCH 2/2] hda - delayed ELD repoll
Date: Wed, 16 Nov 2011 16:57:59 +0100 [thread overview]
Message-ID: <s5h1ut8njp4.wl%tiwai@suse.de> (raw)
In-Reply-To: <74CDBE0F657A3D45AFBB94109FB122FF1740D7487A@HQMAIL01.nvidia.com>
At Wed, 16 Nov 2011 07:51:28 -0800,
Stephen Warren wrote:
>
> > > 250mS almost sounds like it's setting ELDV in the audio HW,
> > > then going and reading the EDID, then writing the EDID to the audio HW;
> > > perhaps the graphics driver is accidentally setting PRESENT+ELDV when it's
> > > meant to be setting just PRESENT, and later setting ELDV?
> >
> > From the debug dmesg, I'm pretty sure that the ELDV events are
> > triggered exactly by the "eld_valid = 0" and "eld_valid = 1" register
> > writes. Since the ELD data is already prepared, there is no EDID read
> > in between.
> >
> > Below is the dmesg representing a video mode set.
> >
> > ELD writes from the graphics driver
> >
> > [ 424.254958] [drm:intel_write_eld], ELD on [CONNECTOR:12:HDMI-A-2], [ENCODER:11:TMDS-11]
> > [ 424.257670] [drm:ironlake_write_eld], ELD on pipe B
> > [ 424.259833] [drm:ironlake_write_eld], Audio directed to unknown port
> > [ 424.262156] [drm:ironlake_write_eld], ELD size 13
> >
> > ELD events received by audio driver (eld reads 0)
> >
> > [ 424.263258] HDMI hot plug event: Codec=3 Pin=6 Presence_Detect=1 ELD_Valid=0
>
> That line makes sense.
>
> > [ 424.265877] HDMI status: Codec=3 Pin=6 Presence_Detect=1 ELD_Valid=1
>
> I don't /think/ it's related to this issue, but I wonder why ELDV==1 in
> that message; it seems that the unsolicited response contains the correct
> data, but AC_VERB_GET_PIN_SENSE contains ELDV==1 all the time. That's odd.
Note that this bit isn't from GET_PIN_SENSE verb but the bit in the
unsol event argument.
Takashi
next prev parent reply other threads:[~2011-11-16 15:57 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-15 14:31 [PATCH 1/2] hda - fix ELD memory leak Wu Fengguang
2011-11-15 14:33 ` [PATCH 2/2] hda - delayed ELD repoll Wu Fengguang
2011-11-15 16:56 ` Wu Fengguang
2011-11-15 16:57 ` [PATCH 2/2 v2] " Wu Fengguang
2011-11-15 17:10 ` Takashi Iwai
2011-11-16 2:35 ` Wu Fengguang
2011-11-16 7:02 ` Paul Menzel
2011-11-15 18:25 ` [PATCH 2/2] " Stephen Warren
2011-11-16 2:48 ` [alsa-devel] " Wu Fengguang
2011-11-16 15:51 ` Stephen Warren
2011-11-16 15:57 ` Takashi Iwai [this message]
2011-11-16 16:12 ` Stephen Warren
2011-11-16 16:16 ` [alsa-devel] " Takashi Iwai
2011-11-16 22:46 ` Wu Fengguang
2011-11-16 23:12 ` Wu Fengguang
2011-11-15 14:35 ` [PATCH 1/2] hda - fix ELD memory leak Takashi Iwai
2011-11-15 14:41 ` Wu Fengguang
2011-11-15 14:45 ` Takashi Iwai
2011-11-15 16:56 ` Wu Fengguang
2011-11-15 18:19 ` Stephen Warren
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=s5h1ut8njp4.wl%tiwai@suse.de \
--to=tiwai@suse.de \
--cc=alsa-devel@alsa-project.org \
--cc=c.white@pulseforce.com \
--cc=contractfrombelow@gmail.com \
--cc=fengguang.wu@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jesse.barnes@intel.com \
--cc=pierre-louis.bossart@intel.com \
--cc=swarren@nvidia.com \
--cc=zhenyu.z.wang@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox