diff options
author | José Roberto de Souza <jose.souza@intel.com> | 2018-10-23 13:57:02 -0700 |
---|---|---|
committer | José Roberto de Souza <jose.souza@intel.com> | 2019-01-14 14:33:15 -0800 |
commit | 465001c775a516ac1c7c24302593b02c0f73f486 (patch) | |
tree | c5f160d85a706fd5d4cddd41d5685a45dd19e3be /tests | |
parent | 7d3fdd0fdddb0993ad25d1416e77ed812f284e32 (diff) |
lib/psr: Add a macro with the maximum lenght of i915_edp_psr_status and use it
So every function reading i915_edp_psr_status can allocate a buffer
long enough.
Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Reviewed-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Signed-off-by: José Roberto de Souza <jose.souza@intel.com>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/kms_fbcon_fbt.c | 6 | ||||
-rw-r--r-- | tests/kms_frontbuffer_tracking.c | 2 | ||||
-rw-r--r-- | tests/kms_psr.c | 2 |
3 files changed, 5 insertions, 5 deletions
diff --git a/tests/kms_fbcon_fbt.c b/tests/kms_fbcon_fbt.c index 24d3ad90..0ad53266 100644 --- a/tests/kms_fbcon_fbt.c +++ b/tests/kms_fbcon_fbt.c @@ -170,7 +170,7 @@ static void set_mode_for_one_screen(struct drm_info *drm, struct igt_fb *fb, static bool psr_supported_on_chipset(int debugfs_fd) { - char buf[256]; + char buf[PSR_STATUS_MAX_LEN]; int ret; ret = igt_debugfs_simple_read(debugfs_fd, "i915_edp_psr_status", @@ -188,7 +188,7 @@ static bool connector_can_psr(drmModeConnectorPtr connector) static void psr_print_status(int debugfs_fd) { - static char buf[256]; + static char buf[PSR_STATUS_MAX_LEN]; igt_debugfs_simple_read(debugfs_fd, "i915_edp_psr_status", buf, sizeof(buf)); @@ -197,7 +197,7 @@ static void psr_print_status(int debugfs_fd) static bool psr_is_enabled(int debugfs_fd) { - char buf[256]; + char buf[PSR_STATUS_MAX_LEN]; igt_debugfs_simple_read(debugfs_fd, "i915_edp_psr_status", buf, sizeof(buf)); diff --git a/tests/kms_frontbuffer_tracking.c b/tests/kms_frontbuffer_tracking.c index 5ab28319..c366fecf 100644 --- a/tests/kms_frontbuffer_tracking.c +++ b/tests/kms_frontbuffer_tracking.c @@ -1419,7 +1419,7 @@ static void teardown_fbc(void) static bool psr_sink_has_support(void) { - char buf[256]; + char buf[PSR_STATUS_MAX_LEN]; debugfs_read("i915_edp_psr_status", buf); if (*buf == '\0') /* !HAS_PSR -> -ENODEV*/ diff --git a/tests/kms_psr.c b/tests/kms_psr.c index d00e552f..20b69892 100644 --- a/tests/kms_psr.c +++ b/tests/kms_psr.c @@ -191,7 +191,7 @@ static void fill_render(data_t *data, uint32_t handle, unsigned char color) static bool sink_support(data_t *data) { - char buf[512]; + char buf[PSR_STATUS_MAX_LEN]; igt_debugfs_simple_read(data->debugfs_fd, "i915_edp_psr_status", buf, sizeof(buf)); |