diff options
| author | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2021-10-12 03:12:17 +0300 |
|---|---|---|
| committer | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2021-10-18 16:28:16 +0300 |
| commit | 4a0d5d5c7b0ce18d44f259db8c5ecf27919e85c7 (patch) | |
| tree | 186b1b2d00f1995e9c2c1a650e26eb4c2c6f6655 /lib/igt_kms.c | |
| parent | 9db2886c2111eae9591fbff62ac12e13cbaf9941 (diff) | |
lib/kms: Have igt_std_1024_mode_get() return a copy of the mode
We want to provide override modes with different refresh rates.
Start by making igt_std_1024_mode_get() return a copy rather
than a pointer to the static const mode directly. And sprinkle
the necessary free() calls, and some igt_memdups() into parallel
codepaths, so we are consistnetly allocating and freeing everything.
Reviewed-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Diffstat (limited to 'lib/igt_kms.c')
| -rw-r--r-- | lib/igt_kms.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 6b0639f6..f833785b 100644 --- a/lib/igt_kms.c +++ b/lib/igt_kms.c @@ -2478,7 +2478,7 @@ igt_output_t *igt_output_from_connector(igt_display_t *display, return found; } -const drmModeModeInfo *igt_std_1024_mode_get(void) +drmModeModeInfo *igt_std_1024_mode_get(void) { static const drmModeModeInfo std_1024_mode = { .clock = 65000, @@ -2498,7 +2498,7 @@ const drmModeModeInfo *igt_std_1024_mode_get(void) .name = "Custom 1024x768", }; - return &std_1024_mode; + return igt_memdup(&std_1024_mode, sizeof(std_1024_mode)); } /* |