fix: inverted modifiers check in DRM plane allocation

ardera · ardera · commit f6e104f7acf4 · 2025-09-14T13:20:45.000+02:00
diff --git a/src/kms/req_builder.c b/src/kms/req_builder.c
@@ -115,7 +115,7 @@ static bool plane_qualifies(
     }
 
     if (has_modifier) {
-        if (drm_plane_supports_modified_formats(plane)) {
+        if (!drm_plane_supports_modified_formats(plane)) {
             // return false if we want a modified format but the plane doesn't support modified formats
             LOG_DRM_PLANE_ALLOCATION_DEBUG(
                 "    does not qualify: framebuffer has modifier %" PRIu64 " but plane does not support modified formats\n",

Original file line number	Diff line number	Diff line change
`@@ -115,7 +115,7 @@ static bool plane_qualifies(`
`115`	`115`	`}`
`116`	`116`
`117`	`117`	`if (has_modifier) {`
`118`		`- if (drm_plane_supports_modified_formats(plane)) {`
	`118`	`+ if (!drm_plane_supports_modified_formats(plane)) {`
`119`	`119`	`// return false if we want a modified format but the plane doesn't support modified formats`
`120`	`120`	`LOG_DRM_PLANE_ALLOCATION_DEBUG(`
`121`	`121`	`" does not qualify: framebuffer has modifier %" PRIu64 " but plane does not support modified formats\n",`