[coreboot-gerrit] Change in coreboot[master]: arch/x86/acpi: Add os.asl

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33652 )

Change subject: arch/x86/acpi: Add os.asl
......................................................................

Patch Set 1:
> Only useful if supporting Windows. I doubt an open source project would use $$$ Microsoft.

Many do, but I assume mostly only support for a single version
is necessary. Linux has its bugs too, so one cannot say that
this is only about Windows. But generally, I prefer to add work-
arounds for specific OS's only case-by-case when one really has
the means to test it. And then the workaround should be commen-
ted as what it is. That's much better than a choice of per OS
paths by default where it's not recognizable which path has to
stay as is (workaround) and which can be adapted/unified (not
saying that this change would have led to the latter, it's just
what happened in some firmware in the past).