Looks like you've taking my suggestion in some parts of the patch here but not yet in other parts... please address the remaining open comments and indicate if you consider this ready for review again.

View Change

1 comment:

To view, visit change 35506. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I020d1e4d4f5c948948e1b39dd18af1d0e860c279
Gerrit-Change-Number: 35506
Gerrit-PatchSet: 36
Gerrit-Owner: mturney mturney <mturney@codeaurora.org>
Gerrit-Reviewer: Doug Anderson <dianders@chromium.org>
Gerrit-Reviewer: Ravi kumar <rbokka@codeaurora.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Reviewer: mturney mturney <mturney@codeaurora.org>
Gerrit-CC: Julius Werner <jwerner@chromium.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-CC: Rishabh Sharma <rishabh435@gmail.com>
Gerrit-CC: ashk@codeaurora.org
Gerrit-Comment-Date: Thu, 06 Feb 2020 01:51:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment