Re: [Intel-gfx] [PATCH 2/2] drm/i915: Make use of intel_fb_obj() (v2)

2014-07-09 Thread Daniel Vetter
On Wed, Jul 09, 2014 at 10:29:09AM +0100, Chris Wilson wrote: > On Tue, Jul 08, 2014 at 07:50:07AM -0700, Matt Roper wrote: > > This should hopefully simplify the display code slightly and also > > solves at least one mistake in intel_pipe_set_base() where > > to_intel_framebuffer(fb)->obj is refer

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Make use of intel_fb_obj() (v2)

2014-07-09 Thread Chris Wilson
On Tue, Jul 08, 2014 at 07:50:07AM -0700, Matt Roper wrote: > This should hopefully simplify the display code slightly and also > solves at least one mistake in intel_pipe_set_base() where > to_intel_framebuffer(fb)->obj is referenced during local variable > initialization, before 'if (!fb)' gets c

[Intel-gfx] [PATCH 2/2] drm/i915: Make use of intel_fb_obj() (v2)

2014-07-08 Thread Matt Roper
This should hopefully simplify the display code slightly and also solves at least one mistake in intel_pipe_set_base() where to_intel_framebuffer(fb)->obj is referenced during local variable initialization, before 'if (!fb)' gets checked. Potential uses of this macro were identified via the follow