On Tue, 2025-03-11 at 21:56 +0200, Ville Syrjala wrote: > From: Ville Syrjälä <ville.syrj...@linux.intel.com> > > Since this switcheroo garbage bypasses all the core pm we > have to manually manage the pci state. To that end add the > missing pci_restore_state() to the switcheroo resume hook. > We already have the pci_save_state() counterpart on the > suspend side. > > Arguably none of this code should exist in the driver > in the first place, and instead the entire switcheroo > mechanism should be rewritten and properly integrated into > core pm code... > > Cc: Rodrigo Vivi <rodrigo.v...@intel.com> > Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
Reviewed-by: Jouni Högander <jouni.hogan...@intel.com> > --- > drivers/gpu/drm/i915/i915_driver.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/i915/i915_driver.c > b/drivers/gpu/drm/i915/i915_driver.c > index 995205e24ebf..a199e21c7bb4 100644 > --- a/drivers/gpu/drm/i915/i915_driver.c > +++ b/drivers/gpu/drm/i915/i915_driver.c > @@ -1340,6 +1340,8 @@ int i915_driver_resume_switcheroo(struct > drm_i915_private *i915) > if (ret) > return ret; > > + pci_restore_state(pdev); > + > ret = i915_drm_resume_early(&i915->drm); > if (ret) > return ret;
