diff options
Diffstat (limited to 'video/out/hwdec/dmabuf_interop_wl.c')
-rw-r--r-- | video/out/hwdec/dmabuf_interop_wl.c | 32 |
1 files changed, 29 insertions, 3 deletions
diff --git a/video/out/hwdec/dmabuf_interop_wl.c b/video/out/hwdec/dmabuf_interop_wl.c index 74b421d583..eede6bb0f2 100644 --- a/video/out/hwdec/dmabuf_interop_wl.c +++ b/video/out/hwdec/dmabuf_interop_wl.c @@ -21,9 +21,6 @@ static bool mapper_init(struct ra_hwdec_mapper *mapper, const struct ra_imgfmt_desc *desc) { - struct dmabuf_interop_priv *p = mapper->priv; - - p->num_planes = 1; return true; } @@ -35,6 +32,32 @@ static bool map(struct ra_hwdec_mapper *mapper, struct dmabuf_interop *dmabuf_interop, bool probing) { + // 1. only validate format when composed layers is enabled (i.e. vaapi) + // 2. for drmprime, just return true for now, as this use case + // has not been tested. + if (!dmabuf_interop->composed_layers) + return true; + + int layer_no = 0; + struct dmabuf_interop_priv *mapper_p = mapper->priv; + uint32_t drm_format = mapper_p->desc.layers[layer_no].format; + + if (mapper_p->desc.nb_layers != 1) { + MP_VERBOSE(mapper, "Mapped surface has separate layers - expected composed layers.\n"); + return false; + } else if (!ra_compatible_format(mapper->ra, drm_format, + mapper_p->desc.objects[0].format_modifier)) { + MP_VERBOSE(mapper, "Mapped surface with format %s; drm format '%s(%016lx)' " + "is not supported by compositor.\n", + mp_imgfmt_to_name(mapper->src->params.hw_subfmt), + mp_tag_str(drm_format), + mapper_p->desc.objects[0].format_modifier); + return false; + } + + MP_VERBOSE(mapper, "Supported Wayland display format: '%s(%016lx)'\n", + mp_tag_str(drm_format), mapper_p->desc.objects[0].format_modifier); + return true; } @@ -48,6 +71,9 @@ bool dmabuf_interop_wl_init(const struct ra_hwdec *hw, if (!ra_is_wldmabuf(hw->ra)) return false; + if (strstr(hw->driver->name, "vaapi") != NULL) + dmabuf_interop->composed_layers = true; + dmabuf_interop->interop_init = mapper_init; dmabuf_interop->interop_uninit = mapper_uninit; dmabuf_interop->interop_map = map; |