[PATCH v2 0/3] Fixup some coccicheck warnings

Harry Wentland harry.wentland at amd.com
Wed Aug 16 13:50:31 UTC 2023


On 2023-08-16 09:15, Lazar, Lijo wrote:
> This series decreases the readability of the code. Is it really a rule to fix such warnings or is it only a generic pointer to double check (not to be followed every time)?
> 

I'm a little tired seeing these changes come in and agree. I see
no value in this change. The old lines in the change actually
have fewer characters and more whitespace, making them more
readable.

Harry

> Thanks,
> Lijo
> 
> On 8/16/2023 5:02 PM, Srinivasan Shanmugam wrote:
>> v2:
>>
>> Updated to max_t/min_t along with commit title message
>>
>> Srinivasan Shanmugam (3):
>>    drm/amd/display: Update ternary operator with min_t() in
>>      'do_aquire_global_lock'
>>    drm/amdgpu: Update ternary operator with max_t() in
>>      'amdgpu_fence_wait_polling'
>>    drm/amdgpu: Update ternary operator with min_t() in
>>      'amdgpu_ras_eeprom_init'
>>
>>   drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c         | 2 +-
>>   drivers/gpu/drm/amd/amdgpu/amdgpu_ras_eeprom.c    | 2 +-
>>   drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +-
>>   3 files changed, 3 insertions(+), 3 deletions(-)
>>



More information about the amd-gfx mailing list