On Fri, Feb 24, 2023 at 11:34:44AM -0500, Rodrigo Vivi wrote:
> Some PCI IDs got accidentally removed when subplatform was added.

I don't think they got removed?  INTEL_RPLP_IDS still includes the
entire INTEL_RPLU_IDS list:

        /* RPL-P */
        #define INTEL_RPLP_IDS(info) \
                INTEL_RPLU_IDS(info), \
                INTEL_VGA_DEVICE(0xA720, info), \
                INTEL_VGA_DEVICE(0xA7A0, info), \
                INTEL_VGA_DEVICE(0xA7A8, info)

so the change below is just adding them to the list a second time.


Matt

> 
> Fixes: 61b795a9c352 ("drm/i915: Add RPL-U sub platform")
> Cc: Chaitanya Kumar Borah <[email protected]>
> Cc: Jani Nikula <[email protected]>
> Signed-off-by: Rodrigo Vivi <[email protected]>
> ---
>  drivers/gpu/drm/i915/i915_pci.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
> index a8d942b16223..d2dc81848e3b 100644
> --- a/drivers/gpu/drm/i915/i915_pci.c
> +++ b/drivers/gpu/drm/i915/i915_pci.c
> @@ -1235,6 +1235,7 @@ static const struct pci_device_id pciidlist[] = {
>       INTEL_DG1_IDS(&dg1_info),
>       INTEL_RPLS_IDS(&adl_s_info),
>       INTEL_RPLP_IDS(&adl_p_info),
> +     INTEL_RPLU_IDS(&adl_p_info),
>       INTEL_DG2_IDS(&dg2_info),
>       INTEL_ATS_M_IDS(&ats_m_info),
>       INTEL_MTL_IDS(&mtl_info),
> -- 
> 2.39.1
> 

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

Reply via email to