drm/i915/display/psr: Fix cppcheck warnings
authorJosé Roberto de Souza <jose.souza@intel.com>
Fri, 9 Apr 2021 23:17:38 +0000 (16:17 -0700)
committerJani Nikula <jani.nikula@intel.com>
Mon, 26 Apr 2021 08:54:21 +0000 (11:54 +0300)
Fix redundant condition, caught in cppcheck by kernel test robot.

Reported-by: kernel test robot <lkp@intel.com>
Cc: Gwan-gyeong Mun <gwan-gyeong.mun@intel.com>
Fixes: b64d6c51380b ("drm/i915/display: Support PSR Multiple Instances")
Signed-off-by: José Roberto de Souza <jose.souza@intel.com>
Reviewed-by: Harish Chegondi <harish.chegondi@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210409231738.238682-1-jose.souza@intel.com
(cherry picked from commit 1884b579c0cfbb52a92462184406558ac633cafb)
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/display/intel_psr.c

index 1d56181..8ada4f8 100644 (file)
@@ -1519,8 +1519,7 @@ void intel_psr_wait_for_idle(const struct intel_crtc_state *new_crtc_state)
                u32 psr_status;
 
                mutex_lock(&intel_dp->psr.lock);
-               if (!intel_dp->psr.enabled ||
-                   (intel_dp->psr.enabled && intel_dp->psr.psr2_enabled)) {
+               if (!intel_dp->psr.enabled || intel_dp->psr.psr2_enabled) {
                        mutex_unlock(&intel_dp->psr.lock);
                        continue;
                }