[coreboot-gerrit] Change in coreboot[master]: util/autoport: Adapt logmaker for newer ACPI versions

Arthur Heymans (Code Review) gerrit at coreboot.org
Sun Aug 12 00:22:19 CEST 2018


Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/28054


Change subject: util/autoport: Adapt logmaker for newer ACPI versions
......................................................................

util/autoport: Adapt logmaker for newer ACPI versions

acpidump now creates dumps with 4 spaces instead of 2 in front of the hex dump.

Change-Id: I8d48c09cdff9432f394b350540ea9765fc942781
Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
---
M util/autoport/log_reader.go
1 file changed, 3 insertions(+), 0 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/28054/1

diff --git a/util/autoport/log_reader.go b/util/autoport/log_reader.go
index c94d182..a356f97 100644
--- a/util/autoport/log_reader.go
+++ b/util/autoport/log_reader.go
@@ -112,6 +112,9 @@
 			Tables[curTable] = make([]byte, 0, 100000)
 		case len(line) > 7 && line[0:2] == "  " && isXDigit(line[2]) && isXDigit(line[3]) && isXDigit(line[4]) && isXDigit(line[5]) && line[6] == ':':
 			Tables[curTable] = l.AssignHexLine(line, Tables[curTable])
+		/* acpidump has 4 spaces on newer versions */
+		case len(line) > 7 && line[0:4] == "    " && isXDigit(line[4]) && isXDigit(line[5]) && isXDigit(line[6]) && isXDigit(line[7]) && line[8] == ':':
+			Tables[curTable] = l.AssignHexLine(line, Tables[curTable])
 		}
 	}
 

-- 
To view, visit https://review.coreboot.org/28054
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8d48c09cdff9432f394b350540ea9765fc942781
Gerrit-Change-Number: 28054
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180811/2c97bc1c/attachment.html>


More information about the coreboot-gerrit mailing list