Krystian Hebel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31086 )
Change subject: payloads/ipxe: Enable HTTPS support ......................................................................
Patch Set 6:
(2 comments)
https://review.coreboot.org/c/coreboot/+/31086/6/payloads/external/iPXE/Kcon... File payloads/external/iPXE/Kconfig:
https://review.coreboot.org/c/coreboot/+/31086/6/payloads/external/iPXE/Kcon... PS6, Line 118: default n Any reason for 'n' here? Are we worried about size of binary?
https://review.coreboot.org/c/coreboot/+/31086/6/payloads/external/iPXE/Make... File payloads/external/iPXE/Makefile:
https://review.coreboot.org/c/coreboot/+/31086/6/payloads/external/iPXE/Make... PS6, Line 70: ifeq ($(CONFIG_PXE_HAS_HTTPS),y) Inconsistent spacing above this and preceding 'if(n)eq's