07:15 <steev> lumag: did you want that tested too? i only see RFC/RFC in the intro email (but figured i'd test anyway...) i see the PDM messages - http://paste.debian.net/1302181/ - i'm not getting any battery info, but the led does at least blink when plugged in on the thinkpad x13s

09:27 <z3ntu1> lumag: yes, https://raw.githubusercontent.com/FairBlobs/FP5-firmware/main/adsps.jsn

12:20 <lumag> steev, ack, thanks for the test. I see, sc8280xp used the sm8250 adsp resources, so no battery

12:35 <lumag> steev, could you please also add https://pastebin.ubuntu.com/p/VKNsmzMXx8/ ?

12:43 <lumag> z3ntu1, ack, thanks!

12:44 <lumag> generally I have mixed feelings towards tying pd maps to platform compatibles. Another alternative is to describe all domains in some top-level node, but it might cause objections from DT maintainers

17:39 <steev> lumag: even with that change, no battery level

17:47 <lumag> steev, coud you pleas post PDM messages with this change in place ?

17:47 <steev> sure

17:47 <lumag> thanks!

17:47 <steev> https://paste.debian.net/hidden/1ce99a3a/

17:50 <lumag> steev, strange. [ 4.757623] PDM: returning msm/adsp/charger_pd / 74

17:50 <lumag> It works here on sm8350.

17:51 <lumag> I'll run the tests on sc8280xp later in the next few days.

17:51 <lumag> Thank you!

17:52 <steev> one other thing i noticed, not sure if this is anything or just being silly, but when i start pd-mapper service, i get messages in dmesg about usb 4-1

18:02 <lumag> steev, you shouldn't need userspace pd-mapper

18:03 <lumag> I mean, this thing replaces it.

18:04 <steev> right, i only started it because of lack of battery status :)

18:06 <lumag> Ack

18:12 <steev> but also, thank you for this! it will be exciting to not need userspace services once this is all working

18:12 <steev> at least on the thinkpad