Re: [Intel-gfx] [PATCH 3/4] drm/i915/opregion: Register display debugfs later, after initialization steps

[Jani Nikula <jani.nikula@linux.intel.com>]

On Tue, 21 Feb 2023, Imre Deak <imre.deak@intel.com> 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@intel.com>

Patches 1-3,

Reviewed-by: Jani Nikula <jani.nikula@intel.com>


> ---
>  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