Commit 8e3967a7 authored by Perry Yuan's avatar Perry Yuan Committed by Alex Deucher
Browse files

drm/amdgpu: Fix build error when CONFIG_SUSPEND is disabled 



The variable `pm_suspend_target_state` is conditionally defined only when
`CONFIG_SUSPEND` is enabled (see `include/linux/suspend.h`). Directly
referencing it without guarding by `#ifdef CONFIG_SUSPEND` causes build
failures when suspend functionality is disabled (e.g., `CONFIG_SUSPEND=n`).

Reviewed-by: default avatarLijo Lazar <lijo.lazar@amd.com>
Signed-off-by: default avatarPerry Yuan <perry.yuan@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 6716a823

drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c

+4 −0
Original line number Diff line number Diff line
@@ -2597,6 +2597,7 @@ static int amdgpu_pmops_suspend(struct device *dev) 
	else if (amdgpu_acpi_is_s3_active(adev))

		adev->in_s3 = true;

	if (!adev->in_s0ix && !adev->in_s3) {

#if IS_ENABLED(CONFIG_SUSPEND)

		/* don't allow going deep first time followed by s2idle the next time */

		if (adev->last_suspend_state != PM_SUSPEND_ON &&

		    adev->last_suspend_state != pm_suspend_target_state) {
@@ -2604,11 +2605,14 @@ static int amdgpu_pmops_suspend(struct device *dev) 
				     pm_suspend_target_state);

			return -EINVAL;

		}

#endif

		return 0;

	}



#if IS_ENABLED(CONFIG_SUSPEND)

	/* cache the state last used for suspend */

	adev->last_suspend_state = pm_suspend_target_state;

#endif



	return amdgpu_device_suspend(drm_dev, true);

}