diff options
| author | Danilo M. <danix@danix.xyz> | 2026-09-20 11:45:08 +0200 |
|---|---|---|
| committer | Danilo M. <danix@danix.xyz> | 2026-09-20 11:45:08 +0200 |
| commit | 59c4a841b7107f6b21bcd1c8f120c8caf1bd2d7c (patch) | |
| tree | 868891e386c2e7597a3364aaed8c752c7a245469 /desktop/modules/bluetooth/BluetoothModule.qml | |
| parent | eba0c13ad2f50964b6f88950366cb659c0d8b055 (diff) | |
| download | quickshell-59c4a841b7107f6b21bcd1c8f120c8caf1bd2d7c.tar.gz quickshell-59c4a841b7107f6b21bcd1c8f120c8caf1bd2d7c.zip | |
feat(desktop): read gamepad battery from power_supply
BluetoothRow only read BluetoothDevice.battery, which is BlueZ's
org.bluez.Battery1. A gamepad never publishes that interface: a Sony pad
exposes its charge as a kernel power_supply instead, so the percentage never
appeared.
Read every power_supply whose name ends in a MAC on a 30s timer while
anything is connected, keyed by address. BlueZ still wins when it has a
value.
Diffstat (limited to 'desktop/modules/bluetooth/BluetoothModule.qml')
| -rw-r--r-- | desktop/modules/bluetooth/BluetoothModule.qml | 55 |
1 files changed, 55 insertions, 0 deletions
diff --git a/desktop/modules/bluetooth/BluetoothModule.qml b/desktop/modules/bluetooth/BluetoothModule.qml index 4fd600f..6c4f587 100644 --- a/desktop/modules/bluetooth/BluetoothModule.qml +++ b/desktop/modules/bluetooth/BluetoothModule.qml @@ -33,6 +33,61 @@ Module { readonly property bool anyConnected: connectedDevices.length > 0 + // Battery for devices BlueZ does not publish through org.bluez.Battery1. + // Gamepads are the case: the Sony pads expose their charge as a kernel + // power_supply rather than a BlueZ interface, so BluetoothDevice.battery + // reads nothing for them. The DualSense driver names the supply + // "ps-controller-battery-<mac>"; the parser takes any supply whose name + // ends in a MAC, so the separator ("-" here, "_" under hid-sony) and the + // driver do not matter. This map is keyed by lowercase MAC and holds 0-100. + property var batteryPct: ({}) + function batteryFor(address) { + if (!address) return null; + const v = mod.batteryPct[address.toLowerCase()]; + return v === undefined ? null : v; + } + + function refreshBatteries() { + batProc.running = false; + batProc.running = true; + } + + // Reads every power_supply once. A supply earns a line only when its name + // ends in "xx:xx:xx:xx:xx:xx", which excludes the system battery (BAT0) + // and AC adapter while catching the controller batteries. The MAC is the + // trailing 17 characters; the separator before it is not assumed. + property Process batProc: Process { + command: ["sh", "-c", + "for d in /sys/class/power_supply/*; do n=${d##*/}; " + + "case $n in *??:??:??:??:??:??) " + + "[ -r \"$d/capacity\" ] && printf '%s %s\\n' \"$n\" \"$(cat \"$d/capacity\")\";; " + + "esac; done"] + stdout: StdioCollector { + onStreamFinished: { + const map = {}; + for (const line of text.trim().split("\n")) { + const i = line.lastIndexOf(" "); + if (i <= 0) continue; + const pct = parseInt(line.slice(i + 1), 10); + if (!isNaN(pct)) map[line.slice(0, i).slice(-17).toLowerCase()] = pct; + } + mod.batteryPct = map; + } + } + } + + // ponytail: 30s poll, matches how slowly a gamepad charge moves. Move to a + // uevent watch only if a lagging percentage is ever noticed. + property Timer batTimer: Timer { + interval: 30000 + repeat: true + running: mod.anyConnected + onTriggered: mod.refreshBatteries() + } + + onAnyConnectedChanged: if (mod.anyConnected) mod.refreshBatteries() + Component.onCompleted: mod.refreshBatteries() + // Null for the first ~2s, then filled. Safe navigation everywhere. icon: mod.adapter && mod.adapter.enabled ? "\uf293" : "\uf05e" |
