[Public] > PM_SUSPEND_TO_IDLE should be under a compile guard
It is actually. All of the amdgpu_acpi_* are. It's not obvious though looking at the patch, you need to apply it to notice it. > It makes sense to rename to something like amdgpu_need_reset_on_suspend() as > it decides on reset only for a suspend situation. Remember it's in amdgpu_acpi.c. So it would be amdgpu_acpi_need_reset_on_suspend()? It's a bit lengthy. From: Lazar, Lijo <lijo.la...@amd.com> Sent: Tuesday, May 17, 2022 11:25 To: Limonciello, Mario <mario.limoncie...@amd.com>; amd-gfx@lists.freedesktop.org Cc: Limonciello, Mario <mario.limoncie...@amd.com> Subject: Re: [PATCH] drm/amd: Don't reset dGPUs if the system is going to s2idle [Public] A couple of things - PM_SUSPEND_TO_IDLE should be under a compile guard It makes sense to rename to something like amdgpu_need_reset_on_suspend() as it decides on reset only for a suspend situation. Thanks, Lijo