build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56442 )
Change subject: mb/siemens/mc_apl{1,2,3,5,6}: Set PCI bus master bit only if allowed ......................................................................
Patch Set 1:
(2 comments)
File src/mainboard/siemens/mc_apl1/mainboard.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-124608): https://review.coreboot.org/c/coreboot/+/56442/comment/0f99b56b_5a28c626 PS1, Line 221: if (dev) { braces {} are not necessary for single statement blocks
File src/mainboard/siemens/mc_apl1/variants/mc_apl2/mainboard.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-124608): https://review.coreboot.org/c/coreboot/+/56442/comment/0d4317c0_53b00b01 PS1, Line 16: if (dev) { braces {} are not necessary for single statement blocks