Attention is currently required from: Lance Zhao, Bora Guvendik, Hannah Williams, Cliff Huang, Jakub Czapiga, Paul Menzel, Tim Wawrzynczak, Nick Vaccaro.
Hello Bora Guvendik, Lance Zhao, build bot (Jenkins), Hannah Williams, Jérémy Compostella, Tim Wawrzynczak, Nick Vaccaro,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/73113
to look at the new patch set (#9).
Change subject: acpigen: Add print-like function for debug string ......................................................................
acpigen: Add print-like function for debug string
Generate formatted string and generate code for printing this string in ACPI.
ex: pcie_rp = 1 acpigen_write_debug_sprintf("calling _ON for RP: %u", pcie_rp);
will generate ACPI code:
Debug = "calling _ON for RP: 1"
With this new function, the following functions won't be needed and therefore to be removed: acpigen_concatenate_string_string() acpigen_concatenate_string_int() acpigen_write_debug_concatenate_string_string() acpigen_write_debug_concatenate_string_int()
BRANCH=firmware-brya-14505.B TEST=Add above functions in the acpigen code and check the generated SSDT table after OS boot. Check the debug messages is in the kernel log when /sys/modules/acpi/parameters/aml_debug_output is set to '1'.
Signed-off-by: Cliff Huang cliff.huang@intel.com Change-Id: Id4a42e5854516a22b7bc4559c2ed08680722c5ba --- M src/acpi/acpigen.c M src/include/acpi/acpigen.h 2 files changed, 51 insertions(+), 42 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/13/73113/9