Re: [PATCH v2] drm/i915/gsc: Only initialize GSC in tile 0

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> From: José Roberto de Souza <jose.souza@xxxxxxxxx>
> 
> For multi-tile setups the GSC operational only on the tile 0.
> Skip GSC auxiliary device creation for all other tiles in GSC device init code.
> Initialize basic GSC fields and use the same path as HECI1 (HECI_PXP) device
> disable.
> 
> Cc: Tomas Winkler <tomas.winkler@xxxxxxxxx>
> Cc: Vitaly Lubart <vitaly.lubart@xxxxxxxxx>
> Signed-off-by: José Roberto de Souza <jose.souza@xxxxxxxxx>
> Signed-off-by: Alexander Usyskin <alexander.usyskin@xxxxxxxxx>

Acked-by: Tomas Winkler <tomas.winkler@xxxxxxxxx>
> ---
> V2: Move decision to skip initialization into GSC device init code.
>     This initializes basic GSC fields and uses the same path
>     as HECI1 (HECI_PXP) device disable.
>     It is simpler and protects interrupt handler too.
> 
>  drivers/gpu/drm/i915/gt/intel_gsc.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/gt/intel_gsc.c
> b/drivers/gpu/drm/i915/gt/intel_gsc.c
> index 976fdf27e790..bcc3605158db 100644
> --- a/drivers/gpu/drm/i915/gt/intel_gsc.c
> +++ b/drivers/gpu/drm/i915/gt/intel_gsc.c
> @@ -174,6 +174,14 @@ static void gsc_init_one(struct drm_i915_private
> *i915, struct intel_gsc *gsc,
>  	intf->irq = -1;
>  	intf->id = intf_id;
> 
> +	/*
> +	 * On the multi-tile setups the GSC is functional on the first tile only
> +	 */
> +	if (gsc_to_gt(gsc)->info.id != 0) {
> +		drm_dbg(&i915->drm, "Not initializing gsc for remote
> tiles\n");
> +		return;
> +	}
> +
>  	if (intf_id == 0 && !HAS_HECI_PXP(i915))
>  		return;
> 
> --
> 2.34.1





[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux