On Fri, Jul 29, 2016 at 05:06:13PM +0300, Ville Syrjälä wrote:
> On Thu, Jul 28, 2016 at 12:12:17PM -0700, Carlos Santa wrote:
> > Based on the GEN7_FEATURES changes from Ben W.
> >
> > Use it for snb.
> >
> > Signed-off-by: Carlos Santa <carlos.santa@xxxxxxxxx>
> > Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
> > ---
> > drivers/gpu/drm/i915/i915_pci.c | 26 ++++++++++++--------------
> > 1 file changed, 12 insertions(+), 14 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
> > index 08cdeb3..8b1311d 100644
> > --- a/drivers/gpu/drm/i915/i915_pci.c
> > +++ b/drivers/gpu/drm/i915/i915_pci.c
> > @@ -194,24 +194,22 @@ static const struct intel_device_info intel_ironlake_m_info = {
> > CURSOR_OFFSETS,
> > };
> >
> > +#define GEN6_FEATURES \
> > + .gen = 6, .num_pipes = 2, \
> > + .need_gfx_hws = 1, .has_hotplug = 1, \
>
> BTW .need_gfx_hws has been bugging me for a while now. For the other
> similar things we make "physical" the exception, so I think we flip this
> flag over as well. Especially as the majority of our platforms falls in
> the gtt hws, not physical hws camp.
Yes. Very much please. need_phys_* should be the exception.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
