On Fri, Nov 18, 2016 at 09:52:53PM +0200, [email protected] wrote: > From: Ville Syrjälä <[email protected]> > > We want the fbs inherited from the BIOS to be more or less fully working > prior to actually registering them. This will allow us to just pass the > fb to various helper function instead of having to pass all the > different parameters separately. > > Cc: [email protected] > Signed-off-by: Ville Syrjälä <[email protected]>
I guess eventually we'll grow a WARN_ON(!fb->dev) in drm_framebuffer_init() to make sure these don't escape? Anyway: Reviewed-by: Daniel Vetter <[email protected]> > --- > drivers/gpu/drm/i915/intel_display.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/drivers/gpu/drm/i915/intel_display.c > b/drivers/gpu/drm/i915/intel_display.c > index 12af936a402d..74a638c8de61 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -8704,6 +8704,8 @@ i9xx_get_initial_plane_config(struct intel_crtc *crtc, > > fb = &intel_fb->base; > > + fb->dev = dev; > + > if (INTEL_GEN(dev_priv) >= 4) { > if (val & DISPPLANE_TILED) { > plane_config->tiling = I915_TILING_X; > @@ -9734,6 +9736,8 @@ skylake_get_initial_plane_config(struct intel_crtc > *crtc, > > fb = &intel_fb->base; > > + fb->dev = dev; > + > val = I915_READ(PLANE_CTL(pipe, 0)); > if (!(val & PLANE_CTL_ENABLE)) > goto error; > @@ -9846,6 +9850,8 @@ ironlake_get_initial_plane_config(struct intel_crtc > *crtc, > > fb = &intel_fb->base; > > + fb->dev = dev; > + > if (INTEL_GEN(dev_priv) >= 4) { > if (val & DISPPLANE_TILED) { > plane_config->tiling = I915_TILING_X; > -- > 2.7.4 > > _______________________________________________ > Intel-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
