Re: [PATCH] tests: debugfs_psr_status

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

 



On Tue, Oct 1, 2013 at 4:17 PM, Daniel Vetter <daniel@xxxxxxxx> wrote:
> 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 ...

in any case source doesn't match all conditions to get PSR enabled,
igt test will skip on this:
 if (!source)
          igt_skip("This Hardware does not support or isn't ready for PSR\n");

> -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



-- 
Rodrigo Vivi
Blog: http://blog.vivi.eng.br
_______________________________________________
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