On Thu, 27 Jul 2017 14:07:31 +0300 Pekka Paalanen <[email protected]> wrote:
> On Thu, 27 Jul 2017 11:54:43 +0100 > Daniel Stone <[email protected]> wrote: > > > Hi, > > > > On 27 July 2017 at 11:51, Pekka Paalanen <[email protected]> wrote: > > > @@ -2810,6 +2820,7 @@ create_output_for_connector(struct drm_backend *b, > > > if (output == NULL) > > > goto err; > > > > > > + output->state_invalid = true; > > > output->connector = connector; > > > output->crtc_id = resources->crtcs[i]; > > > output->pipe = i; > > > @@ -3130,6 +3141,10 @@ session_notify(struct wl_listener *listener, void > > > *data) > > > weston_log("activating session\n"); > > > weston_compositor_wake(compositor); > > > weston_compositor_damage_all(compositor); > > > + > > > + wl_list_for_each(output, &compositor->output_list, > > > base.link) > > > + output->state_invalid = true; > > > + > > > > In addition to output creation, and on VT enter, should we also be > > resetting the state on output enable? Or maybe just moving it there > > from create_output_for_connector, rather than both. Either way: > > Hi, > > yes, we should indeed. But I don't think enable is called on VT enter, > so we still need it on VT enter explicitly. I'll remove it from create > output, as that path always goes through enable, and have it in enable. > > > Reviewed-by: Daniel Stone <[email protected]> > > Will test and push. Done: f981d695..6b65d8f1 master -> master Thanks, pq
pgppu9_1DpQss.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
