All of lore.kernel.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 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.