On Mon, Sep 18, 2017 at 12:36 PM, Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> wrote:
> On Sat, Sep 16, 2017 at 12:00:08AM +0000, Radhakrishna Sripada wrote:
>> Platforms do not support psr and drrs simultaneously.
>> Adding a subtest to make the check.
>
> I wasn't confident this was the right place for that, but
> this is definitely the easisest and cleanest way.
>
>>
>> Cc: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
>> Cc: Daniel Vetter <daniel.vetter@xxxxxxxx>
>> Signed-off-by: Radhakrishna Sripada <radhakrishna.sripada@xxxxxxxxx>
>
> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
merged to master. thanks for the patch,.
>
>
>> ---
>> tests/kms_psr_sink_crc.c | 14 ++++++++++++++
>> 1 file changed, 14 insertions(+)
>>
>> diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c
>> index bd3fa5e94d85..1c25f2c81a34 100644
>> --- a/tests/kms_psr_sink_crc.c
>> +++ b/tests/kms_psr_sink_crc.c
>> @@ -284,6 +284,15 @@ static void assert_or_manual(bool condition, const char *expected)
>> igt_assert(igt_interactive_debug || condition);
>> }
>>
>> +static bool drrs_disabled(data_t *data)
>> +{
>> + char buf[512];
>> +
>> + igt_debugfs_read(data->drm_fd, "i915_drrs_status", buf);
>> +
>> + return strstr(buf, "DRRS Support: No\n");
>> +}
>> +
>> static void run_test(data_t *data)
>> {
>> uint32_t handle = data->fb_white.gem_handle;
>> @@ -524,6 +533,11 @@ int main(int argc, char *argv[])
>> igt_assert(wait_psr_entry(&data));
>> }
>>
>> + igt_subtest("psr_drrs") {
>> + setup_test_plane(&data);
>> + igt_assert(drrs_disabled(&data));
>> + }
>> +
>> for (op = PAGE_FLIP; op <= RENDER; op++) {
>> igt_subtest_f("primary_%s", op_str(op)) {
>> data.test_plane = DRM_PLANE_TYPE_PRIMARY;
>> --
>> 2.9.3
>>
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Rodrigo Vivi
Blog: http://blog.vivi.eng.br
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
