Re: [Intel-gfx] [PATCH 4/9] drm/i915: Introduce intel_prepare_cursor_plane()

2014-12-01 Thread Ander Conselvan de Oliveira
On 11/24/2014 09:53 PM, Matt Roper wrote: Primary and sprite planes have already been refactored to include a 'prepare' step which handles all the commit-time operations that could fail (i.e., pinning buffers and such). Refactor the cursor commit in a similar manner. For simplicity and consiste

Re: [Intel-gfx] [PATCH 4/9] drm/i915: Introduce intel_prepare_cursor_plane()

2014-11-28 Thread Daniel Vetter
On Fri, Nov 28, 2014 at 02:15:08PM +0200, Ander Conselvan de Oliveira wrote: > On 11/24/2014 09:53 PM, Matt Roper wrote: > >Primary and sprite planes have already been refactored to include a > >'prepare' step which handles all the commit-time operations that could > >fail (i.e., pinning buffers an

Re: [Intel-gfx] [PATCH 4/9] drm/i915: Introduce intel_prepare_cursor_plane()

2014-11-28 Thread Ander Conselvan de Oliveira
On 11/28/2014 02:15 PM, Ander Conselvan de Oliveira wrote: On 11/24/2014 09:53 PM, Matt Roper wrote: Primary and sprite planes have already been refactored to include a 'prepare' step which handles all the commit-time operations that could fail (i.e., pinning buffers and such). Refactor the cur

Re: [Intel-gfx] [PATCH 4/9] drm/i915: Introduce intel_prepare_cursor_plane()

2014-11-28 Thread Ander Conselvan de Oliveira
On 11/24/2014 09:53 PM, Matt Roper wrote: Primary and sprite planes have already been refactored to include a 'prepare' step which handles all the commit-time operations that could fail (i.e., pinning buffers and such). Refactor the cursor commit in a similar manner. For simplicity and consiste

[Intel-gfx] [PATCH 4/9] drm/i915: Introduce intel_prepare_cursor_plane()

2014-11-24 Thread Matt Roper
Primary and sprite planes have already been refactored to include a 'prepare' step which handles all the commit-time operations that could fail (i.e., pinning buffers and such). Refactor the cursor commit in a similar manner. For simplicity and consistency with other plane types, we also switch t