summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2017-01-12 13:52:35 +0100
committerwm4 <wm4@nowhere>2017-01-12 14:01:09 +0100
commit5174add43a77f2c4444cbe272d8360f36da06198 (patch)
tree5c577c1c198c4e382cd882fc0beedcd8f4604505
parent3d7b3996d287014697d9080ef20285c6e804e09e (diff)
downloadmpv-5174add43a77f2c4444cbe272d8360f36da06198.tar.bz2
mpv-5174add43a77f2c4444cbe272d8360f36da06198.tar.xz
vo_opengl: hwdec_vaegl: add experimental P010 support
This does not work, because Mesa has no support for the proposed DRM_FORMAT_R16 and DRM_FORMAT_GR16 formats. It's also untested of course. As long as video/decode/vaapi.c doesn't hand down P010 surfaces, this is fine anyway. This can be tested by removing the code that disables P010 output: diff --git a/video/decode/vaapi.c b/video/decode/vaapi.c --- a/video/decode/vaapi.c +++ b/video/decode/vaapi.c @@ -55,13 +55,6 @@ static int init_decoder(struct lavc_ctx *ctx, int w, int h) assert(!ctx->avctx->hw_frames_ctx); - // If we use direct rendering, disallow 10 bit - it's probably not - // implemented yet, and our downstream components can't deal with it. - if (!p->own_ctx && required_sw_format != AV_PIX_FMT_NV12) { - MP_WARN(ctx, "10 bit surfaces are currently supported.\n"); - return -1; - } -
-rw-r--r--video/out/opengl/hwdec_vaegl.c23
1 files changed, 17 insertions, 6 deletions
diff --git a/video/out/opengl/hwdec_vaegl.c b/video/out/opengl/hwdec_vaegl.c
index 96a35fcb22..8a971a020b 100644
--- a/video/out/opengl/hwdec_vaegl.c
+++ b/video/out/opengl/hwdec_vaegl.c
@@ -257,6 +257,7 @@ static int reinit(struct gl_hwdec *hw, struct mp_image_params *params)
p->current_mpfmt = params->hw_subfmt;
if (p->current_mpfmt != IMGFMT_NV12 &&
+ p->current_mpfmt != IMGFMT_P010 &&
p->current_mpfmt != IMGFMT_420P)
{
MP_FATAL(p, "unsupported VA image format %s\n",
@@ -313,17 +314,27 @@ static int map_frame(struct gl_hwdec *hw, struct mp_image *hw_image,
mp_image_set_params(&layout, &hw_image->params);
mp_image_setfmt(&layout, mpfmt);
- // (it would be nice if we could use EGL_IMAGE_INTERNAL_FORMAT_EXT)
- int drm_fmts[4] = {MP_FOURCC('R', '8', ' ', ' '), // DRM_FORMAT_R8
- MP_FOURCC('G', 'R', '8', '8'), // DRM_FORMAT_GR88
- MP_FOURCC('R', 'G', '2', '4'), // DRM_FORMAT_RGB888
- MP_FOURCC('R', 'A', '2', '4')}; // DRM_FORMAT_RGBA8888
+ int drm_fmts[8] = {
+ // 1 bytes per pixel, 1-4 components
+ MP_FOURCC('R', '8', ' ', ' '), // DRM_FORMAT_R8
+ MP_FOURCC('G', 'R', '8', '8'), // DRM_FORMAT_GR88
+ MP_FOURCC('R', 'G', '2', '4'), // DRM_FORMAT_RGB888
+ MP_FOURCC('R', 'A', '2', '4'), // DRM_FORMAT_RGBA8888
+ // 2 bytes per pixel, 1-4 components
+ MP_FOURCC('R', '1', '6', ' '), // proposed DRM_FORMAT_R16
+ MP_FOURCC('G', 'R', '1', '6'), // proposed DRM_FORMAT_GR16
+ 0, // N/A
+ 0, // N/A
+ };
for (int n = 0; n < layout.num_planes; n++) {
int attribs[20] = {EGL_NONE};
int num_attribs = 0;
- ADD_ATTRIB(EGL_LINUX_DRM_FOURCC_EXT, drm_fmts[layout.fmt.bytes[n] - 1]);
+ int fmt_index = layout.fmt.components[n] - 1 +
+ 4 * (layout.fmt.component_full_bits / 8 - 1);
+
+ ADD_ATTRIB(EGL_LINUX_DRM_FOURCC_EXT, drm_fmts[fmt_index]);
ADD_ATTRIB(EGL_WIDTH, mp_image_plane_w(&layout, n));
ADD_ATTRIB(EGL_HEIGHT, mp_image_plane_h(&layout, n));
ADD_ATTRIB(EGL_DMA_BUF_PLANE0_FD_EXT, buffer_info.handle);