On Wed, Aug 27, 2025 at 1:43 AM Jesse.Zhang <jesse.zh...@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.zh...@amd.com>
Acked-by: Alex Deucher <alexander.deuc...@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
>
