Attachment 129441 Details for Bug 98404 – [PATCH 2/3] UpKbdBacklight: up_kbd_backlight_brightness_read: pass in fd

[patch] [PATCH 2/3] UpKbdBacklight: up_kbd_backlight_brightness_read: pass in fd

0002-UpKbdBacklight-up_kbd_backlight_brightness_read-pass.patch (text/plain), 2.10 KB, created by Hans de Goede on 2017-02-09 16:09:59 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Hans de Goede

Created: 2017-02-09 16:09:59 UTC

Size: 2.10 KB

patch

obsolete

>From 0c4e3a9ddcf24f4e21161383ea5f1cd9d97f53fd Mon Sep 17 00:00:00 2001
>From: Hans de Goede <hdegoede@redhat.com>
>Date: Wed, 25 Jan 2017 08:34:40 +0100
>Subject: [PATCH 2/3] UpKbdBacklight: up_kbd_backlight_brightness_read: pass in
> fd
>
>Pass the fd to use into up_kbd_backlight_brightness_read so that it can
>be used with multiple fds.
>
>This is a preparation patch for adding support for the new
>brightness_hw_changed sysfs attribute.
>---
> src/up-kbd-backlight.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
>diff --git a/src/up-kbd-backlight.c b/src/up-kbd-backlight.c
>index f693983..57d9524 100644
>--- a/src/up-kbd-backlight.c
>+++ b/src/up-kbd-backlight.c
>@@ -62,17 +62,17 @@ up_kbd_backlight_emit_change(UpKbdBacklight *kbd_backlight, int value, const cha
>  * up_kbd_backlight_brightness_read:
>  **/
> static gint
>-up_kbd_backlight_brightness_read (UpKbdBacklight *kbd_backlight)
>+up_kbd_backlight_brightness_read (UpKbdBacklight *kbd_backlight, int fd)
> {
> 	gchar buf[16];
> 	gchar *end = NULL;
> 	ssize_t len;
> 	gint64 brightness = -1;
> 
>-	g_return_val_if_fail (kbd_backlight->priv->fd >= 0, brightness);
>+	g_return_val_if_fail (fd >= 0, brightness);
> 
>-	lseek (kbd_backlight->priv->fd, 0, SEEK_SET);
>-	len = read (kbd_backlight->priv->fd, buf, G_N_ELEMENTS (buf) - 1);
>+	lseek (fd, 0, SEEK_SET);
>+	len = read (fd, buf, G_N_ELEMENTS (buf) - 1);
> 
> 	if (len > 0) {
> 		buf[len] = '\0';
>@@ -143,7 +143,7 @@ up_kbd_backlight_get_brightness (UpExportedKbdBacklight *skeleton,
> {
> 	gint brightness;
> 
>-	brightness = up_kbd_backlight_brightness_read (kbd_backlight);
>+	brightness = up_kbd_backlight_brightness_read (kbd_backlight, kbd_backlight->priv->fd);
> 
> 	if (brightness >= 0) {
> 		up_exported_kbd_backlight_complete_get_brightness (skeleton, invocation,
>@@ -270,7 +270,7 @@ up_kbd_backlight_find (UpKbdBacklight *kbd_backlight)
> 	kbd_backlight->priv->fd = open (path_now, O_RDWR);
> 
> 	/* read brightness and check if it has an acceptable value */
>-	if (up_kbd_backlight_brightness_read (kbd_backlight) < 0)
>+	if (up_kbd_backlight_brightness_read (kbd_backlight, kbd_backlight->priv->fd) < 0)
> 		goto out;
> 
> 	/* success */
>-- 
>2.9.3
>

Actions: View | Splinter Review

Attachments on bug 98404: 127509 | 127510 | 128051 | 128052 | 129440 | 129441 | 129442 | 129640 | 129641 | 129642 | 129644 | 129645 | 129646

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.