| Message ID | 20220129010413.97801-1-yang.lee@linux.alibaba.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [-next] drm/amd/display: clean up some inconsistent indenting | expand |
On 2022-01-28 20:04, Yang Li wrote: > Eliminate the follow smatch warning: > drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:2246 > dp_perform_8b_10b_link_training() warn: inconsistent indenting > > Reported-by: Abaci Robot <abaci@linux.alibaba.com> > Signed-off-by: Yang Li <yang.lee@linux.alibaba.com> Reviewed-by: Harry Wentland <harry.wentland@amd.com> Harry > --- > drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c b/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c > index daaec3164875..34ffcd5bb1d7 100644 > --- a/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c > +++ b/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c > @@ -2243,11 +2243,11 @@ static enum link_training_result dp_perform_8b_10b_link_training( > > if (status == LINK_TRAINING_SUCCESS) { > status = perform_clock_recovery_sequence(link, link_res, lt_settings, DPRX); > - if (status == LINK_TRAINING_SUCCESS) { > - status = perform_channel_equalization_sequence(link, > - link_res, > - lt_settings, > - DPRX); > + if (status == LINK_TRAINING_SUCCESS) { > + status = perform_channel_equalization_sequence(link, > + link_res, > + lt_settings, > + DPRX); > } > } >
Applied. Thanks! Alex On Mon, Jan 31, 2022 at 10:17 AM Harry Wentland <harry.wentland@amd.com> wrote: > > On 2022-01-28 20:04, Yang Li wrote: > > Eliminate the follow smatch warning: > > drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:2246 > > dp_perform_8b_10b_link_training() warn: inconsistent indenting > > > > Reported-by: Abaci Robot <abaci@linux.alibaba.com> > > Signed-off-by: Yang Li <yang.lee@linux.alibaba.com> > > Reviewed-by: Harry Wentland <harry.wentland@amd.com> > > Harry > > > --- > > drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c | 10 +++++----- > > 1 file changed, 5 insertions(+), 5 deletions(-) > > > > diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c b/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c > > index daaec3164875..34ffcd5bb1d7 100644 > > --- a/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c > > +++ b/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c > > @@ -2243,11 +2243,11 @@ static enum link_training_result dp_perform_8b_10b_link_training( > > > > if (status == LINK_TRAINING_SUCCESS) { > > status = perform_clock_recovery_sequence(link, link_res, lt_settings, DPRX); > > - if (status == LINK_TRAINING_SUCCESS) { > > - status = perform_channel_equalization_sequence(link, > > - link_res, > > - lt_settings, > > - DPRX); > > + if (status == LINK_TRAINING_SUCCESS) { > > + status = perform_channel_equalization_sequence(link, > > + link_res, > > + lt_settings, > > + DPRX); > > } > > } > > >
diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c b/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c index daaec3164875..34ffcd5bb1d7 100644 --- a/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c +++ b/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c @@ -2243,11 +2243,11 @@ static enum link_training_result dp_perform_8b_10b_link_training( if (status == LINK_TRAINING_SUCCESS) { status = perform_clock_recovery_sequence(link, link_res, lt_settings, DPRX); - if (status == LINK_TRAINING_SUCCESS) { - status = perform_channel_equalization_sequence(link, - link_res, - lt_settings, - DPRX); + if (status == LINK_TRAINING_SUCCESS) { + status = perform_channel_equalization_sequence(link, + link_res, + lt_settings, + DPRX); } }
Eliminate the follow smatch warning: drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:2246 dp_perform_8b_10b_link_training() warn: inconsistent indenting Reported-by: Abaci Robot <abaci@linux.alibaba.com> Signed-off-by: Yang Li <yang.lee@linux.alibaba.com> --- drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-)