[PATCH] drm/amd/display: Clean up coding style

Xichao Zhao posted 1 patch 2 months ago
.../amd/display/dc/link/protocols/link_dp_capability.c    | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
[PATCH] drm/amd/display: Clean up coding style
Posted by Xichao Zhao 2 months ago
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@vivo.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);
 
-- 
2.34.1
Re: [PATCH] drm/amd/display: Clean up coding style
Posted by Alex Hung 1 month, 2 weeks ago
Reviewed-by: Alex Hung <alex.hung@amd.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@vivo.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);
>