Re: [PATCH] tests: debugfs_psr_status

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Oct 01, 2013 at 03:26:43PM -0300, Rodrigo Vivi wrote:
> Check on debugfs if PSR is supported by panel and matching all conditions in
> hardware. In this case PSR must be enabled and performance counting increasing
> 
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>

Does this work if the console is blanked or someone just never enabled the
screen? I.e. I think we need the usual boilerplate here to make sure that
the eDP port is lit up on pipe A and dpms is ON ...
-Daniel

> ---
>  tests/Makefile.am          |  1 +
>  tests/debugfs_psr_status.c | 79 ++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 80 insertions(+)
>  create mode 100644 tests/debugfs_psr_status.c
> 
> diff --git a/tests/Makefile.am b/tests/Makefile.am
> index 99d98d6..82bc936 100644
> --- a/tests/Makefile.am
> +++ b/tests/Makefile.am
> @@ -116,6 +116,7 @@ TESTS_progs = \
>  	prime_udl \
>  	sysfs_rc6_residency \
>  	sysfs_rps \
> +	debugfs_psr_status \
>  	$(NULL)
>  
>  # IMPORTANT: The ZZ_ tests need to be run last!
> diff --git a/tests/debugfs_psr_status.c b/tests/debugfs_psr_status.c
> new file mode 100644
> index 0000000..44b621f
> --- /dev/null
> +++ b/tests/debugfs_psr_status.c
> @@ -0,0 +1,79 @@
> +/*
> + * Copyright (c) 2013 Intel Corporation
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice (including the next
> + * paragraph) shall be included in all copies or substantial portions of the
> + * Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> + * IN THE SOFTWARE.
> + *
> + * Authors:
> + *    Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
> + */
> +
> +#define _GNU_SOURCE
> +#include <stdio.h>
> +#include <stdlib.h>
> +#include <string.h>
> +#include <unistd.h>
> +#include "drmtest.h"
> +
> +int main(int argc, char *argv[])
> +{
> +	FILE *file;
> +	int ret, perf;
> +	int device = drm_get_card();
> +	char *str;
> +	bool sink, source, enabled;
> +
> +	igt_skip_on_simulation();
> +
> +	ret = asprintf(&str, "/sys/kernel/debug/dri/%d/i915_edp_psr_status", device);
> +	igt_assert(ret != -1);
> +
> +	file = fopen(str, "r");
> +	igt_require(file);
> +
> +	ret = fscanf(file, "Sink_Support: %s\n", str);
> +	if (ret == 0)
> +	    igt_skip("i915_edp_psr_status format not supported by this test case\n");
> +	sink = strcmp(str, "yes") == 0;
> +	ret = fscanf(file, "Source_OK: %s\n", str);
> +	igt_assert(ret != 0);
> +	source = strcmp(str, "yes") == 0;
> +	ret = fscanf(file, "Enabled: %s\n", str);
> +	igt_assert(ret != 0);
> +	enabled = strcmp(str, "yes") == 0;
> +	ret = fscanf(file, "Performance_Counter: %i", &perf);
> +
> +	if (!sink)
> +	    igt_skip("This panel does not support PSR.\n");
> +
> +	if (!source)
> +	    igt_skip("This Hardware does not support or isn't ready for PSR\n");
> +
> +	if (!enabled) {
> +	    fprintf(stderr, "PSR should be enabled\n");
> +	    igt_fail(1);
> +	}
> +
> +	if (perf == 0) {
> +	    fprintf(stderr, "PSR state never achieved\n");
> +	    igt_fail(1);
> +	}
> +
> +	igt_success();
> +}
> -- 
> 1.7.11.7
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux