On Tue, Apr 25, 2023 at 2:52 PM Srinivasan Shanmugam
<srinivasan.shanmugam@xxxxxxx> 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@xxxxxxx>
> Cc: Alex Deucher <alexander.deucher@xxxxxxx>
> Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@xxxxxxx>
Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx>
> ---
> 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
>
