> -----Original Message-----
> From: Intel-gfx <intel-gfx-boun...@lists.freedesktop.org> On Behalf Of Ville
> Syrjala
> Sent: Friday, February 9, 2024 8:38 PM
> To: intel-gfx@lists.freedesktop.org
> Subject: [PATCH 5/5] drm/i915: Enable fastboot across the board
>
> From: Ville Syrjälä <ville.syrj...@linux.intel.com>
>
> There's nothing magical about vlv+ platforms vs. fastboot.
> If it works somewhere it should work everywhere, assuming we've not missed
> any crucial state checks. That seems unlikely on
> older platforms with less state to check anyway.
>
> Just enable fastboot across the board, and the remove the remnants of the
> optional stuff (we already removed the modparam for
> fastboot anyway).
>
Reviewed-by: Mika Kahola <mika.kah...@intel.com>
> Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_display.c | 22 --------------------
> 1 file changed, 22 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_display.c
> b/drivers/gpu/drm/i915/display/intel_display.c
> index 66ee6749fdae..00ac65a14029 100644
> --- a/drivers/gpu/drm/i915/display/intel_display.c
> +++ b/drivers/gpu/drm/i915/display/intel_display.c
> @@ -4937,20 +4937,6 @@ pipe_config_pll_mismatch(bool fastset,
> }
> }
>
> -static bool fastboot_enabled(struct drm_i915_private *dev_priv) -{
> - /* Enable fastboot by default on Skylake and newer */
> - if (DISPLAY_VER(dev_priv) >= 9)
> - return true;
> -
> - /* Enable fastboot by default on VLV and CHV */
> - if (IS_VALLEYVIEW(dev_priv) || IS_CHERRYVIEW(dev_priv))
> - return true;
> -
> - /* Disabled by default on all others */
> - return false;
> -}
> -
> bool
> intel_pipe_config_compare(const struct intel_crtc_state *current_config,
> const struct intel_crtc_state *pipe_config, @@
> -4959,14 +4945,6 @@ intel_pipe_config_compare(const
> struct intel_crtc_state *current_config,
> struct drm_i915_private *dev_priv =
> to_i915(current_config->uapi.crtc->dev);
> struct intel_crtc *crtc = to_intel_crtc(pipe_config->uapi.crtc);
> bool ret = true;
> - bool fixup_inherited = fastset &&
> - current_config->inherited && !pipe_config->inherited;
> -
> - if (fixup_inherited && !fastboot_enabled(dev_priv)) {
> - drm_dbg_kms(&dev_priv->drm,
> - "initial modeset and fastboot not set\n");
> - ret = false;
> - }
>
> #define PIPE_CONF_CHECK_X(name) do { \
> if (current_config->name != pipe_config->name) { \
> --
> 2.43.0
