Skip to content

Conversation

@edenir-anschau
Copy link

While installing TLP on my laptop running Void Linux,I realized this was a good opportunity to contribute to the documentation.

Copy link
Owner

@linrunner linrunner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution. See my comments.


Install them with the command:

*Version 1.9 and newer* ::
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I understand the official documentation correctly, Void has a rolling release scheme. This means that there are no longer any TLP packages < 1.9, and the whole case distinction for 1.9 can be dropped, which would make the instructions more straightforward.


sudo xbps-install tlp tlp-pd tlp-rdw

If you have installed tpl-rwd, you also need to install NetWorkManager ::
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd prefer to phrase the deduction differently: “If you want to use the functionality of tlp-rdw, you need NetworkManager. You can install it with ...”. "Please read the Void instructions (link below) before installing"_


sudo ln -s /etc/sv/tlp /var/service

*For version 1.9 and newer* with tlp-pd, additionally: ::
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Case distinction unnecessary, see my comment above.

sudo xbps-install NetworkManager

Enabling the Services
-------------
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

void.rst:24: WARNING: Title underline too short.


then install the approriate package with the command ::

sudo xbps-install tp_smapi-dkmsk
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the package name "tp_smapi-dkmsk" really correct or a typo?


.. note::

* You must disable Secure Boot to use the ThinkPad specific packages
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is about tp_smapi-dkms, right?

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

Successfully merging this pull request may close these issues.

2 participants