[PATCH 5/7] drm/amd/display: Fix hdcp2_execution.c codestyle

[Marcelo Mendes Spessoto Junior]

Remove braces for single statement if expressions and change comparison
order for hdcp2_execution.c file

Signed-off-by: Marcelo Mendes Spessoto Junior 
---
 drivers/gpu/drm/amd/display/modules/hdcp/hdcp2_execution.c | 6 ++
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/modules/hdcp/hdcp2_execution.c 
b/drivers/gpu/drm/amd/display/modules/hdcp/hdcp2_execution.c
index 91c22b96e..733f22bed 100644
--- a/drivers/gpu/drm/amd/display/modules/hdcp/hdcp2_execution.c
+++ b/drivers/gpu/drm/amd/display/modules/hdcp/hdcp2_execution.c
@@ -208,9 +208,8 @@ static inline uint8_t get_device_count(struct mod_hdcp 
*hdcp)
 static enum mod_hdcp_status check_device_count(struct mod_hdcp *hdcp)
 {
/* Avoid device count == 0 to do authentication */
-   if (0 == get_device_count(hdcp)) {
+   if (get_device_count(hdcp) == 0)
return MOD_HDCP_STATUS_HDCP1_DEVICE_COUNT_MISMATCH_FAILURE;
-   }
 
/* Some MST display may choose to report the internal panel as an HDCP 
RX.   */
/* To update this condition with 1(because the immediate repeater's 
internal */
@@ -689,9 +688,8 @@ static enum mod_hdcp_status validate_stream_ready(struct 
mod_hdcp *hdcp,
if (is_hdmi_dvi_sl_hdcp(hdcp)) {
if (!process_rxstatus(hdcp, event_ctx, input, &status))
goto out;
-   if (event_ctx->rx_id_list_ready) {
+   if (event_ctx->rx_id_list_ready)
goto out;
-   }
}
if (is_hdmi_dvi_sl_hdcp(hdcp))
if (!mod_hdcp_execute_and_set(check_stream_ready_available,
-- 
2.39.2

Re: [PATCH 5/7] drm/amd/display: Fix hdcp2_execution.c codestyle

[Alex Deucher]

Applied.  Thanks!

On Fri, Dec 29, 2023 at 12:43 PM Marcelo Mendes Spessoto Junior
 wrote:
>
> Remove braces for single statement if expressions and change comparison
> order for hdcp2_execution.c file
>
> Signed-off-by: Marcelo Mendes Spessoto Junior 
> ---
>  drivers/gpu/drm/amd/display/modules/hdcp/hdcp2_execution.c | 6 ++
>  1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/modules/hdcp/hdcp2_execution.c 
> b/drivers/gpu/drm/amd/display/modules/hdcp/hdcp2_execution.c
> index 91c22b96e..733f22bed 100644
> --- a/drivers/gpu/drm/amd/display/modules/hdcp/hdcp2_execution.c
> +++ b/drivers/gpu/drm/amd/display/modules/hdcp/hdcp2_execution.c
> @@ -208,9 +208,8 @@ static inline uint8_t get_device_count(struct mod_hdcp 
> *hdcp)
>  static enum mod_hdcp_status check_device_count(struct mod_hdcp *hdcp)
>  {
> /* Avoid device count == 0 to do authentication */
> -   if (0 == get_device_count(hdcp)) {
> +   if (get_device_count(hdcp) == 0)
> return MOD_HDCP_STATUS_HDCP1_DEVICE_COUNT_MISMATCH_FAILURE;
> -   }
>
> /* Some MST display may choose to report the internal panel as an 
> HDCP RX.   */
> /* To update this condition with 1(because the immediate repeater's 
> internal */
> @@ -689,9 +688,8 @@ static enum mod_hdcp_status validate_stream_ready(struct 
> mod_hdcp *hdcp,
> if (is_hdmi_dvi_sl_hdcp(hdcp)) {
> if (!process_rxstatus(hdcp, event_ctx, input, &status))
> goto out;
> -   if (event_ctx->rx_id_list_ready) {
> +   if (event_ctx->rx_id_list_ready)
> goto out;
> -   }
> }
> if (is_hdmi_dvi_sl_hdcp(hdcp))
> if (!mod_hdcp_execute_and_set(check_stream_ready_available,
> --
> 2.39.2
>