Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40515 )
Change subject: ec/google/chromeec: Add driver for i2c_tunnel device under Chrome EC
......................................................................
Patch Set 7:
(1 comment)
Patch Set 7:
the build failure is just jenkins, seems like it's been really flaky today...
Yeah. I just noticed one issue with acpi_fill_ssdt() name being used. I will fix that and push a change. Hopefully, jenkins would be happier by then.
https://review.coreboot.org/c/coreboot/+/40515/7/src/ec/google/chromeec/i2c_...
File src/ec/google/chromeec/i2c_tunnel/i2c_tunnel.c:
https://review.coreboot.org/c/coreboot/+/40515/7/src/ec/google/chromeec/i2c_...
PS7, Line 59: acpi_fill_ssdt_generator
This should be acpi_fill_ssdt since it was recently changed.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/40515
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Icfc0ec3725d7f1d20bcb5cb43a0a23aac72bf4eb
Gerrit-Change-Number: 40515
Gerrit-PatchSet: 7
Gerrit-Owner: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Aaron Durbin
adurbin@chromium.org
Gerrit-Reviewer: Karthik Ramasubramanian
kramasub@google.com
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Reviewer: Tim Wawrzynczak
twawrzynczak@chromium.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Comment-Date: Tue, 21 Apr 2020 19:50:50 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment