Re: universal planes in drm backend

2019-09-19 Thread zou lan
Hi all Thank you for your feedback. >From your opinions, the drm backend reserve primary plane for each CRTC to avoid the failure in hotplug cases. However, as weston already enabled DRM_CLIENT_CAP_ATOMIC and DRM_CLIENT_CAP_UNIVERSAL_PLANES, the concept of primary/overlay is no necessary

Re: universal planes in drm backend

2019-09-18 Thread Pekka Paalanen
On Tue, 17 Sep 2019 10:02:44 + Simon Ser wrote: > On Tuesday, September 17, 2019 12:29 PM, Pekka Paalanen > wrote: > > > On Tue, 17 Sep 2019 19:50:01 +1200 > > Scott Anderson scott.ander...@collabora.com wrote: > > > > > On 17/09/19 7:38 pm, zou lan wrote: > > > > > > > Hi Daniel &

Re: universal planes in drm backend

2019-09-17 Thread Simon Ser
On Tuesday, September 17, 2019 12:29 PM, Pekka Paalanen wrote: > On Tue, 17 Sep 2019 19:50:01 +1200 > Scott Anderson scott.ander...@collabora.com wrote: > > > On 17/09/19 7:38 pm, zou lan wrote: > > > > > Hi Daniel & all > > > I find the function drm_output_prepare_overlay_view() only use the

Re: universal planes in drm backend

2019-09-17 Thread Pekka Paalanen
On Tue, 17 Sep 2019 19:50:01 +1200 Scott Anderson wrote: > On 17/09/19 7:38 pm, zou lan wrote: > > Hi Daniel & all > > > > I find the function drm_output_prepare_overlay_view() only use the plane > > type of WDRM_PLANE_TYPE_OVERLAY. it could be a waste for some planes of > > type

Re: universal planes in drm backend

2019-09-17 Thread zou lan
Hi scott yes, one CRTC bind to one primary plane. But if not all the crtcs are connected and universal planes is enable, the spare primary plane could be an overlay plane. for example, plane 0 ~ 5 for 6 CRTCs, they are all primary planes. But weston only enable 3 output, then plane 3 ~ 5 could

Re: universal planes in drm backend

2019-09-17 Thread Scott Anderson
On 17/09/19 7:38 pm, zou lan wrote: Hi Daniel & all I find the function drm_output_prepare_overlay_view() only use the plane type of WDRM_PLANE_TYPE_OVERLAY. it could be a waste for some planes of type WDRM_PLANE_TYPE_PRIMARY if the universal planes is enable. For example, the kernel define

universal planes in drm backend

2019-09-17 Thread zou lan
Hi Daniel & all I find the function drm_output_prepare_overlay_view() only use the plane type of WDRM_PLANE_TYPE_OVERLAY. it could be a waste for some planes of type WDRM_PLANE_TYPE_PRIMARY if the universal planes is enable. For example, the kernel define 6 crtcs, and each crtc will have one