[Intel-gfx] [PATCH] drm/i915/mtl: Remove the 'force_probe' requirement for Meteor Lake

[Karthik B S]

Hi,

On 10/8/2023 10:18 PM, Andi Shyti wrote:
> From: Radhakrishna Sripada <radhakrishna.sripada at intel.com>
>
> Meteor Lake has demonstrated consistent stability for some time.
> All user-space API modifications tide to its core platform
> functions are operational.
>
> The necessary firmware components are set up and comprehensive
> testing has been condused over a period.
>
> Given the recent faborable CI results, as well, we believe it's
> time to eliminate the 'force_probe' prerequisite and activate the
> platform by default.
>
> Signed-off-by: Aditya Chauhan <aditya.chauhan at intel.com>
> Signed-off-by: Andrzej Hajda <andrzej.hajda at intel.com>
> Signed-off-by: Chris Wilson <chris.p.wilson at linux.intel.com>
> Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik at linux.intel.com>
> Signed-off-by: Jonathan Cavitt <jonathan.cavitt at intel.com>
> Signed-off-by: Nirmoy Das <nirmoy.das at intel.com>
> Signed-off-by: Radhakrishna Sripada <radhakrishna.sripada at intel.com>
> Signed-off-by: Andi Shyti <andi.shyti at linux.intel.com>
> Cc: Jani Nikula <jani.nikula at linux.intel.com>
> Cc: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
> Cc: Rodrigo Vivi <rodrigo.vivi at intel.com>
> Cc: Tvrtko Ursulin <tvrtko.ursulin at intel.com>

Verified with the latest drmtip(CI_DRM_13736) on eDP+HDMI config on MTL. 
System is booting seamlessly into Ubuntu UI and played around with a few 
display settings as well.(Single display, clone and extended modes).

Also tried hot-unplug and plug for the HDMI and is working as expected. 
Basic video playback was also verified on both eDP and HDMI.

Tested-by: Karthik B S <karthik.b.s at intel.com>

> ---
> Hello,
>
> This patch eliminates the 'force probe' for the MTL platforms. Over the recent
> weeks, MTL has demonstrated stability, consistently passing BAT tests with
> success rates ranging from 98% to 100%.
>
> There's a single issue hindering us from achieving a 100% BAT test coverage.
> Fortunately, we've identified the issue, and the proposed solution can be found
> here[*]. The CI results are encouraging.
>
> Once all reviews are addressed, we plan to submit this series with the "Fixes:"
> tag.
>
> Thank you and best regards,
> Andi and Radhakrishna
>
> [*] https://patchwork.freedesktop.org/series/124744/
>
>   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 df7c261410f7..fe748906c06f 100644
> --- a/drivers/gpu/drm/i915/i915_pci.c
> +++ b/drivers/gpu/drm/i915/i915_pci.c
> @@ -836,7 +836,6 @@ static const struct intel_device_info mtl_info = {
>   	.has_pxp = 1,
>   	.memory_regions = REGION_SMEM | REGION_STOLEN_LMEM,
>   	.platform_engine_mask = BIT(RCS0) | BIT(BCS0) | BIT(CCS0),
> -	.require_force_probe = 1,
>   	MTL_CACHELEVEL,
>   };
>