Re: [PATCH -next] drm/amdgpu: Remove a lot of unnecessary ternary operators

[Tom Rix]

On 7/31/23 6:26 AM, Ruan Jinjie wrote:

Ther are many ternary operators, the true or false judgement
of which is unnecessary in C language semantics.

Signed-off-by: Ruan Jinjie<ruanjin...@huawei.com>
---

snip

        data->registry_data.avfs_support =
-               hwmgr->feature_mask & PP_AVFS_MASK ? true : false;
+               hwmgr->feature_mask & PP_AVFS_MASK;
        data->registry_data.led_dpm_enabled = 1;

These are not equivalent, consider 0xffff & 0x1000 != 1

Tom