Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-07-05 Thread Pekka Paalanen
On Thu, 4 Jul 2019 16:12:10 +0530 Ramalingam C wrote: > On 2019-07-04 at 14:12:27 +0300, Pekka Paalanen wrote: > > On Tue, 7 May 2019 21:57:43 +0530 > > Ramalingam C wrote: > > > > > DRM API for generating uevent for a status changes of connector's > > > property. > > > > > > This uevent wi

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-07-04 Thread Ramalingam C
On 2019-07-04 at 14:12:27 +0300, Pekka Paalanen wrote: > On Tue, 7 May 2019 21:57:43 +0530 > Ramalingam C wrote: > > > DRM API for generating uevent for a status changes of connector's > > property. > > > > This uevent will have following details related to the status change: > > > > HOTPLUG

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-07-04 Thread Pekka Paalanen
On Tue, 7 May 2019 21:57:43 +0530 Ramalingam C wrote: > DRM API for generating uevent for a status changes of connector's > property. > > This uevent will have following details related to the status change: > > HOTPLUG=1, CONNECTOR= and PROPERTY= > > Need ACK from this uevent from userspac

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-06-04 Thread Pekka Paalanen
On Mon, 3 Jun 2019 15:19:13 + "Ser, Simon" wrote: > On Mon, 2019-06-03 at 17:08 +0200, Daniel Vetter wrote: > > It's definitely a very suboptimal situation. Not sure there's a good way > > out. The trouble here is that i915 ended up configuring crtc/connectors > > differently than -modesetti

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-06-03 Thread Ser, Simon
On Mon, 2019-06-03 at 17:08 +0200, Daniel Vetter wrote: > On Mon, Jun 03, 2019 at 11:50:53AM +0200, Michel Dänzer wrote: > > On 2019-05-21 9:52 a.m., Daniel Vetter wrote: > > > On Tue, May 21, 2019 at 8:55 AM Pekka Paalanen > > > wrote: > > > > On Mon, 20 May 2019 18:11:07 +0200 > > > > Daniel Ve

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-06-03 Thread Daniel Vetter
On Mon, Jun 03, 2019 at 11:50:53AM +0200, Michel Dänzer wrote: > On 2019-05-21 9:52 a.m., Daniel Vetter wrote: > > On Tue, May 21, 2019 at 8:55 AM Pekka Paalanen wrote: > >> On Mon, 20 May 2019 18:11:07 +0200 > >> Daniel Vetter wrote: > >> > >>> There's also a fairly easy fix for that -modesettin

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-06-03 Thread Michel Dänzer
On 2019-05-21 9:52 a.m., Daniel Vetter wrote: > On Tue, May 21, 2019 at 8:55 AM Pekka Paalanen wrote: >> On Mon, 20 May 2019 18:11:07 +0200 >> Daniel Vetter wrote: >> >>> There's also a fairly easy fix for that -modesetting issue: We don't >>> expose atomic if the compositor has a process name of

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-21 Thread Daniel Vetter
On Tue, May 21, 2019 at 12:01:29PM +0300, Pekka Paalanen wrote: > Hi Daniel, > > what says the assumption of the only monitor being driven by CRTC 0 > was a bad one? :-p > > It's probably not obvious that userspace needs to explicitly try to > avoid invalid configuration combinations by inspectin

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-21 Thread Pekka Paalanen
On Tue, 21 May 2019 09:52:50 +0200 Daniel Vetter wrote: > On Tue, May 21, 2019 at 8:55 AM Pekka Paalanen wrote: > > > > On Mon, 20 May 2019 18:11:07 +0200 > > Daniel Vetter wrote: > > > > > On Fri, May 17, 2019 at 01:08:24PM +0300, Pekka Paalanen wrote: > > > > On Thu, 16 May 2019 14:24:55

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-21 Thread Daniel Vetter
On Tue, May 21, 2019 at 8:55 AM Pekka Paalanen wrote: > > On Mon, 20 May 2019 18:11:07 +0200 > Daniel Vetter wrote: > > > On Fri, May 17, 2019 at 01:08:24PM +0300, Pekka Paalanen wrote: > > > On Thu, 16 May 2019 14:24:55 +0200 > > > Daniel Vetter wrote: > > > > > > > On Thu, May 16, 2019 at 11:2

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-20 Thread Pekka Paalanen
On Mon, 20 May 2019 18:11:07 +0200 Daniel Vetter wrote: > On Fri, May 17, 2019 at 01:08:24PM +0300, Pekka Paalanen wrote: > > On Thu, 16 May 2019 14:24:55 +0200 > > Daniel Vetter wrote: > > > > > On Thu, May 16, 2019 at 11:22:11AM +0300, Pekka Paalanen wrote: > > > > On Wed, 15 May 2019 10:

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-20 Thread Paul Kocialkowski
Le lundi 20 mai 2019 à 18:11 +0200, Daniel Vetter a écrit : > On Fri, May 17, 2019 at 01:08:24PM +0300, Pekka Paalanen wrote: > > On Thu, 16 May 2019 14:24:55 +0200 > > Daniel Vetter wrote: > > > > > On Thu, May 16, 2019 at 11:22:11AM +0300, Pekka Paalanen wrote: > > > > On Wed, 15 May 2019 10:24

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-20 Thread Daniel Vetter
On Fri, May 17, 2019 at 01:08:24PM +0300, Pekka Paalanen wrote: > On Thu, 16 May 2019 14:24:55 +0200 > Daniel Vetter wrote: > > > On Thu, May 16, 2019 at 11:22:11AM +0300, Pekka Paalanen wrote: > > > On Wed, 15 May 2019 10:24:49 +0200 > > > Daniel Vetter wrote: > > > > > > > On Wed, May 15, 2

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-17 Thread Pekka Paalanen
On Thu, 16 May 2019 14:24:55 +0200 Daniel Vetter wrote: > On Thu, May 16, 2019 at 11:22:11AM +0300, Pekka Paalanen wrote: > > On Wed, 15 May 2019 10:24:49 +0200 > > Daniel Vetter wrote: > > > > > On Wed, May 15, 2019 at 10:37:31AM +0300, Pekka Paalanen wrote: > > > > On Tue, 14 May 2019 16:

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-16 Thread Daniel Vetter
On Thu, May 16, 2019 at 11:22:11AM +0300, Pekka Paalanen wrote: > On Wed, 15 May 2019 10:24:49 +0200 > Daniel Vetter wrote: > > > On Wed, May 15, 2019 at 10:37:31AM +0300, Pekka Paalanen wrote: > > > On Tue, 14 May 2019 16:34:01 +0200 > > > Daniel Vetter wrote: > > > > > > > On Tue, May 14, 2

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-16 Thread Pekka Paalanen
On Wed, 15 May 2019 10:24:49 +0200 Daniel Vetter wrote: > On Wed, May 15, 2019 at 10:37:31AM +0300, Pekka Paalanen wrote: > > On Tue, 14 May 2019 16:34:01 +0200 > > Daniel Vetter wrote: > > > > > On Tue, May 14, 2019 at 3:36 PM Pekka Paalanen > > > wrote: > > > > > > > > On Tue, 14 May 20

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-15 Thread Paul Kocialkowski
Hi, On Tue, 2019-05-14 at 16:28 +0200, Daniel Vetter wrote: > On Tue, May 14, 2019 at 4:13 PM Paul Kocialkowski > wrote: > > Hi, > > > > On Tue, 2019-05-14 at 13:09 +0200, Daniel Vetter wrote: > > > On Mon, May 13, 2019 at 7:14 PM Paul Kocialkowski > > > wrote: > > > > Hey, > > > > > > > > Le

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-15 Thread Daniel Vetter
On Wed, May 15, 2019 at 10:37:31AM +0300, Pekka Paalanen wrote: > On Tue, 14 May 2019 16:34:01 +0200 > Daniel Vetter wrote: > > > On Tue, May 14, 2019 at 3:36 PM Pekka Paalanen wrote: > > > > > > On Tue, 14 May 2019 13:02:09 +0200 > > > Daniel Vetter wrote: > > > > > > > On Tue, May 14, 2019

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-15 Thread Paul Kocialkowski
Hi, On Wed, 2019-05-15 at 10:37 +0300, Pekka Paalanen wrote: > On Tue, 14 May 2019 16:34:01 +0200 > Daniel Vetter wrote: > > > On Tue, May 14, 2019 at 3:36 PM Pekka Paalanen wrote: > > > On Tue, 14 May 2019 13:02:09 +0200 > > > Daniel Vetter wrote: > > > > > > > On Tue, May 14, 2019 at 10:18

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-15 Thread Daniel Vetter
On Wed, May 15, 2019 at 09:43:24AM +0200, Paul Kocialkowski wrote: > Hi, > > On Tue, 2019-05-14 at 16:28 +0200, Daniel Vetter wrote: > > On Tue, May 14, 2019 at 4:13 PM Paul Kocialkowski > > wrote: > > > Hi, > > > > > > On Tue, 2019-05-14 at 13:09 +0200, Daniel Vetter wrote: > > > > On Mon, May

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-15 Thread Pekka Paalanen
On Tue, 14 May 2019 16:34:01 +0200 Daniel Vetter wrote: > On Tue, May 14, 2019 at 3:36 PM Pekka Paalanen wrote: > > > > On Tue, 14 May 2019 13:02:09 +0200 > > Daniel Vetter wrote: > > > > > On Tue, May 14, 2019 at 10:18 AM Ser, Simon wrote: > > > > > > > > On Tue, 2019-05-14 at 11:02 +0300

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-14 Thread Daniel Vetter
On Tue, May 14, 2019 at 3:36 PM Pekka Paalanen wrote: > > On Tue, 14 May 2019 13:02:09 +0200 > Daniel Vetter wrote: > > > On Tue, May 14, 2019 at 10:18 AM Ser, Simon wrote: > > > > > > On Tue, 2019-05-14 at 11:02 +0300, Pekka Paalanen wrote: > > > > On Mon, 13 May 2019 11:34:58 +0200 > > > > Dan

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-14 Thread Daniel Vetter
On Tue, May 14, 2019 at 4:13 PM Paul Kocialkowski wrote: > > Hi, > > On Tue, 2019-05-14 at 13:09 +0200, Daniel Vetter wrote: > > On Mon, May 13, 2019 at 7:14 PM Paul Kocialkowski > > wrote: > > > Hey, > > > > > > Le lundi 13 mai 2019 à 11:34 +0200, Daniel Vetter a écrit : > > > > On Mon, May 13,

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-14 Thread Paul Kocialkowski
Hi, On Tue, 2019-05-14 at 13:09 +0200, Daniel Vetter wrote: > On Mon, May 13, 2019 at 7:14 PM Paul Kocialkowski > wrote: > > Hey, > > > > Le lundi 13 mai 2019 à 11:34 +0200, Daniel Vetter a écrit : > > > On Mon, May 13, 2019 at 11:02 AM Paul Kocialkowski > > > wrote: > > > > Hi, > > > > > > >

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-14 Thread Paul Kocialkowski
Hi, On Tue, 2019-05-14 at 16:36 +0300, Pekka Paalanen wrote: > On Tue, 14 May 2019 13:02:09 +0200 > Daniel Vetter wrote: > > > On Tue, May 14, 2019 at 10:18 AM Ser, Simon wrote: > > > On Tue, 2019-05-14 at 11:02 +0300, Pekka Paalanen wrote: > > > > On Mon, 13 May 2019 11:34:58 +0200 > > > > D

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-14 Thread Pekka Paalanen
On Tue, 14 May 2019 13:02:09 +0200 Daniel Vetter wrote: > On Tue, May 14, 2019 at 10:18 AM Ser, Simon wrote: > > > > On Tue, 2019-05-14 at 11:02 +0300, Pekka Paalanen wrote: > > > On Mon, 13 May 2019 11:34:58 +0200 > > > Daniel Vetter wrote: > > > > > > > On Mon, May 13, 2019 at 11:02 AM Pa

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-14 Thread Daniel Vetter
On Mon, May 13, 2019 at 11:20 PM Lyude Paul wrote: > > Hi-just wanted to give some general thoughts here. > > First off I'm 100% behind the epoch idea, that was one of the ideas I had been > thinking of proposing here in the first place but probably forgot at some > point down the road. > > A coup

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-14 Thread Daniel Vetter
On Mon, May 13, 2019 at 7:14 PM Paul Kocialkowski wrote: > > Hey, > > Le lundi 13 mai 2019 à 11:34 +0200, Daniel Vetter a écrit : > > On Mon, May 13, 2019 at 11:02 AM Paul Kocialkowski > > wrote: > > > Hi, > > > > > > On Fri, 2019-05-10 at 16:54 +0200, Daniel Vetter wrote: > > > > On Fri, May 10,

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-14 Thread Daniel Vetter
On Tue, May 14, 2019 at 10:18 AM Ser, Simon wrote: > > On Tue, 2019-05-14 at 11:02 +0300, Pekka Paalanen wrote: > > On Mon, 13 May 2019 11:34:58 +0200 > > Daniel Vetter wrote: > > > > > On Mon, May 13, 2019 at 11:02 AM Paul Kocialkowski > > > wrote: > > > > Hi, > > > > > > > > On Fri, 2019-05-10

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-14 Thread Ser, Simon
On Tue, 2019-05-14 at 11:02 +0300, Pekka Paalanen wrote: > On Mon, 13 May 2019 11:34:58 +0200 > Daniel Vetter wrote: > > > On Mon, May 13, 2019 at 11:02 AM Paul Kocialkowski > > wrote: > > > Hi, > > > > > > On Fri, 2019-05-10 at 16:54 +0200, Daniel Vetter wrote: > > > > On Fri, May 10, 2019 a

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-14 Thread Pekka Paalanen
On Mon, 13 May 2019 11:34:58 +0200 Daniel Vetter wrote: > On Mon, May 13, 2019 at 11:02 AM Paul Kocialkowski > wrote: > > > > Hi, > > > > On Fri, 2019-05-10 at 16:54 +0200, Daniel Vetter wrote: > > > On Fri, May 10, 2019 at 2:12 PM Paul Kocialkowski > > > wrote: > > > > Hi, > > > > > > > >

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-13 Thread Ser, Simon
On Mon, 2019-05-13 at 17:04 +0200, Daniel Vetter wrote: > On Mon, May 13, 2019 at 10:11:01AM +, Ser, Simon wrote: > > On Mon, 2019-05-13 at 11:34 +0200, Daniel Vetter wrote: > > > On Mon, May 13, 2019 at 11:02 AM Paul Kocialkowski > > > wrote: > > > > Hi, > > > > > > > > On Fri, 2019-05-10 at

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-13 Thread Lyude Paul
Hi-just wanted to give some general thoughts here. First off I'm 100% behind the epoch idea, that was one of the ideas I had been thinking of proposing here in the first place but probably forgot at some point down the road. A couple of other things: * I think it would also probably be good to h

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-13 Thread Paul Kocialkowski
Hey, Le lundi 13 mai 2019 à 11:34 +0200, Daniel Vetter a écrit : > On Mon, May 13, 2019 at 11:02 AM Paul Kocialkowski > wrote: > > Hi, > > > > On Fri, 2019-05-10 at 16:54 +0200, Daniel Vetter wrote: > > > On Fri, May 10, 2019 at 2:12 PM Paul Kocialkowski > > > wrote: > > > > Hi, > > > > > > >

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-13 Thread Daniel Vetter
On Mon, May 13, 2019 at 10:11:01AM +, Ser, Simon wrote: > On Mon, 2019-05-13 at 11:34 +0200, Daniel Vetter wrote: > > On Mon, May 13, 2019 at 11:02 AM Paul Kocialkowski > > wrote: > > > Hi, > > > > > > On Fri, 2019-05-10 at 16:54 +0200, Daniel Vetter wrote: > > > > On Fri, May 10, 2019 at 2:1

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-13 Thread Ser, Simon
On Mon, 2019-05-13 at 11:34 +0200, Daniel Vetter wrote: > On Mon, May 13, 2019 at 11:02 AM Paul Kocialkowski > wrote: > > Hi, > > > > On Fri, 2019-05-10 at 16:54 +0200, Daniel Vetter wrote: > > > On Fri, May 10, 2019 at 2:12 PM Paul Kocialkowski > > > wrote: > > > > Hi, > > > > > > > > On Tue,

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-13 Thread Paul Kocialkowski
Hi, On Fri, 2019-05-10 at 16:54 +0200, Daniel Vetter wrote: > On Fri, May 10, 2019 at 2:12 PM Paul Kocialkowski > wrote: > > Hi, > > > > On Tue, 2019-05-07 at 21:57 +0530, Ramalingam C wrote: > > > DRM API for generating uevent for a status changes of connector's > > > property. > > > > > > Thi

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-13 Thread Daniel Vetter
On Mon, May 13, 2019 at 11:02 AM Paul Kocialkowski wrote: > > Hi, > > On Fri, 2019-05-10 at 16:54 +0200, Daniel Vetter wrote: > > On Fri, May 10, 2019 at 2:12 PM Paul Kocialkowski > > wrote: > > > Hi, > > > > > > On Tue, 2019-05-07 at 21:57 +0530, Ramalingam C wrote: > > > > DRM API for generatin

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-10 Thread Daniel Vetter
On Fri, May 10, 2019 at 2:12 PM Paul Kocialkowski wrote: > > Hi, > > On Tue, 2019-05-07 at 21:57 +0530, Ramalingam C wrote: > > DRM API for generating uevent for a status changes of connector's > > property. > > > > This uevent will have following details related to the status change: > > > > HO

Re: [Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-10 Thread Paul Kocialkowski
Hi, On Tue, 2019-05-07 at 21:57 +0530, Ramalingam C wrote: > DRM API for generating uevent for a status changes of connector's > property. > > This uevent will have following details related to the status change: > > HOTPLUG=1, CONNECTOR= and PROPERTY= > > Need ACK from this uevent from users

[Intel-gfx] [PATCH v7 09/11] drm: uevent for connector status change

2019-05-07 Thread Ramalingam C
DRM API for generating uevent for a status changes of connector's property. This uevent will have following details related to the status change: HOTPLUG=1, CONNECTOR= and PROPERTY= Need ACK from this uevent from userspace consumer. v2: Minor fixes at KDoc comments [Daniel] v3: Check the