Message ID | 1390969547-1018-11-git-send-email-benjamin.widawsky@intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Wed, Jan 29, 2014 at 2:25 AM, Ben Widawsky <benjamin.widawsky@intel.com> 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@bwidawsk.net> > --- > 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@gmail.com> > /* > * PCU communication is slow and this doesn't need to be > * done at any specific time, so do this out of our fast path > -- > 1.8.5.3 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
On Wed, Jan 29, 2014 at 9:55 AM, Ben Widawsky > <benjamin.widawsky@intel.com <mailto:benjamin.widawsky@intel.com>> 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@bwidawsk.net <mailto:ben@bwidawsk.net>> > --- > 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)) { > /* > * PCU communication is slow and this doesn't need > to be > * done at any specific time, so do this out of our > fast path > -- > 1.8.5.3 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org <mailto:Intel-gfx@lists.freedesktop.org> > http://lists.freedesktop.org/mailman/listinfo/intel-gfx > Reviewed-by: Deepak S <deepak.s@intel.com>
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@intel.com> 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@bwidawsk.net> > > --- > > 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@gmail.com> > 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. 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
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@intel.com> 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@bwidawsk.net> > > > --- > > > 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@gmail.com> > > > > 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@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
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)) { /* * PCU communication is slow and this doesn't need to be * done at any specific time, so do this out of our fast path
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@bwidawsk.net> --- drivers/gpu/drm/i915/intel_pm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)