Yes,
reverting comit 81d55cf solved the problem with current measurement on BAT state on lenovo T420.
 
Angel, thanks!
Also, thanks to all coreboot developers!
 
06.04.2021, 21:57, "Angel Pons" <th3fanbus@gmail.com>:

Hi Andrew, list

On Tue, Apr 6, 2021 at 6:52 PM Andrew A. I. <aidron@yandex.ru> wrote:


 Hello, Elyes!
 After "Convert to ASL 2.0 syntax" set of commits to current coreboot branch, i lost the data of battery current on BAT state from:
 cat /sys/class/power_supply/BAT0/current_now
 0
 On AC state (charging) data is present:
 cat /sys/class/power_supply/BAT0/current_now
 1346000
 Checkout to coreboot tag/4.13 is fixing the problem.
 But system was very unstable with:

 Apr 4 23:45:06 tp0 kernel: [ 60.868922] BUG: unable to handle page fault for address: 000000ccfffffffc
 Apr 4 23:45:06 tp0 kernel: [ 60.868992] #PF: supervisor read access in kernel mode
 Apr 4 23:45:06 tp0 kernel: [ 60.869033] #PF: error_code(0x0000) - not-present page
 Apr 4 23:45:06 tp0 kernel: [ 60.869074] PGD 0 P4D 0
 Apr 4 23:45:06 tp0 kernel: [ 60.869102] Oops: 0000 [#1] SMP PTI
 Apr 4 23:45:06 tp0 kernel: [ 60.869134] CPU: 0 PID: 1960 Comm: git Tainted: G U 5.10.0-5-amd64 #1 Debian 5.10.24-1
 Apr 4 23:45:06 tp0 kernel: [ 60.869261] RIP: 0010:__d_lookup_rcu+0x82/0x170

 System:
 Lenovo T420
 coreboot-4.13-2951-g0972871a23-T420
 Debian bullseye/sid
 Battery in main slot

 Any ideas?


It's most likely this commit, and I already noticed an issue with it:
https://review.coreboot.org/50317
You can try reverting it to see if the problem gets fixed.
 

 _______________________________________________
 coreboot mailing list -- coreboot@coreboot.org
 To unsubscribe send an email to coreboot-leave@coreboot.org


Best regards,

Angel
_______________________________________________
coreboot mailing list -- coreboot@coreboot.org
To unsubscribe send an email to coreboot-leave@coreboot.org