/sys/class/dmi/id/board_name allows identification of librebooted devices.

[karstenes]

Is your feature request related to a problem? Please describe.

This will allow proper identification of librebooted thinkpads.

As discussed in this issue, tlp does not identify thinkpads with coreboot due to an improper /sys/class/dmi/id/product_version string. The /sys/class/dmi/id/board_name is properly populated however, which will allow the identification.

Describe the solution you'd like

Utilization of the /sys/class/dmi/id/board_name string for tlp battery driver identification.

Additional context
Here is a gist of all the strings in /sys/class/dmi/id/*

https://gist.github.com/karstenes/c15397843e097bc273bf507a216ef031

[linrunner]

Hi,

I confess that frequent incorporation of workarounds is tiring. In this case I would really prefer Libreboot to fix its stuff. But so be it.

Please test and report.

[karstenes]

Works great. With tp_smapi loaded the thinkpad is detected properly.

[linrunner]

Hold on. Here no one gets away without outputs! Show:

sudo tlp-stat -s -b

[karstenes]

Sorry!

Here's the tlp-stat

https://gist.github.com/karstenes/b0a04a0cd609fb980560bdb044cd6a97

[linrunner]

This output does not fit yet

+++ System Info
System = LENOVO 1.0 ThinkPad T440p

Stay tuned.

[linrunner]

Done. Please test again and show

sudo tlp-stat -s -b

Btw: you don't need external kernel modules with your ThinkPad model

• tp_smapi is for T420 and older only
• acpi_call is obsolete with kernel 5.17 or newer

Does setting the charge thresholds work with Libreboot?

As for force-discharge (recalibration), I expect from your output that with Libreboot it does not work:

/sys/class/power_supply/BAT0/charge_behaviour = [auto]

You don't need to try external kernel modules, the necessary logic is available from kernel 5.17 on. If it is to work, it must look like this:

/sys/class/power_supply/BAT0/charge_behaviour = [auto] inhibit-charge force-discharge

[karstenes]

Here's that gist with that output.
https://gist.github.com/karstenes/e440b94d3d244b954c7c65ce71d7587f

[linrunner]

Output looks fine. Thanks for testing.

[linrunner]

Hi @karstenes : TLP 1.6 Beta 1 is out and contains the change -> #700