For ternary operators in the form of "a ? false : true", if 'a' itself
returns a boolean result, the ternary operator can be omitted. Remove
redundant ternary operators to clean up the code.

Signed-off-by: Liao Yuanhong <liaoyuanh...@vivo.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_cper.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cper.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_cper.c
index 6c266f18c598..c4590ec35cad 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cper.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cper.c
@@ -174,7 +174,7 @@ int amdgpu_cper_entry_fill_runtime_section(struct 
amdgpu_device *adev,
        struct cper_sec_nonstd_err *section;
        bool poison;
 
-       poison = (sev == CPER_SEV_NON_FATAL_CORRECTED) ? false : true;
+       poison = sev != CPER_SEV_NON_FATAL_CORRECTED;
        section_desc = (struct cper_sec_desc *)((uint8_t *)hdr + 
SEC_DESC_OFFSET(idx));
        section = (struct cper_sec_nonstd_err *)((uint8_t *)hdr +
                   NONSTD_SEC_OFFSET(hdr->sec_cnt, idx));
-- 
2.34.1

Reply via email to