From bff72840e9c69b3e5bec7264c95b3ac245a70fc0 Mon Sep 17 00:00:00 2001
From: Ahmed Abdelhalim <pm@a14m.dev>
Date: Mon, 9 Feb 2026 22:56:05 +0100
Subject: Use concurrent locking to avoid stacking ddcutil

Co-Authored-By: Claude.ai
---
 roles/hyprland/files/scripts/brightness.sh | 28 +++++++++++++++-------------
 1 file changed, 15 insertions(+), 13 deletions(-)

diff --git a/roles/hyprland/files/scripts/brightness.sh b/roles/hyprland/files/scripts/brightness.sh
index 851fba77..f25495f6 100755
--- a/roles/hyprland/files/scripts/brightness.sh
+++ b/roles/hyprland/files/scripts/brightness.sh
@@ -17,12 +17,13 @@ set_brightness_ddcutil() {
     local display_serial="$1"
     local step="$2"
     local direction="$3"
-    ddcutil \
-        --noconfig \
-        --skip-ddc-checks \
-        --sleep-multiplier=0.1 \
-        --sn="$display_serial" \
-        setvcp 10 $direction "$step"
+    flock -n /tmp/ddcutil-brightness.lock \
+        ddcutil \
+            --noconfig \
+            --skip-ddc-checks \
+            --sleep-multiplier=0.1 \
+            --sn="$display_serial" \
+            setvcp 10 $direction "$step"
 }
 
 set_brightness_brightnessctl() {
@@ -33,13 +34,14 @@ set_brightness_brightnessctl() {
 
 get_brightness_ddcutil() {
     local display_serial="$1"
-    ddcutil \
-        --noconfig \
-        --skip-ddc-checks \
-        --sleep-multiplier=0.1 \
-        --sn="$display_serial" \
-        --terse getvcp 10 \
-        | awk '{print $4}'
+    flock -n /tmp/ddcutil-brightness.lock \
+        ddcutil \
+            --noconfig \
+            --skip-ddc-checks \
+            --sleep-multiplier=0.1 \
+            --sn="$display_serial" \
+            --terse getvcp 10 \
+            | awk '{print $4}'
 }
 
 get_brightness_brightnessctl() {
-- 
cgit v1.2.3