-----Original Message----- From: Intel-gfx <[email protected]> On Behalf Of Rodrigo Vivi Sent: Tuesday, September 24, 2024 1:36 PM To: [email protected]; [email protected] Cc: Deak, Imre <[email protected]>; Vivi, Rodrigo <[email protected]>; Govindapillai, Vinod <[email protected]> Subject: [PATCH 24/31] drm/xe/display: Call intel_hpd_init on every runtime resume > > We need to ensure this is called on every kind of runtime > resume and not only on d3cold is possible. Aligning with > the hpd sequence from i915. > > Cc: Vinod Govindapillai <[email protected]> > Signed-off-by: Rodrigo Vivi <[email protected]>
LGTM. Reviewed-by: Jonathan Cavitt <[email protected]> -Jonathan Cavitt > --- > drivers/gpu/drm/xe/display/xe_display.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/xe/display/xe_display.c > b/drivers/gpu/drm/xe/display/xe_display.c > index 1ab4dd51094f..4d1dd5294b89 100644 > --- a/drivers/gpu/drm/xe/display/xe_display.c > +++ b/drivers/gpu/drm/xe/display/xe_display.c > @@ -330,8 +330,6 @@ static void xe_display_from_d3cold(struct xe_device *xe) > > intel_display_driver_init_hw(xe); > > - intel_hpd_init(xe); > - > intel_opregion_resume(display); > > intel_power_domains_enable(xe); > @@ -470,6 +468,8 @@ void xe_display_pm_runtime_resume(struct xe_device *xe) > > if (xe->d3cold.allowed) > xe_display_from_d3cold(xe); > + > + intel_hpd_init(xe); > } > > static void display_device_remove(struct drm_device *dev, void *arg) > -- > 2.46.0 > >
