intel_audio_dump/hsw: remove misuse of PCH transcoder configuration register
authorMengdong Lin <mengdong.lin@intel.com>
Mon, 9 Sep 2013 19:38:20 +0000 (15:38 -0400)
committerBen Widawsky <benjamin.widawsky@intel.com>
Mon, 16 Sep 2013 21:20:28 +0000 (14:20 -0700)
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@intel.com>
Reviewed-by: Haihao Xiang <haihao.xiang@intel.com>
Signed-off-by: Ben Widawsky <benjamin.widawsky@intel.com>
tools/intel_audio_dump.c

index 8b9da30..cb2641f 100644 (file)
@@ -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");