Re: [PATCH v14 03/41] compositor-drm: Introduce drm_plane_state structure

2018-02-01 Thread Pekka Paalanen
On Tue, 30 Jan 2018 18:59:35 + Daniel Stone wrote: > Hi Philipp, > > On 29 January 2018 at 10:19, Philipp Zabel wrote: > > On Wed, 2017-12-20 at 12:26 +, Daniel Stone wrote: > >> + wl_list_for_each(ps, &src->plane_list, link) { > >> + /* Don't carry planes which are no

Re: [PATCH v14 03/41] compositor-drm: Introduce drm_plane_state structure

2018-01-30 Thread Daniel Stone
Hi Philipp, On 29 January 2018 at 10:19, Philipp Zabel wrote: > On Wed, 2017-12-20 at 12:26 +, Daniel Stone wrote: >> + wl_list_for_each(ps, &src->plane_list, link) { >> + /* Don't carry planes which are now disabled; these should be >> + * free for other outputs

Re: [PATCH v14 03/41] compositor-drm: Introduce drm_plane_state structure

2018-01-29 Thread Philipp Zabel
Hi Daniel, On Wed, 2017-12-20 at 12:26 +, Daniel Stone wrote: > Track dynamic plane state (CRTC, FB, position) in separate structures, > rather than as part of the plane. This will make it easier to handle > state management later, and much more closely tracks what the kernel > does with atomi

Re: [PATCH v14 03/41] compositor-drm: Introduce drm_plane_state structure

2018-01-15 Thread Pekka Paalanen
On Wed, 20 Dec 2017 12:26:20 + Daniel Stone wrote: > Track dynamic plane state (CRTC, FB, position) in separate structures, > rather than as part of the plane. This will make it easier to handle > state management later, and much more closely tracks what the kernel > does with atomic modesets

[PATCH v14 03/41] compositor-drm: Introduce drm_plane_state structure

2017-12-20 Thread Daniel Stone
Track dynamic plane state (CRTC, FB, position) in separate structures, rather than as part of the plane. This will make it easier to handle state management later, and much more closely tracks what the kernel does with atomic modesets. The fb_last pointer previously used in drm_plane now becomes p