Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-24 Thread Marius-cristian Vlad
On Fri, 2018-08-24 at 11:58 +0200, Philipp Zabel wrote: > Hi Marius, > > On Fri, 2018-08-24 at 09:21 +, Marius-cristian Vlad wrote: > > On Fri, 2018-08-24 at 10:57 +0200, Philipp Zabel wrote: > > [...] > > > > yes, sending one event per connector is a good design, but see > > > > below if we

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-24 Thread Philipp Zabel
Hi Marius, On Fri, 2018-08-24 at 09:21 +, Marius-cristian Vlad wrote: > On Fri, 2018-08-24 at 10:57 +0200, Philipp Zabel wrote: [...] > > > yes, sending one event per connector is a good design, but see > > > below if we actually might want to extend that to creating an > > > object per

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-24 Thread Marius-cristian Vlad
On Fri, 2018-08-24 at 10:57 +0200, Philipp Zabel wrote: > Hi Pekka, > > On Thu, 2018-08-23 at 14:39 +0300, Pekka Paalanen wrote: > > Sorry for the potentially duplicate email, my MUA messed up the CC > > list > > on the first go. > > > > > > On Thu, 23 Aug 2018 08:41:30 +0200 > > Philipp Zabel

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-24 Thread Philipp Zabel
Hi Pekka, On Thu, 2018-08-23 at 14:39 +0300, Pekka Paalanen wrote: > Sorry for the potentially duplicate email, my MUA messed up the CC list > on the first go. > > > On Thu, 23 Aug 2018 08:41:30 +0200 > Philipp Zabel wrote: > > > Hi, > > > > On Wed, 2018-08-22 at 11:12 +, Marius-cristian

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-24 Thread Pekka Paalanen
On Thu, 23 Aug 2018 15:12:03 + Marius-cristian Vlad wrote: > On Thu, 2018-08-23 at 14:39 +0300, Pekka Paalanen wrote: > > > > > Especially EDID info would be most useful for finding the right > > > > > VR > > > > > headset.   > > > > Sharing EDID could be tricky mechanically. If we

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-23 Thread Marius-cristian Vlad
On Thu, 2018-08-23 at 08:41 +0200, Philipp Zabel wrote: > Hi, > > On Wed, 2018-08-22 at 11:12 +, Marius-cristian Vlad wrote: > [...] > > > Why not just send the connectors one by one, a single event with > > > all > > > relevant information for each? > > > > Hmm, okay, I'll try do that.  >

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-23 Thread Marius-cristian Vlad
On Thu, 2018-08-23 at 14:39 +0300, Pekka Paalanen wrote: > Sorry for the potentially duplicate email, my MUA messed up the CC > list > on the first go. > > > On Thu, 23 Aug 2018 08:41:30 +0200 > Philipp Zabel wrote: > > > Hi, > > > > On Wed, 2018-08-22 at 11:12 +, Marius-cristian Vlad

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-23 Thread Pekka Paalanen
Sorry for the potentially duplicate email, my MUA messed up the CC list on the first go. On Thu, 23 Aug 2018 08:41:30 +0200 Philipp Zabel wrote: > Hi, > > On Wed, 2018-08-22 at 11:12 +, Marius-cristian Vlad wrote: > [...] > > > Why not just send the connectors one by one, a single event

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-23 Thread Philipp Zabel
On Mon, 2018-08-20 at 23:00 +0300, Marius Vlad wrote: [...] > + > + > + This interface is used for managing zwp_kms_lease_v1 object. It is used > + to create a zwp_kms_lease_v1 object (the actual lease object) and also > to > + revoke the lease. > + > + > + > +

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-23 Thread Philipp Zabel
Hi, On Wed, 2018-08-22 at 11:12 +, Marius-cristian Vlad wrote: [...] > > Why not just send the connectors one by one, a single event with all > > relevant information for each? > > Hmm, okay, I'll try do that. I'm wondering what should be used to identify a connector to a hypothetical

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-23 Thread Philipp Zabel
On Wed, 2018-08-22 at 09:24 -0700, Keith Packard wrote: > Marius-cristian Vlad writes: > > > > Can't the client query available modes on the passed connector via > > > the > > > leased fd? > > > > That's how the client does it now, it uses the leased fd to query > > available modes. Presumably

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-22 Thread Keith Packard
Marius-cristian Vlad writes: >> Can't the client query available modes on the passed connector via >> the >> leased fd? > > That's how the client does it now, it uses the leased fd to query > available modes. Presumably the client should have/receive all the data > from the compositor Just

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-22 Thread Marius-cristian Vlad
On Wed, 2018-08-22 at 09:17 +0200, Philipp Zabel wrote: > Hi Marius, Hi, > > On Mon, 2018-08-20 at 23:00 +0300, Marius Vlad wrote: > > Simple protocol extension to manage DRM lease. Based on the work by > > Keith > > Packard in [1], respectively [2]. > > > > [1]

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-22 Thread Philipp Zabel
Hi Marius, On Mon, 2018-08-20 at 23:00 +0300, Marius Vlad wrote: > Simple protocol extension to manage DRM lease. Based on the work by Keith > Packard in [1], respectively [2]. > > [1] > https://cgit.freedesktop.org/mesa/drm/commit/?id=c4171535389d72e9135c9615cecd07b346fd6d7e > [2] >

[PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-20 Thread Marius Vlad
Simple protocol extension to manage DRM lease. Based on the work by Keith Packard in [1], respectively [2]. [1] https://cgit.freedesktop.org/mesa/drm/commit/?id=c4171535389d72e9135c9615cecd07b346fd6d7e [2]