<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Dec 18, 2018 at 3:07 AM Kirithika Kalirathnam <<a href="mailto:kirithika@multicorewareinc.com">kirithika@multicorewareinc.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div dir="ltr"><div># HG changeset patch</div><div># User kirithika <<a href="mailto:kirithika@multicorewareinc.com" target="_blank">kirithika@multicorewareinc.com</a>></div><div># Date 1544426306 -19800</div><div>#      Mon Dec 10 12:48:26 2018 +0530</div><div># Node ID 827668cf88a85f8d14b8943c53e3f4c87196e96d</div><div># Parent  81373aab81dfe2e31a5ef353b1073d8bf1e22502</div><div>Add support for Dolby Vision profile 8.1</div><div><br></div><div>diff -r 81373aab81df -r 827668cf88a8 doc/reST/cli.rst</div><div>--- a/doc/reST/cli.rst<span style="white-space:pre-wrap">      </span>Thu Dec 13 10:55:15 2018 +0530</div><div>+++ b/doc/reST/cli.rst<span style="white-space:pre-wrap">     </span>Mon Dec 10 12:48:26 2018 +0530</div><div>@@ -2214,7 +2214,7 @@</div><div>     The value is specified as a float or as an integer with the profile times 10,</div><div>     for example profile 5 is specified as "5" or "5.0" or "50".</div><div>     </div><div>-    Currently only profile 5 enabled, Default 0 (disabled)</div><div>+    Currently only profile 5 and profile 8.1 enabled , Default 0 (disabled)</div><div> </div><div> .. option:: --dolby-vision-rpu <filename></div><div> </div><div>diff -r 81373aab81df -r 827668cf88a8 source/common/param.cpp</div><div>--- a/source/common/param.cpp<span style="white-space:pre-wrap"> </span>Thu Dec 13 10:55:15 2018 +0530</div><div>+++ b/source/common/param.cpp<span style="white-space:pre-wrap">      </span>Mon Dec 10 12:48:26 2018 +0530</div><div>@@ -1418,14 +1418,17 @@</div><div>         "Invalid refine-intra value, refine-intra levels 0 to 3 supported");</div><div>     CHECK(param->maxAUSizeFactor < 0.5 || param->maxAUSizeFactor > 1.0,</div><div>         "Supported factor for controlling max AU size is from 0.5 to 1");</div><div>-    CHECK((param->dolbyProfile != 0) && (param->dolbyProfile != 50),</div><div>-        "Unsupported Dolby Vision profile, only profile 5 enabled");</div><div>-    if (param->dolbyProfile == 50)</div><div>+    CHECK((param->dolbyProfile != 0) && (param->dolbyProfile != 50) && (param->dolbyProfile != 81),</div><div>+        "Unsupported Dolby Vision profile, only profile 5 and profile 8.1 enabled");</div><div>+    if (param->dolbyProfile)</div><div>     {</div><div>-        CHECK((param->rc.vbvMaxBitrate < 0 && param->rc.vbvBufferSize < 0), "Dolby Vision requires VBV settings to enable HRD.\n");</div><div>-        CHECK((param->sourceWidth > 3840 || param->sourceHeight > 2160), "Maximum supported resolution for Dolby Vision profile - 5 is 4k UHD\n");</div><div>-        CHECK((param->internalBitDepth != 10), "Dolby Vision profile - 5 is Main10 only\n");</div><div>-        CHECK((param->internalCsp != X265_CSP_I420), "Dolby Vision profile - 5 requires YCbCr 4:2:0 color space\n");</div><div>+        CHECK((param->rc.vbvMaxBitrate <= 0 && param->rc.vbvBufferSize <= 0), "Dolby Vision requires VBV settings to enable HRD.\n");</div></div></div></blockquote><div><br></div><div>This is still wrong, the condition should be (param->rc.vbvMaxBitrate <= 0 && param->rc.vbvBufferSize <= 0) since you need both values set to something in order to have a working VBV. I mentioned this in a previous review but it got ignored and the wrong version got pushed...<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div dir="ltr"><div>+        CHECK((param->sourceWidth > 3840 || param->sourceHeight > 2160), "Maximum supported resolution for Dolby Vision profile - 5 and profile  - 8.1 is 4k UHD\n");</div></div></div></blockquote><div><br></div><div>nit: double space before "- 8.1"</div><div>also this constraint is a bit strange, what about 4k dci? seems odd that it is not supported...</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div dir="ltr"><div>+        CHECK((param->internalBitDepth != 10), "Dolby Vision profile - 5 and profile - 8.1 is Main10 only\n");</div><div>+        CHECK((param->internalCsp != X265_CSP_I420), "Dolby Vision profile - 5 and profile  - 8.1  requires YCbCr 4:2:0 color space\n");</div><div>+</div><div>+        if (param->dolbyProfile == 81)</div><div>+            CHECK(!(param->masteringDisplayColorVolume), "Dolby Vision profile - 8.1 requires Mastering display color volume information\n");</div><div>     }</div><div> #if !X86_64<br></div></div></div></blockquote></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature">Vittorio</div></div>