linux-kernel - Re: [PATCH 0/6] drm/amdgpu: Remove redundant ternary operators

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <3dd20bc8-f258-40f8-9b91-ba3d77ce6bc8@amd.com>
Date: Wed, 3 Sep 2025 14:41:36 +0200
From: Christian König <christian.koenig@....com>
To: Liao Yuanhong <liaoyuanhong@...o.com>,
 Alex Deucher <alexander.deucher@....com>, David Airlie <airlied@...il.com>,
 Simona Vetter <simona@...ll.ch>, Tao Zhou <tao.zhou1@....com>,
 Xiang Liu <xiang.liu@....com>, Hawking Zhang <Hawking.Zhang@....com>,
 Yang Wang <kevinyang.wang@....com>,
 Victor Skvortsov <victor.skvortsov@....com>,
 Colin Ian King <colin.i.king@...il.com>, Sunil Khatri
 <sunil.khatri@....com>, Lijo Lazar <lijo.lazar@....com>,
 Shashank Sharma <shashank.sharma@....com>,
 Srinivasan Shanmugam <srinivasan.shanmugam@....com>,
 Prike Liang <Prike.Liang@....com>, Rodrigo Siqueira <siqueira@...lia.com>,
 "Jesse.Zhang" <Jesse.Zhang@....com>, Felix Kuehling
 <felix.kuehling@....com>, Boyuan Zhang <boyuan.zhang@....com>,
 Kent Russell <kent.russell@....com>, André Almeida
 <andrealmeid@...lia.com>, Tim Huang <tim.huang@....com>,
 Yifan Zhang <yifan1.zhang@....com>, Natalie Vock <natalie.vock@....de>,
 Candice Li <candice.li@....com>, Shaoyun Liu <shaoyun.liu@....com>,
 David Belanger <david.belanger@....com>, Emily Deng <Emily.Deng@....com>,
 Sathishkumar S <sathishkumar.sundararaju@....com>,
 "David (Ming Qiang) Wu" <David.Wu3@....com>, Leo Liu <leo.liu@....com>,
 "Stanley.Yang" <Stanley.Yang@....com>, Mangesh Gadre
 <Mangesh.Gadre@....com>, fanhuang <FangSheng.Huang@....com>,
 Saleemkhan Jamadar <saleemkhan.jamadar@....com>,
 "open list:RADEON and AMDGPU DRM DRIVERS" <amd-gfx@...ts.freedesktop.org>,
 "open list:DRM DRIVERS" <dri-devel@...ts.freedesktop.org>,
 open list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 0/6] drm/amdgpu: Remove redundant ternary operators

Reviewed-by: Christian König <christian.koenig@....com> for the entire series.

On 03.09.25 14:03, Liao Yuanhong wrote:
> For ternary operators in the form of "a ? true : false" or
> "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.
> 
> Liao Yuanhong (6):
>   drm/amdgpu/amdgpu_cper: Remove redundant ternary operators
>   drm/amdgpu/gfx: Remove redundant ternary operators
>   drm/amdgpu/gmc: Remove redundant ternary operators
>   drm/amdgpu/ih: Remove redundant ternary operators
>   drm/amdgpu/jpeg: Remove redundant ternary operators
>   drm/amdgpu/vcn: Remove redundant ternary operators
> 
>  drivers/gpu/drm/amd/amdgpu/amdgpu_cper.c | 2 +-
>  drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c   | 3 +--
>  drivers/gpu/drm/amd/amdgpu/gfx_v12_0.c   | 3 +--
>  drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c   | 3 +--
>  drivers/gpu/drm/amd/amdgpu/gmc_v11_0.c   | 3 +--
>  drivers/gpu/drm/amd/amdgpu/gmc_v12_0.c   | 3 +--
>  drivers/gpu/drm/amd/amdgpu/ih_v6_0.c     | 3 +--
>  drivers/gpu/drm/amd/amdgpu/ih_v6_1.c     | 3 +--
>  drivers/gpu/drm/amd/amdgpu/ih_v7_0.c     | 3 +--
>  drivers/gpu/drm/amd/amdgpu/jpeg_v4_0_5.c | 2 +-
>  drivers/gpu/drm/amd/amdgpu/jpeg_v5_0_0.c | 2 +-
>  drivers/gpu/drm/amd/amdgpu/jpeg_v5_0_1.c | 2 +-
>  drivers/gpu/drm/amd/amdgpu/vcn_v4_0_5.c  | 2 +-
>  drivers/gpu/drm/amd/amdgpu/vcn_v5_0_0.c  | 2 +-
>  14 files changed, 14 insertions(+), 22 deletions(-)
>