12:57 <digetx> jenneron[m]: applied

13:15 <jenneron[m]> thanks

13:23 <jenneron[m]> i will submit dts soon

13:30 <digetx> so it's copied from arm64, is it working properly?

13:31 <jenneron[m]> it works on surface rt

13:31 <digetx> I'll need to fix the missing acpi check because it breaks non-acpi

13:33 <jenneron[m]> didn't know it breaks something.. thanks

13:48 <digetx> jenneron[m]: I added new CONFIG_ARM_ACPI_PARKING_PROTOCOL

13:48 <digetx> but it needs to be implemented properly

13:56 <digetx> is it possible for DT and ACPI to coexist?

13:57 <digetx> without special hacks

13:58 <digetx> ACPI is a grey area for me

14:48 <jenneron[m]> > ACPI Parking Protocol can coexist with DT.

14:48 <jenneron[m]> > Not sure if ACPI as a whole can coexist with DT.

14:48 <jenneron[m]> Leander

14:53 <jenneron[m]> btw he shared his sign-off

14:53 <jenneron[m]> `Leander Wollersberger <leander.knampf@gmx.at>`

14:53 <digetx> thanks, I'll update commit

14:54 <digetx> perhaps not a big problem if ACPI is needed only SMP; is there anything else that will require to use ACPI? like system suspend-resume for example

14:55 <digetx> is cpuidle working properly?

14:56 <digetx> since ACPI is needed for cpu hotplugging, I assume LP2 cpuidle may not work without ACPI

15:05 <leanderglanda[m]> Jenneron brought me here.

15:05 <leanderglanda[m]> I'm Leander, the guy who did the ACPI Parking Protocol implementation for arm32.

15:05 <leanderglanda[m]> Hello there!

15:06 <digetx> hi

15:09 <digetx> could you please very briefly tell the current status of the SurfaceRT support, what works

15:10 <jenneron[m]> digetx: https://openrt.gitbook.io/open-surfacert/#current-project-status

15:10 <jenneron[m]> i'm currently working on device-tree clean up before submitting

15:12 <digetx> will it be possibly to use DT for a full-featured device support, or a proper ACPI support will become needed at some point?

15:13 <leanderglanda[m]> <digetx "will it be possibly to use DT fo"> A full DT is likelier.

15:14 <leanderglanda[m]> If the Trustzone takeover works as expected, no ACPI/UEFI is needed.

15:15 <leanderglanda[m]> The firmware of the Surface RT is a bit buggy too, so that makes ACPI support a bit worse.

15:15 <leanderglanda[m]> From what I have seen ACPI support requires changing a lot in /arch/arm, but no so much in /driver/acpi. Here and there modifications somewhere else are needed.

15:16 <leanderglanda[m]> * From what I have seen ACPI support requires changing a lot in /arch/arm, but no so much in /driver/acpi. Sometimes modifications somewhere else are needed.

15:16 <digetx> ok

15:20 <digetx> buggy firmware is quite usual, but often it's not a huge problem to add workarounds for it; will be good if DT-only will work, certainly it will simplify everything

15:24 <leanderglanda[m]> In our case patching ACPI tables is the first thing that would need to be done.

15:24 <leanderglanda[m]> Back then Microsoft didn't want to compatible, and liked to break specs.

15:24 <leanderglanda[m]> So they compiled there ACPI tables with a custom Microsoft ASL compiler. It needs to be used to decompile the SSDT001. iASL (that is used *everywhere* else), doesn't work. It ends in a segmentation fault.

15:34 <digetx> thank you for the clarification

15:49 <digetx> jenneron[m]: please open PR once DT will be ready

15:50 <jenneron[m]> ok

18:34 <leanderglanda[m]> <digetx "thank you for the clarification"> You are welcome. If you want to know any more details, just ask 👍️