surface-control icon indicating copy to clipboard operation
surface-control copied to clipboard

Udev rules are for old performance mode driver

Open rotech opened this issue 2 years ago • 3 comments

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"

rotech avatar Jun 12 '23 09:06 rotech

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 avatar Jun 12 '23 10:06 qzed

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

qzed avatar Jun 12 '23 11:06 qzed

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

qzed avatar Jun 12 '23 11:06 qzed