Re: [Intel-gfx] [PATCH] drm/i915/adlp: Remove require_force_probe protection

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Mon, Nov 15, 2021 at 03:53:45PM -0800, clinton.a.taylor@intel.com wrote:
> From: Clint Taylor <clinton.a.taylor@intel.com>
> 
>     drm/i915/adlp: Remove require_force_probe protection
> 
>     Removing force probe protection from ADL_P platform. Did
>     not observe warnings, errors, flickering or any visual
>     defects while doing ordinary tasks like browsing and
>     editing documents in a two monitor setup.

some strange alignment here... please remove the extra
tabs here.

> 
>     For more info drm-tip idle run results :
>     https://intel-gfx-ci.01.org/tree/drm-tip/drmtip.html?

hmm... I could swear that I had seen the ADL-P green there a few
days ago as well... But right now I couldn't see ADL-P there...

So that fails on having a *reliable* green CI picture...
Any idea why that is down at this moment?

> 
> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> 
> Signed-off-by: Clint Taylor <clinton.a.taylor@intel.com>
> ---
>  drivers/gpu/drm/i915/i915_pci.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
> index 4c7fcc5f9a97..af9f4988bd88 100644
> --- a/drivers/gpu/drm/i915/i915_pci.c
> +++ b/drivers/gpu/drm/i915/i915_pci.c
> @@ -970,7 +970,6 @@ static const struct intel_device_info adl_p_info = {
>  	GEN12_FEATURES,
>  	XE_LPD_FEATURES,
>  	PLATFORM(INTEL_ALDERLAKE_P),
> -	.require_force_probe = 1,
>  	.cpu_transcoder_mask = BIT(TRANSCODER_A) | BIT(TRANSCODER_B) |
>  			       BIT(TRANSCODER_C) | BIT(TRANSCODER_D) |
>  			       BIT(TRANSCODER_DSI_0) | BIT(TRANSCODER_DSI_1),
> -- 
> 2.33.1
>