Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Udev rules are for old performance mode driver #20

Open
rotech opened this issue Jun 12, 2023 · 3 comments
Open

Udev rules are for old performance mode driver #20

rotech opened this issue Jun 12, 2023 · 3 comments

Comments

@rotech
Copy link

rotech commented Jun 12, 2023

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
Copy link
Member

qzed commented Jun 12, 2023

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
Copy link
Member

qzed commented Jun 12, 2023

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

@qzed
Copy link
Member

qzed commented Jun 12, 2023

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants