[PATCH] drm/amdgpu: Fix logic to determine TOS reload

[Bhardwaj, Rajneesh]

[Public]

Looks good to me.
Acked-by: Rajneesh Bhardwaj <Rajneesh.bhardwaj at amd.com>

-----Original Message-----
From: Lazar, Lijo <Lijo.Lazar at amd.com>
Sent: Monday, September 30, 2024 8:21 AM
To: amd-gfx at lists.freedesktop.org
Cc: Zhang, Hawking <Hawking.Zhang at amd.com>; Deucher, Alexander <Alexander.Deucher at amd.com>; Bhardwaj, Rajneesh <Rajneesh.Bhardwaj at amd.com>; Xu, Feifei <Feifei.Xu at amd.com>
Subject: [PATCH] drm/amdgpu: Fix logic to determine TOS reload

Avoid comparing TOS version on APUs. On APUs driver doesn't take care of TOS load.

Fixes: 2edc5ecbf1a9 ("drm/amdgpu: Add interface for TOS reload cases")

Signed-off-by: Lijo Lazar <lijo.lazar at amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
index 04be0fabb4f5..2a2d34fe8707 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
@@ -2295,7 +2295,7 @@ bool amdgpu_psp_tos_reload_needed(struct amdgpu_device *adev)  {
        struct psp_context *psp = &adev->psp;

-       if (amdgpu_sriov_vf(adev))
+       if (amdgpu_sriov_vf(adev) || (adev->flags & AMD_IS_APU))
                return false;

        if (psp->funcs && psp->funcs->is_reload_needed)
--
2.25.1