From patchwork Fri May 5 21:04:46 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: jim.bride@linux.intel.com X-Patchwork-Id: 9714387 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 316FC6034B for ; Fri, 5 May 2017 21:06:57 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 25021285FD for ; Fri, 5 May 2017 21:06:57 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 19F2728688; Fri, 5 May 2017 21:06:57 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=2.0 tests=BAYES_00, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 622D5285FD for ; Fri, 5 May 2017 21:06:56 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id A94886E71C; Fri, 5 May 2017 21:06:55 +0000 (UTC) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by gabe.freedesktop.org (Postfix) with ESMTPS id DDD0B6E6DF for ; Fri, 5 May 2017 21:06:54 +0000 (UTC) Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 05 May 2017 14:06:54 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.38,294,1491289200"; d="scan'208";a="96196943" Received: from shiv.jf.intel.com ([10.54.75.141]) by orsmga005.jf.intel.com with ESMTP; 05 May 2017 14:06:54 -0700 From: Jim Bride To: intel-gfx@lists.freedesktop.org Date: Fri, 5 May 2017 14:04:46 -0700 Message-Id: <1494018289-30259-3-git-send-email-jim.bride@linux.intel.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1494018289-30259-1-git-send-email-jim.bride@linux.intel.com> References: <1494018289-30259-1-git-send-email-jim.bride@linux.intel.com> MIME-Version: 1.0 Cc: Paulo Zanoni , Rodrigo Vivi Subject: [Intel-gfx] [PATCH IGT 2/5] lib: Add PSR utility functions to igt library. X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" X-Virus-Scanned: ClamAV using ClamSMTP Factor out some code that was replicated in three test utilities into some new IGT library functions so that we are checking PSR status in a consistent fashion across all of our PSR tests. Cc: Rodrigo Vivi Cc: Paulo Zanoni Signed-off-by: Jim Bride --- lib/Makefile.sources | 2 + lib/igt.h | 1 + lib/igt_psr.c | 195 +++++++++++++++++++++++++++++++++++++++++++++++++++ lib/igt_psr.h | 40 +++++++++++ 4 files changed, 238 insertions(+) create mode 100644 lib/igt_psr.c create mode 100644 lib/igt_psr.h diff --git a/lib/Makefile.sources b/lib/Makefile.sources index 6348487..0a8835b 100644 --- a/lib/Makefile.sources +++ b/lib/Makefile.sources @@ -83,6 +83,8 @@ lib_source_list = \ uwildmat/uwildmat.c \ igt_kmod.c \ igt_kmod.h \ + igt_psr.c \ + igt_psr.h \ $(NULL) if HAVE_CHAMELIUM diff --git a/lib/igt.h b/lib/igt.h index a97923e..7f52d6c 100644 --- a/lib/igt.h +++ b/lib/igt.h @@ -37,6 +37,7 @@ #include "igt_gt.h" #include "igt_kms.h" #include "igt_pm.h" +#include "igt_psr.h" #include "igt_stats.h" #include "igt_chamelium.h" #include "instdone.h" diff --git a/lib/igt_psr.c b/lib/igt_psr.c new file mode 100644 index 0000000..cfbd139 --- /dev/null +++ b/lib/igt_psr.c @@ -0,0 +1,195 @@ +/* + * Copyright © 2017 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. + */ + +#include "igt.h" +#include +#include +#include +#include + +/** + * SECTION:igt_psr + * @short_description: Panel Self Refresh helpers + * @title: Panel Self Refresh + * @include: igt.h + * + * This library provides various helpers to enable Panel Self Refresh, + * as well as to check the state of PSR on the system (enabled vs. + * disabled, active vs. inactive) or to wait for PSR to be active + * or inactive. + */ + +/** + * igt_psr_source_support: + * + * Returns true if the source supports PSR. + */ +bool igt_psr_source_support(int fd) +{ + char buf[512]; + + igt_debugfs_read(fd, "i915_edp_psr_status", buf); + + return strstr(buf, "Source_OK: yes\n"); +} + + +/** + * igt_psr_sink_support: + * + * Returns true if the current eDP panel supports PSR. + */ +bool igt_psr_sink_support(int fd) +{ + char buf[256]; + + igt_debugfs_read(fd, "i915_edp_psr_status", buf); + return strstr(buf, "Sink_Support: yes\n"); +} + +/** + * igt_psr_possible: + * + * Returns true if both the source and sink support PSR. + */ +bool igt_psr_possible(int fd) +{ + char buf[512]; + + igt_debugfs_read(fd, "i915_edp_psr_status", buf); + + return igt_psr_source_support(fd) && igt_psr_sink_support(fd); +} + +/** + * igt_psr_active: + * + * Returns true if PSR is active on the panel. + */ +bool igt_psr_active(int fd) +{ + char buf[512]; + bool actret = false; + bool hwactret = false; + + igt_debugfs_read(fd, "i915_edp_psr_status", buf); + hwactret = (strstr(buf, "HW Enabled & Active bit: yes\n") != NULL); + actret = (strstr(buf, "Active: yes\n") != NULL); + igt_debug("hwactret: %s actret: %s\n", hwactret ? "true" : "false", + actret ? "true" : "false"); + return hwactret && actret; +} + +/** + * igt_psr_await_status: + * @active: A boolean that causes the function to wait for PSR to activate + * if set to true, or to wait for PSR to deactivate if false. + * + * Returns true if the requested condition is met. + */ +bool igt_psr_await_status(int fd, bool active) +{ + const int timeout = 10; + int count = 0; + while (count < timeout) { + if (igt_psr_active(fd) == active) { + igt_debug("PSR %s after %d seconds.\n", + active ? "Active" : "Inactive", count); + return true; + } + count++; + sleep(1); + } + return false; +} + +/** + * igt_psr_enabled: + * + * Returns true if the PSR feature is enabled. + */ +bool igt_psr_enabled(int fd) +{ + char buf[256]; + + igt_debugfs_read(fd, "i915_edp_psr_status", buf); + return strstr(buf, "Enabled: yes\n"); +} + +/** + * igt_psr_print_status: + * + * Dumps the contents of i915_edp_psr_status from debugfs. + */ +void igt_psr_print_status(int fd) +{ + char buf[256]; + + igt_debugfs_read(fd, "i915_edp_psr_status", buf); + igt_info("PSR status:\n%s\n", buf); +} + +/** + * igt_psr_valid_connector: + * @connector: a drmModeConnector pointer to check + * + * Returns true if connector is an eDP connector. + */ +bool igt_psr_valid_connector(drmModeConnectorPtr connector) +{ + return (connector->connector_type == DRM_MODE_CONNECTOR_eDP); +} + +/** + * igt_psr_find_good_mode + * @connector: a drmModeConnector pointer to find the mode on + * @mode: a drmModeModePtr pointer that is set to the matching mode + * + * Returns true (and populates *mode with the match) if a valid + * PSR mdoe is found, and false otherwise. + */ +bool igt_psr_find_good_mode(drmModeConnectorPtr connector, + drmModeModeInfoPtr *mode) +{ + int i; + + if (!connector->count_modes) { + igt_warn("no modes for connector %d.\n", + connector->connector_id); + return false; + } else { + igt_debug("Connector has %d modes.\n", connector->count_modes); + } + + for (i = 0; i < connector->count_modes; i++) { + if ((connector->modes[i].vtotal - + connector->modes[i].vdisplay) >= 36) { + igt_debug("Mode %d good for PSR.\n", i); + *mode = &(connector->modes[i]); + return true; + } else { + igt_debug("Throwing out mode %d\n", i); + } + } + return false; +} diff --git a/lib/igt_psr.h b/lib/igt_psr.h new file mode 100644 index 0000000..6f5c8ae --- /dev/null +++ b/lib/igt_psr.h @@ -0,0 +1,40 @@ +/* + * Copyright © 2017 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. + */ + +#ifndef IGT_PSR_H +#define IGT_PSR_H + +#define igt_psr_enable() igt_set_module_param_int("enable_psr", 1) +#define igt_psr_disable() igt_set_module_param_int("enable_psr", 0) + +bool igt_psr_sink_support(int fd); +bool igt_psr_source_support(int fd); +bool igt_psr_possible(int fd); +bool igt_psr_active(int fd); +bool igt_psr_await_status(int fd, bool active); +bool igt_psr_enabled(int fd); +void igt_psr_print_status(int fd); +bool igt_psr_valid_connector(drmModeConnectorPtr connector); +bool igt_psr_find_good_mode(drmModeConnectorPtr connector, + drmModeModeInfoPtr *mode); +#endif /* IGT_PSR_H */