On Mon, 5 Feb 2018 18:44:10 +0000 Daniel Stone <dani...@collabora.com> wrote:
> Rather than a more piecemeal approach at backend creation, explicitly > track connectors and CRTCs we do not intend to use, so we can ensure > they are disabled where appropriate. > > When we have an updated list of connector and CRTC IDs, we add any which > are not owned by an enabled drm_output to the list. We remove them from > the list when drm_output_repaint() is called for that output, and re-add > them when the output is disabled or destroyed. > > Signed-off-by: Daniel Stone <dani...@collabora.com> > --- > libweston/compositor-drm.c | 89 > +++++++++++++++++++++++++++++++++++++++++++++- > 1 file changed, 88 insertions(+), 1 deletion(-) > Hi Daniel, this patch now looks technically correct and logically consistent: Reviewed-by: Pekka Paalanen <pekka.paala...@collabora.co.uk> I was about to claim the logic could be further simplified, but I forgot that connectors can appear and disappear on update_outputs(). Thanks, pq
pgpbPtfCZJask.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel
