Re: [PATCH 9/9] drm/i915/bdw: Enable RC6

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Feb 17, 2014 at 12:04:19PM -0800, Ben Widawsky wrote:
> On Thu, Feb 06, 2014 at 11:54:44AM -0200, Rodrigo Vivi wrote:
> > On Wed, Jan 29, 2014 at 2:25 AM, Ben Widawsky
> > <benjamin.widawsky@xxxxxxxxx> wrote:
> > > It is tested and looking fairly stable now, so turn it on. It wasn't
> > > intentionally turned off originally :P
> > >
> > > Signed-off-by: Ben Widawsky <ben@xxxxxxxxxxxx>
> > > ---
> > >  drivers/gpu/drm/i915/intel_pm.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
> > > index a5c9142..377bd7f 100644
> > > --- a/drivers/gpu/drm/i915/intel_pm.c
> > > +++ b/drivers/gpu/drm/i915/intel_pm.c
> > > @@ -4428,7 +4428,7 @@ void intel_enable_gt_powersave(struct drm_device *dev)
> > >                 ironlake_enable_drps(dev);
> > >                 ironlake_enable_rc6(dev);
> > >                 intel_init_emon(dev);
> > > -       } else if (IS_GEN6(dev) || IS_GEN7(dev)) {
> > > +       } else if (IS_GEN6(dev) || IS_GEN7(dev) || IS_BROADWELL(dev)) {
> > 
> > why not IS_GEN8?
> > or why not use gen >= gen6?
> > 
> > anyway: Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
> > 
> 
> This is my personal preference, though we have some ongoing discussions
> in this area. I do not like to explicitly add support for future
> platforms except for the most trivial things. I don't think everyone
> agrees with me.

s/explicitly/implicitly

> 
> Anyway, thanks for the review. I'll be resending the series with a
> fairly trivial, though large change.
> 
> > 
> > 
> > 
> > -- 
> > Rodrigo Vivi
> > Blog: http://blog.vivi.eng.br
> 
> -- 
> Ben Widawsky, Intel Open Source Technology Center
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Ben Widawsky, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux