[PATCH 1/2 v3] drm/amdgpu: Add fw minimum version check for usermode queue

[Alex Deucher]

On Thu, Apr 10, 2025 at 12:52 PM Arvind Yadav <Arvind.Yadav at amd.com> wrote:
>
> This patch is load usermode queue based on FW support for gfx11.
> CP Ucode FW version: [PFP = 2530, ME = 2390, MEC = 2600, MES = 120]
>
> v2: Addressed review comments from Alex.
>     - Just check the firmware versions directly.
> v3: Firmware version checks only for Navi3x(by Alex).
>
> Cc: Alex Deucher <alexander.deucher at amd.com>
> Cc: Christian Koenig <christian.koenig at amd.com>
> Cc: Shashank Sharma <shashank.sharma at amd.com>
> Cc: Sunil Khatri <sunil.khatri at amd.com>
> Signed-off-by: Arvind Yadav <Arvind.Yadav at amd.com>

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

For some reason I haven't gotten any of the 2/2 patches for any revision yet.

Alex

> ---
>  drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c
> index 91d29f482c3c..460bb06d2ee1 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c
> @@ -1632,7 +1632,10 @@ static int gfx_v11_0_sw_init(struct amdgpu_ip_block *ip_block)
>         case IP_VERSION(11, 0, 3):
>  #ifdef CONFIG_DRM_AMDGPU_NAVI3X_USERQ
>                 /* add firmware version checks here */
> -               if (0) {
> +               if (adev->gfx.me_fw_version  >= 2390 &&
> +                   adev->gfx.pfp_fw_version >= 2530 &&
> +                   adev->gfx.mec_fw_version >= 2600 &&
> +                   adev->mes.fw_version[0] >= 120) {
>                         adev->userq_funcs[AMDGPU_HW_IP_GFX] = &userq_mes_funcs;
>                         adev->userq_funcs[AMDGPU_HW_IP_COMPUTE] = &userq_mes_funcs;
>                 }
> --
> 2.34.1
>