[coreboot-gerrit] Change in coreboot[master]: soc/intel/skylake: Update DTRP ACPI method so both arguments are used
Martin Roth (Code Review)
gerrit at coreboot.org
Tue May 8 04:23:27 CEST 2018
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/26127
to look at the new patch set (#5).
Change subject: soc/intel/skylake: Update DTRP ACPI method so both arguments are used
......................................................................
soc/intel/skylake: Update DTRP ACPI method so both arguments are used
Previously, because the two arguments were wrapped with preprocessor
conditionals, the compiler couldn't see that both were being used. This
uses a local variable for the conditional instead and avoids the
warning.
Change-Id: Id9017894f475250cde6900f46d497cc71ee1bba0
Signed-off-by: Martin Roth <martinroth at google.com>
---
M src/soc/intel/skylake/acpi/dptf/thermal.asl
1 file changed, 8 insertions(+), 6 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/26127/5
--
To view, visit https://review.coreboot.org/26127
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id9017894f475250cde6900f46d497cc71ee1bba0
Gerrit-Change-Number: 26127
Gerrit-PatchSet: 5
Gerrit-Owner: Martin Roth <martinroth at google.com>
Gerrit-Reviewer: Martin Roth <martinroth at google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-CC: Lijian Zhao <lijian.zhao at intel.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180508/922204b2/attachment.html>
More information about the coreboot-gerrit
mailing list