[Intel-xe] [PATCH 1/2] xe/rpls: Add RPLS Support

[Matt Roper]

On Mon, May 22, 2023 at 12:35:40PM -0700, Anusha Srivatsa wrote:
> Add RPLS support that was missing apart from the PCI IDs.
> 
> Signed-off-by: Anusha Srivatsa <anusha.srivatsa at intel.com>

The subject prefix is missing the "drm" part.  But aside from that,

        Reviewed-by: Matt Roper <matthew.d.roper at intel.com>

> ---
>  drivers/gpu/drm/xe/xe_pci.c            | 7 +++++++
>  drivers/gpu/drm/xe/xe_platform_types.h | 1 +
>  2 files changed, 8 insertions(+)
> 
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index e789a50a1310..cb748b062e54 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -194,6 +194,8 @@ static const struct xe_device_desc rkl_desc = {
>  	.require_force_probe = true,
>  };
>  
> +static const u16 adls_rpls_ids[] = { XE_RPLS_IDS(NOP), 0 };
> +
>  static const struct xe_device_desc adl_s_desc = {
>  	.graphics = &graphics_xelp,
>  	.media = &media_xem,
> @@ -201,6 +203,10 @@ static const struct xe_device_desc adl_s_desc = {
>  	.has_display = true,
>  	.has_llc = true,
>  	.require_force_probe = true,
> +	.subplatforms = (const struct xe_subplatform_desc[]) {
> +		{ XE_SUBPLATFORM_ADLS_RPLS, "RPLS", adls_rpls_ids },
> +		{},
> +	},
>  };
>  
>  static const u16 adlp_rplu_ids[] = { XE_RPLU_IDS(NOP), 0 };
> @@ -339,6 +345,7 @@ static const struct pci_device_id pciidlist[] = {
>  	XE_ADLP_IDS(INTEL_VGA_DEVICE, &adl_p_desc),
>  	XE_ADLN_IDS(INTEL_VGA_DEVICE, &adl_n_desc),
>  	XE_RPLP_IDS(INTEL_VGA_DEVICE, &adl_p_desc),
> +	XE_RPLS_IDS(INTEL_VGA_DEVICE, &adl_s_desc),
>  	XE_DG1_IDS(INTEL_VGA_DEVICE, &dg1_desc),
>  	XE_ATS_M_IDS(INTEL_VGA_DEVICE, &ats_m_desc),
>  	XE_DG2_IDS(INTEL_VGA_DEVICE, &dg2_desc),
> diff --git a/drivers/gpu/drm/xe/xe_platform_types.h b/drivers/gpu/drm/xe/xe_platform_types.h
> index abbb8a1f29a8..394149838d06 100644
> --- a/drivers/gpu/drm/xe/xe_platform_types.h
> +++ b/drivers/gpu/drm/xe/xe_platform_types.h
> @@ -26,6 +26,7 @@ enum xe_platform {
>  enum xe_subplatform {
>  	XE_SUBPLATFORM_UNINITIALIZED = 0,
>  	XE_SUBPLATFORM_NONE,
> +	XE_SUBPLATFORM_ADLS_RPLS,
>  	XE_SUBPLATFORM_ADLP_RPLU,
>  	XE_SUBPLATFORM_DG2_G10,
>  	XE_SUBPLATFORM_DG2_G11,
> -- 
> 2.25.1
> 

-- 
Matt Roper
Graphics Software Engineer
Linux GPU Platform Enablement
Intel Corporation