Attachment 100492 Details for Bug 79699 – Hook up a backlight udev monitor

[patch] Hook up a backlight udev monitor

0001-sna-Hook-up-a-backlight-udev-monitor-for-external-ch.patch (text/plain), 4.02 KB, created by Chris Wilson on 2014-06-05 21:48:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Chris Wilson

Created: 2014-06-05 21:48:44 UTC

Size: 4.02 KB

patch

obsolete

>From 11baf900f5d26635e407d9d69bf8a206a983580d Mon Sep 17 00:00:00 2001
>From: Chris Wilson <chris@chris-wilson.co.uk>
>Date: Thu, 5 Jun 2014 22:43:37 +0100
>Subject: [PATCH] sna: Hook up a backlight udev monitor for external changes
>
>Changes to the backlights are notified through uevents. Hooking up a
>udev monitor to listen out for external changes to the backlight (e.g.
>through ACPI function keys, or by the user writing to
>/sys/class/backlight directly) is easier than enabling polling on the
>backlight sysfs file using X's select() mechanism.
>
>Reported-by: Alexander Mezin <mezin.alexander@gmail.com>
>Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=79699
>Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
>---
> src/sna/sna.h         |  5 +++
> src/sna/sna_display.c | 94 +++++++++++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 99 insertions(+)
>
>diff --git a/src/sna/sna.h b/src/sna/sna.h
>index 7845cd1..0739f7b 100644
>--- a/src/sna/sna.h
>+++ b/src/sna/sna.h
>@@ -298,6 +298,11 @@ struct sna {
> 		unsigned serial;
> 
> 		uint32_t *encoders;
>+
>+#if HAVE_UDEV
>+		struct udev_monitor *backlight_monitor;
>+		pointer backlight_handler;
>+#endif
> 	} mode;
> 
> 	struct {
>diff --git a/src/sna/sna_display.c b/src/sna/sna_display.c
>index 3453c95..a5381c8 100644
>--- a/src/sna/sna_display.c
>+++ b/src/sna/sna_display.c
>@@ -4790,6 +4790,98 @@ sna_crtc_config_notify(ScreenPtr screen)
> #define sna_setup_provider(scrn)
> #endif
> 
>+#if HAVE_UDEV
>+static void
>+backlight_uevent(int fd, void *closure)
>+{
>+	struct sna *sna = closure;
>+	xf86CrtcConfigPtr config = XF86_CRTC_CONFIG_PTR(sna->scrn);
>+	int i;
>+
>+	for (i = 0; i < sna->mode.num_real_output; i++) {
>+		xf86OutputPtr output = config->output[i];
>+		struct sna_output *sna_output = to_sna_output(output);
>+		int val;
>+
>+		if (sna_output->dpms_mode != DPMSModeOn)
>+			continue;
>+
>+		val = backlight_get(&sna_output->backlight);
>+		if (val < 0)
>+			continue;
>+
>+		if (val == sna_output->backlight_active_level)
>+			continue;
>+
>+		sna_output->backlight_active_level = val;
>+
>+		RRChangeOutputProperty(output->randr_output,
>+				       backlight_atom, XA_INTEGER,
>+				       32, PropModeReplace, 1, &val,
>+				       FALSE, FALSE);
>+		RRChangeOutputProperty(output->randr_output,
>+				       backlight_deprecated_atom, XA_INTEGER,
>+				       32, PropModeReplace, 1, &val,
>+				       FALSE, FALSE);
>+	}
>+}
>+
>+static void sna_backlight_pre_init(struct sna *sna)
>+{
>+	struct udev *u;
>+	struct udev_monitor *mon;
>+
>+	u = udev_new();
>+	if (!u)
>+		return;
>+
>+	mon = udev_monitor_new_from_netlink(u, "udev");
>+	if (!mon)
>+		goto free_udev;
>+
>+	if (udev_monitor_filter_add_match_subsystem_devtype(mon, "backlight", NULL))
>+		goto free_monitor;
>+
>+	if (udev_monitor_enable_receiving(mon))
>+		goto free_monitor;
>+
>+	sna->mode.backlight_handler =
>+		xf86AddGeneralHandler(udev_monitor_get_fd(mon),
>+				      backlight_uevent, sna);
>+	if (!sna->mode.backlight_handler)
>+		goto free_monitor;
>+
>+	sna->mode.backlight_monitor = mon;
>+
>+	return;
>+
>+free_monitor:
>+	udev_monitor_unref(mon);
>+free_udev:
>+	udev_unref(u);
>+}
>+
>+static void sna_backlight_close(struct sna *sna)
>+{
>+	struct udev *u;
>+
>+	if (sna->mode.backlight_handler == NULL)
>+		return;
>+
>+	xf86RemoveGeneralHandler(sna->mode.backlight_handler);
>+
>+	u = udev_monitor_get_udev(sna->mode.backlight_monitor);
>+	udev_monitor_unref(sna->mode.backlight_monitor);
>+	udev_unref(u);
>+
>+	sna->mode.backlight_handler = NULL;
>+	sna->mode.backlight_monitor = NULL;
>+}
>+#else
>+static void sna_backlight_pre_init(struct sna *sna) { }
>+static void sna_backlight_close(struct sna *sna) { }
>+#endif
>+
> bool sna_mode_pre_init(ScrnInfoPtr scrn, struct sna *sna)
> {
> 	drmModeResPtr res;
>@@ -4849,6 +4941,7 @@ bool sna_mode_pre_init(ScrnInfoPtr scrn, struct sna *sna)
> 		drmModeFreeResources(res);
> 
> 		sna_cursor_pre_init(sna);
>+		sna_backlight_pre_init(sna);
> 	} else {
> 		if (num_fake == 0)
> 			num_fake = 1;
>@@ -4885,6 +4978,7 @@ sna_mode_close(struct sna *sna)
> 	if (sna->flags & SNA_IS_HOSTED)
> 		return;
> 
>+	sna_backlight_close(sna);
> 	sna_cursor_close(sna);
> 
> 	for (i = 0; i < sna->mode.num_real_crtc; i++)
>-- 
>2.0.0
>

Actions: View | Splinter Review

Attachments on bug 79699: 100485 | 100492 | 100493 | 100503

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.