Attention is currently required from: Lance Zhao, jacz@semihalf.com, Paul Menzel, Duncan Laurie.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49834 )
Change subject: acpi/acpigen.c: Remove unused functions
......................................................................
Patch Set 4: Code-Review+2
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/49834/comment/da6e13e6_acfa53a6
PS4, Line 7: acpi/acpigen.c: Remove unused functions
:
: acpigen_write_name_zero() and acpigen_write_name_one() are not used
: anywhere. These functions are a subset of the more flexible function
: acpigen_write_name_integer().
Please also mention that they are implemented wrongly:
acpi/acpigen.c: Remove unused and incorrect functions
acpigen_write_name_zero() and acpigen_write_name_one() are not
implemented correctly, and are not used anywhere. Drop them in
favor of the more flexible acpigen_write_name_integer(). function.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/49834
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I116fd41624a8e8b536d18d747f21d3131b734dfc
Gerrit-Change-Number: 49834
Gerrit-PatchSet: 4
Gerrit-Owner: jacz@semihalf.com
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Duncan Laurie
dlaurie@chromium.org
Gerrit-Reviewer: Frans Hendriks
fhendriks@eltan.com
Gerrit-Reviewer: HAOUAS Elyes
ehaouas@noos.fr
Gerrit-Reviewer: Lance Zhao
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Attention: Lance Zhao
Gerrit-Attention: jacz@semihalf.com
Gerrit-Attention: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Attention: Duncan Laurie
dlaurie@chromium.org
Gerrit-Comment-Date: Tue, 26 Jan 2021 11:50:28 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment