Skip to content

feat: Add tp_smapi from tlp#274

Open
p4p4j0hn wants to merge 4 commits intoublue-os:mainfrom
p4p4j0hn:add-tp_smapi
Open

feat: Add tp_smapi from tlp#274
p4p4j0hn wants to merge 4 commits intoublue-os:mainfrom
p4p4j0hn:add-tp_smapi

Conversation

@p4p4j0hn
Copy link

Older Thinkpads (2011 and older) require this kernel module for battery charge thresholds and recalibration with tlp. You only need this if the Recommendations section from running tlp-stat -b includes Install tp-smapi kernel modules for ThinkPad battery thresholds and recalibration.

@p4p4j0hn p4p4j0hn requested a review from castrojo as a code owner November 19, 2024 19:23
@p4p4j0hn
Copy link
Author

I just checked out the linrunner repo and it doesn't look like this is available for F41 yet.

@p4p4j0hn p4p4j0hn marked this pull request as draft November 19, 2024 20:05
@p4p4j0hn p4p4j0hn changed the title feat: Add tp_smapi from tlp [WIP] feat: Add tp_smapi from tlp Nov 19, 2024
@p4p4j0hn p4p4j0hn changed the title [WIP] feat: Add tp_smapi from tlp feat: Add tp_smapi from tlp Dec 17, 2024
@p4p4j0hn p4p4j0hn marked this pull request as ready for review December 17, 2024 21:05
@p4p4j0hn
Copy link
Author

I just checked out the linrunner repo and it doesn't look like this is available for F41 yet.

It is available in the updates-testing repo.

Older Thinkpads (2011 and older) require this kernel module for battery
charge thresholds and recalibration with tlp. You only need this if the
Recommendations section from running `tlp-stat -b` includes `Install
tp-smapi kernel modules for ThinkPad battery thresholds and
recalibration`.

fix: Add double quotes to prevent globbing and word-splitting.

fix: move tp_smapi to extra image.

tp_smapi kmod is for niche users with old thinkpads. It should be part
of extras, not common.

Signed-off-by: John Schmidt <john@schmidthaus.rocks>

fix: use testing repo for F41

Signed-off-by: John Schmidt <john@schmidthaus.rocks>
@p4p4j0hn
Copy link
Author

@castrojo is this something that could go in here? Or is it too niche?

@castrojo
Copy link
Contributor

I think having a diverse set of maintained ones will always be useful for custom images, sorry the review is taking so long! (Outside my skillset)

KyleGospo
KyleGospo previously approved these changes Dec 27, 2024
@KyleGospo KyleGospo enabled auto-merge December 27, 2024 21:18
@p4p4j0hn
Copy link
Author

p4p4j0hn commented Dec 28, 2024

Closer! It is building the image just fine, however, it looks like in order to install the akmod during the test, the linrunner repo needs to be added in order to pull in tp_smapi-kmod-common.

auto-merge was automatically disabled December 29, 2024 20:59

Head branch was pushed to by a user without write access

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.

3 participants