Re: [PATCH 5/9] drm/xe: Add platform-level step info to xe_probed_info

[Violet Monti <violet.monti@intel.com>]

On Tue, Jun 09, 2026 at 05:17:37PM -0300, Gustavo Sousa wrote:
> The platform-level step information depends on the PCI revid and, as
> such, should be probed in xe_probe_info_early() instead of
> xe_info_init_early().  Move the code accordingly.
> 
> Note that we currently only update probed_info->step.platform as part
> of this change.  We will deal with the other fields of
> probed_info->step as a follow-up change, which will be tied to the
> probing of graphics and media IPs.
> 
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>

Reviewed-by: Violet Monti <violet.monti@intel.com>

> ---
>  drivers/gpu/drm/xe/xe_pci.c | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index c5c75fbfa0e2..1c53a25442a9 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -730,12 +730,14 @@ static int handle_gmdid(struct xe_device *xe,
>  struct xe_probed_info {
>  	u16 devid;
>  	u8 revid;
> +	struct xe_step_info step;
>  };
>  
>  /*
>   * Probe from the hardware the info required by xe_info_init_early().
>   */
>  static int xe_probe_info_early(struct xe_device *xe,
> +			       const struct xe_device_desc *desc,
>  			       struct xe_probed_info *probed_info)
>  {
>  	struct pci_dev *pdev = to_pci_dev(xe->drm.dev);
> @@ -743,6 +745,8 @@ static int xe_probe_info_early(struct xe_device *xe,
>  	probed_info->devid = pdev->device;
>  	probed_info->revid = pdev->revision;
>  
> +	xe_step_platform_get(desc->platform, probed_info->revid, &probed_info->step);
> +
>  	return 0;
>  }
>  
> @@ -759,6 +763,7 @@ static int xe_info_init_early(struct xe_device *xe,
>  
>  	xe->info.devid = probed_info->devid;
>  	xe->info.revid = probed_info->revid;
> +	xe->info.step.platform = probed_info->step.platform;
>  
>  	xe->info.platform_name = desc->platform_name;
>  	xe->info.platform = desc->platform;
> @@ -808,8 +813,6 @@ static int xe_info_init_early(struct xe_device *xe,
>  	xe->info.max_gt_per_tile = desc->max_gt_per_tile;
>  	xe->info.tile_count = 1 + desc->max_remote_tiles;
>  
> -	xe_step_platform_get(xe->info.platform, xe->info.revid, &xe->info.step);
> -
>  	err = xe_tile_init_early(xe_device_get_root_tile(xe), xe, 0);
>  	if (err)
>  		return err;
> @@ -1139,7 +1142,7 @@ static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>  
>  	pci_set_master(pdev);
>  
> -	err = xe_probe_info_early(xe, &probed_info);
> +	err = xe_probe_info_early(xe, desc, &probed_info);
>  	if (err)
>  		return err;
>  
> 
> -- 
> 2.53.0
> 

-- 
--
Violet Monti