Re: [Freedreno] [PATCH v2 2/3] DRM: Create new Content Protection connector property

2023-04-24 Thread Dave Stevenson
Hi Mark (and Dmitry) On Fri, 21 Apr 2023 at 18:07, Dmitry Baryshkov wrote: > > On 21/04/2023 19:27, Mark Yacoub wrote: > > From: Mark Yacoub > > Nit: is there a reason for this header? My first impression is that it > matches your outgoing name & email address and as such is not necessary. > > N

Re: [Freedreno] [PATCH v2 4/5] drm/msm/mdss: Handle the reg bus ICC path

2023-04-24 Thread Georgi Djakov
Hi Konrad, On 18.04.23 15:10, Konrad Dybcio wrote: Apart from the already handled data bus (MAS_MDP_Pn<->DDR), there's another path that needs to be handled to ensure MDSS functions properly, namely the "reg bus", a.k.a the CPU-MDSS interconnect. Gating that path may have a variety of effects..

Re: [Freedreno] [PATCH v2 01/17] drm/msm/dpu: Remove unused INTF0 interrupt mask from SM6115/QCM2290

2023-04-24 Thread Abhinav Kumar
On 4/17/2023 1:21 PM, Marijn Suijten wrote: Neither of these SoCs has INTF0, they only have a DSI interface on index 1. Stop enabling an interrupt that can't fire. Fixes: 3581b7062cec ("drm/msm/disp/dpu1: add support for display on SM6115") Fixes: 5334087ee743 ("drm/msm: add support for QCM2

Re: [Freedreno] [PATCH v1 5/5] drm/msm/dpu: add DSC 1.2 hw blocks for relevant chipsets

2023-04-24 Thread Kuogee Hsieh
On 4/21/2023 4:22 PM, Dmitry Baryshkov wrote: On 22/04/2023 02:16, Kuogee Hsieh wrote: On 4/21/2023 4:11 PM, Dmitry Baryshkov wrote: On 22/04/2023 02:08, Kuogee Hsieh wrote: On 4/21/2023 3:16 PM, Dmitry Baryshkov wrote: On 22/04/2023 01:05, Kuogee Hsieh wrote: On 4/20/2023 5:07 PM, Dmit

Re: [Freedreno] [PATCH v2 02/17] drm/msm/dpu: Remove TE2 block and feature from DPU >= 7.0.0 hardware

2023-04-24 Thread Abhinav Kumar
On 4/17/2023 1:21 PM, Marijn Suijten wrote: No hardware beyond kona (sm8250) defines the TE2 PINGPONG sub-block offset downstream. Even though neither downstream nor upstream utilizes these registers in any way, remove the erroneous specification for SC8280XP, SM8350 and SM8450 to prevent con

Re: [Freedreno] [PATCH v2 03/17] drm/msm/dpu: Move non-MDP_TOP INTF_INTR offsets out of hwio header

2023-04-24 Thread Abhinav Kumar
On 4/17/2023 1:21 PM, Marijn Suijten wrote: These offsets do not fall under the MDP TOP block and do not fit the comment right above. Move them to dpu_hw_interrupts.c next to the repsective MDP_INTF_x_OFF interrupt block offsets. Fixes: 25fdd5933e4c ("drm/msm: Add SDM845 DPU support") Signed

Re: [Freedreno] [PATCH v2 04/17] drm/msm/dpu: Fix PP_BLK_DIPHER -> DITHER typo

2023-04-24 Thread Abhinav Kumar
On 4/17/2023 1:21 PM, Marijn Suijten wrote: SM8550 only comes with a DITHER subblock inside the PINGPONG block, hence the name and a block length of zero. However, the PP_BLK macro name was typo'd to DIPHER rather than DITHER. Fixes: efcd0107727c ("drm/msm/dpu: add support for SM8550") Signe

Re: [Freedreno] [PATCH v2 3/3] drm/msm/dpu: Pass catalog pointers directly from RM instead of IDs

2023-04-24 Thread Marijn Suijten
On 2023-04-21 16:25:15, Abhinav Kumar wrote: > > > On 4/21/2023 1:53 PM, Marijn Suijten wrote: > > The Resource Manager already iterates over all available blocks from the > > catalog, only to pass their ID to a dpu_hw_xxx_init() function which > > uses an _xxx_offset() helper to search for and f

Re: [Freedreno] [PATCH v2 02/17] drm/msm/dpu: Remove TE2 block and feature from DPU >= 7.0.0 hardware

2023-04-24 Thread Marijn Suijten
On 2023-04-24 13:41:07, Abhinav Kumar wrote: > > > On 4/17/2023 1:21 PM, Marijn Suijten wrote: > > No hardware beyond kona (sm8250) defines the TE2 PINGPONG sub-block > > offset downstream. Even though neither downstream nor upstream utilizes > > these registers in any way, remove the erroneous

Re: [Freedreno] [PATCH v2 03/17] drm/msm/dpu: Move non-MDP_TOP INTF_INTR offsets out of hwio header

2023-04-24 Thread Marijn Suijten
On 2023-04-24 13:44:55, Abhinav Kumar wrote: > > > On 4/17/2023 1:21 PM, Marijn Suijten wrote: > > These offsets do not fall under the MDP TOP block and do not fit the > > comment right above. Move them to dpu_hw_interrupts.c next to the > > repsective MDP_INTF_x_OFF interrupt block offsets. > >

Re: [Freedreno] [PATCH v2 04/17] drm/msm/dpu: Fix PP_BLK_DIPHER -> DITHER typo

2023-04-24 Thread Marijn Suijten
On 2023-04-24 13:53:13, Abhinav Kumar wrote: > > > On 4/17/2023 1:21 PM, Marijn Suijten wrote: > > SM8550 only comes with a DITHER subblock inside the PINGPONG block, > > hence the name and a block length of zero. However, the PP_BLK macro > > name was typo'd to DIPHER rather than DITHER. > > >

Re: [Freedreno] [PATCH v2 3/3] drm/msm/dpu: Pass catalog pointers directly from RM instead of IDs

2023-04-24 Thread Dmitry Baryshkov
On Tue, 25 Apr 2023 at 01:03, Marijn Suijten wrote: > > On 2023-04-21 16:25:15, Abhinav Kumar wrote: > > > > > > On 4/21/2023 1:53 PM, Marijn Suijten wrote: > > > The Resource Manager already iterates over all available blocks from the > > > catalog, only to pass their ID to a dpu_hw_xxx_init() fu

Re: [Freedreno] [PATCH v2 04/17] drm/msm/dpu: Fix PP_BLK_DIPHER -> DITHER typo

2023-04-24 Thread Abhinav Kumar
On 4/24/2023 3:30 PM, Marijn Suijten wrote: On 2023-04-24 13:53:13, Abhinav Kumar wrote: On 4/17/2023 1:21 PM, Marijn Suijten wrote: SM8550 only comes with a DITHER subblock inside the PINGPONG block, hence the name and a block length of zero. However, the PP_BLK macro name was typo'd to

Re: [Freedreno] [PATCH v2 3/3] drm/msm/dpu: Pass catalog pointers directly from RM instead of IDs

2023-04-24 Thread Abhinav Kumar
On 4/24/2023 3:54 PM, Dmitry Baryshkov wrote: On Tue, 25 Apr 2023 at 01:03, Marijn Suijten wrote: On 2023-04-21 16:25:15, Abhinav Kumar wrote: On 4/21/2023 1:53 PM, Marijn Suijten wrote: The Resource Manager already iterates over all available blocks from the catalog, only to pass their

Re: [Freedreno] [PATCH v2 03/17] drm/msm/dpu: Move non-MDP_TOP INTF_INTR offsets out of hwio header

2023-04-24 Thread Abhinav Kumar
On 4/24/2023 3:25 PM, Marijn Suijten wrote: On 2023-04-24 13:44:55, Abhinav Kumar wrote: On 4/17/2023 1:21 PM, Marijn Suijten wrote: These offsets do not fall under the MDP TOP block and do not fit the comment right above. Move them to dpu_hw_interrupts.c next to the repsective MDP_INTF_x

Re: [Freedreno] [PATCH v2 05/17] drm/msm/dpu: Remove duplicate register defines from INTF

2023-04-24 Thread Abhinav Kumar
On 4/17/2023 1:21 PM, Marijn Suijten wrote: The INTF_FRAME_LINE_COUNT_EN, INTF_FRAME_COUNT and INTF_LINE_COUNT registers are already defined higher up, in the right place when sorted numerically. Fixes: 25fdd5933e4c ("drm/msm: Add SDM845 DPU support") Signed-off-by: Marijn Suijten --- Revi

Re: [Freedreno] [PATCH v2 06/17] drm/msm/dpu: Remove extraneous register define indentation

2023-04-24 Thread Abhinav Kumar
On 4/17/2023 1:21 PM, Marijn Suijten wrote: A bunch of registers are indented with two extra spaces, looking as if these are values corresponding to the previous register which is not the case, rather these are simply also register offsets and should only have a single space separating them an

Re: [Freedreno] [PATCH v2 00/17] drm/msm/dpu: Implement tearcheck support on INTF block

2023-04-24 Thread Abhinav Kumar
On 4/17/2023 1:21 PM, Marijn Suijten wrote: Since DPU 5.0.0 the TEARCHECK registers and interrupts moved out of the PINGPONG block and into the INTF. Implement the necessary callbacks in the INTF block, and use these callbacks together with the INTF_TEAR interrupts. Additionally, disable pre

Re: [Freedreno] [PATCH 10/11] drm/msm/dpu: tweak lm pairings in msm8998 hw catalog

2023-04-24 Thread Abhinav Kumar
On 4/19/2023 7:41 AM, Arnaud Vrac wrote: Change lm blocks pairs so that lm blocks with the same features are paired together: LM_0 and LM_1 with PP and DSPP LM_2 and LM_5 with PP LM_3 and LM_4 This matches the sdm845 configuration and allows using pp or dspp when 2 lm blocks are needed in th

Re: [Freedreno] [PATCH v2 04/17] drm/msm/dpu: Fix PP_BLK_DIPHER -> DITHER typo

2023-04-24 Thread Marijn Suijten
On 2023-04-24 16:09:45, Abhinav Kumar wrote: > >> dither block should be present on many other chipsets too but looks like > >> on sm8550 was enabling it. Not sure how it was validated there. But we > >> are enabling dither, even other chipsets have this block. > > > > Correct, they all seem to h