build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46661 )
Change subject: sc7180: Add Modem region in memlayout to avoid modem cleanup in Secboot reboot. ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/46661/3/src/soc/qualcomm/sc7180/car... File src/soc/qualcomm/sc7180/carve_out.c:
https://review.coreboot.org/c/coreboot/+/46661/3/src/soc/qualcomm/sc7180/car... PS3, Line 18: switch (modem_id) { switch and case should be at the same indent
https://review.coreboot.org/c/coreboot/+/46661/3/src/soc/qualcomm/sc7180/soc... File src/soc/qualcomm/sc7180/soc.c:
https://review.coreboot.org/c/coreboot/+/46661/3/src/soc/qualcomm/sc7180/soc... PS3, Line 21: if (soc_modem_carve_out(&start, &end)) { braces {} are not necessary for single statement blocks