Change in coreboot[master]: payloads/Makefile.inc: Add warning for image built with no payload

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/60025 )

Change subject: payloads/Makefile.inc: Add warning for image built with no payload
......................................................................

payloads/Makefile.inc: Add warning for image built with no payload

Writing a coreboot image without a payload to a board's flash chip will
result in a non-bootable system, so warn the user if this is the case.

Signed-off-by: Nicholas Chin 
Change-Id: I15ae9548a45e9f566c84db41e8e171c6bc179057
Reviewed-on: https://review.coreboot.org/c/coreboot/+/60025
Tested-by: build bot (Jenkins) 
Reviewed-by: Nico Huber 
Reviewed-by: Tim Wawrzynczak 
Reviewed-by: Angel Pons 
---
M payloads/Makefile.inc
1 file changed, 12 insertions(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Nico Huber: Looks good to me, approved
  Angel Pons: Looks good to me, approved
  Tim Wawrzynczak: Looks good to me, but someone else must approve

diff --git a/payloads/Makefile.inc b/payloads/Makefile.inc
index d89b5ba..83194c2 100644
--- a/payloads/Makefile.inc
+++ b/payloads/Makefile.inc
@@ -48,5 +48,16 @@
 print-repo-info-payloads:
    -$(foreach payload, $(PAYLOADS_LIST), $(MAKE) -C $(payload) print-repo-info 2>/dev/null; )

+ifeq ($(CONFIG_PAYLOAD_NONE),y)
+files_added:: warn_no_payload
+endif
+
+warn_no_payload:
+	printf "\n\t** WARNING **\n"
+	printf "coreboot has been built without a payload. Writing\n"
+	printf "a coreboot image without a payload to your board's\n"
+	printf "flash chip will result in a non-booting system. You\n"
+	printf "can use cbfstool to add a payload to the image.\n\n"
+
 .PHONY: force-payload coreinfo nvramcui
-.PHONY: clean-payloads distclean-payloads print-repo-info-payloads
+.PHONY: clean-payloads distclean-payloads print-repo-info-payloads warn_no_payload