On Tue, 07 Nov 2017, Daniel Vetter <[email protected]> wrote: > Now that we have CI, and that pm_rpm fully passes (I guess the audio > folks have implemented proper runtime pm for snd-hda, hooray, pls > confirm) it's time to enable this again by default. > > Real goal here is to have 1 configuration only that we fully support, > instead of tons of different codes with every user/customer tuning it > differently. And really, power stuff should work by default, and > should be enabled by everywhere where it is save to do so. > > v2: Completely new commit message, a few years passed since v1 ...
I suppose this is something that could use more than a single round of IGT CI before merging...? BR, Jani. > > Cc: Takashi Iwai <[email protected]> > Cc: Liam Girdwood <[email protected]> > Cc: "Yang, Libin" <[email protected]> > Cc: "Lin, Mengdong" <[email protected]> > Cc: "Li, Jocelyn" <[email protected]> > Cc: "Kaskinen, Tanu" <[email protected]> > Cc: "Zanoni, Paulo R" <[email protected]> > Signed-off-by: Daniel Vetter <[email protected]> > --- > drivers/gpu/drm/i915/intel_runtime_pm.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_runtime_pm.c > b/drivers/gpu/drm/i915/intel_runtime_pm.c > index 8315499452dc..dc24d008d8d4 100644 > --- a/drivers/gpu/drm/i915/intel_runtime_pm.c > +++ b/drivers/gpu/drm/i915/intel_runtime_pm.c > @@ -3232,7 +3232,7 @@ void intel_runtime_pm_enable(struct drm_i915_private > *dev_priv) > struct pci_dev *pdev = dev_priv->drm.pdev; > struct device *kdev = &pdev->dev; > > - pm_runtime_set_autosuspend_delay(kdev, 10000); /* 10s */ > + pm_runtime_set_autosuspend_delay(kdev, 100); > pm_runtime_mark_last_busy(kdev); > > /* > @@ -3251,6 +3251,8 @@ void intel_runtime_pm_enable(struct drm_i915_private > *dev_priv) > pm_runtime_use_autosuspend(kdev); > } > > + pm_runtime_allow(kdev); > + > /* > * The core calls the driver load handler with an RPM reference held. > * We drop that here and will reacquire it during unloading in -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
