fdo-mirrors/weston
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/wayland/weston.git synced 2025-12-20 03:30:19 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

backend-drm: Set coefficients and range plane properties

Browse source 
Using values from the color representation protocol. Only supported for
YCbCr formats in KMS drivers for now.

Signed-off-by: Robert Mader <robert.mader@collabora.com>
This commit is contained in:
Robert Mader 2025-02-03 11:43:22 +01:00
parent 104cd2e813
commit adaffa980e
1 changed files with 23 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

35
libweston/backend-drm/state-propose.c
View file

@ -39,6 +39,7 @@
#include "drm-internal.h" #include "drm-internal.h"
#include "color.h" #include "color.h"
#include "color-representation.h"
#include "linux-dmabuf.h" #include "linux-dmabuf.h"
#include "presentation-time-server-protocol.h" #include "presentation-time-server-protocol.h"
#include "linux-dmabuf-unstable-v1-server-protocol.h" #include "linux-dmabuf-unstable-v1-server-protocol.h"
@ -139,23 +140,33 @@ drm_output_try_paint_node_on_plane(struct drm_plane *plane,
state->in_fence_fd = ev->surface->acquire_fence_fd; state->in_fence_fd = ev->surface->acquire_fence_fd;
if (fb->format && fb->format->color_model == COLOR_MODEL_YUV) { if (fb->format && fb->format->color_model == COLOR_MODEL_YUV) {
enum wdrm_plane_color_encoding color_encoding; struct weston_color_representation color_rep;
enum wdrm_plane_color_range color_range; const struct weston_color_matrix_coef_info *matrix_coef_info;
const struct weston_color_quant_range_info *quant_range_info;
color_rep =
weston_fill_color_representation(&surface->color_representation,
fb->format);
matrix_coef_info =
weston_color_matrix_coef_info_get(color_rep.matrix_coefficients);
assert(matrix_coef_info);
assert(matrix_coef_info->wdrm != WDRM_PLANE_COLOR_ENCODING__COUNT);
quant_range_info =
weston_color_quant_range_info_get(color_rep.quant_range);
assert(quant_range_info);
assert(quant_range_info->wdrm != WDRM_PLANE_COLOR_RANGE__COUNT);
/* These values will become dynamic once we implement the
* color-representation protocol. */
color_encoding = WDRM_PLANE_COLOR_ENCODING_DEFAULT;
color_range = WDRM_PLANE_COLOR_RANGE_DEFAULT;
if (plane->props[WDRM_PLANE_COLOR_ENCODING].prop_id == 0) { if (plane->props[WDRM_PLANE_COLOR_ENCODING].prop_id == 0) {
if (color_encoding != WDRM_PLANE_COLOR_ENCODING_DEFAULT) { if (matrix_coef_info->wdrm != WDRM_PLANE_COLOR_ENCODING_DEFAULT) {
drm_debug(b, "\t\t\t[view] not placing view %p on plane %lu: " drm_debug(b, "\t\t\t[view] not placing view %p on plane %lu: "
"non-default color encoding not supported\n", "non-default color encoding not supported\n",
ev, (unsigned long) plane->plane_id); ev, (unsigned long) plane->plane_id);
goto out; goto out;
} }
} else if (!drm_plane_supports_color_encoding(plane, } else if (!drm_plane_supports_color_encoding(plane,
color_encoding)) { matrix_coef_info->wdrm)) {
drm_debug(b, "\t\t\t[view] not placing view %p on plane %lu: " drm_debug(b, "\t\t\t[view] not placing view %p on plane %lu: "
"color encoding not supported\n", ev, "color encoding not supported\n", ev,
(unsigned long) plane->plane_id); (unsigned long) plane->plane_id);
@ -163,22 +174,22 @@ drm_output_try_paint_node_on_plane(struct drm_plane *plane,
} }
if (plane->props[WDRM_PLANE_COLOR_RANGE].prop_id == 0) { if (plane->props[WDRM_PLANE_COLOR_RANGE].prop_id == 0) {
if (color_range != WDRM_PLANE_COLOR_RANGE_DEFAULT) { if (quant_range_info->wdrm != WDRM_PLANE_COLOR_RANGE_DEFAULT) {
drm_debug(b, "\t\t\t[view] not placing view %p on plane %lu: " drm_debug(b, "\t\t\t[view] not placing view %p on plane %lu: "
"non-default color range not supported\n", "non-default color range not supported\n",
ev, (unsigned long) plane->plane_id); ev, (unsigned long) plane->plane_id);
goto out; goto out;
} }
} else if (!drm_plane_supports_color_range(plane, } else if (!drm_plane_supports_color_range(plane,
color_range)) { quant_range_info->wdrm)) {
drm_debug(b, "\t\t\t[view] not placing view %p on plane %lu: " drm_debug(b, "\t\t\t[view] not placing view %p on plane %lu: "
"color range not supported\n", ev, "color range not supported\n", ev,
(unsigned long) plane->plane_id); (unsigned long) plane->plane_id);
goto out; goto out;
} }
state->color_encoding = color_encoding; state->color_encoding = matrix_coef_info->wdrm;
state->color_range = color_range; state->color_range = quant_range_info->wdrm;
} }
/* In planes-only mode, we don't have an incremental state to /* In planes-only mode, we don't have an incremental state to