Re: [PATCH 1/2] drm/amd/display: dc_link: code clean up on enable_link_dp function

2020-02-27 Thread Rodrigo Siqueira
Hi, First of all, thank you for your patch. I just have one tiny comment inline. On 02/26, Melissa Wen wrote: > Coding style clean up on enable_link_dp function as suggested by > checkpatch.pl: > > CHECK: Lines should not end with a '(' > WARNING: line over 80 characters > WARNING: suspect code

[PATCH 1/2] drm/amd/display: dc_link: code clean up on enable_link_dp function

2020-02-26 Thread Melissa Wen
Coding style clean up on enable_link_dp function as suggested by checkpatch.pl: CHECK: Lines should not end with a '(' WARNING: line over 80 characters WARNING: suspect code indent for conditional statements (8, 24) CHECK: braces {} should be used on all arms of this statement ERROR: else should f