On Thu, Dec 12, 2013 at 04:19:34PM +0200, Imre Deak wrote:
> On Thu, 2013-11-28 at 17:30 +0200, ville.syrj...@linux.intel.com wrote:
> > From: Ville Syrjälä <ville.syrj...@linux.intel.com>
> > 
> > All mobile gen2 and gen3 chipsets should have FBC1, and the code
> > should now handle them all. So just set has_fbc=true for all such
> > chipsets.
> > 
> > Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
> 
> Based on the above we could also enable fbc for i830. In any case:
> Reviewed-by: Imre Deak <imre.d...@intel.com>

According to the spec there's no FBC on 830, so I guess my commit
message was a tad incorrect. Some part of the spec hints that FBC
might be avilable on 865 too, but in another part only 85x is
mentioned.

> 
> > ---
> >  drivers/gpu/drm/i915/i915_drv.c | 3 +++
> >  1 file changed, 3 insertions(+)
> > 
> > diff --git a/drivers/gpu/drm/i915/i915_drv.c 
> > b/drivers/gpu/drm/i915/i915_drv.c
> > index 0ec0fb3..923c8b6 100644
> > --- a/drivers/gpu/drm/i915/i915_drv.c
> > +++ b/drivers/gpu/drm/i915/i915_drv.c
> > @@ -172,6 +172,7 @@ static const struct intel_device_info intel_i85x_info = 
> > {
> >     .gen = 2, .is_i85x = 1, .is_mobile = 1, .num_pipes = 2,
> >     .cursor_needs_physical = 1,
> >     .has_overlay = 1, .overlay_needs_physical = 1,
> > +   .has_fbc = 1,
> >     .ring_mask = RENDER_RING,
> >  };
> >  
> > @@ -191,6 +192,7 @@ static const struct intel_device_info intel_i915gm_info 
> > = {
> >     .cursor_needs_physical = 1,
> >     .has_overlay = 1, .overlay_needs_physical = 1,
> >     .supports_tv = 1,
> > +   .has_fbc = 1,
> >     .ring_mask = RENDER_RING,
> >  };
> >  static const struct intel_device_info intel_i945g_info = {
> > @@ -203,6 +205,7 @@ static const struct intel_device_info intel_i945gm_info 
> > = {
> >     .has_hotplug = 1, .cursor_needs_physical = 1,
> >     .has_overlay = 1, .overlay_needs_physical = 1,
> >     .supports_tv = 1,
> > +   .has_fbc = 1,
> >     .ring_mask = RENDER_RING,
> >  };
> >  
> 



-- 
Ville Syrjälä
Intel OTC
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to