Re: [Freedreno] [PATCH RFC v5 09/10] drm/msm/dpu: Use DRM solid_fill property

2023-07-30 Thread Dmitry Baryshkov
On 28/07/2023 20:02, Jessica Zhang wrote: Drop DPU_PLANE_COLOR_FILL_FLAG and check the DRM solid_fill property to determine if the plane is solid fill. In addition drop the DPU plane color_fill field as we can now use drm_plane_state.solid_fill instead, and pass in drm_plane_state.alpha to

Re: [Freedreno] [PATCH RFC v5 08/10] drm/msm/dpu: Allow NULL FBs in atomic commit

2023-07-30 Thread Dmitry Baryshkov
On 28/07/2023 20:02, Jessica Zhang wrote: Since solid fill planes allow for a NULL framebuffer in a valid commit, add NULL framebuffer checks to atomic commit calls within DPU. Signed-off-by: Jessica Zhang --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 9 ++-

Re: [Freedreno] [PATCH RFC v5 07/10] drm/atomic: Loosen FB atomic checks

2023-07-30 Thread Dmitry Baryshkov
On 28/07/2023 20:02, Jessica Zhang wrote: Loosen the requirements for atomic and legacy commit so that, in cases where pixel_source != FB, the commit can still go through. This includes adding framebuffer NULL checks in other areas to account for FB being NULL when non-FB pixel sources are

Re: [Freedreno] [PATCH RFC v5 06/10] drm/atomic: Move framebuffer checks to helper

2023-07-30 Thread Dmitry Baryshkov
On 28/07/2023 20:02, Jessica Zhang wrote: Currently framebuffer checks happen directly in drm_atomic_plane_check(). Move these checks into their own helper method. Signed-off-by: Jessica Zhang --- drivers/gpu/drm/drm_atomic.c | 130 --- 1 file

Re: [Freedreno] [PATCH RFC v5 03/10] drm: Add solid fill pixel source

2023-07-30 Thread Dmitry Baryshkov
On 28/07/2023 20:02, Jessica Zhang wrote: Add "SOLID_FILL" as a valid pixel source. If the pixel_source property is set to "SOLID_FILL", it will display data from the drm_plane "solid_fill" blob property. Signed-off-by: Jessica Zhang --- drivers/gpu/drm/drm_blend.c | 10 +-

Re: [Freedreno] [PATCH RFC v5 02/10] drm: Introduce solid fill DRM plane property

2023-07-30 Thread Dmitry Baryshkov
On 28/07/2023 20:02, Jessica Zhang wrote: Document and add support for solid_fill property to drm_plane. In addition, add support for setting and getting the values for solid_fill. To enable solid fill planes, userspace must assign a property blob to the "solid_fill" plane property containing

[Freedreno] [PATCH 3/5] drm_dbg: add trailing newlines

2023-07-30 Thread Jim Cromie
Signed-off-by: Jim Cromie --- drivers/gpu/drm/msm/msm_fb.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/msm/msm_fb.c b/drivers/gpu/drm/msm/msm_fb.c index e3f61c39df69..80166f702a0d 100644 --- a/drivers/gpu/drm/msm/msm_fb.c +++

Re: [Freedreno] [PATCH v2 8/8] drm/msm/dpu: move INTF tearing checks to dpu_encoder_phys_cmd_init

2023-07-30 Thread Dmitry Baryshkov
On 30/07/2023 22:52, Marijn Suijten wrote: On 2023-07-30 03:35:18, Dmitry Baryshkov wrote: As the INTF is fixed at the encoder creation time, we can move the check whether INTF supports tearchck to dpu_encoder_phys_cmd_init(). This function can return an error if INTF doesn't have required

Re: [Freedreno] [PATCH v2 8/8] drm/msm/dpu: move INTF tearing checks to dpu_encoder_phys_cmd_init

2023-07-30 Thread Marijn Suijten
On 2023-07-30 03:35:18, Dmitry Baryshkov wrote: > As the INTF is fixed at the encoder creation time, we can move the > check whether INTF supports tearchck to dpu_encoder_phys_cmd_init(). > This function can return an error if INTF doesn't have required feature. > Performing this check in

Re: [Freedreno] [PATCH v2 2/8] drm/msm/dpu: enable PINGPONG TE operations only when supported by HW

2023-07-30 Thread Marijn Suijten
On 2023-07-30 03:35:12, Dmitry Baryshkov wrote: > The DPU_PINGPONG_TE bit is set for all PINGPONG blocks on DPU < 5.0. > Rather than checking for the flag, check for the presense of the > corresponding interrupt line. > > Reviewed-by: Marijn Suijten This patch changed significantly since the

Re: [Freedreno] [PATCH v2 1/8] drm/msm/dpu: inline _setup_pingpong_ops()

2023-07-30 Thread Marijn Suijten
On 2023-07-30 03:35:11, Dmitry Baryshkov wrote: > Inline the _setup_pingpong_ops() function, it makes it easier to handle > different conditions involving PINGPONG configuration. > > Signed-off-by: Dmitry Baryshkov Reviewed-by: Marijn Suijten > --- >

Re: [Freedreno] [PATCH v4 6/7] drm/msm/dpu: stop using raw IRQ indices in the kernel traces

2023-07-30 Thread Dmitry Baryshkov
On Sun, 30 Jul 2023 at 22:39, Marijn Suijten wrote: > > On 2023-07-30 01:27:07, Dmitry Baryshkov wrote: > > In preparation to reworking IRQ indcies, stop using raw indices in > > kernel traces. Instead use a pair of register index and bit. This > > corresponds closer to the values in HW catalog.

Re: [Freedreno] [PATCH v4 6/7] drm/msm/dpu: stop using raw IRQ indices in the kernel traces

2023-07-30 Thread Marijn Suijten
On 2023-07-30 01:27:07, Dmitry Baryshkov wrote: > In preparation to reworking IRQ indcies, stop using raw indices in > kernel traces. Instead use a pair of register index and bit. This > corresponds closer to the values in HW catalog. > > Signed-off-by: Dmitry Baryshkov > --- >

Re: [Freedreno] [PATCH v4 5/7] drm/msm/dpu: stop using raw IRQ indices in the kernel output

2023-07-30 Thread Marijn Suijten
On 2023-07-30 01:27:06, Dmitry Baryshkov wrote: > In preparation to reworking IRQ indcies, stop using raw indices in IRQ indices* > kernel output (both printk and debugfs). Instead use a pair of register > index and bit. This corresponds closer to the values in HW catalog. > > Signed-off-by:

Re: [Freedreno] [PATCH 4/7] drm/msm/dpu: enable INTF TE operations only when supported by HW

2023-07-30 Thread Marijn Suijten
On 2023-07-30 03:22:46, Dmitry Baryshkov wrote: > On 27/07/2023 23:12, Marijn Suijten wrote: > > On 2023-07-27 19:21:01, Dmitry Baryshkov wrote: > >> The DPU_INTF_TE bit is set for all INTF blocks on DPU >= 5.0, however > >> only INTF_1 and INTF_2 actually support tearing control. Rather than > >>

Re: [Freedreno] [PATCH 7/7] drm/msm/dpu: move INTF tearing checks to dpu_encoder_phys_cmd_init

2023-07-30 Thread Marijn Suijten
On 2023-07-30 03:16:59, Dmitry Baryshkov wrote: > >>> + if (!phys_enc->has_intf_te && > >>> + (!phys_enc->hw_pp || > >>> + !phys_enc->hw_pp->ops.enable_tearcheck)) { > >> > >> when is hw_pp assigned? Can't we also check that somewhere in an init > >> phase? > > > > It would happen

Re: [Freedreno] [PATCH 1/7] drm/msm/dpu: enable PINGPONG TE operations only when supported by HW

2023-07-30 Thread Marijn Suijten
On 2023-07-30 02:18:10, Dmitry Baryshkov wrote: > On 29/07/2023 21:31, Marijn Suijten wrote: > > On 2023-07-29 02:59:32, Dmitry Baryshkov wrote: > >> On 27/07/2023 23:03, Marijn Suijten wrote: > >>> On 2023-07-27 19:20:58, Dmitry Baryshkov wrote: > The DPU_PINGPONG_TE bit is set for all

Re: [Freedreno] [PATCH v2 3/4] drm/msm/dpu: deduplicate some (most) of SSPP sub-blocks

2023-07-30 Thread Abhinav Kumar
On 7/29/2023 5:19 PM, Dmitry Baryshkov wrote: On 30/07/2023 03:00, Abhinav Kumar wrote: On 7/13/2023 6:55 PM, Dmitry Baryshkov wrote: As we have dropped the variadic parts of SSPP sub-blocks declarations, deduplicate them now, reducing memory cruft. Signed-off-by: Dmitry Baryshkov ---