linux-kernel - Re: [PATCH] drm/amd/display: Clean up coding style

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] [day] [month] [year] [list]

Message-ID: <70c3f221-04ca-4ae5-8726-7c5782641822@amd.com>
Date: Sun, 17 Aug 2025 18:01:30 -0600
From: Alex Hung <alex.hung@....com>
To: Xichao Zhao <zhao.xichao@...o.com>, harry.wentland@....com,
 sunpeng.li@....com, alexander.deucher@....com, christian.koenig@....com,
 airlied@...il.com, simona@...ll.ch
Cc: siqueira@...lia.com, linux-kernel@...r.kernel.org,
 amd-gfx@...ts.freedesktop.org, dri-devel@...ts.freedesktop.org
Subject: Re: [PATCH] drm/amd/display: Clean up coding style

Reviewed-by: Alex Hung <alex.hung@....com>

On 7/31/25 01:54, Xichao Zhao wrote:
> Adjust whitespace around operators to improve code readability
> and comply with kernel coding style guidelines.
> 
> These changes are purely stylistic and introduce no
> functional modifications.
> 
> Signed-off-by: Xichao Zhao <zhao.xichao@...o.com>
> ---
>   .../amd/display/dc/link/protocols/link_dp_capability.c    | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c b/drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c
> index e0c4416993d9..caddb7dfb133 100644
> --- a/drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c
> +++ b/drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c
> @@ -1525,8 +1525,8 @@ bool read_is_mst_supported(struct dc_link *link)
>   		return false;
>   	}
>   
> -	rev.raw  = 0;
> -	cap.raw  = 0;
> +	rev.raw = 0;
> +	cap.raw = 0;
>   
>   	st = core_link_read_dpcd(link, DP_DPCD_REV, &rev.raw,
>   			sizeof(rev));
> @@ -2125,13 +2125,13 @@ void detect_edp_sink_caps(struct dc_link *link)
>   						&backlight_adj_cap, sizeof(backlight_adj_cap));
>   
>   	link->dpcd_caps.dynamic_backlight_capable_edp =
> -				(backlight_adj_cap & DP_EDP_DYNAMIC_BACKLIGHT_CAP) ? true:false;
> +				(backlight_adj_cap & DP_EDP_DYNAMIC_BACKLIGHT_CAP) ? true : false;
>   
>   	core_link_read_dpcd(link, DP_EDP_GENERAL_CAP_1,
>   						&general_edp_cap, sizeof(general_edp_cap));
>   
>   	link->dpcd_caps.set_power_state_capable_edp =
> -				(general_edp_cap & DP_EDP_SET_POWER_CAP) ? true:false;
> +				(general_edp_cap & DP_EDP_SET_POWER_CAP) ? true : false;
>   
>   	set_default_brightness_aux(link);
>