Re: [PATCH v3 1/7] drm/msm/dpu: merge dpu_encoder_init() and dpu_encoder_setup()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>, Rob Clark <robdclark@xxxxxxxxx>, Sean Paul <sean@xxxxxxxxxx>, Marijn Suijten <marijn.suijten@xxxxxxxxxxxxxx>
Subject: Re: [PATCH v3 1/7] drm/msm/dpu: merge dpu_encoder_init() and dpu_encoder_setup()
From: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>
Date: Thu, 1 Jun 2023 12:32:12 -0700
Cc: Stephen Boyd <swboyd@xxxxxxxxxxxx>, David Airlie <airlied@xxxxxxxxx>, Daniel Vetter <daniel@xxxxxxxx>, Bjorn Andersson <andersson@xxxxxxxxxx>, <linux-arm-msm@xxxxxxxxxxxxxxx>, <dri-devel@xxxxxxxxxxxxxxxxxxxxx>, <freedreno@xxxxxxxxxxxxxxxxxxxxx>
In-reply-to: <20230601172236.564445-2-dmitry.baryshkov@linaro.org>
References: <20230601172236.564445-1-dmitry.baryshkov@linaro.org> <20230601172236.564445-2-dmitry.baryshkov@linaro.org>
User-agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0



On 6/1/2023 10:22 AM, Dmitry Baryshkov wrote:

There is no reason to split the dpu_encoder interface into separate
_init() and _setup() phases. Merge them into a single function.

Reviewed-by: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
---



Tested-by: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx> # sc7280

References:
- [PATCH v3 0/7] drm/msm/dpu: simplify DPU encoder init
  - From: Dmitry Baryshkov
- [PATCH v3 1/7] drm/msm/dpu: merge dpu_encoder_init() and dpu_encoder_setup()
  - From: Dmitry Baryshkov

Prev by Date: Re: [Freedreno] [PATCH] Revert "drm/msm/dp: Remove INIT_SETUP delay"
Next by Date: Re: [Freedreno] [PATCH v3 2/7] drm/msm/dpu: separate common function to init physical encoder
Previous by thread: [PATCH v3 1/7] drm/msm/dpu: merge dpu_encoder_init() and dpu_encoder_setup()
Next by thread: [PATCH v3 2/7] drm/msm/dpu: separate common function to init physical encoder
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [Linux for Sparc] [IETF Annouce] [Security] [Bugtraq] [Linux MIPS] [ECOS] [Asterisk Internet PBX] [Linux API]