Re: [PATCH v3 2/4] drm/msm/dpu: support binding to the mdp5 devices

2024-02-07 Thread Abhinav Kumar
On 1/5/2024 3:34 PM, Dmitry Baryshkov wrote: Existing MDP5 devices have slightly different bindings. The main register region is called `mdp_phys' instead of `mdp'. Also vbif register regions are a part of the parent, MDSS device. Add support for handling this binding differences.

Re: [PATCH v3 1/4] drm/msm/mdss: generate MDSS data for MDP5 platforms

2024-02-07 Thread Abhinav Kumar
On 1/5/2024 3:34 PM, Dmitry Baryshkov wrote: Older (mdp5) platforms do not use per-SoC compatible strings. Instead they use a single compat entry 'qcom,mdss'. To facilitate migrating these platforms to the DPU driver provide a way to generate the MDSS / UBWC data at runtime, when the DPU

Re: [Freedreno] [PATCH v3] drm/msm/dsi: Document DSC related pclk_rate and hdisplay calculations

2024-02-07 Thread Abhinav Kumar
On 12/2/2023 3:59 PM, Dmitry Baryshkov wrote: Provide actual documentation for the pclk and hdisplay calculations in the case of DSC compression being used. Signed-off-by: Dmitry Baryshkov --- Changes since v2: - Followed suggestion by Abhinav and Marijn to improve documentatry

Re: [PATCH v2] drm/msm/dpu: remove CRTC frame event callback registration

2024-02-07 Thread Abhinav Kumar
On 10/5/2023 3:06 PM, Dmitry Baryshkov wrote: The frame event callback is always set to dpu_crtc_frame_event_cb() (or to NULL) and the data is always either the CRTC itself or NULL (correpondingly). Thus drop the event callback registration, call the dpu_crtc_frame_event_cb() directly and

Re: [PATCH 11/17] drm/msm/dp: add VSC SDP support for YUV420 over DP

2024-02-01 Thread Abhinav Kumar
On 1/31/2024 8:36 PM, Dmitry Baryshkov wrote: On Thu, 1 Feb 2024 at 03:56, Abhinav Kumar wrote: On 1/27/2024 9:39 PM, Dmitry Baryshkov wrote: On Sun, 28 Jan 2024 at 07:34, Paloma Arellano wrote: On 1/25/2024 1:48 PM, Dmitry Baryshkov wrote: On 25/01/2024 21:38, Paloma Arellano

Re: [PATCH 14/17] drm/msm/dpu: modify encoder programming for CDM over DP

2024-02-01 Thread Abhinav Kumar
On 1/31/2024 7:17 PM, Dmitry Baryshkov wrote: On Thu, 1 Feb 2024 at 03:30, Abhinav Kumar wrote: On 1/29/2024 3:44 PM, Dmitry Baryshkov wrote: On Mon, 29 Jan 2024 at 09:08, Abhinav Kumar wrote: On 1/28/2024 10:12 PM, Dmitry Baryshkov wrote: On Mon, 29 Jan 2024 at 07:03, Abhinav Kumar

Re: [PATCH 11/17] drm/msm/dp: add VSC SDP support for YUV420 over DP

2024-01-31 Thread Abhinav Kumar
On 1/27/2024 9:39 PM, Dmitry Baryshkov wrote: On Sun, 28 Jan 2024 at 07:34, Paloma Arellano wrote: On 1/25/2024 1:48 PM, Dmitry Baryshkov wrote: On 25/01/2024 21:38, Paloma Arellano wrote: Add support to pack and send the VSC SDP packet for DP. This therefore allows the transmision of

Re: [PATCH 14/17] drm/msm/dpu: modify encoder programming for CDM over DP

2024-01-31 Thread Abhinav Kumar
On 1/29/2024 3:44 PM, Dmitry Baryshkov wrote: On Mon, 29 Jan 2024 at 09:08, Abhinav Kumar wrote: On 1/28/2024 10:12 PM, Dmitry Baryshkov wrote: On Mon, 29 Jan 2024 at 07:03, Abhinav Kumar wrote: On 1/28/2024 7:42 PM, Dmitry Baryshkov wrote: On Mon, 29 Jan 2024 at 04:58, Abhinav

Re: [PATCH] drm/msm/dpu: fix the programming of INTF_CFG2_DATA_HCTL_EN

2024-01-31 Thread Abhinav Kumar
On 1/31/2024 5:05 PM, Dmitry Baryshkov wrote: On Thu, 1 Feb 2024 at 02:48, Abhinav Kumar wrote: Currently INTF_CFG2_DATA_HCTL_EN is coupled with the enablement of widebus but this is incorrect because we should be enabling this bit independent of widebus except for cases where compression

[PATCH] drm/msm/dpu: fix the programming of INTF_CFG2_DATA_HCTL_EN

2024-01-31 Thread Abhinav Kumar
and enabling INTF_CFG2_DATA_HCTL_EN for all other cases when supported by DPU. Fixes: 3309a7563971 ("drm/msm/dpu: revise timing engine programming to support widebus feature") Suggested-by: Dmitry Baryshkov Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c

Re: [PATCH 07/17] drm/msm/dpu: disallow widebus en in INTF_CONFIG2 when DP is YUV420

2024-01-29 Thread Abhinav Kumar
On 1/29/2024 9:28 PM, Dmitry Baryshkov wrote: On Tue, 30 Jan 2024 at 06:10, Abhinav Kumar wrote: On 1/29/2024 5:43 PM, Dmitry Baryshkov wrote: On Tue, 30 Jan 2024 at 03:07, Abhinav Kumar wrote: On 1/29/2024 4:03 PM, Dmitry Baryshkov wrote: On Tue, 30 Jan 2024 at 01:51, Abhinav

Re: [PATCH 07/17] drm/msm/dpu: disallow widebus en in INTF_CONFIG2 when DP is YUV420

2024-01-29 Thread Abhinav Kumar
On 1/29/2024 5:43 PM, Dmitry Baryshkov wrote: On Tue, 30 Jan 2024 at 03:07, Abhinav Kumar wrote: On 1/29/2024 4:03 PM, Dmitry Baryshkov wrote: On Tue, 30 Jan 2024 at 01:51, Abhinav Kumar wrote: On 1/27/2024 9:33 PM, Dmitry Baryshkov wrote: On Sun, 28 Jan 2024 at 07:16, Paloma

Re: [PATCH 07/17] drm/msm/dpu: disallow widebus en in INTF_CONFIG2 when DP is YUV420

2024-01-29 Thread Abhinav Kumar
On 1/29/2024 4:03 PM, Dmitry Baryshkov wrote: On Tue, 30 Jan 2024 at 01:51, Abhinav Kumar wrote: On 1/27/2024 9:33 PM, Dmitry Baryshkov wrote: On Sun, 28 Jan 2024 at 07:16, Paloma Arellano wrote: On 1/25/2024 1:26 PM, Dmitry Baryshkov wrote: On 25/01/2024 21:38, Paloma Arellano

Re: [PATCH 07/17] drm/msm/dpu: disallow widebus en in INTF_CONFIG2 when DP is YUV420

2024-01-29 Thread Abhinav Kumar
On 1/27/2024 9:33 PM, Dmitry Baryshkov wrote: On Sun, 28 Jan 2024 at 07:16, Paloma Arellano wrote: On 1/25/2024 1:26 PM, Dmitry Baryshkov wrote: On 25/01/2024 21:38, Paloma Arellano wrote: INTF_CONFIG2 register cannot have widebus enabled when DP format is YUV420. Therefore, program the

Re: [PATCH 14/17] drm/msm/dpu: modify encoder programming for CDM over DP

2024-01-28 Thread Abhinav Kumar
On 1/28/2024 10:12 PM, Dmitry Baryshkov wrote: On Mon, 29 Jan 2024 at 07:03, Abhinav Kumar wrote: On 1/28/2024 7:42 PM, Dmitry Baryshkov wrote: On Mon, 29 Jan 2024 at 04:58, Abhinav Kumar wrote: On 1/27/2024 9:55 PM, Dmitry Baryshkov wrote: On Sun, 28 Jan 2024 at 07:48, Paloma

Re: [PATCH 17/17] drm/msm/dp: allow YUV420 mode for DP connector when VSC SDP supported

2024-01-28 Thread Abhinav Kumar
On 1/28/2024 9:05 PM, Dmitry Baryshkov wrote: On Mon, 29 Jan 2024 at 06:30, Abhinav Kumar wrote: On 1/28/2024 7:52 PM, Dmitry Baryshkov wrote: On Mon, 29 Jan 2024 at 05:17, Abhinav Kumar wrote: On 1/25/2024 2:05 PM, Dmitry Baryshkov wrote: On 25/01/2024 21:38, Paloma Arellano

Re: [PATCH 14/17] drm/msm/dpu: modify encoder programming for CDM over DP

2024-01-28 Thread Abhinav Kumar
On 1/28/2024 7:42 PM, Dmitry Baryshkov wrote: On Mon, 29 Jan 2024 at 04:58, Abhinav Kumar wrote: On 1/27/2024 9:55 PM, Dmitry Baryshkov wrote: On Sun, 28 Jan 2024 at 07:48, Paloma Arellano wrote: On 1/25/2024 1:57 PM, Dmitry Baryshkov wrote: On 25/01/2024 21:38, Paloma Arellano

Re: [PATCH 01/17] drm/msm/dpu: allow dpu_encoder_helper_phys_setup_cdm to work for DP

2024-01-28 Thread Abhinav Kumar
On 1/28/2024 8:12 PM, Dmitry Baryshkov wrote: On Mon, 29 Jan 2024 at 06:01, Abhinav Kumar wrote: On 1/28/2024 7:23 PM, Dmitry Baryshkov wrote: On Mon, 29 Jan 2024 at 05:06, Abhinav Kumar wrote: On 1/26/2024 4:39 PM, Paloma Arellano wrote: On 1/25/2024 1:14 PM, Dmitry Baryshkov

Re: [PATCH 17/17] drm/msm/dp: allow YUV420 mode for DP connector when VSC SDP supported

2024-01-28 Thread Abhinav Kumar
On 1/28/2024 7:52 PM, Dmitry Baryshkov wrote: On Mon, 29 Jan 2024 at 05:17, Abhinav Kumar wrote: On 1/25/2024 2:05 PM, Dmitry Baryshkov wrote: On 25/01/2024 21:38, Paloma Arellano wrote: All the components of YUV420 over DP are added. Therefore, let's mark the connector property

Re: [PATCH 01/17] drm/msm/dpu: allow dpu_encoder_helper_phys_setup_cdm to work for DP

2024-01-28 Thread Abhinav Kumar
On 1/28/2024 7:23 PM, Dmitry Baryshkov wrote: On Mon, 29 Jan 2024 at 05:06, Abhinav Kumar wrote: On 1/26/2024 4:39 PM, Paloma Arellano wrote: On 1/25/2024 1:14 PM, Dmitry Baryshkov wrote: On 25/01/2024 21:38, Paloma Arellano wrote: Generalize dpu_encoder_helper_phys_setup_cdm

Re: [PATCH 17/17] drm/msm/dp: allow YUV420 mode for DP connector when VSC SDP supported

2024-01-28 Thread Abhinav Kumar
On 1/25/2024 2:05 PM, Dmitry Baryshkov wrote: On 25/01/2024 21:38, Paloma Arellano wrote: All the components of YUV420 over DP are added. Therefore, let's mark the connector property as true for DP connector when the DP type is not eDP and when VSC SDP is supported. Signed-off-by: Paloma

Re: [PATCH 01/17] drm/msm/dpu: allow dpu_encoder_helper_phys_setup_cdm to work for DP

2024-01-28 Thread Abhinav Kumar
On 1/26/2024 4:39 PM, Paloma Arellano wrote: On 1/25/2024 1:14 PM, Dmitry Baryshkov wrote: On 25/01/2024 21:38, Paloma Arellano wrote: Generalize dpu_encoder_helper_phys_setup_cdm to be compatible with DP. Signed-off-by: Paloma Arellano ---   .../gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h 

Re: [PATCH 14/17] drm/msm/dpu: modify encoder programming for CDM over DP

2024-01-28 Thread Abhinav Kumar
On 1/27/2024 9:55 PM, Dmitry Baryshkov wrote: On Sun, 28 Jan 2024 at 07:48, Paloma Arellano wrote: On 1/25/2024 1:57 PM, Dmitry Baryshkov wrote: On 25/01/2024 21:38, Paloma Arellano wrote: Adjust the encoder format programming in the case of video mode for DP to accommodate CDM related

Re: [PATCH 05/17] drm/msm/dp: add an API to indicate if sink supports VSC SDP

2024-01-26 Thread Abhinav Kumar
On 1/26/2024 6:40 PM, Dmitry Baryshkov wrote: On Sat, 27 Jan 2024 at 02:58, Paloma Arellano wrote: On 1/25/2024 1:23 PM, Dmitry Baryshkov wrote: On 25/01/2024 21:38, Paloma Arellano wrote: YUV420 format is supported only in the VSC SDP packet and not through MSA. Hence add an API which

Re: [PATCH] drm/msm/dpu: make "vblank timeout" more useful

2024-01-25 Thread Abhinav Kumar
On 1/5/2024 3:50 PM, Dmitry Baryshkov wrote: We have several reports of vblank timeout messages. However after some debugging it was found that there might be different causes to that. Include the actual CTL_FLUSH value into the timeout message. This allows us to identify the DPU block that

Re: [PATCH v1] drm/msms/dp: fixed link clock divider bits be over written in BPC unknown case

2024-01-24 Thread Abhinav Kumar
On 1/10/2024 12:18 PM, Kuogee Hsieh wrote: Since the value of DP_TEST_BIT_DEPTH_8 is already left shifted, in the BPC unknown case, the additional shift causes spill over to the other bits of the [DP_CONFIGURATION_CTRL] register. Fix this by changing the return value of

Re: [PATCH] drm/msm/dpu: fix kernel-doc warnings

2024-01-24 Thread Abhinav Kumar
' Signed-off-by: Randy Dunlap Cc: Rob Clark Cc: Abhinav Kumar Cc: Dmitry Baryshkov Cc: Sean Paul Cc: Marijn Suijten Cc: linux-arm-...@vger.kernel.org Cc: dri-de...@lists.freedesktop.org Cc: freedreno@lists.freedesktop.org Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Thomas Zimmermann Cc

Re: [PATCH] drm/msm/dpu: drop obsolete documentation for dpu_encoder_virt

2024-01-24 Thread Abhinav Kumar
On 12/18/2023 9:57 AM, Abhinav Kumar wrote: On 12/16/2023 4:01 PM, Dmitry Baryshkov wrote: Drop obsolete kerneldoc for several fields in struct dpu_encoder_virt Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202312170641.5exlvqqx-...@intel.com/ Fixes

Re: [PATCH v3 4/5] drm/msm/dpu: move writeback's atomic_check to dpu_writeback.c

2024-01-23 Thread Abhinav Kumar
On 12/25/2023 5:08 AM, Dmitry Baryshkov wrote: dpu_encoder_phys_wb is the only user of encoder's atomic_check callback. Move corresponding checks to drm_writeback_connector's implementation and drop the dpu_encoder_phys_wb_atomic_check() function. Signed-off-by: Dmitry Baryshkov ---

[PATCH] drm/msm/dpu: check for valid hw_pp in dpu_encoder_helper_phys_cleanup

2024-01-17 Thread Abhinav Kumar
the other conditional block by making sure hw_pp is valid before dereferencing it. Reported-by: Dan Carpenter Fixes: ae4d721ce100 ("drm/msm/dpu: add an API to reset the encoder related hw blocks") Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 4 ++-

Re: [PATCH v1] drm/msm/dp: remove mdss_dp_test_bit_depth_to_bpc()

2024-01-11 Thread Abhinav Kumar
files changed, 1 insertion(+), 24 deletions(-) Reviewed-by: Abhinav Kumar

Re: [PATCH v1] drm/msms/dp: fixed link clock divider bits be over written in BPC unknown case

2024-01-10 Thread Abhinav Kumar
2 files changed, 7 insertions(+), 8 deletions(-) Reviewed-by: Abhinav Kumar

Re: [PATCH 5/5] drm/msm/mdp5: drop split display support

2024-01-10 Thread Abhinav Kumar
. If someone wants to add that support, I guess they have to start by reverting this commit first. If thats the plan and agreement, Reviewed-by: Abhinav Kumar

Re: [PATCH 4/5] drm/msm/dsi: remove msm_dsi::encoder

2024-01-10 Thread Abhinav Kumar
++ drivers/gpu/drm/msm/dsi/dsi_manager.c | 8 +++- 3 files changed, 6 insertions(+), 12 deletions(-) Reviewed-by: Abhinav Kumar

Re: [PATCH 3/5] drm/msm/dsi: stop calling set_split_display

2024-01-10 Thread Abhinav Kumar
s changed, 40 deletions(-) Reviewed-by: Abhinav Kumar

Re: [PATCH 2/5] drm/msm/dsi: drop msm_dsi_device_connected() function

2024-01-10 Thread Abhinav Kumar
Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/dsi/dsi.h | 6 -- drivers/gpu/drm/msm/dsi/dsi_manager.c | 5 - 2 files changed, 11 deletions(-) Reviewed-by: Abhinav Kumar

Re: [PATCH 1/5] drm/msm/dsi: do not store internal bridge pointer

2024-01-10 Thread Abhinav Kumar
| 8 +--- drivers/gpu/drm/msm/dsi/dsi.h | 7 ++- drivers/gpu/drm/msm/dsi/dsi_manager.c | 19 --- 3 files changed, 15 insertions(+), 19 deletions(-) Reviewed-by: Abhinav Kumar

Re: [PATCH] drm/ci: Add msm tests

2024-01-09 Thread Abhinav Kumar
On 1/9/2024 7:31 AM, Rob Clark wrote: On Mon, Jan 8, 2024 at 6:13 PM Rob Clark wrote: On Mon, Jan 8, 2024 at 2:58 PM Abhinav Kumar wrote: On 1/8/2024 11:50 AM, Rob Clark wrote: From: Rob Clark The msm tests should skip on non-msm hw, so I think it should be safe to enable

Re: [PATCH] drm/ci: Add msm tests

2024-01-08 Thread Abhinav Kumar
On 1/8/2024 11:50 AM, Rob Clark wrote: From: Rob Clark The msm tests should skip on non-msm hw, so I think it should be safe to enable everywhere. Signed-off-by: Rob Clark --- drivers/gpu/drm/ci/testlist.txt | 49 + 1 file changed, 49 insertions(+) I

Re: [PATCH v3 0/5] drm/msm/dpu: remove dpu_encoder_phys_ops::atomic_mode_set callback

2024-01-08 Thread Abhinav Kumar
On 12/25/2023 5:08 AM, Dmitry Baryshkov wrote: The dpu_encoder_phys_ops::atomic_mode_set() callback is mostly redundant. Implementations only set the IRQ indices there. Move statically allocated IRQs to dpu_encoder_phys_*_init() and set dynamically allocated IRQs in the irq_enable() callback.

Re: [PATCH v3 5/5] drm/msm/dpu: drop dpu_encoder_phys_ops::atomic_check()

2024-01-08 Thread Abhinav Kumar
/dpu_encoder.c | 15 --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h | 4 2 files changed, 19 deletions(-) Same comment as prev change but otherwise Reviewed-by: Abhinav Kumar

Re: [PATCH v3 4/5] drm/msm/dpu: move writeback's atomic_check to dpu_writeback.c

2024-01-08 Thread Abhinav Kumar
(-) I am fine with this change with respect to how the code is today. Hence, Reviewed-by: Abhinav Kumar But if we start noticing a pattern like below in dpu_encoder.c's atomic_check, if (INTF_WB) . else if (INTF_DP || INTF_DSI) . then, it will demand bringing back a phys specific

Re: [PATCH v3 2/5] drm/msm/dpu: split _dpu_encoder_resource_control_helper()

2024-01-08 Thread Abhinav Kumar
+ drivers/gpu/drm/msm/disp/dpu1/dpu_trace.h | 12 -- 2 files changed, 37 insertions(+), 20 deletions(-) Reviewed-by: Abhinav Kumar

Re: [PATCH v3 1/5] drm/msm/dpu: split irq_control into irq_enable and _disable

2024-01-08 Thread Abhinav Kumar
++--- drivers/gpu/drm/msm/disp/dpu1/dpu_trace.h | 62 ++ 6 files changed, 158 insertions(+), 86 deletions(-) Reviewed-by: Abhinav Kumar

Re: [PATCH] drm/msm/mdss: specify cfg bandwidth for SDM670

2023-12-18 Thread Abhinav Kumar
path") Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/msm_mdss.c | 1 + 1 file changed, 1 insertion(+) Reviewed-by: Abhinav Kumar

Re: [PATCH] drm/msm/dpu: drop obsolete documentation for dpu_encoder_virt

2023-12-18 Thread Abhinav Kumar
tatus to standard encoder debugfs dir") Fixes: 25fdd5933e4c ("drm/msm: Add SDM845 DPU support") Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 4 1 file changed, 4 deletions(-) Reviewed-by: Abhinav Kumar

Re: [PATCH v4 2/2] drm/msm/dpu: Drop enable and frame_count parameters from dpu_hw_setup_misr()

2023-12-13 Thread Abhinav Kumar
es changed, 22 insertions(+), 33 deletions(-) Reviewed-by: Abhinav Kumar

Re: [PATCH v4 1/2] drm/msm/dpu: Set input_sel bit for INTF

2023-12-13 Thread Abhinav Kumar
deletions(-) Reviewed-by: Abhinav Kumar

Re: [RFT PATCH v2 3/4] drm/msm/dpu: enable writeback on SM6125

2023-12-13 Thread Abhinav Kumar
en = 0x2c8, + .features = WB_SDM845_MASK, + .format_list = wb2_formats, + .num_formats = ARRAY_SIZE(wb2_formats), + .clk_ctrl = DPU_CLK_CTRL_WB2, This should now be wb2_formats_rgb. With that fixed, Reviewed-by: Abhinav Kumar + .

Re: [RFT PATCH v2 2/4] drm/msm/dpu: enable writeback on SC8108X

2023-12-13 Thread Abhinav Kumar
en = 0x2c8, + .features = WB_SDM845_MASK, + .format_list = wb2_formats, + .num_formats = ARRAY_SIZE(wb2_formats), This should now be wb2_formats_rgb. With that fixed, Reviewed-by: Abhinav Kumar + .clk_ctrl = DPU_CLK_CTRL_WB2, + .

Re: [RFT PATCH v2 1/4] drm/msm/dpu: enable writeback on SM8150

2023-12-13 Thread Abhinav Kumar
en = 0x2c8, + .features = WB_SDM845_MASK, + .format_list = wb2_formats, + .num_formats = ARRAY_SIZE(wb2_formats), This should now be wb2_formats_rgb. With that fixed, Reviewed-by: Abhinav Kumar + .clk_ctrl = DPU_CLK_CTRL_WB2, + .

Re: [RFT PATCH v2 0/4] drm/msm/dpu: enable writeback on the other platforms

2023-12-13 Thread Abhinav Kumar
On 12/11/2023 10:23 PM, Dmitry Baryshkov wrote: On Tue, 12 Dec 2023 at 02:30, Abhinav Kumar wrote: On 12/2/2023 4:31 PM, Dmitry Baryshkov wrote: I was not able to test it on my own, this is a call for testing for the owners of these platforms. The git version of modetest now fully

Re: [PATCH v2 1/2] drm/msm/dpu: Set input_sel bit for INTF

2023-12-13 Thread Abhinav Kumar
On 12/13/2023 12:51 PM, Jessica Zhang wrote: Set the input_sel bit for encoders as it was missed in the initial implementation. Reported-by: Rob Clark Closes: https://gitlab.freedesktop.org/drm/msm/-/issues/39 Fixes: 91143873a05d ("drm/msm/dpu: Add MISR register support for interface")

Re: [PATCH v2 2/2] drm/msm/dpu: Drop enable and frame_count parameters from dpu_hw_setup_misr()

2023-12-13 Thread Abhinav Kumar
On 12/13/2023 12:51 PM, Jessica Zhang wrote: Drop the enable and frame_count parameters from dpu_hw_setup_misr() as they are always set to the same values. In addition, replace MISR_FRAME_COUNT_MASK with MISR_FRAME_COUNT as frame_count is always set to the same value. Fixes: 7b37523fb1d1

Re: [PATCH] drm/msm/dpu: move CSC tables to dpu_hw_util.c

2023-12-12 Thread Abhinav Kumar
/dpu_hw_util.h | 44 ++--- 2 files changed, 47 insertions(+), 41 deletions(-) Strange, I didnt hit this but change LGTM, Reviewed-by: Abhinav Kumar

Re: [PATCH v4 01/15] drm/msm/dpu: add formats check for writeback encoder

2023-12-12 Thread Abhinav Kumar
On 12/12/2023 2:03 PM, Dmitry Baryshkov wrote: On Tue, 12 Dec 2023 at 23:30, Dmitry Baryshkov wrote: On Tue, 12 Dec 2023 at 22:53, Abhinav Kumar wrote: In preparation for adding more formats to dpu writeback add format validation to it to fail any unsupported formats. changes in v4

[PATCH v4 13/15] drm/msm/dpu: reserve cdm blocks for writeback in case of YUV output

2023-12-12 Thread Abhinav Kumar
into this change changes in v2: - use needs_cdm from topology struct - drop fb related checks from atomic_mode_set() Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 37 + 1 file changed, 37 insertions

[PATCH v4 14/15] drm/msm/dpu: introduce separate wb2_format arrays for rgb and yuv

2023-12-12 Thread Abhinav Kumar
support for CDM block will use the newly added wb2_formats_rgb_yuv array. changes in v3: - change type of wb2_formats_rgb/wb2_formats_rgb_yuv to u32 to fix checkpatch warnings Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- .../msm/disp/dpu1/catalog

[PATCH v4 15/15] drm/msm/dpu: add cdm blocks to dpu snapshot

2023-12-12 Thread Abhinav Kumar
Now that CDM block support has been added to DPU lets also add its entry to the DPU snapshot to help debugging. Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/msm

[PATCH v4 11/15] drm/msm/dpu: add an API to setup the CDM block for writeback

2023-12-12 Thread Abhinav Kumar
as they both have been merged into enable() - drop reduntant hw_cdm and hw_pp checks Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202312102149.qmbcdsg2-...@intel.com/ Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- .../gpu/drm/msm/disp/dpu1

[PATCH v4 12/15] drm/msm/dpu: plug-in the cdm related bits to writeback setup

2023-12-12 Thread Abhinav Kumar
-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c index 3e5dbaa9c896

[PATCH v4 10/15] drm/msm/dpu: add CDM related logic to dpu_hw_ctl layer

2023-12-12 Thread Abhinav Kumar
: - remove unused empty line - pass in cdm_num to update_pending_flush_cdm() Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202312102047.s0i69pcs-...@intel.com/ Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp

[PATCH v4 09/15] drm/msm/dpu: add support to allocate CDM from RM

2023-12-12 Thread Abhinav Kumar
struct Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_mdss.h | 1 + drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h | 1 + drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c | 38 +++-- drivers/gpu/drm/msm/msm_drv.h | 2 ++ 4

[PATCH v4 07/15] drm/msm/dpu: add dpu_hw_cdm abstraction for CDM block

2023-12-12 Thread Abhinav Kumar
nto enable() Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202312101815.b3zh7pfy-...@intel.com/ Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/Makefile| 1 + drivers/gpu/drm/msm/disp/dpu1/dpu_hw_cdm.c | 245 drivers/g

[PATCH v4 08/15] drm/msm/dpu: add cdm blocks to RM

2023-12-12 Thread Abhinav Kumar
Add the RM APIs necessary to initialize and allocate CDM blocks to be used by the rest of the DPU pipeline. changes in v2: - treat cdm_init() failure as fatal - fixed the commit text Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1

[PATCH v4 06/15] drm/msm/dpu: add cdm blocks to sm8250 dpu_hw_catalog

2023-12-12 Thread Abhinav Kumar
Add CDM blocks to the sm8250 dpu_hw_catalog to support YUV format output from writeback block. changes in v2: - re-use the cdm definition from sc7280 Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/catalog/dpu_6_0_sm8250.h | 1 + 1 file

[PATCH v4 05/15] drm/msm/dpu: add cdm blocks to sc7280 dpu_hw_catalog

2023-12-12 Thread Abhinav Kumar
catalog file as its definition can be re-used Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- .../gpu/drm/msm/disp/dpu1/catalog/dpu_7_2_sc7280.h | 1 + drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 10 ++ drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h

[PATCH v4 04/15] drm/msm/dpu: move csc matrices to dpu_hw_util

2023-12-12 Thread Abhinav Kumar
the extra wrapper and export the matrices directly Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h | 30 drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 31 + 2 files changed, 31 insertions(+), 30

[PATCH v4 03/15] drm/msm/dpu: fix writeback programming for YUV cases

2023-12-12 Thread Abhinav Kumar
For YUV cases, setting the required format bits was missed out in the register programming. Lets fix it now in preparation of adding YUV formats support for writeback. changes in v2: - dropped the fixes tag as its not a fix but adding new functionality Signed-off-by: Abhinav Kumar

[PATCH v4 01/15] drm/msm/dpu: add formats check for writeback encoder

2023-12-12 Thread Abhinav Kumar
drm_atomic_helper_check_wb_encoder_state() with drm_atomic_helper_check_wb_connector_state() due to the rebase changes in v2: - correct some grammar in the commit text Fixes: d7d0e73f7de3 ("drm/msm/dpu: introduce the dpu_encoder_phys_* for writeback") Signed-off-by: Abhinav Kumar --- drive

[PATCH v4 02/15] drm/msm/dpu: rename dpu_encoder_phys_wb_setup_cdp to match its functionality

2023-12-12 Thread Abhinav Kumar
phys_* for writeback") Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c b/drivers/gpu/drm/msm

[PATCH v4 00/15] Add CDM support for MSM writeback

2023-12-12 Thread Abhinav Kumar
they both have been merged into enable() - drop reduntant hw_cdm and hw_pp checks - drop fb related checks from dpu_encoder::atomic_mode_set() - introduce separate wb2_format arrays for rgb and yuv Abhinav Kumar (15): drm/msm/dpu: add formats check for writeback encoder

Re: [PATCH v3 07/15] drm/msm/dpu: add dpu_hw_cdm abstraction for CDM block

2023-12-12 Thread Abhinav Kumar
On 12/12/2023 1:40 AM, Dmitry Baryshkov wrote: On Tue, 12 Dec 2023 at 02:23, Abhinav Kumar wrote: CDM block comes with its own set of registers and operations which can be done. In-line with other hardware blocks, this change adds the dpu_hw_cdm abstraction for the CDM block. changes

Re: [PATCH v3 01/15] drm/msm/dpu: add formats check for writeback encoder

2023-12-12 Thread Abhinav Kumar
On 12/11/2023 10:40 PM, Dmitry Baryshkov wrote: On Tue, 12 Dec 2023 at 02:23, Abhinav Kumar wrote: In preparation for adding more formats to dpu writeback add format validation to it to fail any unsupported formats. changes in v3: - rebase on top of msm-next - replace

Re: [RFT PATCH v2 0/4] drm/msm/dpu: enable writeback on the other platforms

2023-12-11 Thread Abhinav Kumar
On 12/2/2023 4:31 PM, Dmitry Baryshkov wrote: I was not able to test it on my own, this is a call for testing for the owners of these platforms. The git version of modetest now fully supports writeback. Use libdrm >= 2.4.117, run modetest -ac to determine the writeback connector, cat

[PATCH v3 10/15] drm/msm/dpu: add CDM related logic to dpu_hw_ctl layer

2023-12-11 Thread Abhinav Kumar
: - remove unused empty line - pass in cdm_num to update_pending_flush_cdm() Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202312102047.s0i69pcs-...@intel.com/ Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp

[PATCH v3 13/15] drm/msm/dpu: reserve cdm blocks for writeback in case of YUV output

2023-12-11 Thread Abhinav Kumar
into this change changes in v2: - use needs_cdm from topology struct - drop fb related checks from atomic_mode_set() Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 37 + 1 file changed, 37 insertions(+) diff --git a/drivers/gpu

[PATCH v3 07/15] drm/msm/dpu: add dpu_hw_cdm abstraction for CDM block

2023-12-11 Thread Abhinav Kumar
- drop setup_csc_data() and setup_cdwn() ops as they are merged into enable() Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202312101815.b3zh7pfy-...@intel.com/ Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/Makefile| 1 + drivers/gpu/dr

[PATCH v3 12/15] drm/msm/dpu: plug-in the cdm related bits to writeback setup

2023-12-11 Thread Abhinav Kumar
-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c index 85cb8596737d

[PATCH v3 15/15] drm/msm/dpu: add cdm blocks to dpu snapshot

2023-12-11 Thread Abhinav Kumar
Now that CDM block support has been added to DPU lets also add its entry to the DPU snapshot to help debugging. Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/msm

[PATCH v3 14/15] drm/msm/dpu: introduce separate wb2_format arrays for rgb and yuv

2023-12-11 Thread Abhinav Kumar
support for CDM block will use the newly added wb2_formats_rgb_yuv array. changes in v3: - change type of wb2_formats_rgb/wb2_formats_rgb_yuv to u32 to fix checkpatch warnings Signed-off-by: Abhinav Kumar --- .../msm/disp/dpu1/catalog/dpu_10_0_sm8650.h | 4 +- .../msm/disp

[PATCH v3 06/15] drm/msm/dpu: add cdm blocks to sm8250 dpu_hw_catalog

2023-12-11 Thread Abhinav Kumar
Add CDM blocks to the sm8250 dpu_hw_catalog to support YUV format output from writeback block. changes in v2: - re-use the cdm definition from sc7280 Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/catalog/dpu_6_0_sm8250.h | 1 + 1 file

[PATCH v3 11/15] drm/msm/dpu: add an API to setup the CDM block for writeback

2023-12-11 Thread Abhinav Kumar
as they both have been merged into enable() - drop reduntant hw_cdm and hw_pp checks Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202312102149.qmbcdsg2-...@intel.com/ Signed-off-by: Abhinav Kumar --- .../gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h | 6

[PATCH v3 09/15] drm/msm/dpu: add support to allocate CDM from RM

2023-12-11 Thread Abhinav Kumar
struct Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_mdss.h | 1 + drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h | 1 + drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c | 38 +++-- drivers/gpu/drm/msm/msm_drv.h | 2 ++ 4

[PATCH v3 08/15] drm/msm/dpu: add cdm blocks to RM

2023-12-11 Thread Abhinav Kumar
Add the RM APIs necessary to initialize and allocate CDM blocks to be used by the rest of the DPU pipeline. changes in v2: - treat cdm_init() failure as fatal - fixed the commit text Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1

[PATCH v3 04/15] drm/msm/dpu: move csc matrices to dpu_hw_util

2023-12-11 Thread Abhinav Kumar
the extra wrapper and export the matrices directly Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h | 30 drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 31 + 2 files changed, 31 insertions(+), 30 deletions(-) diff --git

[PATCH v3 02/15] drm/msm/dpu: rename dpu_encoder_phys_wb_setup_cdp to match its functionality

2023-12-11 Thread Abhinav Kumar
phys_* for writeback") Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c b/drivers/gpu/drm/msm

[PATCH v3 05/15] drm/msm/dpu: add cdm blocks to sc7280 dpu_hw_catalog

2023-12-11 Thread Abhinav Kumar
catalog file as its definition can be re-used Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- .../gpu/drm/msm/disp/dpu1/catalog/dpu_7_2_sc7280.h | 1 + drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 10 ++ drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h

[PATCH v3 03/15] drm/msm/dpu: fix writeback programming for YUV cases

2023-12-11 Thread Abhinav Kumar
For YUV cases, setting the required format bits was missed out in the register programming. Lets fix it now in preparation of adding YUV formats support for writeback. changes in v2: - dropped the fixes tag as its not a fix but adding new functionality Signed-off-by: Abhinav Kumar

[PATCH v3 01/15] drm/msm/dpu: add formats check for writeback encoder

2023-12-11 Thread Abhinav Kumar
to the rebase changes in v2: - correct some grammar in the commit text Fixes: d7d0e73f7de3 ("drm/msm/dpu: introduce the dpu_encoder_phys_* for writeback") Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c | 7 +++ 1 file changed, 7

[PATCH v3 00/15] Add CDM support for MSM writeback

2023-12-11 Thread Abhinav Kumar
- drop reduntant hw_cdm and hw_pp checks - drop fb related checks from dpu_encoder::atomic_mode_set() - introduce separate wb2_format arrays for rgb and yuv Abhinav Kumar (15): drm/msm/dpu: add formats check for writeback encoder drm/msm/dpu: rename dpu_encoder_phys_wb

Re: [PATCH v2 05/16] drm/msm/dpu: add cdm blocks to sc7280 dpu_hw_catalog

2023-12-11 Thread Abhinav Kumar
On 12/11/2023 1:42 PM, Dmitry Baryshkov wrote: On Mon, 11 Dec 2023 at 23:32, Abhinav Kumar wrote: On 12/11/2023 1:31 PM, Dmitry Baryshkov wrote: On Mon, 11 Dec 2023 at 23:16, Abhinav Kumar wrote: On 12/8/2023 3:19 AM, Dmitry Baryshkov wrote: On Fri, 8 Dec 2023 at 07:07, Abhinav

Re: [PATCH v2 05/16] drm/msm/dpu: add cdm blocks to sc7280 dpu_hw_catalog

2023-12-11 Thread Abhinav Kumar
On 12/11/2023 1:31 PM, Dmitry Baryshkov wrote: On Mon, 11 Dec 2023 at 23:16, Abhinav Kumar wrote: On 12/8/2023 3:19 AM, Dmitry Baryshkov wrote: On Fri, 8 Dec 2023 at 07:07, Abhinav Kumar wrote: Add CDM blocks to the sc7280 dpu_hw_catalog to support YUV format output from writeback

Re: [PATCH] drm/msm/dpu: remove extra drm_encoder_cleanup from the error path

2023-12-11 Thread Abhinav Kumar
() calls. Fixes: cd42c56d9c0b ("drm/msm/dpu: use drmm-managed allocation for dpu_encoder_virt") Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 3 --- 1 file changed, 3 deletions(-) Reviewed-by: Abhinav Kumar Tested-by: Abhinav Kumar #sm8250 CI

Re: [PATCH v2 05/16] drm/msm/dpu: add cdm blocks to sc7280 dpu_hw_catalog

2023-12-11 Thread Abhinav Kumar
On 12/8/2023 3:19 AM, Dmitry Baryshkov wrote: On Fri, 8 Dec 2023 at 07:07, Abhinav Kumar wrote: Add CDM blocks to the sc7280 dpu_hw_catalog to support YUV format output from writeback block. changes in v2: - remove explicit zero assignment for features - move sc7280_cdm

Re: [PATCH] drm/msm/dpu: Ratelimit framedone timeout msgs

2023-12-11 Thread Abhinav Kumar
/dpu1/dpu_encoder.c | 5 - drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h | 1 + 2 files changed, 5 insertions(+), 1 deletion(-) Reviewed-by: Abhinav Kumar

Re: [PATCH v2 15/16] drm/msm/dpu: introduce separate wb2_format arrays for rgb and yuv

2023-12-08 Thread Abhinav Kumar
On 12/8/2023 12:45 PM, Dmitry Baryshkov wrote: On Fri, 8 Dec 2023 at 19:53, Abhinav Kumar wrote: On 12/8/2023 3:44 AM, Dmitry Baryshkov wrote: On Fri, 8 Dec 2023 at 07:07, Abhinav Kumar wrote: Lets rename the existing wb2_formats array wb2_formats_rgb to indicate that it has only RGB

Re: [PATCH v2 12/16] drm/msm/dpu: add an API to setup the CDM block for writeback

2023-12-08 Thread Abhinav Kumar
On 12/8/2023 12:55 PM, Dmitry Baryshkov wrote: On Fri, 8 Dec 2023 at 19:28, Abhinav Kumar wrote: On 12/8/2023 3:52 AM, Dmitry Baryshkov wrote: On Fri, 8 Dec 2023 at 07:07, Abhinav Kumar wrote: Add an API dpu_encoder_helper_phys_setup_cdm() which can be used by the writeback encoder

Re: [PATCH v2 3/3] drm/msm/dpu: enable writeback on SM8450

2023-12-08 Thread Abhinav Kumar
On 12/2/2023 4:27 PM, Dmitry Baryshkov wrote: Enable WB2 hardware block, enabling writeback support on this platform. Signed-off-by: Dmitry Baryshkov --- .../drm/msm/disp/dpu1/catalog/dpu_8_1_sm8450.h | 18 ++ 1 file changed, 18 insertions(+) Reviewed-by: Abhinav Kumar

Re: [PATCH v2 2/3] drm/msm/dpu: enable writeback on SM8350

2023-12-08 Thread Abhinav Kumar
On 12/2/2023 4:27 PM, Dmitry Baryshkov wrote: Enable WB2 hardware block, enabling writeback support on this platform. Signed-off-by: Dmitry Baryshkov --- .../drm/msm/disp/dpu1/catalog/dpu_7_0_sm8350.h | 18 ++ 1 file changed, 18 insertions(+) Reviewed-by: Abhinav

<    1   2   3   4   5   6   7   8   9   10   >