On Tue, 21 Feb 2023, Imre Deak <imre.deak@xxxxxxxxx> wrote: > Move the display debugfs registration later, after initializing steps > for opregion/acpi/audio. These latter ones don't depend on the debugfs > entries, OTOH some debugfs entries may depend on the initialized state. > > Signed-off-by: Imre Deak <imre.deak@xxxxxxxxx> Patches 1-3, Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx> > --- > drivers/gpu/drm/i915/display/intel_display.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c > index a1fbdf32bd214..7827ef3e2d1e6 100644 > --- a/drivers/gpu/drm/i915/display/intel_display.c > +++ b/drivers/gpu/drm/i915/display/intel_display.c > @@ -8885,14 +8885,14 @@ void intel_display_driver_register(struct drm_i915_private *i915) > if (!HAS_DISPLAY(i915)) > return; > > - intel_display_debugfs_register(i915); > - > /* Must be done after probing outputs */ > intel_opregion_register(i915); > intel_acpi_video_register(i915); > > intel_audio_init(i915); > > + intel_display_debugfs_register(i915); > + > /* > * Some ports require correctly set-up hpd registers for > * detection to work properly (leading to ghost connected -- Jani Nikula, Intel Open Source Graphics Center
