From: "José Roberto de Souza" <jose.souza@intel.com>
To: igt-dev@lists.freedesktop.org
Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Subject: [igt-dev] [PATCH i-g-t 8/8] tests/fbcon_fbt: Do not print debugfs of features
Date: Mon, 7 Jan 2019 11:10:56 -0800 [thread overview]
Message-ID: <20190107191056.22759-8-jose.souza@intel.com> (raw)
In-Reply-To: <20190107191056.22759-1-jose.souza@intel.com>
Print the debugfs of the features is only helpful for debug not for
the automated runs, so lets remove it and if needed for debug the one
debugging it can add it back in his/her machine.
Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: José Roberto de Souza <jose.souza@intel.com>
---
tests/kms_fbcon_fbt.c | 27 ++-------------------------
1 file changed, 2 insertions(+), 25 deletions(-)
diff --git a/tests/kms_fbcon_fbt.c b/tests/kms_fbcon_fbt.c
index cdba1c6e..7d1cdd0e 100644
--- a/tests/kms_fbcon_fbt.c
+++ b/tests/kms_fbcon_fbt.c
@@ -105,15 +105,6 @@ static bool connector_can_fbc(drmModeConnectorPtr connector)
return true;
}
-static void fbc_print_status(int debugfs_fd)
-{
- static char buf[128];
-
- igt_debugfs_simple_read(debugfs_fd, "i915_fbc_status", buf,
- sizeof(buf));
- igt_debug("FBC status: %s\n", buf);
-}
-
static bool fbc_is_enabled(int debugfs_fd)
{
char buf[128];
@@ -125,9 +116,7 @@ static bool fbc_is_enabled(int debugfs_fd)
static bool fbc_wait_until_enabled(int debugfs_fd)
{
- bool r = igt_wait(fbc_is_enabled(debugfs_fd), 5000, 1);
- fbc_print_status(debugfs_fd);
- return r;
+ return igt_wait(fbc_is_enabled(debugfs_fd), 5000, 1);
}
static bool fbc_wait_until_update(int debugfs)
@@ -179,21 +168,9 @@ static bool connector_can_psr(drmModeConnectorPtr connector)
return (connector->connector_type == DRM_MODE_CONNECTOR_eDP);
}
-static void psr_print_status(int debugfs_fd)
-{
- static char buf[PSR_STATUS_MAX_LEN];
-
- igt_debugfs_simple_read(debugfs_fd, "i915_edp_psr_status", buf,
- sizeof(buf));
- igt_debug("PSR status: %s\n", buf);
-}
-
static bool psr_wait_until_enabled(int debugfs_fd)
{
- bool r = psr_wait_entry(debugfs_fd);
-
- psr_print_status(debugfs_fd);
- return r;
+ return psr_wait_entry(debugfs_fd);
}
static bool psr_wait_until_update(int debugfs_fd)
--
2.20.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
next prev parent reply other threads:[~2019-01-07 19:11 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-07 19:10 [igt-dev] [PATCH i-g-t 1/8] lib/psr: Add a macro with the maximum lenght of i915_edp_psr_status and use it José Roberto de Souza
2019-01-07 19:10 ` [igt-dev] [PATCH i-g-t 2/8] lib/psr: Share the code check if sink supports PSR José Roberto de Souza
2019-01-07 19:10 ` [igt-dev] [PATCH i-g-t 3/8] tests/fbcon_fbt: Use psr_wait_entry() to wait PSR be enabled José Roberto de Souza
2019-01-07 19:10 ` [igt-dev] [PATCH i-g-t 4/8] tests/fbcon_fbt: Allow fbcon to bind when running this test José Roberto de Souza
2019-01-07 19:10 ` [igt-dev] [PATCH i-g-t 5/8] tests/fbcon_fbt: Add wait_until_update() callback to features José Roberto de Souza
2019-01-07 19:10 ` [igt-dev] [PATCH i-g-t 6/8] tests/fbcon_fbt: Use psr_wait_exit() to test updates in PSR José Roberto de Souza
2019-01-07 19:10 ` [igt-dev] [PATCH i-g-t 7/8] tests/fbcon_fbt: Add and use psr_long_wait_exit() " José Roberto de Souza
2019-01-07 20:41 ` Chris Wilson
2019-01-08 14:01 ` Souza, Jose
2019-01-07 19:10 ` José Roberto de Souza [this message]
2019-01-07 20:44 ` [igt-dev] [PATCH i-g-t 8/8] tests/fbcon_fbt: Do not print debugfs of features Chris Wilson
2019-01-07 19:52 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/8] lib/psr: Add a macro with the maximum lenght of i915_edp_psr_status and use it Patchwork
2019-01-07 20:38 ` [igt-dev] [PATCH i-g-t 1/8] " Chris Wilson
2019-01-08 13:52 ` Souza, Jose
2019-01-07 22:52 ` [igt-dev] ✓ Fi.CI.IGT: success for series starting with [i-g-t,1/8] " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190107191056.22759-8-jose.souza@intel.com \
--to=jose.souza@intel.com \
--cc=dhinakaran.pandiyan@intel.com \
--cc=igt-dev@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox