[coreboot-gerrit] Change in coreboot[master]: acpi/acpigen.h: Add more intuitive AML package closing functions

[Angel Pons (Code Review)]

Attention is currently required from: [e-mail], Jan Dabros, Duncan Laurie.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/50910 )

Change subject: acpi/acpigen.h: Add more intuitive AML package closing functions
......................................................................

Patch Set 1:

(2 comments)

File src/include/acpi/acpigen.h:

https://review.coreboot.org/c/coreboot/+/50910/comment/f873385a_cce18a2d 
PS1, Line 412: void acpigen_write_else(void);
> It would require a lot of changes to be applied to existing code. […]

It would require some changes, and people would also need to get used to it. I'd like to know what others think about it. I don't really write acpigen myself.

https://review.coreboot.org/c/coreboot/+/50910/comment/3d807098_c9e1379c 
PS1, Line 413: inline void acpigen_write_else_end(void)
> For me it looks good. Like #endif in C preprocessor macros.  […]

IMHO, yes