Attention is currently required from: Angel Pons, Arthur Heymans, Christian Walter, Felix Held, Johnny Lin, Jonathan Zhang, Lean Sheng Tan, Nico Huber, Patrick Rudolph, Tim Chu.
Shuo Liu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/82034?usp=email )
Change subject: soc/intel/xeon_sp: Add _OSC ASL generation utils for IIO stacks ......................................................................
Patch Set 3:
(1 comment)
Patchset:
PS2:
Thanks, Nico. […]
There are indeed some valuable learnings obtained from this long journey (thanks for all of you being with me on tackling many many issues).
For complex ASL objects, there are many corner cases cannot be covered by one or two system tests (TEST=...), we have to rely on code review to make sure no defects. Compared to the AML generating codes, ASL is more readable to human beings and hence should be considered first.
P.S. the reason for the many many issues captured in the previous round of review is that, I directly used the iasl decoded ASL from the previous generated AML, and it surprised myself as well that so many details are not handled correctly.
Have parameterized ASL handlers is a good solution to cover this case and future usages as well (Thanks for Nico to raise this as a suggestion in the mail discussion). I think we should move ahead to make this even more general,
e.g. Can we pass a buffer list instead of being limited to use 8 parameters of an ACPI method? e.g. Can we improve the name space isolation so that we might have a concept of library of many handlers in future?
This will definitely make coreboot as a stronger solution to cover multi-segment usage in the long run, which is the common target of all. Look forward to more good ideas from you.