Re: [RFC PATCH v2 2/3] drm: Adjust atomic checks for solid fill color

2022-12-23 Thread Dmitry Baryshkov
On 23/12/2022 00:14, Jessica Zhang wrote: Loosen the requirements for atomic and legacy commit so that, in cases where solid fill planes is enabled (and FB_ID is NULL), the commit can still go through. In addition, add framebuffer NULL checks in other areas to account for FB being NULL when soli

Re: [RFC PATCH v2 3/3] drm/msm/dpu: Use color_fill property for DPU planes

2022-12-23 Thread Dmitry Baryshkov
On 23/12/2022 00:14, Jessica Zhang wrote: Initialize and use the color_fill properties for planes in DPU driver. In addition, relax framebuffer requirements within atomic commit path and add checks for NULL framebuffers. Finally, drop DPU_PLANE_COLOR_FILL_FLAG as it's unused. Changes since V2: -

Re: [RFC PATCH v3 1/3] drm: Introduce solid fill property for drm plane

2023-01-04 Thread Dmitry Baryshkov
On 05/01/2023 01:40, Jessica Zhang wrote: Add support for solid_fill property to drm_plane. In addition, add support for setting and getting the values for solid_fill. solid_fill holds data for supporting solid fill planes. The property accepts an RGB323232 value and the driver data is formatted

Re: [RFC PATCH v3 2/3] drm: Adjust atomic checks for solid fill color

2023-01-04 Thread Dmitry Baryshkov
On 05/01/2023 01:40, Jessica Zhang wrote: Loosen the requirements for atomic and legacy commit so that, in cases where solid fill planes is enabled (and FB_ID is NULL), the commit can still go through. In addition, add framebuffer NULL checks in other areas to account for FB being NULL when soli

Re: [RFC PATCH v3 1/3] drm: Introduce solid fill property for drm plane

2023-01-04 Thread Dmitry Baryshkov
On 05/01/2023 01:40, Jessica Zhang wrote: Add support for solid_fill property to drm_plane. In addition, add support for setting and getting the values for solid_fill. solid_fill holds data for supporting solid fill planes. The property accepts an RGB323232 value and the driver data is formatted

Re: [RFC PATCH v3 3/3] drm/msm/dpu: Use color_fill property for DPU planes

2023-01-04 Thread Dmitry Baryshkov
On 05/01/2023 01:40, Jessica Zhang wrote: Initialize and use the color_fill properties for planes in DPU driver. In addition, relax framebuffer requirements within atomic commit path and add checks for NULL framebuffers. Finally, drop DPU_PLANE_COLOR_FILL_FLAG as it's unused. Changes since V2: -

Re: [RFC PATCH v3 0/3] Support for Solid Fill Planes

2023-01-05 Thread Dmitry Baryshkov
On Fri, 6 Jan 2023 at 02:38, Jessica Zhang wrote: > > > > On 1/5/2023 3:33 AM, Daniel Vetter wrote: > > On Wed, Jan 04, 2023 at 03:40:33PM -0800, Jessica Zhang wrote: > >> Introduce and add support for a solid_fill property. When the solid_fill > >> property is set, and the framebuffer is set to N

Re: [RFC PATCH v3 0/3] Support for Solid Fill Planes

2023-01-06 Thread Dmitry Baryshkov
On Fri, 6 Jan 2023 at 20:41, Daniel Vetter wrote: > > On Fri, Jan 06, 2023 at 05:43:23AM +0200, Dmitry Baryshkov wrote: > > On Fri, 6 Jan 2023 at 02:38, Jessica Zhang > > wrote: > > > > > > > > > > > > On 1/5/2023 3:33 AM, Daniel Vetter w

Re: [Freedreno] [RFC PATCH v3 3/3] drm/msm/dpu: Use color_fill property for DPU planes

2023-01-06 Thread Dmitry Baryshkov
On Fri, 6 Jan 2023 at 22:57, Jessica Zhang wrote: > On 1/4/2023 6:16 PM, Dmitry Baryshkov wrote: > > On 05/01/2023 01:40, Jessica Zhang wrote: > >> Initialize and use the color_fill properties for planes in DPU driver. In > >> addition, relax framebuffer requirements wi

Re: [RFC PATCH 0/3] Support for Solid Fill Planes

2023-06-26 Thread Dmitry Baryshkov
On 27/06/2023 02:02, Jessica Zhang wrote: On 11/7/2022 11:37 AM, Ville Syrjälä wrote: On Fri, Oct 28, 2022 at 03:59:49PM -0700, Jessica Zhang wrote: Introduce and add support for COLOR_FILL and COLOR_FILL_FORMAT properties. When the color fill value is set, and the framebuffer is set to NULL,

Re: [RFC PATCH 0/3] Support for Solid Fill Planes

2023-06-26 Thread Dmitry Baryshkov
On Tue, 27 Jun 2023 at 03:45, Jessica Zhang wrote: > > > > On 6/26/2023 5:06 PM, Dmitry Baryshkov wrote: > > On 27/06/2023 02:02, Jessica Zhang wrote: > >> > >> > >> On 11/7/2022 11:37 AM, Ville Syrjälä wrote: > >>> On Fri, Oct 28, 2022 a

Re: [RFC PATCH 0/3] Support for Solid Fill Planes

2023-06-27 Thread Dmitry Baryshkov
On 28/06/2023 00:27, Jessica Zhang wrote: On 6/27/2023 12:58 AM, Pekka Paalanen wrote: On Mon, 26 Jun 2023 16:02:50 -0700 Jessica Zhang wrote: On 11/7/2022 11:37 AM, Ville Syrjälä wrote: On Fri, Oct 28, 2022 at 03:59:49PM -0700, Jessica Zhang wrote: Introduce and add support for COLOR_FIL

Re: [PATCH RFC v4 2/7] drm: Introduce pixel_source DRM plane property

2023-06-29 Thread Dmitry Baryshkov
On 30/06/2023 03:25, Jessica Zhang wrote: Add support for pixel_source property to drm_plane and related documentation. This enum property will allow user to specify a pixel source for the plane. Possible pixel sources will be defined in the drm_plane_pixel_source enum. The current possible pix

Re: [PATCH RFC v4 3/7] drm/atomic: Move framebuffer checks to helper

2023-06-29 Thread Dmitry Baryshkov
On 30/06/2023 03:25, 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 changed,

Re: [PATCH RFC v4 4/7] drm/atomic: Loosen FB atomic checks

2023-06-29 Thread Dmitry Baryshkov
On 30/06/2023 03:25, Jessica Zhang wrote: Loosen the requirements for atomic and legacy commit so that, in cases where solid fill planes is enabled but no FB is set, the commit can still go through. This includes adding framebuffer NULL checks in other areas to account for FB being NULL when sol

Re: [PATCH RFC v4 5/7] drm/msm/dpu: Add solid fill and pixel source properties

2023-06-29 Thread Dmitry Baryshkov
On 30/06/2023 03:25, Jessica Zhang wrote: Add solid_fill and pixel_source properties to DPU plane Signed-off-by: Jessica Zhang --- drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 2 ++ 1 file changed, 2 insertions(+) This should be the last commit. Otherwise: Reviewed-by: Dmitry Baryshkov

Re: [PATCH RFC v4 6/7] drm/msm/dpu: Allow NULL FBs in atomic commit

2023-06-29 Thread Dmitry Baryshkov
On 30/06/2023 03:25, 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 ++- drivers/gpu/drm/msm/disp/d

Re: [PATCH RFC v4 7/7] drm/msm/dpu: Use DRM solid_fill property

2023-06-29 Thread Dmitry Baryshkov
_dpu_plane_color_fill_pipe() to allow userspace to configure the alpha value for the solid fill color. Signed-off-by: Jessica Zhang Reviewed-by: Dmitry Baryshkov Minor suggestion below. --- drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 21 +++-- 1 file changed, 15 insertions

Re: [PATCH RFC v4 1/7] drm: Introduce solid fill DRM plane property

2023-06-30 Thread Dmitry Baryshkov
On 30/06/2023 03:25, 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 th

Re: [PATCH RFC v4 2/7] drm: Introduce pixel_source DRM plane property

2023-07-10 Thread Dmitry Baryshkov
On 10/07/2023 22:51, Jessica Zhang wrote: On 6/30/2023 1:27 AM, Pekka Paalanen wrote: On Fri, 30 Jun 2023 03:42:28 +0300 Dmitry Baryshkov wrote: On 30/06/2023 03:25, Jessica Zhang wrote: Add support for pixel_source property to drm_plane and related documentation. This enum property will

Re: [PATCH RFC v4 2/7] drm: Introduce pixel_source DRM plane property

2023-07-11 Thread Dmitry Baryshkov
On 12/07/2023 01:07, Jessica Zhang wrote: On 7/10/2023 1:11 PM, Dmitry Baryshkov wrote: On 10/07/2023 22:51, Jessica Zhang wrote: On 6/30/2023 1:27 AM, Pekka Paalanen wrote: On Fri, 30 Jun 2023 03:42:28 +0300 Dmitry Baryshkov wrote: On 30/06/2023 03:25, Jessica Zhang wrote: Add

Re: [PATCH RFC v4 2/7] drm: Introduce pixel_source DRM plane property

2023-07-11 Thread Dmitry Baryshkov
On Wed, 12 Jul 2023 at 01:42, Abhinav Kumar wrote: > > > > On 7/11/2023 3:19 PM, Dmitry Baryshkov wrote: > > On 12/07/2023 01:07, Jessica Zhang wrote: > >> > >> > >> On 7/10/2023 1:11 PM, Dmitry Baryshkov wrote: > >>> On 10/07/2023 22:51,

Re: [PATCH RFC v5 01/10] drm: Introduce pixel_source DRM plane property

2023-07-28 Thread Dmitry Baryshkov
On Fri, 28 Jul 2023 at 20:03, Jessica Zhang wrote: > > Add support for pixel_source property to drm_plane and related > documentation. In addition, force pixel_source to > DRM_PLANE_PIXEL_SOURCE_FB in DRM_IOCTL_MODE_SETPLANE as to not break > legacy userspace. > > This enum property will allow use

Re: [PATCH RFC v5 04/10] drm/atomic: Add pixel source to plane state dump

2023-07-28 Thread Dmitry Baryshkov
On 28/07/2023 20:02, Jessica Zhang wrote: Add pixel source to the atomic plane state dump Signed-off-by: Jessica Zhang --- drivers/gpu/drm/drm_atomic.c| 1 + drivers/gpu/drm/drm_crtc_internal.h | 2 ++ drivers/gpu/drm/drm_plane.c | 12 3 files changed, 15 ins

Re: [PATCH RFC v5 05/10] drm/atomic: Add solid fill data to plane state dump

2023-07-28 Thread Dmitry Baryshkov
On 28/07/2023 20:02, Jessica Zhang wrote: Add solid_fill property data to the atomic plane state dump. Signed-off-by: Jessica Zhang --- drivers/gpu/drm/drm_atomic.c | 4 drivers/gpu/drm/drm_plane.c | 10 ++ include/drm/drm_plane.h | 3 +++ 3 files changed, 17 insertion

Re: [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 th

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

2023-07-30 Thread Dmitry Baryshkov
nd.c | 10 +- include/drm/drm_plane.h | 1 + 2 files changed, 10 insertions(+), 1 deletion(-) Reviewed-by: Dmitry Baryshkov diff --git a/drivers/gpu/drm/drm_blend.c b/drivers/gpu/drm/drm_blend.c index c632dfcd8a9b..34b1fd3e2310 100644 --- a/drivers/gpu/drm/drm_blend.c +++

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

2023-07-30 Thread Dmitry Baryshkov
*old_plane_state, return true; } +static int drm_atomic_check_fb(const struct drm_plane_state *state) Maybe drm_atomic_plane_check_fb()? Other than that: Reviewed-by: Dmitry Baryshkov +{ + struct drm_plane *plane = state->plane; + const struct drm_framebuffer *fb = state-

Re: [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 enabl

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

2023-07-30 Thread Dmitry Baryshkov
/dpu1/dpu_plane.c | 41 --- 2 files changed, 34 insertions(+), 16 deletions(-) Reviewed-by: Dmitry Baryshkov diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c index 1edf2b6b0a26..cff0f0d26b22 100644 --- a/drivers/gpu

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

2023-07-30 Thread Dmitry Baryshkov
_dpu_plane_color_fill_pipe() to allow userspace to configure the alpha value for the solid fill color. Reviewed-by: Dmitry Baryshkov Signed-off-by: Jessica Zhang --- drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 24 ++-- 1 file changed, 18 insertions(+), 6 deletions(-) diff --git

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

2023-07-31 Thread Dmitry Baryshkov
On 01/08/2023 03:39, Jessica Zhang wrote: On 7/30/2023 9:15 PM, Dmitry Baryshkov wrote: 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

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

2023-08-04 Thread Dmitry Baryshkov
On Fri, 28 Jul 2023 at 20:03, 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 pro

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

2023-08-04 Thread Dmitry Baryshkov
On Fri, 4 Aug 2023 at 16:44, Sebastian Wick wrote: > > On Fri, Aug 4, 2023 at 3:27 PM Dmitry Baryshkov > wrote: > > > > On Fri, 28 Jul 2023 at 20:03, Jessica Zhang > > wrote: > > > > > > Document and add support for solid_fill property to drm_plan

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

2023-08-07 Thread Dmitry Baryshkov
On 8 August 2023 00:41:07 GMT+03:00, Jessica Zhang wrote: > > >On 8/4/2023 6:27 AM, Dmitry Baryshkov wrote: >> On Fri, 28 Jul 2023 at 20:03, Jessica Zhang >> wrote: >>> >>> Document and add support for solid_fill property to drm_plane. In >>&g

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

2023-08-18 Thread Dmitry Baryshkov
On 18/08/2023 13:51, Pekka Paalanen wrote: On Fri, 4 Aug 2023 16:59:00 +0300 Dmitry Baryshkov wrote: On Fri, 4 Aug 2023 at 16:44, Sebastian Wick wrote: On Fri, Aug 4, 2023 at 3:27 PM Dmitry Baryshkov wrote: On Fri, 28 Jul 2023 at 20:03, Jessica Zhang wrote: Document and add support

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

2023-08-21 Thread Dmitry Baryshkov
On Fri, 18 Aug 2023 at 16:55, Pekka Paalanen wrote: > > On Fri, 18 Aug 2023 14:03:14 +0300 > Dmitry Baryshkov wrote: > > > On 18/08/2023 13:51, Pekka Paalanen wrote: > > > On Fri, 4 Aug 2023 16:59:00 +0300 > > > Dmitry Baryshkov wrote: > > > >

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

2023-08-28 Thread Dmitry Baryshkov
On Tue, 29 Aug 2023 at 02:45, Jessica Zhang wrote: > > > > On 8/8/2023 3:57 PM, Jessica Zhang wrote: > > > > > > On 8/7/2023 6:07 PM, Dmitry Baryshkov wrote: > >> > >> > >> On 8 August 2023 00:41:07 GMT+03:00, Jessica Zhang > >&g

Re: [PATCH RFC v6 01/10] drm: Introduce pixel_source DRM plane property

2023-09-24 Thread Dmitry Baryshkov
source for the plane. Possible pixel sources will be defined in the drm_plane_pixel_source enum. Currently, the only pixel sources are DRM_PLANE_PIXEL_SOURCE_FB (the default value) and DRM_PLANE_PIXEL_SOURCE_NONE. Signed-off-by: Jessica Zhang Acked-by: Dmitry Baryshkov Minor question below

Re: [PATCH RFC v6 04/10] drm/atomic: Add pixel source to plane state dump

2023-09-24 Thread Dmitry Baryshkov
(+) Reviewed-by: Dmitry Baryshkov -- With best wishes Dmitry

Re: [PATCH RFC v6 05/10] drm/atomic: Add solid fill data to plane state dump

2023-09-24 Thread Dmitry Baryshkov
(+) Reviewed-by: Dmitry Baryshkov -- With best wishes Dmitry

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

2023-09-24 Thread Dmitry Baryshkov
On 29/08/2023 03:05, 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 enabl

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

2023-09-24 Thread Dmitry Baryshkov
On 22/09/2023 20:49, Jessica Zhang wrote: On 8/29/2023 1:22 AM, Pekka Paalanen wrote: On Mon, 28 Aug 2023 17:05:13 -0700 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 addi

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

2023-09-24 Thread Dmitry Baryshkov
On 29/08/2023 03:05, 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. Reviewed-by: Dmitry Baryshkov Signed-off-by: Jessica Zhang --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 9

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

2023-09-25 Thread Dmitry Baryshkov
gt; and pass in drm_plane_state.alpha to _dpu_plane_color_fill_pipe() to > allow userspace to configure the alpha value for the solid fill color. > > Reviewed-by: Dmitry Baryshkov > Signed-off-by: Jessica Zhang > --- > drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 37 > +

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

2023-10-17 Thread Dmitry Baryshkov
Hi Jessica, On Tue, 17 Oct 2023 at 03:41, Jessica Zhang wrote: > On 9/24/2023 3:23 AM, Dmitry Baryshkov wrote: > > On 22/09/2023 20:49, Jessica Zhang wrote: > >> > >> > >> On 8/29/2023 1:22 AM, Pekka Paalanen wrote: > >>> On Mon, 28 Au

Re: [PATCH] drm/doc: describe PATH format for DP MST

2023-10-23 Thread Dmitry Baryshkov
ble across reboots, but since a KMS object > ID is baked in there that's not the case. So PATH shouldn't be > used as-is in config files and such. > > Signed-off-by: Simon Ser > Cc: Pekka Paalanen > Cc: Dmitry Baryshkov > Cc: Daniel Vetter Reviewed-by: Dmitry Ba

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

2023-11-29 Thread Dmitry Baryshkov
"solid_fill" plane property containing the following information: > > struct drm_mode_solid_fill { > u32 r, g, b, pad; > }; > > Acked-by: Harry Wentland > Acked-by: Sebastian Wick > Signed-off-by: Jessica Zhang Reviewed-by: Dmitry Baryshkov >

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

2023-11-29 Thread Dmitry Baryshkov
drivers/gpu/drm/drm_atomic_helper.c | 39 > + > include/drm/drm_atomic_helper.h | 4 ++-- > include/drm/drm_plane.h | 29 +++ > 4 files changed, 64 insertions(+), 29 deletions(-) Reviewed-by: Dmitry Baryshkov -- With best wishes Dmitry

Re: (subset) [PATCH RFC v7 00/10] Support for Solid Fill Planes

2023-12-02 Thread Dmitry Baryshkov
ic: Loosen FB atomic checks commit: f1e75da5364e780905d9cd6043f9c74cdcf84073 Best regards, -- Dmitry Baryshkov

Re: (subset) [PATCH RFC v7 00/10] Support for Solid Fill Planes

2023-12-03 Thread Dmitry Baryshkov
On Sun, 3 Dec 2023 at 14:15, Simon Ser wrote: > > On Saturday, December 2nd, 2023 at 22:41, Dmitry Baryshkov > wrote: > > > On Fri, 27 Oct 2023 15:32:50 -0700, Jessica Zhang wrote: > > > > > Some drivers support hardware that have optimizations for solid fill