[Intel-xe] [PATCH v1 2/7] drm/xe/uapi: Remove GT_TYPE_REMOTE

[Souza, Jose]

On Fri, 2023-11-10 at 15:41 +0000, Francois Dugast wrote:
> From: Rodrigo Vivi <rodrigo.vivi at intel.com>
> 
> With the split between tile and gt, this is currently unused.
> Also it is bringing confusion because main vs remote would be
> more a concept of the tile itself and not about GT.
> 
> So, the MAIN one is the traditional GT used for every operation
> in older platforms, and for render/graphics and compute on platforms
> that contains the stand-alone Media GT.

Reviewed-by: José Roberto de Souza <jose.souza at intel.com>

> 
> Cc: Matt Roper <matthew.d.roper at intel.com>
> Cc: Francois Dugast <francois.dugast at intel.com>
> Cc: Carl Zhang <carl.zhang at intel.com>
> Cc: José Roberto de Souza <jose.souza at intel.com>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
> Signed-off-by: Francois Dugast <francois.dugast at intel.com>
> ---
>  drivers/gpu/drm/xe/xe_query.c | 2 --
>  include/uapi/drm/xe_drm.h     | 5 ++---
>  2 files changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_query.c b/drivers/gpu/drm/xe/xe_query.c
> index 6b1cd5c68a99..43bc6094855c 100644
> --- a/drivers/gpu/drm/xe/xe_query.c
> +++ b/drivers/gpu/drm/xe/xe_query.c
> @@ -373,8 +373,6 @@ static int query_gt_list(struct xe_device *xe, struct drm_xe_device_query *query
>  	for_each_gt(gt, xe, id) {
>  		if (xe_gt_is_media_type(gt))
>  			gt_list->gt_list[id].type = XE_QUERY_GT_TYPE_MEDIA;
> -		else if (gt_to_tile(gt)->id > 0)
> -			gt_list->gt_list[id].type = XE_QUERY_GT_TYPE_REMOTE;
>  		else
>  			gt_list->gt_list[id].type = XE_QUERY_GT_TYPE_MAIN;
>  		gt_list->gt_list[id].gt_id = gt->info.id;
> diff --git a/include/uapi/drm/xe_drm.h b/include/uapi/drm/xe_drm.h
> index b9a68f8b69f3..8154cecf6f0d 100644
> --- a/include/uapi/drm/xe_drm.h
> +++ b/include/uapi/drm/xe_drm.h
> @@ -343,9 +343,8 @@ struct drm_xe_query_config {
>   */
>  struct drm_xe_query_gt {
>  #define XE_QUERY_GT_TYPE_MAIN		0
> -#define XE_QUERY_GT_TYPE_REMOTE		1
> -#define XE_QUERY_GT_TYPE_MEDIA		2
> -	/** @type: GT type: Main, Remote, or Media */
> +#define XE_QUERY_GT_TYPE_MEDIA		1
> +	/** @type: GT type: Main or Media */
>  	__u16 type;
>  	/** @gt_id: Unique ID of this GT within the PCI Device */
>  	__u16 gt_id;