[PATCH] drm/amdgpu/sdma: bump firmware version checks for user queue support

[Alex Deucher]

On Wed, Aug 27, 2025 at 1:43 AM Jesse.Zhang <Jesse.Zhang at amd.com> wrote:
>
> Using the previous firmware could lead to problems with
> PROTECTED_FENCE_SIGNAL commands, specifically causing register
> conflicts between MCU_DBG0 and MCU_DBG1.
>
> The updated firmware versions ensure proper alignment
> and unification of the SDMA_SUBOP_PROTECTED_FENCE_SIGNAL value with SDMA 7.x,
> resolving these hardware coordination issues
>
> Fixes: 604d44879707 ("drm/amdgpu/sdma6: add ucode version checks for userq support")
>
> Signed-off-by: Jesse Zhang <Jesse.Zhang at amd.com>

Acked-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  drivers/gpu/drm/amd/amdgpu/sdma_v6_0.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v6_0.c b/drivers/gpu/drm/amd/amdgpu/sdma_v6_0.c
> index e6d8eddda2bf..db6e41967f12 100644
> --- a/drivers/gpu/drm/amd/amdgpu/sdma_v6_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/sdma_v6_0.c
> @@ -1377,7 +1377,7 @@ static int sdma_v6_0_sw_init(struct amdgpu_ip_block *ip_block)
>
>         switch (amdgpu_ip_version(adev, SDMA0_HWIP, 0)) {
>         case IP_VERSION(6, 0, 0):
> -               if ((adev->sdma.instance[0].fw_version >= 24) && !adev->sdma.disable_uq)
> +               if ((adev->sdma.instance[0].fw_version >= 27) && !adev->sdma.disable_uq)
>                         adev->userq_funcs[AMDGPU_HW_IP_DMA] = &userq_mes_funcs;
>                 break;
>         case IP_VERSION(6, 0, 1):
> @@ -1385,11 +1385,11 @@ static int sdma_v6_0_sw_init(struct amdgpu_ip_block *ip_block)
>                         adev->userq_funcs[AMDGPU_HW_IP_DMA] = &userq_mes_funcs;
>                 break;
>         case IP_VERSION(6, 0, 2):
> -               if ((adev->sdma.instance[0].fw_version >= 21) && !adev->sdma.disable_uq)
> +               if ((adev->sdma.instance[0].fw_version >= 23) && !adev->sdma.disable_uq)
>                         adev->userq_funcs[AMDGPU_HW_IP_DMA] = &userq_mes_funcs;
>                 break;
>         case IP_VERSION(6, 0, 3):
> -               if ((adev->sdma.instance[0].fw_version >= 25) && !adev->sdma.disable_uq)
> +               if ((adev->sdma.instance[0].fw_version >= 27) && !adev->sdma.disable_uq)
>                         adev->userq_funcs[AMDGPU_HW_IP_DMA] = &userq_mes_funcs;
>                 break;
>         case IP_VERSION(6, 1, 0):
> --
> 2.49.0
>