build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39925 ) Change subject: boot_device: Add boot_device_lockdown ...................................................................... Patch Set 1: (2 comments) https://review.coreboot.org/c/coreboot/+/39925/1/src/include/spi-generic.h File src/include/spi-generic.h: https://review.coreboot.org/c/coreboot/+/39925/1/src/include/spi-generic.h@1... PS1, Line 160: * calling this function. please, no space before tabs https://review.coreboot.org/c/coreboot/+/39925/1/src/southbridge/intel/commo... File src/southbridge/intel/common/spi.c: https://review.coreboot.org/c/coreboot/+/39925/1/src/southbridge/intel/commo... PS1, Line 1115: if (CONFIG(SOUTHBRIDGE_INTEL_I82801GX)) { braces {} are not necessary for any arm of this statement -- To view, visit https://review.coreboot.org/c/coreboot/+/39925 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I9d3a80a2e278c77212e1fba5236ea639ea018837 Gerrit-Change-Number: 39925 Gerrit-PatchSet: 1 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> Gerrit-CC: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-Comment-Date: Mon, 30 Mar 2020 12:40:06 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment