<div dir="ltr">Thank you for working on NV12 plane support. ChromeOS eagerly wants this feature to save power on Youtube. If it's landed soon, I'll really appreciate it.<div><br></div><div>Kristian is working on backporting it to the downstream ChromeOS kernel.</div><div><a href="https://chromium-review.googlesource.com/c/601288">https://chromium-review.googlesource.com/c/601288</a><br></div><div><div><br></div></div><div>I tested it on my Apollo Lake chromebook, but there is blackscreen flickering. NOTE: RGBA, RGBX, BGRA, BGRX, YUYV works without flickering.</div><div><a href="https://goo.gl/photos/tr5VfMiKeyEpDg9K6">https://goo.gl/photos/tr5VfMiKeyEpDg9K6</a><br></div><div><br></div><div>Best regards,</div><div>DS</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 11, 2017 at 9:18 AM, Ville Syrjälä <span dir="ltr"><<a href="mailto:ville.syrjala@linux.intel.com" target="_blank">ville.syrjala@linux.intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, Jul 11, 2017 at 07:40:48PM +0530, Vidya Srinivas wrote:<br>
> This patch series is adding NV12 support for Skylake display after<br>
> rebasing on latest drm-intel-nightly. Initial series of the patches<br>
> can be found here:<br>
> <a href="https://lists.freedesktop.org/archives/intel-gfx/2015-May/066786.html" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>archives/intel-gfx/2015-May/<wbr>066786.html</a><br>
><br>
> Feature has been currently tested with custom linux based test tool<br>
> IGT test development is under progress. Floating these patches for<br>
> initial review. These NV12 patches are dependent on Ville's patches<br>
> mentioned below.<br>
><br>
> Update from last rev:<br>
> Patches were initial reviewed last when floated but<br>
> currently there was a design change with respect to<br>
> - the way fb offset is handled<br>
> - the way rotation is handled<br>
> Rebase of the current NV12 patch series has been done as per the<br>
> current changes on drm-intel-nightly.<br>
> Review comments from Ville (12th June 2017) have been addressed<br>
> Review comments from Clinton A Taylor (7th July 2017) have been<br>
> addressed<br>
> Review comments from Clinton A Taylor (10th July 2017) have been<br>
> addressed. Had missed out tested-by/reviewed-by in the patches.<br>
> Fixed that error in this series.<br>
><br>
> Chandra Konduru (6):<br>
>   drm/i915: Set scaler mode for NV12<br>
>   drm/i915: Update format_is_yuv() to include NV12<br>
>   drm/i915: Upscale scaler max scale for NV12<br>
>   drm/i915: Add NV12 as supported format for primary plane<br>
>   drm/i915: Add NV12 as supported format for sprite plane<br>
>   drm/i915: Add NV12 support to intel_framebuffer_init<br>
><br>
> Ville Syrjälä (2):<br>
>   drm/i915: Implement .get_format_info() hook for CCS<br>
>   drm/i915: Add render decompression support<br>
<br>
</div></div>These two are some old stale versions of my CCS patches. I suppose it<br>
shouldn't matter too much, as long as no one has based any userspace tests<br>
on these since later versions will change how interpret fb->offsets[].<br>
<span class="im HOEnZb"><br>
><br>
>  drivers/gpu/drm/drm_fourcc.c         |   2 +-<br>
>  drivers/gpu/drm/i915/i915_reg.<wbr>h      |  24 +++<br>
>  drivers/gpu/drm/i915/intel_<wbr>atomic.c  |   8 +-<br>
>  drivers/gpu/drm/i915/intel_<wbr>display.c | 313 ++++++++++++++++++++++++++++++<wbr>++---<br>
>  drivers/gpu/drm/i915/intel_<wbr>drv.h     |   3 +-<br>
>  drivers/gpu/drm/i915/intel_pm.<wbr>c      |  29 +++-<br>
>  drivers/gpu/drm/i915/intel_<wbr>sprite.c  |  16 +-<br>
>  include/drm/drm_mode_config.h        |   3 +-<br>
>  include/uapi/drm/drm_fourcc.h        |   3 +<br>
>  9 files changed, 365 insertions(+), 36 deletions(-)<br>
><br>
> --<br>
> 1.9.1<br>
><br>
> ______________________________<wbr>_________________<br>
> Intel-gfx mailing list<br>
> <a href="mailto:Intel-gfx@lists.freedesktop.org">Intel-gfx@lists.freedesktop.<wbr>org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/intel-gfx" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/intel-gfx</a><br>
<br>
</span><span class="HOEnZb"><font color="#888888">--<br>
Ville Syrjälä<br>
Intel OTC<br>
</font></span><div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
Intel-gfx mailing list<br>
<a href="mailto:Intel-gfx@lists.freedesktop.org">Intel-gfx@lists.freedesktop.<wbr>org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/intel-gfx" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/intel-gfx</a><br>
</div></div></blockquote></div><br></div>