tests/psr: Share the code check if sink supports PSR

The same code checking if sink supports PSR was spread into 3 tests,
better move it to lib and reuse.

v2: splitted previous patch into this one and the next one(Dhinakaran)

Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Reviewed-by: Dhinakaran Pandiyan's avatarDhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Signed-off-by: José Roberto de Souza's avatarJosé Roberto de Souza <jose.souza@intel.com>
parent 465001c7
Pipeline #14952 passed with stages
in 2 minutes and 19 seconds
......@@ -129,3 +129,13 @@ bool psr_disable(int debugfs_fd)
{
return psr_set(debugfs_fd, false);
}
bool psr_sink_support(int debugfs_fd)
{
char buf[PSR_STATUS_MAX_LEN];
int ret;
ret = igt_debugfs_simple_read(debugfs_fd, "i915_edp_psr_status", buf,
sizeof(buf));
return ret > 0 && strstr(buf, "Sink_Support: yes\n");
}
......@@ -34,5 +34,6 @@ bool psr_wait_entry(int debugfs_fd);
bool psr_wait_exit(int debugfs_fd);
bool psr_enable(int debugfs_fd);
bool psr_disable(int debugfs_fd);
bool psr_sink_support(int debugfs_fd);
#endif
......@@ -168,19 +168,6 @@ static void set_mode_for_one_screen(struct drm_info *drm, struct igt_fb *fb,
igt_assert_eq(rc, 0);
}
static bool psr_supported_on_chipset(int debugfs_fd)
{
char buf[PSR_STATUS_MAX_LEN];
int ret;
ret = igt_debugfs_simple_read(debugfs_fd, "i915_edp_psr_status",
buf, sizeof(buf));
if (ret < 0)
return false;
return strstr(buf, "Sink_Support: yes\n");
}
static bool connector_can_psr(drmModeConnectorPtr connector)
{
return (connector->connector_type == DRM_MODE_CONNECTOR_eDP);
......@@ -239,7 +226,7 @@ struct feature {
.connector_possible_fn = connector_can_fbc,
.enable = fbc_modparam_enable,
}, psr = {
.supported_on_chipset = psr_supported_on_chipset,
.supported_on_chipset = psr_sink_support,
.wait_until_enabled = psr_wait_until_enabled,
.connector_possible_fn = connector_can_psr,
.enable = psr_debugfs_enable,
......
......@@ -1417,17 +1417,6 @@ static void teardown_fbc(void)
{
}
static bool psr_sink_has_support(void)
{
char buf[PSR_STATUS_MAX_LEN];
debugfs_read("i915_edp_psr_status", buf);
if (*buf == '\0') /* !HAS_PSR -> -ENODEV*/
return false;
return strstr(buf, "Sink_Support: yes\n");
}
static void setup_psr(void)
{
if (prim_mode_params.output->config.connector->connector_type !=
......@@ -1436,7 +1425,7 @@ static void setup_psr(void)
return;
}
if (!psr_sink_has_support()) {
if (!psr_sink_support(drm.debugfs)) {
igt_info("Can't test PSR: not supported by sink.\n");
return;
}
......
......@@ -191,13 +191,7 @@ static void fill_render(data_t *data, uint32_t handle, unsigned char color)
static bool sink_support(data_t *data)
{
char buf[PSR_STATUS_MAX_LEN];
igt_debugfs_simple_read(data->debugfs_fd, "i915_edp_psr_status",
buf, sizeof(buf));
return data->with_psr_disabled ||
strstr(buf, "Sink_Support: yes\n");
return data->with_psr_disabled || psr_sink_support(data->debugfs_fd);
}
static bool psr_wait_entry_if_enabled(data_t *data)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment