I found my check for audio capability within CEA extension might
has some conflict with Adam's patch to add more detailed mode from
CEA ext block. But I haven't found Adam's patch in next tree, so I
just kept my current change.
This fixed incorrect probe for DP monitor's audio. Fengguang helped
To help to determine if digital display port needs to enable
audio output or not. This one adds a helper to get monitor's
audio capability via EDID CEA extension block.
Tested-by: Wu Fengguang fengguang...@intel.com
Signed-off-by: Zhenyu Wang zhen...@linux.intel.com
---
This will turn on DP audio output by checking monitor's audio
capability.
Tested-by: Wu Fengguang fengguang...@intel.com
Signed-off-by: Zhenyu Wang zhen...@linux.intel.com
---
drivers/gpu/drm/i915/intel_dp.c | 61 +++
1 files changed, 36 insertions(+), 25
Rely on monitor's audio capability to turn on audio output for HDMI.
Tested-by: Wu Fengguang fengguang...@intel.com
Signed-off-by: Zhenyu Wang zhen...@linux.intel.com
---
drivers/gpu/drm/i915/intel_hdmi.c | 12
1 files changed, 8 insertions(+), 4 deletions(-)
diff --git
[Lets see if I have a working vpn connection today...]
On Sun, 19 Sep 2010 14:52:06 +0800, Zhenyu Wang zhen...@linux.intel.com wrote:
To help to determine if digital display port needs to enable
audio output or not. This one adds a helper to get monitor's
audio capability via EDID CEA
On Sun, 19 Sep 2010 14:52:07 +0800, Zhenyu Wang zhen...@linux.intel.com wrote:
This will turn on DP audio output by checking monitor's audio
capability.
Tested-by: Wu Fengguang fengguang...@intel.com
Signed-off-by: Zhenyu Wang zhen...@linux.intel.com
---
drivers/gpu/drm/i915/intel_dp.c |
On 2010.09.19 08:38:07 +0100, Chris Wilson wrote:
This should be cc'ed for Adam Jackson's attention as well.
yep, I think I cc-ed ajax in git-send-mail command line...I'll amend the commit.
+bool drm_detect_monitor_audio(struct edid *edid)
drm_edid_has_monitor_audio()? That is a little
On Sun, 19 Sep 2010 14:52:09 +0800, Zhenyu Wang zhen...@linux.intel.com wrote:
Two reasons to add this param, one is for some AV device requiring
HDMI input for audio, but some device might not have sane EDID info
to enable audio on HDMI port. Another is for testing purpose.
I was thinking
On Sun, 19 Sep 2010 16:22:06 +0800, Zhenyu Wang zhen...@linux.intel.com wrote:
On 2010.09.19 09:14:53 +0100, Chris Wilson wrote:
On Sun, 19 Sep 2010 14:52:09 +0800, Zhenyu Wang zhen...@linux.intel.com
wrote:
Two reasons to add this param, one is for some AV device requiring
HDMI input