All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Hwang, Dongseong" <dongseong.hwang@intel.com>
To: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org,
	Vidya Srinivas <vidya.srinivas@intel.com>
Subject: Re: [PATCH 0/8] Adding NV12 support for SKL display
Date: Wed, 9 Aug 2017 17:15:59 -0700	[thread overview]
Message-ID: <CAFVCBbEsE1yNHq7_1f4n+PGwrbXWJESSXg-WKNDuajrgVk8FSA@mail.gmail.com> (raw)
In-Reply-To: <CAFVCBbESyAY5ha_HcchK2DhuKn=04BbRmUWi1_DHs7+9fpKzfg@mail.gmail.com>

I tested it on my Kaby Lake chromebook, and it works perfectly.

Best regards,
DS

On Thu, Aug 3, 2017 at 6:23 PM, Hwang, Dongseong
<dongseong.hwang@intel.com> wrote:
> 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.
>
> Kristian is working on backporting it to the downstream ChromeOS kernel.
> https://chromium-review.googlesource.com/c/601288
>
> I tested it on my Apollo Lake chromebook, but there is blackscreen
> flickering. NOTE: RGBA, RGBX, BGRA, BGRX, YUYV works without flickering.
> https://goo.gl/photos/tr5VfMiKeyEpDg9K6
>
> Best regards,
> DS
>
>
>
> On Tue, Jul 11, 2017 at 9:18 AM, Ville Syrjälä
> <ville.syrjala@linux.intel.com> wrote:
>>
>> On Tue, Jul 11, 2017 at 07:40:48PM +0530, Vidya Srinivas wrote:
>> > This patch series is adding NV12 support for Skylake display after
>> > rebasing on latest drm-intel-nightly. Initial series of the patches
>> > can be found here:
>> > https://lists.freedesktop.org/archives/intel-gfx/2015-May/066786.html
>> >
>> > Feature has been currently tested with custom linux based test tool
>> > IGT test development is under progress. Floating these patches for
>> > initial review. These NV12 patches are dependent on Ville's patches
>> > mentioned below.
>> >
>> > Update from last rev:
>> > Patches were initial reviewed last when floated but
>> > currently there was a design change with respect to
>> > - the way fb offset is handled
>> > - the way rotation is handled
>> > Rebase of the current NV12 patch series has been done as per the
>> > current changes on drm-intel-nightly.
>> > Review comments from Ville (12th June 2017) have been addressed
>> > Review comments from Clinton A Taylor (7th July 2017) have been
>> > addressed
>> > Review comments from Clinton A Taylor (10th July 2017) have been
>> > addressed. Had missed out tested-by/reviewed-by in the patches.
>> > Fixed that error in this series.
>> >
>> > Chandra Konduru (6):
>> >   drm/i915: Set scaler mode for NV12
>> >   drm/i915: Update format_is_yuv() to include NV12
>> >   drm/i915: Upscale scaler max scale for NV12
>> >   drm/i915: Add NV12 as supported format for primary plane
>> >   drm/i915: Add NV12 as supported format for sprite plane
>> >   drm/i915: Add NV12 support to intel_framebuffer_init
>> >
>> > Ville Syrjälä (2):
>> >   drm/i915: Implement .get_format_info() hook for CCS
>> >   drm/i915: Add render decompression support
>>
>> These two are some old stale versions of my CCS patches. I suppose it
>> shouldn't matter too much, as long as no one has based any userspace tests
>> on these since later versions will change how interpret fb->offsets[].
>>
>> >
>> >  drivers/gpu/drm/drm_fourcc.c         |   2 +-
>> >  drivers/gpu/drm/i915/i915_reg.h      |  24 +++
>> >  drivers/gpu/drm/i915/intel_atomic.c  |   8 +-
>> >  drivers/gpu/drm/i915/intel_display.c | 313
>> > ++++++++++++++++++++++++++++++++---
>> >  drivers/gpu/drm/i915/intel_drv.h     |   3 +-
>> >  drivers/gpu/drm/i915/intel_pm.c      |  29 +++-
>> >  drivers/gpu/drm/i915/intel_sprite.c  |  16 +-
>> >  include/drm/drm_mode_config.h        |   3 +-
>> >  include/uapi/drm/drm_fourcc.h        |   3 +
>> >  9 files changed, 365 insertions(+), 36 deletions(-)
>> >
>> > --
>> > 1.9.1
>> >
>> > _______________________________________________
>> > Intel-gfx mailing list
>> > Intel-gfx@lists.freedesktop.org
>> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>>
>> --
>> Ville Syrjälä
>> Intel OTC
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2017-08-10  0:16 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-11 14:10 [PATCH 0/8] Adding NV12 support for SKL display Vidya Srinivas
2017-07-11 14:10 ` [PATCH 1/8] drm/i915: Implement .get_format_info() hook for CCS Vidya Srinivas
2017-07-11 14:10 ` [PATCH 2/8] drm/i915: Add render decompression support Vidya Srinivas
2017-07-11 14:10 ` [PATCH 3/8] drm/i915: Set scaler mode for NV12 Vidya Srinivas
2017-07-11 14:10 ` [PATCH 4/8] drm/i915: Update format_is_yuv() to include NV12 Vidya Srinivas
2017-07-11 14:10 ` [PATCH 5/8] drm/i915: Upscale scaler max scale for NV12 Vidya Srinivas
2017-07-11 16:16   ` Ville Syrjälä
2017-07-11 14:10 ` [PATCH 6/8] drm/i915: Add NV12 as supported format for primary plane Vidya Srinivas
2017-07-11 16:14   ` Ville Syrjälä
2017-07-11 14:10 ` [PATCH 7/8] drm/i915: Add NV12 as supported format for sprite plane Vidya Srinivas
2017-07-11 15:04   ` Clint Taylor
2017-07-11 16:12   ` Ville Syrjälä
2017-07-20 10:05     ` Srinivas, Vidya
2017-07-24  4:24     ` Srinivas, Vidya
2017-07-11 14:10 ` [PATCH 8/8] drm/i915: Add NV12 support to intel_framebuffer_init Vidya Srinivas
2017-07-11 16:13   ` Ville Syrjälä
2017-07-11 14:55 ` ✓ Fi.CI.BAT: success for Adding NV12 support for SKL display (rev5) Patchwork
2017-07-11 16:18 ` [PATCH 0/8] Adding NV12 support for SKL display Ville Syrjälä
2017-08-04  1:23   ` Hwang, Dongseong
2017-08-10  0:15     ` Hwang, Dongseong [this message]
  -- strict thread matches above, loose matches on Subject: below --
2017-07-11  3:26 Vidya Srinivas
2017-07-10  6:53 Vidya Srinivas
2017-06-20  6:10 Vidya Srinivas
2017-06-07  4:51 [PATCH 00/11] " Vidya Srinivas
2017-06-07 10:41 ` [PATCH 0/8] " Vidya Srinivas
2017-06-08  9:07   ` Jani Nikula
2017-06-07 11:40 ` Vidya Srinivas

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAFVCBbEsE1yNHq7_1f4n+PGwrbXWJESSXg-WKNDuajrgVk8FSA@mail.gmail.com \
    --to=dongseong.hwang@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=vidya.srinivas@intel.com \
    --cc=ville.syrjala@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.