Re: [PATCH weston] RFC: libweston/compositor-drm: Add support for drm-lease.

2018-01-26 Thread Pekka Paalanen
t; ; wayland-devel@lists.freedesktop.org > Subject: Re: [PATCH weston] RFC: libweston/compositor-drm: Add support for > drm-lease. > > On Thu, 25 Jan 2018 11:33:34 + > Daniel Stone wrote: > > > Hi Marius, > > > > On 25 J

RE: [PATCH weston] RFC: libweston/compositor-drm: Add support for drm-lease.

2018-01-25 Thread Marius-cristian Vlad
-Original Message- From: Pekka Paalanen [mailto:pekka.paala...@collabora.co.uk] Sent: Thursday, January 25, 2018 2:06 PM To: Daniel Stone Cc: Marius-cristian Vlad ; Keith Packard ; wayland-devel@lists.freedesktop.org Subject: Re: [PATCH weston] RFC: libweston/compositor-drm: Add

Re: [PATCH weston] RFC: libweston/compositor-drm: Add support for drm-lease.

2018-01-25 Thread Pekka Paalanen
On Thu, 25 Jan 2018 11:33:34 + Daniel Stone wrote: > Hi Marius, > > On 25 January 2018 at 11:10, Marius-cristian Vlad > wrote: > > >> + wl_signal_emit(&compositor->wake_signal, > > >> compositor); > > >> + > > >> wl_event_source_timer_update(co

Re: [PATCH weston] RFC: libweston/compositor-drm: Add support for drm-lease.

2018-01-25 Thread Daniel Stone
Hi Marius, On 25 January 2018 at 11:10, Marius-cristian Vlad wrote: > >> +struct drm_display { > >> + uint32_t connector_id; > >> + uint32_t crtc_id; > >> + uint32_t plane_id; > >> +}; > > > > This seems to get stored but not used after that? > > [mvlad] A helper struct to store

RE: [PATCH weston] RFC: libweston/compositor-drm: Add support for drm-lease.

2018-01-25 Thread Marius-cristian Vlad
-Original Message- From: Daniel Stone [mailto:dan...@fooishbar.org] Sent: Thursday, January 25, 2018 12:07 AM To: Marius-cristian Vlad Cc: wayland-devel@lists.freedesktop.org; Pekka Paalanen ; Keith Packard Subject: Re: [PATCH weston] RFC: libweston/compositor-drm: Add support for

Re: [PATCH weston] RFC: libweston/compositor-drm: Add support for drm-lease.

2018-01-24 Thread Daniel Stone
Hi Marius, The protocol changes I suggested would require a fair bit of work here, but I've enclosed a few comments on the implementation. Also, do you have a client you're using for this somewhere, that we could use to test? On 24 January 2018 at 19:11, Marius Vlad wrote: > @@ -1208,6 +1209,15