public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
From: Damien Lespiau <damien.lespiau@intel.com>
To: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Cc: Intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH 0/8] NV12 90/270 rotated GGTT mapping
Date: Fri, 8 May 2015 15:59:29 +0100	[thread overview]
Message-ID: <20150508145928.GA19801@strange.ger.corp.intel.com> (raw)
In-Reply-To: <1431086563-4498-1-git-send-email-tvrtko.ursulin@linux.intel.com>

On Fri, May 08, 2015 at 01:02:35PM +0100, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> 
> First attempt at rotated GGTT mapping for the NV12 format.
> 
> It compiles and even does not crash on first use. But some parts are
> probably too simplistic at the moment and need further polish.
> 
> Sending out early since Chandra needs this to complete his display
> programming patches.
> 
> Depends on Chandra's NV12 patch series.

I don't know how you split up the work, but there's also adjustments to
be done in the DDB allocation and WM code for rotated NV12 buffers and I
don't see that in either of your series (I think).

The natural assignee is probably Chandra as he's already working on the
the NV12 DDB/WM code.

-- 
Damien

> Tvrtko Ursulin (8):
>   drm/i915: Remove duplicated intel_tile_height declaration
>   drm/i915: Support planar formats in tile height calculations
>   drm/i915: Remove pointless calculation in intel_rotate_fb_obj_pages
>   drm/i915: Extract tiled geometry calculation into a helper function
>   drm/i915: Support appending to the rotated pages mapping
>   drm/i915: Support NV12 in rotated GGTT mapping
>   drm/i915: Enable querying offset of UV plane with
>     intel_plane_obj_offset
>   drm/i915: Allow NV12 with 90/270 rotation
> 
>  drivers/gpu/drm/i915/i915_gem_gtt.c       | 96 +++++++++++++++++++++++--------
>  drivers/gpu/drm/i915/i915_gem_gtt.h       |  3 +
>  drivers/gpu/drm/i915/intel_atomic_plane.c |  3 +-
>  drivers/gpu/drm/i915/intel_display.c      | 36 +++++++++---
>  drivers/gpu/drm/i915/intel_drv.h          |  8 +--
>  drivers/gpu/drm/i915/intel_sprite.c       |  4 +-
>  6 files changed, 109 insertions(+), 41 deletions(-)
> 
> -- 
> 2.4.0
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

  parent reply	other threads:[~2015-05-08 14:59 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-08 12:02 [PATCH 0/8] NV12 90/270 rotated GGTT mapping Tvrtko Ursulin
2015-05-08 12:02 ` [PATCH 1/8] drm/i915: Remove duplicated intel_tile_height declaration Tvrtko Ursulin
2015-05-11 10:26   ` Daniel Vetter
2015-05-08 12:02 ` [PATCH 2/8] drm/i915: Support planar formats in tile height calculations Tvrtko Ursulin
2015-05-08 12:02 ` [PATCH 3/8] drm/i915: Remove pointless calculation in intel_rotate_fb_obj_pages Tvrtko Ursulin
2015-05-08 12:02 ` [PATCH 4/8] drm/i915: Extract tiled geometry calculation into a helper function Tvrtko Ursulin
2015-05-08 12:02 ` [PATCH 5/8] drm/i915: Support appending to the rotated pages mapping Tvrtko Ursulin
2015-05-08 12:02 ` [PATCH 6/8] drm/i915: Support NV12 in rotated GGTT mapping Tvrtko Ursulin
2015-05-08 12:02 ` [PATCH 7/8] drm/i915: Enable querying offset of UV plane with intel_plane_obj_offset Tvrtko Ursulin
2015-05-08 12:02 ` [PATCH 8/8] drm/i915: Allow NV12 with 90/270 rotation Tvrtko Ursulin
2015-05-11  3:52   ` shuang.he
2015-05-08 14:59 ` Damien Lespiau [this message]
2015-05-08 15:05   ` [PATCH 0/8] NV12 90/270 rotated GGTT mapping Tvrtko Ursulin

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=20150508145928.GA19801@strange.ger.corp.intel.com \
    --to=damien.lespiau@intel.com \
    --cc=Intel-gfx@lists.freedesktop.org \
    --cc=tvrtko.ursulin@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox