Attention is currently required from: Ravi kumar, Shelley Chen, Paul Menzel, mturney mturney, Julius Werner. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/50587 )
Change subject: sc7280: Reserve wlan & wpss dram memory regions ......................................................................
Patch Set 33:
(4 comments)
File src/soc/qualcomm/sc7280/soc.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-119108): https://review.coreboot.org/c/coreboot/+/50587/comment/f94c7e01_fa934ad1 PS33, Line 16: REGION_SIZE(dram_wlan) / KiB); code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-119108): https://review.coreboot.org/c/coreboot/+/50587/comment/ee3a697b_4dc10f62 PS33, Line 16: REGION_SIZE(dram_wlan) / KiB); please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-119108): https://review.coreboot.org/c/coreboot/+/50587/comment/ca12de4a_84b4fe85 PS33, Line 18: REGION_SIZE(dram_wpss) / KiB); code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-119108): https://review.coreboot.org/c/coreboot/+/50587/comment/afe4ca1f_4bf86310 PS33, Line 18: REGION_SIZE(dram_wpss) / KiB); please, no spaces at the start of a line