summarylogtreecommitdiffstats
path: root/12-fix-intel-audio-powersave-hook.patch
blob: 2d884da1e7b6942e20283d53b6192f55dda74303 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
From: Florian Kriener <florian@kriener.org>
To: submit@bugs.debian.org
Subject: [pm-utils] wrong path in intel-audio-powersave (and a small bug)
Date: Sat, 25 Sep 2010 11:27:30 +0200

In the script intel-audio-powersave is this loop

for dev in /sys/module/snd_*/parameters/power_save; do
	[ -w "$dev/parameters/power_save" ] || continue
     printf "Setting power savings for $s to %d..." "$dev##*/" "$1"
	echo $1 > "$dev/parameters/power_save" && echo Done. || echo Failed.
done

I think it should be

for dev in /sys/module/snd_*; do
	[ -w "$dev/parameters/power_save" ] || continue
     printf "Setting power savings for %s to %d..." "${dev##*/}" "$1"
	echo $1 > "$dev/parameters/power_save" && echo Done. || echo Failed.
done


This fixes the two bugs.

diff --git a/pm/power.d/intel-audio-powersave b/pm/power.d/intel-audio-powersave
index 36675a8..da63e40 100644
--- a/pm/power.d/intel-audio-powersave
+++ b/pm/power.d/intel-audio-powersave
@@ -20,9 +20,9 @@ EOF
 
 audio_powersave() {
     [ "$INTEL_AUDIO_POWERSAVE" = "true" ] || exit $NA
-    for dev in /sys/module/snd_*/parameters/power_save; do
+    for dev in /sys/module/snd_*; do
 	[ -w "$dev/parameters/power_save" ] || continue
-	printf "Setting power savings for $s to %d..." "$dev##*/" "$1"
+	printf "Setting power savings for %s to %d..." "${dev##*/}" "$1"
 	echo $1 > "$dev/parameters/power_save" && echo Done. || echo Failed.
     done
 }