Re: [PATCH] drm: Call sysfs_notify after changing drm_connector::dpms

2017-09-27 Thread Jani Nikula
On Wed, 27 Sep 2017, Karsten Wiese wrote: > 2017-09-27 14:58 GMT+02:00 Jani Nikula : >> It still feels wrong to add sysfs notify support for just one of our >> properties, for just this one use case. In particular if the reason is >> to

Re: [PATCH] drm: Call sysfs_notify after changing drm_connector::dpms

2017-09-27 Thread Karsten Wiese
2017-09-27 14:58 GMT+02:00 Jani Nikula : > On Wed, 27 Sep 2017, Daniel Vetter wrote: > > On Wed, Sep 27, 2017 at 12:20:21PM +0200, Karsten Wiese wrote: > >> 2017-09-27 9:18 GMT+02:00 Jani Nikula : > >> > >> > On Wed, 27

Re: [PATCH] drm: Call sysfs_notify after changing drm_connector::dpms

2017-09-27 Thread Jani Nikula
On Wed, 27 Sep 2017, Daniel Vetter wrote: > On Wed, Sep 27, 2017 at 12:20:21PM +0200, Karsten Wiese wrote: >> 2017-09-27 9:18 GMT+02:00 Jani Nikula : >> >> > On Wed, 27 Sep 2017, Karsten Wiese wrote: >> > > 2017-09-25 15:48

Re: [PATCH] drm: Call sysfs_notify after changing drm_connector::dpms

2017-09-27 Thread Daniel Vetter
On Wed, Sep 27, 2017 at 12:20:21PM +0200, Karsten Wiese wrote: > 2017-09-27 9:18 GMT+02:00 Jani Nikula : > > > On Wed, 27 Sep 2017, Karsten Wiese wrote: > > > 2017-09-25 15:48 GMT+02:00 Jani Nikula : > > > > > >> On

Re: [PATCH] drm: Call sysfs_notify after changing drm_connector::dpms

2017-09-27 Thread Karsten Wiese
2017-09-27 10:45 GMT+02:00 Daniel Vetter : > On Wed, Sep 27, 2017 at 12:29:46AM +0200, Karsten Wiese wrote: > > 2017-09-25 15:48 GMT+02:00 Jani Nikula : > > > > > On Tue, 19 Sep 2017, Karsten Wiese wrote: > > > > This makes

Re: [PATCH] drm: Call sysfs_notify after changing drm_connector::dpms

2017-09-27 Thread Karsten Wiese
2017-09-27 9:18 GMT+02:00 Jani Nikula : > On Wed, 27 Sep 2017, Karsten Wiese wrote: > > 2017-09-25 15:48 GMT+02:00 Jani Nikula : > > > >> On Tue, 19 Sep 2017, Karsten Wiese wrote: > >> > This

Re: [PATCH] drm: Call sysfs_notify after changing drm_connector::dpms

2017-09-27 Thread Daniel Vetter
On Wed, Sep 27, 2017 at 12:29:46AM +0200, Karsten Wiese wrote: > 2017-09-25 15:48 GMT+02:00 Jani Nikula : > > > On Tue, 19 Sep 2017, Karsten Wiese wrote: > > > This makes poll work for the > > > /sys/class/drm/cardX/connectorY/dpms attributes.

Re: [PATCH] drm: Call sysfs_notify after changing drm_connector::dpms

2017-09-27 Thread Jani Nikula
On Wed, 27 Sep 2017, Karsten Wiese wrote: > 2017-09-25 15:48 GMT+02:00 Jani Nikula : > >> On Tue, 19 Sep 2017, Karsten Wiese wrote: >> > This makes poll work for the >> > /sys/class/drm/cardX/connectorY/dpms attributes.

Re: [PATCH] drm: Call sysfs_notify after changing drm_connector::dpms

2017-09-26 Thread Karsten Wiese
2017-09-25 15:48 GMT+02:00 Jani Nikula : > On Tue, 19 Sep 2017, Karsten Wiese wrote: > > This makes poll work for the > > /sys/class/drm/cardX/connectorY/dpms attributes. > > I guess the question is, what are you trying to achieve? What is the

Re: [PATCH] drm: Call sysfs_notify after changing drm_connector::dpms

2017-09-25 Thread Jani Nikula
On Tue, 19 Sep 2017, Karsten Wiese wrote: > This makes poll work for the > /sys/class/drm/cardX/connectorY/dpms attributes. I guess the question is, what are you trying to achieve? What is the problem that this solves? We have zero sysfs_notify in all of drm AFAICT. BR,

[PATCH] drm: Call sysfs_notify after changing drm_connector::dpms

2017-09-19 Thread Karsten Wiese
This makes poll work for the /sys/class/drm/cardX/connectorY/dpms attributes. Tested with i915 suspended by XScreenServer and suspend to RAM. Signed-off-by: Karsten Wiese --- drivers/gpu/drm/drm_atomic.c| 4 drivers/gpu/drm/drm_atomic_helper.c | 6 +- 2