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

[Pekka Paalanen]

On Wed, 20 Dec 2017 12:26:20 +0000
Daniel Stone <daniels at collabora.com> 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.
> 
> The fb_last pointer previously used in drm_plane now becomes part of
> output->state_last, and is not directly visible from the plane itself.
> 
> Signed-off-by: Daniel Stone <daniels at collabora.com>
> ---
>  libweston/compositor-drm.c | 348 ++++++++++++++++++++++++++++++++++++---------
>  1 file changed, 283 insertions(+), 65 deletions(-)

Hi,

my comment from v12 is fixed, so:

Reviewed-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>


Thanks,
pq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20180116/40a842d1/attachment.sig>