On Thu, Mar 16, 2017 at 11:58:09PM +0530, Sagar Arun Kamble wrote:
> From: Tom O'Rourke <Tom.O'Rourke@xxxxxxxxx>
>
> The SLPC interface is dependent on GuC version.
> Only GuC versions known to be compatible are supported here.
>
> SLPC with GuC firmware v9 is supported with this series.
> Other platforms and corresponding checks added later in the series.
>
> v1: Updated with modified sanitize_slpc_option in earlier patch.
>
> v2-v3: Rebase.
>
> v4: Updated support for GuC firmware v9.
>
> v5: Commit subject updated.
>
> v6: Commit subject and message update. Add support condition as >=v9.
>
> Signed-off-by: Tom O'Rourke <Tom.O'Rourke@xxxxxxxxx>
> Signed-off-by: Sagar Arun Kamble <sagar.a.kamble@xxxxxxxxx>
> ---
> drivers/gpu/drm/i915/intel_uc.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/intel_uc.c b/drivers/gpu/drm/i915/intel_uc.c
> index 31b90fb..1dcfbd5 100644
> --- a/drivers/gpu/drm/i915/intel_uc.c
> +++ b/drivers/gpu/drm/i915/intel_uc.c
> @@ -83,6 +83,8 @@ void intel_uc_init_early(struct drm_i915_private *dev_priv)
>
> void sanitize_slpc_option(struct drm_i915_private *dev_priv)
> {
> + struct intel_uc_fw *guc_fw = &dev_priv->guc.fw;
> +
> /* slpc requires hardware support and compatible firmware */
> if (!HAS_SLPC(dev_priv))
> i915.enable_slpc = 0;
> @@ -94,6 +96,11 @@ void sanitize_slpc_option(struct drm_i915_private *dev_priv)
> /* slpc requires guc submission */
> if (!i915.enable_guc_submission)
> i915.enable_slpc = 0;
> +
> + if (guc_fw->major_ver_found < 9) {
> + DRM_INFO("SLPC not supported with current GuC firmware\n");
Say the actual version found, the requirement and the likely consequence.
Is it important enough to direct the user to 01.org?
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
