Re: Re: [PATCH 1/2] drm/radeon/kms: use drm_detect_hdmi_monitor for picking encoder mode

On -10.01.-28163 20:59, Rafał Miłecki wrote:

2012/1/20alexdeucher@gmail.com:

...

From: Alex Deucheralexander.deucher@amd.com

We were previously just checking for audio.

So... Does every hdmi_monitor supports audio? Or should it rather be if (drm_detect_monitor_audio(radeon_connector->edid)&& drm_detect_hdmi_monitor(radeon_connector->edid)) ?

Sorry, I'm not familiar enough with EDID.

IIRC HDMI is more than just Audio, so it is possible for a TV to announce support for HDMI but not Audio.

So setting the decoder moder should depend on whether or not a TV announces HDMI support, but actually sending audio should depend on the supported audio capabilities.

Christian.