[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
<marcelomspessoto at gmail.com> wrote:
>
> Remove braces for single statement if expressions and change comparison
> order for hdcp2_execution.c file
>
> Signed-off-by: Marcelo Mendes Spessoto Junior <marcelomspessoto at gmail.com>
> ---
>  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
>