On Thu, Jun 10, 2021 at 6:16 AM Erik Kaneda <erik.kaneda@xxxxxxxxx> wrote: > > Hi Rafael, I found out that there were several misspelled struct fields > that resulted from merging my PRM code with ACPICA material that caused > build errors. This patch set resolves build issues by > > 1) fixing the spelling errors > 2) defining the struct acpi_prmt_module_header that got left off. > This struct is used by acpi_parse_table_entries and is not used by > ACPICA so it was left out accidentally. > > This patch set also fixes a crash that was caused when I split up > acpi_table_prmt and acpi_table_prmt_header. These structs used to be > defined in a single struct acpi_table_prmt but it was split up in > order to facilitate iASL compiler implementation. The size of the > 36-byte ACPI table header + the size of any additional fields are > needed as parameter to the acpi_parse_table_entries function. When I > split up acpi_table_prmt into two different structs, I did not adjust > the parameter to acpi_parse_table_entries and this ended up breaking > the PRMT parsing. I've fixed this by changing the parameter to > acpi_parse_table_entries to account for this additional struct. > > Sorry for the confusion! This patchset should clear things up and > runs the PRM handler in the edk2 PRM samples without any issues. > > I've based this on rc-5. Applied (instead of the previous version), thanks!
