[PATCH] drm/amd/amdgpu: Fix style errors in amdgpu_display`.c

[Alex Deucher]

On Tue, Apr 25, 2023 at 2:52 PM Srinivasan Shanmugam
<srinivasan.shanmugam at amd.com> wrote:
>
> Fix following checkpatch errors in amdgpu_display.c
>
> ERROR: spaces required around that '=' (ctx:VxW)
> ERROR: that open brace { should be on the previous line
> ERROR: else should follow close brace '}'
>
> Cc: Christian König <christian.koenig at amd.com>
> Cc: Alex Deucher <alexander.deucher at amd.com>
> Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam at amd.com>

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 17 ++++++++---------
>  1 file changed, 8 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> index 8632ab695a6c..389396eac222 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> @@ -97,7 +97,7 @@ static void amdgpu_display_flip_callback(struct dma_fence *f,
>  static bool amdgpu_display_flip_handle_fence(struct amdgpu_flip_work *work,
>                                              struct dma_fence **f)
>  {
> -       struct dma_fence *fence= *f;
> +       struct dma_fence *fence = *f;
>
>         if (fence == NULL)
>                 return false;
> @@ -1251,21 +1251,21 @@ const struct drm_mode_config_funcs amdgpu_mode_funcs = {
>         .fb_create = amdgpu_display_user_framebuffer_create,
>  };
>
> -static const struct drm_prop_enum_list amdgpu_underscan_enum_list[] =
> -{      { UNDERSCAN_OFF, "off" },
> +static const struct drm_prop_enum_list amdgpu_underscan_enum_list[] = {
> +       { UNDERSCAN_OFF, "off" },
>         { UNDERSCAN_ON, "on" },
>         { UNDERSCAN_AUTO, "auto" },
>  };
>
> -static const struct drm_prop_enum_list amdgpu_audio_enum_list[] =
> -{      { AMDGPU_AUDIO_DISABLE, "off" },
> +static const struct drm_prop_enum_list amdgpu_audio_enum_list[] = {
> +       { AMDGPU_AUDIO_DISABLE, "off" },
>         { AMDGPU_AUDIO_ENABLE, "on" },
>         { AMDGPU_AUDIO_AUTO, "auto" },
>  };
>
>  /* XXX support different dither options? spatial, temporal, both, etc. */
> -static const struct drm_prop_enum_list amdgpu_dither_enum_list[] =
> -{      { AMDGPU_FMT_DITHER_DISABLE, "off" },
> +static const struct drm_prop_enum_list amdgpu_dither_enum_list[] = {
> +       { AMDGPU_FMT_DITHER_DISABLE, "off" },
>         { AMDGPU_FMT_DITHER_ENABLE, "on" },
>  };
>
> @@ -1495,8 +1495,7 @@ int amdgpu_display_get_crtc_scanoutpos(struct drm_device *dev,
>                 ret |= DRM_SCANOUTPOS_ACCURATE;
>                 vbl_start = vbl & 0x1fff;
>                 vbl_end = (vbl >> 16) & 0x1fff;
> -       }
> -       else {
> +       } else {
>                 /* No: Fake something reasonable which gives at least ok results. */
>                 vbl_start = mode->crtc_vdisplay;
>                 vbl_end = 0;
> --
> 2.25.1
>