Re: [Freedreno] [PATCH 2/3] drm/msm/dp: Remove pixel_rate from struct dp_ctrl

2022-06-21 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2022-06-17 16:07:58) > On 17/06/2022 23:47, Stephen Boyd wrote: > > This struct member is stored to in the function that calls the function > > which uses it. That's possible with a function argument instead of > > storing to a struct member. Pass the pixel_rate as an

[Freedreno] [PATCH] drm/msm/dpu: Increment vsync_cnt before waking up userspace

2022-06-21 Thread Stephen Boyd
The 'vsync_cnt' is used to count the number of frames for a crtc. Unfortunately, we increment the count after waking up userspace via dpu_crtc_vblank_callback() calling drm_crtc_handle_vblank(). drm_crtc_handle_vblank() wakes up userspace processes that have called drm_wait_vblank_ioctl(), and if

Re: [Freedreno] [v3 5/5] drm/msm/disp/dpu1: add PSR support for eDP interface in dpu driver

2022-06-21 Thread Dmitry Baryshkov
On 21/06/2022 13:53, Vinod Polimera wrote: Enable PSR on eDP interface using drm self-refresh librabry. This patch uses a trigger from self-refresh library to enter/exit into PSR, when there are no updates from framework. Signed-off-by: Kalyan Thota Signed-off-by: Vinod Polimera ---

Re: [Freedreno] [v3 4/5] drm/msm/disp/dpu1: use atomic enable/disable callbacks for encoder functions

2022-06-21 Thread Dmitry Baryshkov
On 21/06/2022 13:53, Vinod Polimera wrote: Use atomic variants for encoder callback functions such that certain states like self-refresh can be accessed as part of enable/disable sequence. Signed-off-by: Kalyan Thota Signed-off-by: Vinod Polimera Reviewed-by: Dmitry Baryshkov ---

Re: [Freedreno] [v3 3/5] drm/bridge: add psr support during panel bridge enable & disable sequence

2022-06-21 Thread Dmitry Baryshkov
On 21/06/2022 13:53, Vinod Polimera wrote: This change avoids panel prepare/unprepare based on self-refresh state. Signed-off-by: Sankeerth Billakanti Signed-off-by: Kalyan Thota Signed-off-by: Vinod Polimera --- drivers/gpu/drm/bridge/panel.c | 102

Re: [Freedreno] [v3 2/5] drm/bridge: use atomic enable/disable callbacks for panel bridge functions

2022-06-21 Thread Dmitry Baryshkov
On 21/06/2022 21:54, Dmitry Baryshkov wrote: On 21/06/2022 13:53, Vinod Polimera wrote: Use atomic variants for panel bridge callback functions such that certain states like self-refresh can be accessed as part of enable/disable sequence. Signed-off-by: Vinod Polimera Reviewed-by: Dmitry

Re: [Freedreno] [v3 2/5] drm/bridge: use atomic enable/disable callbacks for panel bridge functions

2022-06-21 Thread Dmitry Baryshkov
On 21/06/2022 13:53, Vinod Polimera wrote: Use atomic variants for panel bridge callback functions such that certain states like self-refresh can be accessed as part of enable/disable sequence. Signed-off-by: Vinod Polimera Reviewed-by: Dmitry Baryshkov ---

Re: [Freedreno] [v3 1/5] drm/msm/dp: Add basic PSR support for eDP

2022-06-21 Thread Dmitry Baryshkov
On 21/06/2022 13:53, Vinod Polimera wrote: Add support for basic panel self refresh (PSR) feature for eDP. Add a new interface to set PSR state in the sink from DPU. Program the eDP controller to issue PSR enter and exit SDP to the sink. Signed-off-by: Sankeerth Billakanti Signed-off-by: Vinod

Re: [Freedreno] [v3 0/5] Add PSR support for eDP

2022-06-21 Thread Dmitry Baryshkov
Please fix your mailer. You've got the following headers, which clearly confuse the threading: In-Reply-To: References: Also the y...@qualcomm.com address (present in To:) rejects incoming mail. On 21/06/2022 13:53, Vinod Polimera wrote: Changes in v2: - Use dp bridge to set psr

Re: [Freedreno] [PATCH] drm/msm/dpu: Fix variable dereferenced before check

2022-06-21 Thread Dmitry Baryshkov
On Tue, 21 Jun 2022 at 20:44, Abhinav Kumar wrote: > > Copying freedreno The patch needs to be sent to freedreno@. Otherwise it doesn't exist in patchwork. > > On 6/21/2022 10:42 AM, Abhinav Kumar wrote: > > > > > > On 6/21/2022 4:06 AM, sunliming wrote: > >> Fixes the following smatch warning:

Re: [Freedreno] [PATCH] drm/msm/dpu: Fix variable dereferenced before check

2022-06-21 Thread Abhinav Kumar
Copying freedreno On 6/21/2022 10:42 AM, Abhinav Kumar wrote: On 6/21/2022 4:06 AM, sunliming wrote: Fixes the following smatch warning: drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c:261 dpu_encoder_phys_wb_atomic_check() warn: variable dereferenced before check 'conn_state'

Re: [Freedreno] [PATCH v3 4/4] drm/msm/dpu: Add interface support for CRC debugfs

2022-06-21 Thread Jessica Zhang
On 6/20/2022 11:36 PM, Dmitry Baryshkov wrote: On Tue, 21 Jun 2022 at 03:50, Jessica Zhang wrote: Add support for writing CRC values for the interface block to the debugfs by calling the necessary MISR setup/collect methods. Changes since V1: - Set values_cnt to only include phys with

Re: [Freedreno] [PATCH v3 1/4] drm/msm/dpu: Move LM CRC code into separate method

2022-06-21 Thread Jessica Zhang
On 6/20/2022 11:42 PM, Dmitry Baryshkov wrote: On Tue, 21 Jun 2022 at 03:50, Jessica Zhang wrote: Move layer mixer-specific section of dpu_crtc_get_crc() into a separate helper method. This way, we can make it easier to get CRCs from other HW blocks by adding other get_crc helper methods.

Re: [Freedreno] [PATCH] drm/msm: Use div64_ul instead of do_div

2022-06-21 Thread Abhinav Kumar
On 4/26/2022 6:21 AM, Wan Jiabing wrote: Fix following coccicheck warning: drivers/gpu/drm/msm/msm_gpu_devfreq.c:72:1-7: WARNING: do_div() does a 64-by-32 division, please consider using div64_ul instead. Use div64_ul instead of do_div to avoid a possible truncation. Signed-off-by: Wan

Re: [Freedreno] [PATCH v15 2/3] phy: qcom-qmp: add regulator_set_load to dp phy

2022-06-21 Thread Dmitry Baryshkov
On Tue, 21 Jun 2022 at 20:02, Kuogee Hsieh wrote: > > This patch add regulator_set_load() before enable regulator at > DP phy driver. > > Signed-off-by: Kuogee Hsieh > Reviewed-by: Stephen Boyd > Reviewed-by: Douglas Anderson Reviewed-by: Dmitry Baryshkov > --- >

[Freedreno] [PATCH v15 1/3] phy: qcom-edp: add regulator_set_load to edp phy

2022-06-21 Thread Kuogee Hsieh
This patch add regulator_set_load() before enable regulator at eDP phy driver. Signed-off-by: Kuogee Hsieh Reviewed-by: Douglas Anderson Reviewed-by: Dmitry Baryshkov --- drivers/phy/qualcomm/phy-qcom-edp.c | 12 1 file changed, 12 insertions(+) diff --git

[Freedreno] [PATCH v15 2/3] phy: qcom-qmp: add regulator_set_load to dp phy

2022-06-21 Thread Kuogee Hsieh
This patch add regulator_set_load() before enable regulator at DP phy driver. Signed-off-by: Kuogee Hsieh Reviewed-by: Stephen Boyd Reviewed-by: Douglas Anderson --- drivers/phy/qualcomm/phy-qcom-qmp-combo.c | 43 --- 1 file changed, 33 insertions(+), 10

[Freedreno] [PATCH v15 3/3] drm/msm/dp: delete vdda regulator related functions from eDP/DP controller

2022-06-21 Thread Kuogee Hsieh
Vdda regulators are related to both eDP and DP phy so that it should be managed at eDP and DP phy driver instead of controller. This patch removes vdda regulators related functions out of eDP/DP controller. Signed-off-by: Kuogee Hsieh Reviewed-by: Stephen Boyd Reviewed-by: Dmitry Baryshkov

[Freedreno] [PATCH v15 0/3] eDP/DP Phy vdda realted function

2022-06-21 Thread Kuogee Hsieh
0) rebase on https://git.kernel.org/pub/scm/linux/kernel/git/phy/linux-phy.git tree 1) add regulator_set_load() to eDP phy 2) add regulator_set_load() to DP phy 3) remove vdda related function out of eDP/DP controller Kuogee Hsieh (3): phy: qcom-edp: add regulator_set_load to edp phy phy:

Re: [Freedreno] [PATCH v2 1/4] drm/msm/dp: Add basic PSR support for eDP

2022-06-21 Thread Sankeerth Billakanti
Hi Bjorn, >> Add support for basic panel self refresh (PSR) feature for eDP. >> Add a new interface to set PSR state in the sink from DPU. >> Program the eDP controller to issue PSR enter and exit SDP to the >> sink. >> >> Signed-off-by: Sankeerth Billakanti >> >> Changes in v2: >> - Use dp

Re: [Freedreno] [v1 2/2] drm/msm/disp/dpu1: enable crtc color management based on encoder topology

2022-06-21 Thread Dmitry Baryshkov
Generic comment: y...@qualcomm.com address bounces. Please remove it from the cc list. If you need to send a patch for the internal reasons, please use Bcc. On Tue, 21 Jun 2022 at 12:06, Kalyan Thota wrote: > > Crtc color management needs to be registered only for the crtc which has the >

Re: [Freedreno] [PATCH v2 4/4] drm/msm/disp/dpu1: add PSR support for eDP interface in dpu driver

2022-06-21 Thread Vinod Polimera
> -Original Message- > From: Stephen Boyd > Sent: Wednesday, February 23, 2022 2:59 AM > To: quic_vpolimer ; agr...@kernel.org; > airl...@linux.ie; bjorn.anders...@linaro.org; dan...@ffwll.ch; > devicet...@vger.kernel.org; diand...@chromium.org; dri- > de...@lists.freedesktop.org;

Re: [Freedreno] [PATCH v2 1/4] drm/msm/dp: Add basic PSR support for eDP

2022-06-21 Thread Sankeerth Billakanti
Hi Dmitry, >On Mon, 21 Feb 2022 at 17:52, Vinod Polimera >wrote: >> >> Add support for basic panel self refresh (PSR) feature for eDP. >> Add a new interface to set PSR state in the sink from DPU. >> Program the eDP controller to issue PSR enter and exit SDP to the >> sink. >> >> Signed-off-by:

[Freedreno] [v3 2/5] drm/bridge: use atomic enable/disable callbacks for panel bridge functions

2022-06-21 Thread Vinod Polimera
Use atomic variants for panel bridge callback functions such that certain states like self-refresh can be accessed as part of enable/disable sequence. Signed-off-by: Vinod Polimera --- drivers/gpu/drm/bridge/panel.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff

[Freedreno] [v3 0/5] Add PSR support for eDP

2022-06-21 Thread Vinod Polimera
Changes in v2: - Use dp bridge to set psr entry/exit instead of dpu_enocder. - Don't modify whitespaces. - Set self refresh aware from atomic_check. - Set self refresh aware only if psr is supported. - Provide a stub for msm_dp_display_set_psr. - Move dp functions to bridge code.

[Freedreno] [v3 5/5] drm/msm/disp/dpu1: add PSR support for eDP interface in dpu driver

2022-06-21 Thread Vinod Polimera
Enable PSR on eDP interface using drm self-refresh librabry. This patch uses a trigger from self-refresh library to enter/exit into PSR, when there are no updates from framework. Signed-off-by: Kalyan Thota Signed-off-by: Vinod Polimera --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c| 36

[Freedreno] [v3 1/5] drm/msm/dp: Add basic PSR support for eDP

2022-06-21 Thread Vinod Polimera
Add support for basic panel self refresh (PSR) feature for eDP. Add a new interface to set PSR state in the sink from DPU. Program the eDP controller to issue PSR enter and exit SDP to the sink. Signed-off-by: Sankeerth Billakanti Signed-off-by: Vinod Polimera ---

[Freedreno] [v3 3/5] drm/bridge: add psr support during panel bridge enable & disable sequence

2022-06-21 Thread Vinod Polimera
This change avoids panel prepare/unprepare based on self-refresh state. Signed-off-by: Sankeerth Billakanti Signed-off-by: Kalyan Thota Signed-off-by: Vinod Polimera --- drivers/gpu/drm/bridge/panel.c | 102 +++-- 1 file changed, 98 insertions(+), 4

[Freedreno] [v3 4/5] drm/msm/disp/dpu1: use atomic enable/disable callbacks for encoder functions

2022-06-21 Thread Vinod Polimera
Use atomic variants for encoder callback functions such that certain states like self-refresh can be accessed as part of enable/disable sequence. Signed-off-by: Kalyan Thota Signed-off-by: Vinod Polimera --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 10 ++ 1 file changed, 6

Re: [Freedreno] [v1 1/2] drm/msm/disp/dpu1: add dspp support for sc7280

2022-06-21 Thread Dmitry Baryshkov
On Tue, 21 Jun 2022 at 12:06, Kalyan Thota wrote: > > Add destination side post processing hw block support in sc7280. > > This hwblock enablement is necessary to support color features > like CT Matix (Ex: Night Light feature) > > Change-Id: Iba7d5e1693b06cede2891f5b998466070a77c6ef >

[Freedreno] [v1 1/2] drm/msm/disp/dpu1: add dspp support for sc7280

2022-06-21 Thread Kalyan Thota
Add destination side post processing hw block support in sc7280. This hwblock enablement is necessary to support color features like CT Matix (Ex: Night Light feature) Change-Id: Iba7d5e1693b06cede2891f5b998466070a77c6ef Signed-off-by: Kalyan Thota ---

[Freedreno] [v1 2/2] drm/msm/disp/dpu1: enable crtc color management based on encoder topology

2022-06-21 Thread Kalyan Thota
Crtc color management needs to be registered only for the crtc which has the capability to handle it. Since topology decides which encoder will get the dspp hw block, tie up the crtc and the encoder together (encoder->possible_crtcs) Change-Id: If5a0f33547b6f527ca4b8fbb78424b141dbbd711

Re: [Freedreno] [PATCH v3 1/4] drm/msm/dpu: Move LM CRC code into separate method

2022-06-21 Thread Dmitry Baryshkov
On Tue, 21 Jun 2022 at 03:50, Jessica Zhang wrote: > > Move layer mixer-specific section of dpu_crtc_get_crc() into a separate > helper method. This way, we can make it easier to get CRCs from other HW > blocks by adding other get_crc helper methods. > > Changes since V1: > - Move common bitmasks

Re: [Freedreno] [PATCH v3 2/4] drm/msm/dpu: Move MISR methods to dpu_hw_util

2022-06-21 Thread Dmitry Baryshkov
On Tue, 21 Jun 2022 at 03:50, Jessica Zhang wrote: > > Move layer mixer specific MISR methods to generalized helper methods. > This will make it easier to add CRC support for other blocks in the > future. > > Changes since V2: > - Reordered parameters so that offsets are after hw_blk_reg_map > -

Re: [Freedreno] [PATCH v3 4/4] drm/msm/dpu: Add interface support for CRC debugfs

2022-06-21 Thread Dmitry Baryshkov
On Tue, 21 Jun 2022 at 03:50, Jessica Zhang wrote: > > Add support for writing CRC values for the interface block to > the debugfs by calling the necessary MISR setup/collect methods. > > Changes since V1: > - Set values_cnt to only include phys with backing hw_intf > - Loop over all drm_encs