From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wu Fengguang Subject: [PATCH 1/4 v2] drm/i915: fix ELD writing for SandyBridge Date: Tue, 29 Nov 2011 19:53:45 +0800 Message-ID: <20111129115345.GB11973@localhost> References: <20111128073337.815008177@intel.com> <20111128074029.504309527@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20111128074029.504309527@intel.com> Sender: linux-kernel-owner@vger.kernel.org To: Keith Packard Cc: Takashi Iwai , Sander Jansen , Christopher White , "intel-gfx@lists.freedesktop.org" , "Bossart, Pierre-louis" , "Wang, Zhenyu Z" , Jeremy Bush , LKML List-Id: intel-gfx@lists.freedesktop.org SandyBridge should be using the same register addresses as IvyBridge. Signed-off-by: Wu Fengguang --- drivers/gpu/drm/i915/intel_display.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) Sorry, this moves some necessary changes from patch 2/4 to here. --- linux.orig/drivers/gpu/drm/i915/intel_display.c 2011-11-28 15:33:04.000000000 +0800 +++ linux/drivers/gpu/drm/i915/intel_display.c 2011-11-29 19:51:28.000000000 +0800 @@ -5857,14 +5857,14 @@ static void ironlake_write_eld(struct dr int aud_cntl_st; int aud_cntrl_st2; - if (IS_IVYBRIDGE(connector->dev)) { - hdmiw_hdmiedid = GEN7_HDMIW_HDMIEDID_A; - aud_cntl_st = GEN7_AUD_CNTRL_ST_A; - aud_cntrl_st2 = GEN7_AUD_CNTRL_ST2; - } else { + if (HAS_PCH_IBX(connector->dev)) { hdmiw_hdmiedid = GEN5_HDMIW_HDMIEDID_A; aud_cntl_st = GEN5_AUD_CNTL_ST_A; aud_cntrl_st2 = GEN5_AUD_CNTL_ST2; + } else { + hdmiw_hdmiedid = GEN7_HDMIW_HDMIEDID_A; + aud_cntl_st = GEN7_AUD_CNTRL_ST_A; + aud_cntrl_st2 = GEN7_AUD_CNTRL_ST2; } i = to_intel_crtc(crtc)->pipe;