[igt-dev] [i-g-t, v2, 2/2] lib/rpl_p: Add Raptor Lake P platform definition

[Petri Latvala]

On Thu, Apr 21, 2022 at 11:28:19AM +0530, Talla Raviteja Goud wrote:
> From: Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay at intel.com>
> 
> Adding Raptor lake platform definitions
> 
> v2: Modifying intel_device_info.c in seperate commit (Petri)
>     Earlier changes done in i915_pciids.h commit
> 
> Signed-off-by: Adam Miszczak <adam.miszczak at intel.com>
> Signed-off-by: Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay at intel.com>
> Signed-off-by: Raviteja Goud Talla <ravitejax.goud.talla at intel.com>

Acked-by: Petri Latvala <petri.latvala at intel.com>

but,


> ---
>  lib/intel_device_info.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/lib/intel_device_info.c b/lib/intel_device_info.c
> index e55841df..0253a732 100644
> --- a/lib/intel_device_info.c
> +++ b/lib/intel_device_info.c
> @@ -521,6 +521,7 @@ static const struct pci_id_match intel_device_match[] = {
>  	INTEL_ADLS_IDS(&intel_alderlake_s_info),
>  	INTEL_RPLS_IDS(&intel_raptorlake_s_info),
>  	INTEL_ADLP_IDS(&intel_alderlake_p_info),
> +	INTEL_RPLP_IDS(&intel_alderlake_p_info),
>  	INTEL_ADLN_IDS(&intel_alderlake_n_info),

Not necessary to do in this patch, but as also discussed internally,
this needs to be made consistent. RPL-P using ADL-P info but RPL-S
using RPL-S info. Either both use the corresponding ADL info struct or
neither, not this mix of in between.


-- 
Petri Latvala