Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31914 )
Change subject: rockchip/rk3399: Remove obsolete BL31 resource reservation
......................................................................
Patch Set 2:
No self-CR+2
I have done this before on a few patches specific to SoCs or mainboards that I am maintainer for, if none of the other developers familiar with the code have +2 rights and I have a +1 from them. If you object to that I'm happy to send all those kinds of reviews to you in the future. I was just trying to avoid bothering people with trivialities in code they're not interested in.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/31914
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I87458fa09f437b038af10e0fd9d76ef6d9394bc5
Gerrit-Change-Number: 31914
Gerrit-PatchSet: 2
Gerrit-Owner: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: Ting Shen
phoenixshen@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: mturney mturney
mturney@codeaurora.org
Gerrit-Comment-Date: Tue, 19 Mar 2019 22:07:30 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment