<html><head></head><body><div><br></div><div>On Thu, 2016-12-15 at 13:04 +0000, Mark Thompson wrote:</div><blockquote type="cite"><pre>On 05/12/16 17:54, Mark Thompson wrote:
<blockquote type="cite">
Signed-off-by: Mark Thompson <<a href="mailto:sw@jkqxz.net">sw@jkqxz.net</a>>
---
The supported surface formats are already correct (i.e. only P010); this makes the render target format right as well (before this, it declares that it supports only YUV 4:2:0 8-bit).
src/i965_drv_video.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/i965_drv_video.c b/src/i965_drv_video.c
index d83427c..9e1c92a 100644
--- a/src/i965_drv_video.c
+++ b/src/i965_drv_video.c
@@ -880,6 +880,8 @@ i965_get_default_chroma_formats(VADriverContextP ctx, VAProfile profile,
break;
case VAProfileHEVCMain10:
+ if (HAS_HEVC10_ENCODING(i965) && entrypoint == VAEntrypointEncSlice)
+ chroma_formats = VA_RT_FORMAT_YUV420_10BPP;
if (HAS_HEVC10_DECODING(i965) && entrypoint == VAEntrypointVLD)
chroma_formats |= i965->codec_info->hevc_dec_chroma_formats;
break;
</blockquote>
Ping. (The other patches in this series have been considered separately.)
</pre></blockquote><div><br></div><div><br></div><div>Yes this is a good catch. It was overlooked with the addition of 10bit support. We may need to make further adjustment when we roll in new encoder formats too.</div><div><br></div><div>lgtm.</div><div><br></div><div>applied.</div><div><br></div><div>Sean</div><div><br></div><div><br></div><blockquote type="cite"><pre>
Thanks,
- Mark
_______________________________________________
Libva mailing list
<a href="mailto:Libva@lists.freedesktop.org">Libva@lists.freedesktop.org</a>
<a href="https://lists.freedesktop.org/mailman/listinfo/libva">https://lists.freedesktop.org/mailman/listinfo/libva</a>
</pre></blockquote><div><br></div></body></html>