diff mbox

[3/5] igt/kms_psr_sink_crc: Add debug for CRC

Message ID 1409869371-7558-3-git-send-email-rodrigo.vivi@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Rodrigo Vivi Sept. 4, 2014, 10:22 p.m. UTC 
This helps to debug and check if the CRC you are reading correspond
what you are seeing on the screen


Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 tests/kms_psr_sink_crc.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Daniel Vetter Sept. 5, 2014, 7:40 a.m. UTC | #1 
On Thu, Sep 04, 2014 at 06:22:49PM -0400, Rodrigo Vivi wrote:
> This helps to debug and check if the CRC you are reading correspond
> what you are seeing on the screen
> 
> 
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> ---
>  tests/kms_psr_sink_crc.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c
> index bbe18f0..51e54a7 100644
> --- a/tests/kms_psr_sink_crc.c
> +++ b/tests/kms_psr_sink_crc.c
> @@ -328,6 +328,8 @@ static void get_sink_crc(data_t *data, char *crc) {
>  
>  	fclose(file);
>  
> +	igt_debug_wait(1, "%s\n", crc);

I think igt_wait_keypress here is the better option. Together with
igt_check_boolean_env_var it should be fairly pretty. I'll send out a
quick prep patch for this.
-Daniel

> +
>  	/* The important value was already taken.
>  	 * Now give a time for human eyes
>  	 */
> -- 
> 1.9.3
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
diff mbox

Patch

diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c
index bbe18f0..51e54a7 100644
--- a/tests/kms_psr_sink_crc.c
+++ b/tests/kms_psr_sink_crc.c
@@ -328,6 +328,8 @@  static void get_sink_crc(data_t *data, char *crc) {
 
 	fclose(file);
 
+	igt_debug_wait(1, "%s\n", crc);
+
 	/* The important value was already taken.
 	 * Now give a time for human eyes
 	 */