search

linux-surface / surface-control

Control various aspects of Microsoft Surface devices on Linux from the Command-Line.
MIT License
65 stars 7 forks source link

Udev rules are for old performance mode driver #20

Open rotech opened 1 year ago

rotech commented 1 year ago

The udev rules that allow normal users to change the mode are still for the old performance driver /sys/bus/surface_aggregator/devices/01:03:01:00:01/perf_mode

I have tried an updated rules set (below), but unfortunately I can't get it to work, I have never had much success getting my head around udev.

ACTION=="add" KERNEL=="acpi" RUN+="/usr/bin/chmod 664 /sys/firware/acpi/platform_profile"
ACTION=="add" KERNEL=="acpi" RUN+="/usr/bin/chown root:surface-control /sys/firware/acpi/platform_profile"
qzed commented 1 year ago

Oh, you're right. We should probably remove those from the package (and maybe also README) since the platform-profile is not exclusive to Surface devices. Also, there's a Gnome daemon / Desktop integration, so these rules are probably not that useful any more (and I'm not sure if they'd conflict with any other rules distros may ship for that).

qzed commented 1 year ago

I've removed them in ad493624b67deb4e47ba7080025615c907368305 / v0.4.5.

qzed commented 1 year ago

And I have no idea why the rules you posted don't work. Maybe `platform_profile is not a "device" in the udev sense?