public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
From: Wu Fengguang <fengguang.wu@intel.com>
To: Christopher White <c.white@pulseforce.com>
Cc: Jeremy Bush <contractfrombelow@gmail.com>,
	"intel-gfx@lists.freedesktop.org"
	<intel-gfx@lists.freedesktop.org>,
	"Wang, Zhenyu Z" <zhenyu.z.wang@intel.com>,
	"Bossart, Pierre-louis" <pierre-louis.bossart@intel.com>
Subject: Re: [PATCH v5] drm/i915: pass ELD to HDMI/DP audio driver
Date: Thu, 10 Nov 2011 11:27:12 +0800	[thread overview]
Message-ID: <20111110032712.GA10116@localhost> (raw)
In-Reply-To: <4EBB3637.1070606@pulseforce.com>

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.560001] [drm:drm_edid_to_eld], ELD monitor TX-SR607
[    2.560005] [drm:drm_edid_to_eld], ELD size 13, SAD count 8
[    4.636651] [drm:drm_edid_to_eld], ELD monitor TX-SR607
[    4.636654] [drm:drm_edid_to_eld], ELD size 13, SAD count 8
[   10.486889] [drm:drm_edid_to_eld], ELD monitor TX-SR607
[   10.486893] [drm:drm_edid_to_eld], ELD size 13, SAD count 8
[   10.976211] [drm:drm_edid_to_eld], ELD monitor TX-SR607
[   10.976215] [drm:drm_edid_to_eld], ELD size 13, SAD count 8
[   12.034833] [drm:drm_edid_to_eld], ELD monitor TX-SR607
[   12.034836] [drm:drm_edid_to_eld], ELD size 13, SAD count 8
[   12.524615] [drm:drm_edid_to_eld], ELD monitor TX-SR607
[   12.524618] [drm:drm_edid_to_eld], ELD size 13, SAD count 8
[   13.015331] [drm:drm_edid_to_eld], ELD monitor TX-SR607
[   13.015334] [drm:drm_edid_to_eld], ELD size 13, SAD count 8
[   14.374778] [drm:drm_edid_to_eld], ELD monitor TX-SR607
[   14.374781] [drm:drm_edid_to_eld], ELD size 13, SAD count 8
[   16.373820] [drm:drm_edid_to_eld], ELD monitor TX-SR607
[   16.373824] [drm:drm_edid_to_eld], ELD size 13, SAD count 8
[   19.912594] [drm:drm_edid_to_eld], ELD monitor TX-SR607
[   19.912597] [drm:drm_edid_to_eld], ELD size 13, SAD count 8

- all *_write_eld() invocations are writing non-empty ELD to the same pipe A

[    2.674229] [drm:intel_hdmi_mode_set], Enabling HDMI audio on pipe A
[    2.674231] [drm:intel_write_eld], ELD on [CONNECTOR:14:HDMI-A-2], [ENCODER:13:TMDS-13]
[    2.674232] [drm:ironlake_write_eld], ELD on pipe A
[    2.674234] [drm:ironlake_write_eld], Audio directed to unknown port
[    2.674237] [drm:ironlake_write_eld], ELD size 13

[   13.651210] [drm:intel_hdmi_mode_set], Enabling HDMI audio on pipe A
[   13.651214] [drm:intel_write_eld], ELD on [CONNECTOR:14:HDMI-A-2], [ENCODER:13:TMDS-13]
[   13.651218] [drm:ironlake_write_eld], ELD on pipe A
[   13.651221] [drm:ironlake_write_eld], Audio directed to unknown port
[   13.651227] [drm:ironlake_write_eld], ELD size 13

- audio driver first got the good ELD:

[   10.268696] HDMI status: Pin=7 Presence_Detect=1 ELD_Valid=1
[   10.272189] HDMI: detected monitor TX-SR607 at connection type HDMI
[   10.272190] HDMI: available speakers: FL/FR LFE FC RL/RR RLC/RRC
[   10.272193] HDMI: supports coding type LPCM: channels = 2, rates = 44100 48000 88200 176400 192000 384000, bits = 16 20 24
[   10.272196] HDMI: supports coding type LPCM: channels = 8, rates = 44100 48000 88200 176400 192000 384000, bits = 16 20 24
[   10.272198] HDMI: supports coding type AC-3: channels = 8, rates = 44100 48000 88200, max bitrate = 640000
[   10.272199] HDMI: supports coding type DTS: channels = 8, rates = 48000 88200, max bitrate = 1536000
[   10.272201] HDMI: supports coding type DSD (One Bit Audio): channels = 6, rates = 48000
[   10.272202] HDMI: supports coding type E-AC-3/DD+ (Dolby Digital Plus): channels = 8, rates = 48000 88200
[   10.272204] HDMI: supports coding type DTS-HD: channels = 8, rates = 48000 88200 176400 192000 384000
[   10.272206] HDMI: supports coding type MLP (Dolby TrueHD): channels = 8, rates = 88200 192000

[   10.274995] HDMI hot plug event: Pin=7 Presence_Detect=1 ELD_Valid=1
[   10.275049] HDMI status: Pin=7 Presence_Detect=1 ELD_Valid=1
[   10.278541] HDMI: detected monitor TX-SR607 at connection type HDMI
[   10.278543] HDMI: available speakers: FL/FR LFE FC RL/RR RLC/RRC
[   10.278545] HDMI: supports coding type LPCM: channels = 2, rates = 44100 48000 88200 176400 192000 384000, bits = 16 20 24
[   10.278548] HDMI: supports coding type LPCM: channels = 8, rates = 44100 48000 88200 176400 192000 384000, bits = 16 20 24
[   10.278550] HDMI: supports coding type AC-3: channels = 8, rates = 44100 48000 88200, max bitrate = 640000
[   10.278551] HDMI: supports coding type DTS: channels = 8, rates = 48000 88200, max bitrate = 1536000
[   10.278553] HDMI: supports coding type DSD (One Bit Audio): channels = 6, rates = 48000
[   10.278554] HDMI: supports coding type E-AC-3/DD+ (Dolby Digital Plus): channels = 8, rates = 48000 88200
[   10.278556] HDMI: supports coding type DTS-HD: channels = 8, rates = 48000 88200 176400 192000 384000
[   10.278558] HDMI: supports coding type MLP (Dolby TrueHD): channels = 8, rates = 88200 192000

[   10.278612] HDMI hot plug event: Pin=7 Presence_Detect=1 ELD_Valid=1
[   10.278644] HDMI status: Pin=7 Presence_Detect=1 ELD_Valid=1
[   10.282143] HDMI: detected monitor TX-SR607 at connection type HDMI
[   10.282145] HDMI: available speakers: FL/FR LFE FC RL/RR RLC/RRC
[   10.282147] HDMI: supports coding type LPCM: channels = 2, rates = 44100 48000 88200 176400 192000 384000, bits = 16 20 24
[   10.282149] HDMI: supports coding type LPCM: channels = 8, rates = 44100 48000 88200 176400 192000 384000, bits = 16 20 24
[   10.282151] HDMI: supports coding type AC-3: channels = 8, rates = 44100 48000 88200, max bitrate = 640000
[   10.282152] HDMI: supports coding type DTS: channels = 8, rates = 48000 88200, max bitrate = 1536000
[   10.282154] HDMI: supports coding type DSD (One Bit Audio): channels = 6, rates = 48000
[   10.282155] HDMI: supports coding type E-AC-3/DD+ (Dolby Digital Plus): channels = 8, rates = 48000 88200
[   10.282157] HDMI: supports coding type DTS-HD: channels = 8, rates = 48000 88200 176400 192000 384000
[   10.282159] HDMI: supports coding type MLP (Dolby TrueHD): channels = 8, rates = 88200 192000

- audio driver then read 0 ELD, here we should see some error messages
  if you enabled CONFIG_SND_DEBUG.

[   13.651210] [drm:intel_hdmi_mode_set], Enabling HDMI audio on pipe A
[   13.651214] [drm:intel_write_eld], ELD on [CONNECTOR:14:HDMI-A-2], [ENCODER:13:TMDS-13]
[   13.651218] [drm:ironlake_write_eld], ELD on pipe A
[   13.651221] [drm:ironlake_write_eld], Audio directed to unknown port
[   13.651227] [drm:ironlake_write_eld], ELD size 13
[   13.651306] HDMI hot plug event: Pin=7 Presence_Detect=1 ELD_Valid=0
[   13.651342] HDMI status: Pin=7 Presence_Detect=1 ELD_Valid=1
[   13.654884] HDMI hot plug event: Pin=7 Presence_Detect=1 ELD_Valid=1
[   13.654926] HDMI status: Pin=7 Presence_Detect=1 ELD_Valid=1

Thanks,
Fengguang

  reply	other threads:[~2011-11-10  3:27 UTC|newest]

Thread overview: 66+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-02  8:14 [PATCH v4] drm/i915: pass ELD to HDMI/DP audio driver Wu Fengguang
2011-09-02  8:29 ` Wu Fengguang
2011-09-03 21:15 ` [PATCH v5] " Wu Fengguang
2011-09-04 10:57   ` James Cloos
2011-09-05  1:19     ` Wu Fengguang
2011-09-04 11:11   ` [Intel-gfx] " Paul Menzel
2011-09-05  1:06     ` Wu Fengguang
2011-09-04 12:08   ` Chris Wilson
2011-09-05  1:14     ` Wu Fengguang
2011-09-05 11:04       ` Chris Wilson
2011-09-05 12:31         ` Wu Fengguang
     [not found]           ` <4E64C41B.5090309@pulseforce.com>
     [not found]             ` <20110905124730.GB794@localhost>
     [not found]               ` <4EA82DBD.9020301@pulseforce.com>
2011-10-27 19:57                 ` Christopher White
2011-11-09  6:59                   ` Wu Fengguang
2011-11-09  9:00                     ` Christopher White
2011-11-09  9:30                       ` Christopher White
2011-11-09 13:01                         ` Wu Fengguang
     [not found]                 ` <4EA9B6EF.9040305@pulseforce.com>
2011-11-01 11:36                   ` Wu Fengguang
2011-11-01 17:00                     ` Christopher White
2011-11-02  1:45                       ` Wu Fengguang
2011-11-02  6:10                         ` Sander Jansen
2011-11-02  7:35                           ` Paul Menzel
2011-11-02 11:17                             ` Sander Jansen
2011-11-02 14:26                               ` Sander Jansen
2011-11-02  8:52                           ` Wu Fengguang
2011-11-02 17:41                             ` Keith Packard
2011-11-04  0:21                         ` Tony Olivo
2011-11-05  0:20                         ` Christopher White
2011-11-09 13:12                           ` Wu Fengguang
2011-11-10  2:25                             ` Christopher White
2011-11-10  3:27                               ` Wu Fengguang [this message]
2011-11-10  4:10                                 ` Christopher White
2011-11-10  7:06                                   ` Wu Fengguang
2011-11-10  7:33                                   ` Wu Fengguang
2011-11-10  7:55                                     ` Wu Fengguang
2011-11-10  8:50                                       ` Wu Fengguang
2011-11-10  8:55                                       ` Christopher White
2011-11-10 11:00                                         ` Christopher White
2011-11-10 11:22                                           ` Takashi Iwai
2011-11-10 11:50                                             ` Christopher White
2011-11-10 11:53                                               ` Takashi Iwai
2011-11-10 12:39                                                 ` Christopher White
2011-11-10 13:01                                                   ` Takashi Iwai
2011-11-10 12:56                                               ` Wu Fengguang
2011-11-10 13:01                                                 ` Christopher White
2011-11-10 13:17                                                   ` Wu Fengguang
2011-11-10 13:34                                                     ` Christopher White
2011-11-10 13:47                                                       ` Wu Fengguang
2011-11-10 14:12                                                         ` Wu Fengguang
2011-11-10 13:41                                                     ` Takashi Iwai
2011-11-10 13:51                                                       ` Wu Fengguang
2011-11-10 13:53                                                         ` Wu Fengguang
2011-11-10 14:28                                                         ` Takashi Iwai
2011-11-11  2:29                                                           ` Wu Fengguang
2011-11-11  7:40                                                             ` Takashi Iwai
2011-11-11  8:22                                                               ` Wu Fengguang
2011-11-11  8:49                                                                 ` Takashi Iwai
2011-11-11  9:24                                                                   ` Wu Fengguang
2011-11-11 10:17                                                                     ` Takashi Iwai
2011-11-11 11:12                                                                       ` Wu Fengguang
2011-11-11 11:23                                                                         ` Takashi Iwai
2011-11-11 11:32                                                                           ` Wu Fengguang
2011-11-12  2:27                                                                   ` Wu Fengguang
2011-11-14  9:45                                                                     ` Takashi Iwai
2011-11-14 13:25                                                                       ` Wu Fengguang
2011-11-15 17:18                                                                         ` Purushothaman, Vijay A
2011-11-10  6:59                               ` Wu Fengguang

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=20111110032712.GA10116@localhost \
    --to=fengguang.wu@intel.com \
    --cc=c.white@pulseforce.com \
    --cc=contractfrombelow@gmail.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=pierre-louis.bossart@intel.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