Re: [PATCH 5/5] drm/amd/display: Allow HDMI capabilities over DVI connector (DC)

[Jeffrey Smith <whydoubt@xxxxxxxxx>]

This did work for me, including stereo 3D, so that is rather disappointing.

(DVI output of the card -> DVI/HDMI adapter -> HDMI Cable -> HDMI TV input)

Is it that HDMI signaling passing through a DVI connector is dis-allowed

on the whole, or is there something more subtle that I am missing?

Would it be allowed if only accessible through a kernel parameter?

On Wed, Mar 1, 2017 at 5:11 PM, Harry Wentland <harry.wentland@xxxxxxx> wrote:

On 2017-02-24 07:14 PM, Jeff Smith wrote:

Signed-off-by: Jeff Smith <whydoubt@xxxxxxxxx>
---
 drivers/gpu/drm/amd/display/dc/core/dc_link.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link.c b/drivers/gpu/drm/amd/display/dc/core/dc_link.c
index 2b92939..168f260 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc_link.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc_link.c
@@ -706,8 +706,13 @@ bool dc_link_detect(const struct dc_link *dc_link, bool boot)

                /* HDMI-DVI Dongle */
                if (dc_sink->sink_signal == SIGNAL_TYPE_HDMI_TYPE_A &&
-                               !dc_sink->edid_caps.edid_hdmi)
+                               !dc_sink->edid_caps.edid_hdmi) {
                        dc_sink->sink_signal = SIGNAL_TYPE_DVI_SINGLE_LINK;
+               } else if ((dc_sink->sink_signal == SIGNAL_TYPE_DVI_SINGLE_LINK ||
+                               dc_sink->sink_signal == SIGNAL_TYPE_DVI_DUAL_LINK) &&
+                               dc_sink->edid_caps.edid_hdmi) {
+                       dc_sink->sink_signal = SIGNAL_TYPE_HDMI_TYPE_A;
+               }

This won't pass HDMI compliance.

Harry

                /* Connectivity log: detection */
                for (i = 0; i < sink->public.dc_edid.length / EDID_BLOCK_SIZE; i++) {

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel