[PATCH i-g-t 06/25] lib/kms: Extract fill_plane_default_format_mods()

[Ville Syrjala <ville.syrjala@linux.intel.com>]

From: Ville Syrjälä <ville.syrjala@linux.intel.com>

Extract the code that fills the format+modifier arrays when
the plane doesn't have the IN_FORMATs blob. I plan to reuse
this to also fill the async format+modifier list.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
 lib/igt_kms.c | 33 ++++++++++++++++++---------------
 1 file changed, 18 insertions(+), 15 deletions(-)

diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index b39cba8ae5e1..b3ee8a61683e 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -6220,28 +6220,31 @@ static void igt_parse_format_mod_blob(const struct drm_format_modifier_blob *blo
 	igt_assert_eq(idx, format_mods->count);
 }
 
+static void fill_plane_default_format_mods(igt_plane_t *plane,
+					   struct igt_format_mods *format_mods)
+{
+	drmModePlanePtr p = plane->drm_plane;
+
+	format_mods_alloc(format_mods, p->count_formats);
+
+	/*
+	 * We don't know which modifiers are
+	 * supported, so we'll assume linear only.
+	 */
+	for (int i = 0; i < plane->format_mods.count; i++) {
+		format_mods->formats[i] = p->formats[i];
+		format_mods->modifiers[i] = DRM_FORMAT_MOD_LINEAR;
+	}
+}
+
 static void igt_fill_plane_format_mod(igt_display_t *display, igt_plane_t *plane)
 {
 	const struct drm_format_modifier_blob *blob_data;
 	drmModePropertyBlobPtr blob;
 	uint64_t blob_id;
-	int count = 0;
 
 	if (!igt_plane_has_prop(plane, IGT_PLANE_IN_FORMATS)) {
-		drmModePlanePtr p = plane->drm_plane;
-
-		count = p->count_formats;
-
-		format_mods_alloc(&plane->format_mods, count);
-
-		/*
-		 * We don't know which modifiers are
-		 * supported, so we'll assume linear only.
-		 */
-		for (int i = 0; i < count; i++) {
-			plane->format_mods.formats[i] = p->formats[i];
-			plane->format_mods.modifiers[i] = DRM_FORMAT_MOD_LINEAR;
-		}
+		fill_plane_default_format_mods(plane, &plane->format_mods);
 
 		return;
 	}
-- 
2.52.0