2022-07-01 09:22:50

by Yang Li

Subject: [PATCH -next] drm/amd/display: clean up some inconsistent indenting

Eliminate the follow smatch warning:
drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc.c:3151 commit_planes_for_stream() warn: inconsistent indenting

Signed-off-by: Yang Li <[email protected]>
---
drivers/gpu/drm/amd/display/dc/core/dc.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
index 146fd4b864b2..d31da9c0256a 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
@@ -3148,15 +3148,15 @@ static void commit_planes_for_stream(struct dc *dc,
else
dc->hwss.pipe_control_lock(dc, top_pipe_to_program, false);

- if ((update_type != UPDATE_TYPE_FAST) && stream->update_flags.bits.dsc_changed)
- if (top_pipe_to_program->stream_res.tg->funcs->lock_doublebuffer_enable) {
- top_pipe_to_program->stream_res.tg->funcs->wait_for_state(
+ if ((update_type != UPDATE_TYPE_FAST) && stream->update_flags.bits.dsc_changed)
+ if (top_pipe_to_program->stream_res.tg->funcs->lock_doublebuffer_enable) {
+ top_pipe_to_program->stream_res.tg->funcs->wait_for_state(
top_pipe_to_program->stream_res.tg,
CRTC_STATE_VACTIVE);
- top_pipe_to_program->stream_res.tg->funcs->wait_for_state(
+ top_pipe_to_program->stream_res.tg->funcs->wait_for_state(
top_pipe_to_program->stream_res.tg,
CRTC_STATE_VBLANK);
- top_pipe_to_program->stream_res.tg->funcs->wait_for_state(
+ top_pipe_to_program->stream_res.tg->funcs->wait_for_state(
top_pipe_to_program->stream_res.tg,
CRTC_STATE_VACTIVE);

--
2.20.1.7.g153144c

2022-07-05 15:30:12

by Alex Deucher

Subject: Re: [PATCH -next] drm/amd/display: clean up some inconsistent indenting

Applied. Thanks!

Alex

On Fri, Jul 1, 2022 at 5:24 AM Yang Li <[email protected]> wrote:
>
> Eliminate the follow smatch warning:
> drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc.c:3151 commit_planes_for_stream() warn: inconsistent indenting
>
> Signed-off-by: Yang Li <[email protected]>
> ---
> drivers/gpu/drm/amd/display/dc/core/dc.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
> index 146fd4b864b2..d31da9c0256a 100644
> --- a/drivers/gpu/drm/amd/display/dc/core/dc.c
> +++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
> @@ -3148,15 +3148,15 @@ static void commit_planes_for_stream(struct dc *dc,
> else
> dc->hwss.pipe_control_lock(dc, top_pipe_to_program, false);
>
> - if ((update_type != UPDATE_TYPE_FAST) && stream->update_flags.bits.dsc_changed)
> - if (top_pipe_to_program->stream_res.tg->funcs->lock_doublebuffer_enable) {
> - top_pipe_to_program->stream_res.tg->funcs->wait_for_state(
> + if ((update_type != UPDATE_TYPE_FAST) && stream->update_flags.bits.dsc_changed)
> + if (top_pipe_to_program->stream_res.tg->funcs->lock_doublebuffer_enable) {
> + top_pipe_to_program->stream_res.tg->funcs->wait_for_state(
> top_pipe_to_program->stream_res.tg,
> CRTC_STATE_VACTIVE);
> - top_pipe_to_program->stream_res.tg->funcs->wait_for_state(
> + top_pipe_to_program->stream_res.tg->funcs->wait_for_state(
> top_pipe_to_program->stream_res.tg,
> CRTC_STATE_VBLANK);
> - top_pipe_to_program->stream_res.tg->funcs->wait_for_state(
> + top_pipe_to_program->stream_res.tg->funcs->wait_for_state(
> top_pipe_to_program->stream_res.tg,
> CRTC_STATE_VACTIVE);
>
> --
> 2.20.1.7.g153144c
>