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

[Tom Rix <trix@xxxxxxxxxx>]

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 <ruanjinjie@xxxxxxxxxx>
> ---
>     
    
    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