Re: [PATCH v4] drm/omap: plane zpos/zorder management improvements

2018-01-09 Thread Tomi Valkeinen
On 08/01/18 13:07, Laurent Pinchart wrote: >> Yes, this is a subject I have complained a few times: DRM keeping the >> state. I think that will be a problem with many other properties too. An >> app could change a platform specific property, which no other app is >> aware of, and after that no oth

Re: [PATCH v4] drm/omap: plane zpos/zorder management improvements

2018-01-08 Thread Laurent Pinchart
Hi Tomi, On Monday, 8 January 2018 12:58:28 EET Tomi Valkeinen wrote: > On 08/01/18 12:43, Laurent Pinchart wrote: > > On Monday, 8 January 2018 10:59:29 EET Tomi Valkeinen wrote: > >> On 08/01/18 10:20, Peter Ujfalusi wrote: > >>> On 2018-01-05 16:04, Laurent Pinchart wrote: > On Friday, 5 J

Re: [PATCH v4] drm/omap: plane zpos/zorder management improvements

2018-01-08 Thread Tomi Valkeinen
On 08/01/18 12:43, Laurent Pinchart wrote: > Hello, > > On Monday, 8 January 2018 10:59:29 EET Tomi Valkeinen wrote: >> On 08/01/18 10:20, Peter Ujfalusi wrote: >>> On 2018-01-05 16:04, Laurent Pinchart wrote: On Friday, 5 January 2018 13:30:37 EET Peter Ujfalusi wrote: > Use the plane in

Re: [PATCH v4] drm/omap: plane zpos/zorder management improvements

2018-01-08 Thread Laurent Pinchart
Hi Peter, On Monday, 8 January 2018 10:20:24 EET Peter Ujfalusi wrote: > On 2018-01-05 16:04, Laurent Pinchart wrote: > > On Friday, 5 January 2018 13:30:37 EET Peter Ujfalusi wrote: > >> Use the plane index as default zpos for all planes. Even if the > >> application is not setting zpos/zorder ex

Re: [PATCH v4] drm/omap: plane zpos/zorder management improvements

2018-01-08 Thread Laurent Pinchart
Hello, On Monday, 8 January 2018 10:59:29 EET Tomi Valkeinen wrote: > On 08/01/18 10:20, Peter Ujfalusi wrote: > > On 2018-01-05 16:04, Laurent Pinchart wrote: > >> On Friday, 5 January 2018 13:30:37 EET Peter Ujfalusi wrote: > >>> Use the plane index as default zpos for all planes. Even if the >

Re: [PATCH v4] drm/omap: plane zpos/zorder management improvements

2018-01-08 Thread Tomi Valkeinen
On 08/01/18 10:20, Peter Ujfalusi wrote: > Hi Laurent, > > On 2018-01-05 16:04, Laurent Pinchart wrote: >> Hi Peter, >> >> Thank you for the patch and sorry for the late review. >> >> On Friday, 5 January 2018 13:30:37 EET Peter Ujfalusi wrote: >>> Use the plane index as default zpos for all plane

Re: [PATCH v4] drm/omap: plane zpos/zorder management improvements

2018-01-08 Thread Peter Ujfalusi
Hi Laurent, On 2018-01-05 16:04, Laurent Pinchart wrote: > Hi Peter, > > Thank you for the patch and sorry for the late review. > > On Friday, 5 January 2018 13:30:37 EET Peter Ujfalusi wrote: >> Use the plane index as default zpos for all planes. Even if the >> application is not setting zpos/z

Re: [PATCH v4] drm/omap: plane zpos/zorder management improvements

2018-01-05 Thread Laurent Pinchart
Hi Peter, Thank you for the patch and sorry for the late review. On Friday, 5 January 2018 13:30:37 EET Peter Ujfalusi wrote: > Use the plane index as default zpos for all planes. Even if the > application is not setting zpos/zorder explicitly we will have unique zpos > for each plane. > > Enfor

[PATCH v4] drm/omap: plane zpos/zorder management improvements

2018-01-05 Thread Peter Ujfalusi
Use the plane index as default zpos for all planes. Even if the application is not setting zpos/zorder explicitly we will have unique zpos for each plane. Enforce that all planes must have unique zpos on the given crtc. Signed-off-by: Peter Ujfalusi --- Hi, Changes since v3: - Use drm_plane_ind