[Intel-gfx] [PATCH 1/3] intel_audio_dump/hsw: remove misuse of PCH transcoder configuration register

mengdong.lin at intel.com mengdong.lin at intel.com
Mon Sep 9 21:38:20 CEST 2013


From: Mengdong Lin <mengdong.lin at intel.com>

The PCH transcoder config register (PCH_TRANS_CONF, 0xf0008) is not the
correct config register for transcoder A, B or C. This register is in
PCH and for CRT display, nothing to do with display audio.

So This patch removes misuse of it as config register for transcoder A/B/C.

Signed-off-by: Mengdong Lin <mengdong.lin at intel.com>
Reviewed-by: Haihao Xiang <haihao.xiang at intel.com>

diff --git a/tools/intel_audio_dump.c b/tools/intel_audio_dump.c
index 8b9da30..cb2641f 100644
--- a/tools/intel_audio_dump.c
+++ b/tools/intel_audio_dump.c
@@ -1425,11 +1425,6 @@ static void dump_cpt(void)
 #define DP_TP_ST_D          0x64344
 #define DP_TP_ST_E          0x64444
 
-/* Transcoder configuration */
-#define TRANS_CONF_A	    0xF0008
-#define TRANS_CONF_B	    0xF1008
-#define TRANS_CONF_C	    0xF2008
-
 /* DDI Buffer Control */
 #define DDI_BUF_CTL_A	      0x64000
 #define DDI_BUF_CTL_B	      0x64100
@@ -1553,11 +1548,6 @@ static void dump_hsw(void)
     dump_reg(DP_TP_ST_D,		"DisplayPort Transport D Status");
     dump_reg(DP_TP_ST_E,		"DisplayPort Transport E Status");
 
-    /* HSW Transcoder A configuration */
-    dump_reg(TRANS_CONF_A,		"Transcoder A Configuration");
-    dump_reg(TRANS_CONF_B,		"Transcoder B Configuration");
-    dump_reg(TRANS_CONF_C,		"Transcoder C Configuration");
-
     /* HSW North Display Audio */
     dump_reg(AUD_CONFIG_A,		"Audio Configuration - Transcoder A");
     dump_reg(AUD_CONFIG_B,		"Audio Configuration - Transcoder B");
-- 
1.8.1.2




More information about the Intel-gfx mailing list