SymbX/systemd
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

backlight: use device_get_sysattr_unsigned() at one more place

Browse Source
This commit is contained in:
Yu Watanabe
2025-05-14 02:06:02 +09:00
parent 8ebe9f1da2
commit 25151ca2b8
1 changed files with 3 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/backlight/backlight.c
+3 -7
View File
@@ -8,6 +8,7 @@
#include "alloc-util.h"
#include "argv-util.h"
#include "device-private.h"
#include "device-util.h"
#include "escape.h"
#include "fileio.h"
@@ -298,19 +299,14 @@ static int validate_device(sd_device *device) {
static int read_max_brightness(sd_device *device, unsigned *ret) {
unsigned max_brightness;
const char *s;
int r;
assert(device);
assert(ret);
r = sd_device_get_sysattr_value(device, "max_brightness", &s);
r = device_get_sysattr_unsigned(device, "max_brightness", &max_brightness);
if (r < 0)
return log_device_warning_errno(device, r, "Failed to read 'max_brightness' attribute: %m");
r = safe_atou(s, &max_brightness);
if (r < 0)
return log_device_warning_errno(device, r, "Failed to parse 'max_brightness' \"%s\": %m", s);
return log_device_warning_errno(device, r, "Failed to read/parse 'max_brightness' attribute: %m");
/* If max_brightness is 0, then there is no actual backlight device. This happens on desktops
* with Asus mainboards that load the eeepc-wmi module. */