Martin Roth has uploaded this change for review. ( https://review.coreboot.org/24914
Change subject: ec/google/chromeec: Add note before error message
......................................................................
ec/google/chromeec: Add note before error message
When clearing events from the EC, an error is returned when we try
to clear an event that doesn't exist. This is normal, but can
be distracting when trying to track down an error, so add a message
saying that the error is expected.
BUG=None
Test=Build Grunt with SMM debug enabled. See message before
"EC returned error result code 1".
Change-Id: Ib2e684e357e821c795de4b59658432c91a8d63fc
Signed-off-by: Martin Roth <martinroth(a)google.com>
---
M src/ec/google/chromeec/smihandler.c
1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/14/24914/1
diff --git a/src/ec/google/chromeec/smihandler.c b/src/ec/google/chromeec/smihandler.c
index 54aa4bf..cab7192 100644
--- a/src/ec/google/chromeec/smihandler.c
+++ b/src/ec/google/chromeec/smihandler.c
@@ -53,6 +53,7 @@
while (google_chromeec_get_event() != 0)
;
+ printk(BIOS_DEBUG,"Clearing pending EC events. Error code 1 is expected.\n");
while (google_chromeec_get_mkbp_event(&mkbp_event) == 0)
;
}
--
To view, visit https://review.coreboot.org/24914
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: Ib2e684e357e821c795de4b59658432c91a8d63fc
Gerrit-Change-Number: 24914
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth <martinroth(a)google.com>
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/23869 )
Change subject: util/cbfstool: Add ARM64 Image support
......................................................................
Patch Set 6: Verified+1
Build Successful
https://qa.coreboot.org/job/coreboot-gerrit/68064/ : SUCCESS
https://qa.coreboot.org/job/coreboot-checkpatch/22532/ : SUCCESS
--
To view, visit https://review.coreboot.org/23869
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a8183a87de4dc3a4c4f038fe5b4912ada5d75f9
Gerrit-Change-Number: 23869
Gerrit-PatchSet: 6
Gerrit-Owner: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Wed, 28 Feb 2018 16:53:34 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/23873 )
Change subject: security/vboot: Make vboot first class citizen
......................................................................
Patch Set 5:
Build Successful
https://qa.coreboot.org/job/coreboot-gerrit/68063/ : SUCCESS
https://qa.coreboot.org/job/coreboot-checkpatch/22531/ : SUCCESS
--
To view, visit https://review.coreboot.org/23873
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If1ac6db16647eba4ddb28b74d1e0501224c841df
Gerrit-Change-Number: 23873
Gerrit-PatchSet: 5
Gerrit-Owner: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Wed, 28 Feb 2018 16:26:30 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/23869 )
Change subject: util/cbfstool: Add ARM64 Image support
......................................................................
Patch Set 5: Verified-1
Build Failed
https://qa.coreboot.org/job/coreboot-gerrit/68054/ : FAILURE
https://qa.coreboot.org/job/coreboot-checkpatch/22523/ : SUCCESS
--
To view, visit https://review.coreboot.org/23869
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a8183a87de4dc3a4c4f038fe5b4912ada5d75f9
Gerrit-Change-Number: 23869
Gerrit-PatchSet: 5
Gerrit-Owner: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Wed, 28 Feb 2018 13:23:16 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes