[PATCH] drm/amdgpu: filter out radeon secondary ids as well

[PATCH] drm/amdgpu: filter out radeon secondary ids as well

[Alex Deucher]

Older radeon boards (r2xx-r5xx) had secondary PCI functions
which we solely there for supporting multi-head on OSs with
special requirements.  Add them to the unsupported list
as well so we don't attempt to bind to them.  The driver
would fail to bind to them anyway, but this does so
in a cleaner way that should not confuse the user.

Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 81 +++++++++++++++++++++++++
 1 file changed, 81 insertions(+)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
index 1527decd7e30..75ceb43392b1 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
@@ -1499,6 +1499,87 @@ static const u16 amdgpu_unsupported_pciidlist[] = {
 	0x99A0,
 	0x99A2,
 	0x99A4,
+	/* radeon secondary ids */
+	0x3171,
+	0x3e70,
+	0x4164,
+	0x4165,
+	0x4166,
+	0x4168,
+	0x4170,
+	0x4171,
+	0x4172,
+	0x4173,
+	0x496e,
+	0x4a69,
+	0x4a6a,
+	0x4a6b,
+	0x4a70,
+	0x4a74,
+	0x4b69,
+	0x4b6b,
+	0x4b6c,
+	0x4c6e,
+	0x4e64,
+	0x4e65,
+	0x4e66,
+	0x4e67,
+	0x4e68,
+	0x4e69,
+	0x4e6a,
+	0x4e71,
+	0x4f73,
+	0x5569,
+	0x556b,
+	0x556d,
+	0x556f,
+	0x5571,
+	0x5854,
+	0x5874,
+	0x5940,
+	0x5941,
+	0x5b72,
+	0x5b73,
+	0x5b74,
+	0x5b75,
+	0x5d44,
+	0x5d45,
+	0x5d6d,
+	0x5d6f,
+	0x5d72,
+	0x5d77,
+	0x5e6b,
+	0x5e6d,
+	0x7120,
+	0x7124,
+	0x7129,
+	0x712e,
+	0x712f,
+	0x7162,
+	0x7163,
+	0x7166,
+	0x7167,
+	0x7172,
+	0x7173,
+	0x71a0,
+	0x71a1,
+	0x71a3,
+	0x71a7,
+	0x71bb,
+	0x71e0,
+	0x71e1,
+	0x71e2,
+	0x71e6,
+	0x71e7,
+	0x71f2,
+	0x7269,
+	0x726b,
+	0x726e,
+	0x72a0,
+	0x72a8,
+	0x72b1,
+	0x72b3,
+	0x793f,
 };
 
 static const struct pci_device_id pciidlist[] = {
-- 
2.34.1

Re: [PATCH] drm/amdgpu: filter out radeon secondary ids as well

[Christian König]

Am 20.01.22 um 18:48 schrieb Alex Deucher:
> Older radeon boards (r2xx-r5xx) had secondary PCI functions
> which we solely there for supporting multi-head on OSs with
> special requirements.  Add them to the unsupported list
> as well so we don't attempt to bind to them.  The driver
> would fail to bind to them anyway, but this does so
> in a cleaner way that should not confuse the user.
>
> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

Acked-by: Christian König <christian.koenig@amd.com>

> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 81 +++++++++++++++++++++++++
>   1 file changed, 81 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
> index 1527decd7e30..75ceb43392b1 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
> @@ -1499,6 +1499,87 @@ static const u16 amdgpu_unsupported_pciidlist[] = {
>   	0x99A0,
>   	0x99A2,
>   	0x99A4,
> +	/* radeon secondary ids */
> +	0x3171,
> +	0x3e70,
> +	0x4164,
> +	0x4165,
> +	0x4166,
> +	0x4168,
> +	0x4170,
> +	0x4171,
> +	0x4172,
> +	0x4173,
> +	0x496e,
> +	0x4a69,
> +	0x4a6a,
> +	0x4a6b,
> +	0x4a70,
> +	0x4a74,
> +	0x4b69,
> +	0x4b6b,
> +	0x4b6c,
> +	0x4c6e,
> +	0x4e64,
> +	0x4e65,
> +	0x4e66,
> +	0x4e67,
> +	0x4e68,
> +	0x4e69,
> +	0x4e6a,
> +	0x4e71,
> +	0x4f73,
> +	0x5569,
> +	0x556b,
> +	0x556d,
> +	0x556f,
> +	0x5571,
> +	0x5854,
> +	0x5874,
> +	0x5940,
> +	0x5941,
> +	0x5b72,
> +	0x5b73,
> +	0x5b74,
> +	0x5b75,
> +	0x5d44,
> +	0x5d45,
> +	0x5d6d,
> +	0x5d6f,
> +	0x5d72,
> +	0x5d77,
> +	0x5e6b,
> +	0x5e6d,
> +	0x7120,
> +	0x7124,
> +	0x7129,
> +	0x712e,
> +	0x712f,
> +	0x7162,
> +	0x7163,
> +	0x7166,
> +	0x7167,
> +	0x7172,
> +	0x7173,
> +	0x71a0,
> +	0x71a1,
> +	0x71a3,
> +	0x71a7,
> +	0x71bb,
> +	0x71e0,
> +	0x71e1,
> +	0x71e2,
> +	0x71e6,
> +	0x71e7,
> +	0x71f2,
> +	0x7269,
> +	0x726b,
> +	0x726e,
> +	0x72a0,
> +	0x72a8,
> +	0x72b1,
> +	0x72b3,
> +	0x793f,
>   };
>   
>   static const struct pci_device_id pciidlist[] = {