On Mon, Oct 07, 2024 at 09:29:46PM +0300, Cavitt, Jonathan wrote: > -----Original Message----- > From: Intel-xe <intel-xe-boun...@lists.freedesktop.org> On Behalf Of Imre Deak > Sent: Monday, October 7, 2024 7:06 AM > To: intel...@lists.freedesktop.org; intel-gfx@lists.freedesktop.org > Cc: Vivi, Rodrigo <rodrigo.v...@intel.com> > Subject: [PATCH 2/2] drm/xe: Add missing HPD interrupt enabling during > non-d3cold RPM resume > > > > Atm the display HPD interrupts that got disabled during runtime > > suspend, are re-enabled only if d3cold is enabled. Fix things by > > also re-enabling the interrupts if d3cold is disabled. > > > > Cc: Rodrigo Vivi <rodrigo.v...@intel.com> > > Signed-off-by: Imre Deak <imre.d...@intel.com> > > Is there a reason this ended up as a separate patch?
On the principle of doing one thing in one patch: the first is refactoring, this one is a fix. > Not blocking, just curious. > Reviewed-by: Jonathan Cavitt <jonathan.cav...@intel.com> > -Jonathan Cavitt > > > --- > > drivers/gpu/drm/xe/display/xe_display.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/gpu/drm/xe/display/xe_display.c > > b/drivers/gpu/drm/xe/display/xe_display.c > > index cb2449b7921ac..695c27ac6b0f8 100644 > > --- a/drivers/gpu/drm/xe/display/xe_display.c > > +++ b/drivers/gpu/drm/xe/display/xe_display.c > > @@ -486,6 +486,7 @@ void xe_display_pm_runtime_resume(struct xe_device *xe) > > return; > > } > > > > + intel_hpd_init(xe); > > intel_hpd_poll_disable(xe); > > } > > > > -- > > 2.44.2 > > > >
