[PATCH] tools/intel_audio_dump: add details dump for Cherryview

[PATCH] tools/intel_audio_dump: add details dump for Cherryview

[libin.yang]

From: Libin Yang <libin.yang@intel.com>

This patch adds the details dump for audio registers of Cherryview.

Signed-off-by: Libin Yang <libin.yang@intel.com>
---
 tools/intel_audio_dump.c | 50 ++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 50 insertions(+)

diff --git a/tools/intel_audio_dump.c b/tools/intel_audio_dump.c
index b673288..945b136 100644
--- a/tools/intel_audio_dump.c
+++ b/tools/intel_audio_dump.c
@@ -2349,6 +2349,56 @@ static void dump_braswell(void)
 	dump_aud_reg(AUD_TCC_M_CTS,            "Audio M CTS Read Back Transcoder C");
 
 	printf("\n");
+
+	printf("\nDetails:\n\n");
+
+	dump_aud_vendor_device_id();
+	dump_aud_revision_id();
+
+	dump_hdmi_port_ctrl(PORT_B);
+	dump_hdmi_port_ctrl(PORT_C);
+	if (!IS_VALLEYVIEW(devid))
+		dump_hdmi_port_ctrl(PORT_D);
+
+	dump_dp_port_ctrl(PORT_B);
+	dump_dp_port_ctrl(PORT_C);
+	if (!IS_VALLEYVIEW(devid))
+		dump_dp_port_ctrl(PORT_D);
+
+	dump_aud_config(PIPE_A);
+	dump_aud_config(PIPE_B);
+
+	dump_aud_m_cts_enable(PIPE_A);
+	dump_aud_m_cts_enable(PIPE_B);
+
+	dump_aud_misc_control(PIPE_A);
+	dump_aud_misc_control(PIPE_B);
+
+	dump_aud_power_state();
+	dump_aud_port_en_hd_cfg();
+
+	dump_aud_dig_cnvt(PIPE_A);
+	dump_aud_dig_cnvt(PIPE_B);
+
+	dump_aud_out_ch_str();
+
+	dump_aud_str_desc(PIPE_A);
+	dump_aud_str_desc(PIPE_B);
+
+	dump_aud_connect_list();
+	dump_aud_connect_select();
+
+	dump_aud_ctrl_state(PIPE_A);
+	dump_aud_ctrl_state(PIPE_B);
+	dump_aud_ctrl_state2();
+
+	dump_aud_hdmi_status();
+
+	dump_aud_edid_data(PIPE_A);
+	dump_aud_edid_data(PIPE_B);
+
+	dump_aud_infoframe(PIPE_A);
+	dump_aud_infoframe(PIPE_B);
 }
 
 int main(int argc, char **argv)
-- 
1.9.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH] tools/intel_audio_dump: add details dump for Cherryview

[Yang, Libin]

Any comments?

Regards,
Libin


> -----Original Message-----
> From: Yang, Libin
> Sent: Thursday, January 15, 2015 9:21 AM
> To: intel-gfx@lists.freedesktop.org
> Cc: Yang, Libin
> Subject: [PATCH] tools/intel_audio_dump: add details dump for
> Cherryview
> 
> From: Libin Yang <libin.yang@intel.com>
> 
> This patch adds the details dump for audio registers of Cherryview.
> 
> Signed-off-by: Libin Yang <libin.yang@intel.com>
> ---
>  tools/intel_audio_dump.c | 50
> ++++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 50 insertions(+)
> 
> diff --git a/tools/intel_audio_dump.c b/tools/intel_audio_dump.c
> index b673288..945b136 100644
> --- a/tools/intel_audio_dump.c
> +++ b/tools/intel_audio_dump.c
> @@ -2349,6 +2349,56 @@ static void dump_braswell(void)
>  	dump_aud_reg(AUD_TCC_M_CTS,            "Audio M CTS Read
> Back Transcoder C");
> 
>  	printf("\n");
> +
> +	printf("\nDetails:\n\n");
> +
> +	dump_aud_vendor_device_id();
> +	dump_aud_revision_id();
> +
> +	dump_hdmi_port_ctrl(PORT_B);
> +	dump_hdmi_port_ctrl(PORT_C);
> +	if (!IS_VALLEYVIEW(devid))
> +		dump_hdmi_port_ctrl(PORT_D);
> +
> +	dump_dp_port_ctrl(PORT_B);
> +	dump_dp_port_ctrl(PORT_C);
> +	if (!IS_VALLEYVIEW(devid))
> +		dump_dp_port_ctrl(PORT_D);
> +
> +	dump_aud_config(PIPE_A);
> +	dump_aud_config(PIPE_B);
> +
> +	dump_aud_m_cts_enable(PIPE_A);
> +	dump_aud_m_cts_enable(PIPE_B);
> +
> +	dump_aud_misc_control(PIPE_A);
> +	dump_aud_misc_control(PIPE_B);
> +
> +	dump_aud_power_state();
> +	dump_aud_port_en_hd_cfg();
> +
> +	dump_aud_dig_cnvt(PIPE_A);
> +	dump_aud_dig_cnvt(PIPE_B);
> +
> +	dump_aud_out_ch_str();
> +
> +	dump_aud_str_desc(PIPE_A);
> +	dump_aud_str_desc(PIPE_B);
> +
> +	dump_aud_connect_list();
> +	dump_aud_connect_select();
> +
> +	dump_aud_ctrl_state(PIPE_A);
> +	dump_aud_ctrl_state(PIPE_B);
> +	dump_aud_ctrl_state2();
> +
> +	dump_aud_hdmi_status();
> +
> +	dump_aud_edid_data(PIPE_A);
> +	dump_aud_edid_data(PIPE_B);
> +
> +	dump_aud_infoframe(PIPE_A);
> +	dump_aud_infoframe(PIPE_B);
>  }
> 
>  int main(int argc, char **argv)
> --
> 1.9.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH] tools/intel_audio_dump: add details dump for Cherryview

[Zhenyu Wang]

[-- Attachment #1.1: Type: text/plain, Size: 285 bytes --]

On 2015.01.26 01:15:36 +0000, Yang, Libin wrote:
> Any comments?
> 

Looks fine to me. Reviewed-by: Zhenyu Wang <zhenyuw@linux.intel.com>

I will help to push this later.

-- 
Open Source Technology Center, Intel ltd.

$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx