On 20.06.2023 02:08, Dmitry Baryshkov wrote:
> dpu_core_perf.c contains several multi-line conditions which are hard to
> comprehent because of the indentation. Rework the identation of these
> conditions to make it easier to understand them.
>
> Signed-off-by: Dmitry Baryshkov <[email protected]>
> ---
Acked-by: Konrad Dybcio <[email protected]>
Konrad
> drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c | 13 +++++--------
> 1 file changed, 5 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c
> b/drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c
> index ba146af73bc5..f8d5c87d0915 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c
> @@ -148,8 +148,8 @@ int dpu_core_perf_crtc_check(struct drm_crtc *crtc,
>
> drm_for_each_crtc(tmp_crtc, crtc->dev) {
> if (tmp_crtc->enabled &&
> - (dpu_crtc_get_client_type(tmp_crtc) ==
> - curr_client_type) && (tmp_crtc != crtc)) {
> + dpu_crtc_get_client_type(tmp_crtc) == curr_client_type &&
> + tmp_crtc != crtc) {
> struct dpu_crtc_state *tmp_cstate =
> to_dpu_crtc_state(tmp_crtc->state);
>
> @@ -194,8 +194,7 @@ static int _dpu_core_perf_crtc_update_bus(struct dpu_kms
> *kms,
>
> drm_for_each_crtc(tmp_crtc, crtc->dev) {
> if (tmp_crtc->enabled &&
> - curr_client_type ==
> - dpu_crtc_get_client_type(tmp_crtc)) {
> + curr_client_type == dpu_crtc_get_client_type(tmp_crtc)) {
> dpu_cstate = to_dpu_crtc_state(tmp_crtc->state);
>
> perf.bw_ctl += dpu_cstate->new_perf.bw_ctl;
> @@ -325,10 +324,8 @@ int dpu_core_perf_crtc_update(struct drm_crtc *crtc,
> update_bus = true;
> }
>
> - if ((params_changed &&
> - (new->core_clk_rate > old->core_clk_rate)) ||
> - (!params_changed &&
> - (new->core_clk_rate < old->core_clk_rate))) {
> + if ((params_changed && new->core_clk_rate > old->core_clk_rate)
> ||
> + (!params_changed && new->core_clk_rate <
> old->core_clk_rate)) {
> old->core_clk_rate = new->core_clk_rate;
> update_clk = true;
> }
