Re: [Intel-gfx] [PATCH 8/8] drm/i915: remove early fb allocation dependency on CONFIG_FB v2

2014-03-09 Thread Daniel Vetter
On Sun, Mar 9, 2014 at 6:33 PM, Jesse Barnes wrote: > On Sat, 8 Mar 2014 11:33:15 +0100 > Daniel Vetter wrote: > >> On Fri, Mar 07, 2014 at 08:57:55AM -0800, Jesse Barnes wrote: >> > By stuffing the fb allocation into the crtc, we get mode set lifetime >> > refcounting for free, but have to handl

Re: [Intel-gfx] [PATCH 8/8] drm/i915: remove early fb allocation dependency on CONFIG_FB v2

2014-03-09 Thread Jesse Barnes
On Sat, 8 Mar 2014 11:33:15 +0100 Daniel Vetter wrote: > On Fri, Mar 07, 2014 at 08:57:55AM -0800, Jesse Barnes wrote: > > By stuffing the fb allocation into the crtc, we get mode set lifetime > > refcounting for free, but have to handle the initial pin & fence > > slightly differently. It also

Re: [Intel-gfx] [PATCH 8/8] drm/i915: remove early fb allocation dependency on CONFIG_FB v2

2014-03-08 Thread Daniel Vetter
On Fri, Mar 07, 2014 at 08:57:55AM -0800, Jesse Barnes wrote: > By stuffing the fb allocation into the crtc, we get mode set lifetime > refcounting for free, but have to handle the initial pin & fence > slightly differently. It also means we can move the shared fb handling > into the core rather t

[Intel-gfx] [PATCH 8/8] drm/i915: remove early fb allocation dependency on CONFIG_FB v2

2014-03-07 Thread Jesse Barnes
By stuffing the fb allocation into the crtc, we get mode set lifetime refcounting for free, but have to handle the initial pin & fence slightly differently. It also means we can move the shared fb handling into the core rather than leaving it out in the fbdev code. v2: null out crtc->fb on error