Change in coreboot[master]: sc7280: Add Modem region in memlayout to avoid modem cleanup in Secbo... - coreboot-gerrit

26 Nov 2021


      Attention is currently required from: Ravi kumar.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58545 )
Change subject: sc7280: Add Modem region in memlayout to avoid modem cleanup in Secboot reboot.
......................................................................
Patch Set 11: Verified-1
(1 comment)
File src/soc/qualcomm/sc7280/soc.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134454):
https://review.coreboot.org/c/coreboot/+/58545/comment/547bc546_d35fbf98 
PS11, Line 27: 	if (soc_modem_carve_out(&start, &end))
that open brace { should be on the previous line
-- 
To view, visit https://review.coreboot.org/c/coreboot/+/58545
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I56bfb210606b08893ff71dd1b6679f1ec102ec95
Gerrit-Change-Number: 58545
Gerrit-PatchSet: 11
Gerrit-Owner: Ravi kumar rbokka@codeaurora.org
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-CC: Paul Menzel paulepanter@mailbox.org
Gerrit-Attention: Ravi kumar rbokka@codeaurora.org
Gerrit-Comment-Date: Fri, 26 Nov 2021 07:08:38 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment