From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933343AbcASW0q (ORCPT ); Tue, 19 Jan 2016 17:26:46 -0500 Received: from mail-qg0-f44.google.com ([209.85.192.44]:35760 "EHLO mail-qg0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933301AbcASW02 (ORCPT ); Tue, 19 Jan 2016 17:26:28 -0500 Date: Tue, 19 Jan 2016 17:26:22 -0500 From: Nick Bowler To: linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org Cc: Sonika Jindal , Daniel Vetter Subject: PROBLEM: Intel HDMI output busticated on 4.4 (regression) Message-ID: <20160119222622.GA16397@draconx.ca> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, Upgrading from 4.3 to 4.4 breaks my HDMI output on my G45 machine. As soon as the intel driver is loaded, the monitor shuts off (standby mode). Inspecting /sys/class/drm/card0-HDMI-A-1/status reports "disconnected". When it is working, this attribute says "connected". There is nothing unusual printed to dmesg. Bisection pinpoints the following: 237ed86c693d8a8e4db476976aeb30df4deac74b is the first bad commit commit 237ed86c693d8a8e4db476976aeb30df4deac74b Author: Sonika Jindal Date: Tue Sep 15 09:44:20 2015 +0530 drm/i915: Check live status before reading edid [...] Signed-off-by: Shashank Sharma Signed-off-by: Sonika Jindal Reviewed-by: Rodrigo Vivi Signed-off-by: Daniel Vetter The commit does not revert cleanly, but this patch resolves the issue: diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/drivers/gpu/drm/i915/intel_hdmi.c index e6c035b0fc1c..8cefb9105f26 100644 --- a/drivers/gpu/drm/i915/intel_hdmi.c +++ b/drivers/gpu/drm/i915/intel_hdmi.c @@ -1393,7 +1393,7 @@ intel_hdmi_detect(struct drm_connector *connector, bool force) intel_hdmi_unset_edid(connector); - if (intel_hdmi_set_edid(connector, live_status)) { + if (intel_hdmi_set_edid(connector, true)) { struct intel_hdmi *intel_hdmi = intel_attached_hdmi(connector); hdmi_to_dig_port(intel_hdmi)->base.type = INTEL_OUTPUT_HDMI; Let me know if you need any more info. Thanks, Nick