diff options
| author | Francois Dugast <francois.dugast@intel.com> | 2023-09-05 19:49:42 +0000 |
|---|---|---|
| committer | Rodrigo Vivi <rodrigo.vivi@intel.com> | 2023-12-21 11:44:30 -0500 |
| commit | 04316b4ae6e094569737bababac6f2ef130c0020 (patch) | |
| tree | ecbeae065b6a61624929fa4ca1c236f692fbdefc /drivers/gpu | |
| parent | 9914e19cc215d339b618ccae993e16ed7aafb54e (diff) | |
drm/xe/display: Use acpi_target_system_state only if ACPI_SLEEP is enabled
This fixes the build error below with CONFIG_ACPI_SLEEP=n:
drivers/gpu/drm/xe/xe_display.c:334:23: error: implicit declaration of function ‘acpi_target_system_state’; did you mean ‘acpi_get_system_info’? [-Werror=implicit-function-declaration]
334 | bool s2idle = acpi_target_system_state() < ACPI_STATE_S3;
Signed-off-by: Francois Dugast <francois.dugast@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Diffstat (limited to 'drivers/gpu')
| -rw-r--r-- | drivers/gpu/drm/xe/xe_display.c | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/drivers/gpu/drm/xe/xe_display.c b/drivers/gpu/drm/xe/xe_display.c index fa20faf3cc83..da10f16e1c12 100644 --- a/drivers/gpu/drm/xe/xe_display.c +++ b/drivers/gpu/drm/xe/xe_display.c @@ -319,9 +319,18 @@ static void intel_suspend_encoders(struct xe_device *xe) drm_modeset_unlock_all(dev); } +static bool suspend_to_idle(void) +{ +#if IS_ENABLED(CONFIG_ACPI_SLEEP) + if (acpi_target_system_state() < ACPI_STATE_S3) + return true; +#endif + return false; +} + void xe_display_pm_suspend(struct xe_device *xe) { - bool s2idle = acpi_target_system_state() < ACPI_STATE_S3; + bool s2idle = suspend_to_idle(); if (!xe->info.enable_display) return; @@ -350,7 +359,7 @@ void xe_display_pm_suspend(struct xe_device *xe) void xe_display_pm_suspend_late(struct xe_device *xe) { - bool s2idle = acpi_target_system_state() < ACPI_STATE_S3; + bool s2idle = suspend_to_idle(); if (!xe->info.enable_display) return; |