Martin L Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/70297?usp=email )
Change subject: include/acpi/acpi_pld.h: Remove comment on PLD ......................................................................
include/acpi/acpi_pld.h: Remove comment on PLD
Remove comment on PLD horizontal position as ACPI spec 6.5 define that field: https://uefi.org/specs/ACPI/6.5/06_Device_Configuration.html?highlight=pld%2...
Change-Id: I228e0780699c223f1e3227fd45ec094e0c46205e Signed-off-by: Elyes Haouas ehaouas@noos.fr Reviewed-on: https://review.coreboot.org/c/coreboot/+/70297 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Martin L Roth gaumless@gmail.com --- M src/include/acpi/acpi_pld.h 1 file changed, 0 insertions(+), 6 deletions(-)
Approvals: build bot (Jenkins): Verified Martin L Roth: Looks good to me, approved
diff --git a/src/include/acpi/acpi_pld.h b/src/include/acpi/acpi_pld.h index f89a680..98d7f9e 100644 --- a/src/include/acpi/acpi_pld.h +++ b/src/include/acpi/acpi_pld.h @@ -22,12 +22,6 @@ PLD_VERTICAL_POSITION_LOWER };
-/* - * The ACPI spec 6.2A does not define the horizontal position field. - * These values are taken from the IASL compiler: - * https://github.com/acpica/acpica/blob/master/source/components/utilities/utg... - */ - enum acpi_pld_horizontal_position { PLD_HORIZONTAL_POSITION_LEFT, PLD_HORIZONTAL_POSITION_CENTER,