c467334620
When using microcode header and updating the header (due to a new release of microcode during early development) the build system doesn't detect the header change. This commit fixes this by adding the appropriate dependency. Change-Id: I4211a3e39f67da727ef7cddbbee6d8c4718dee4a Signed-off-by: Julien Viard de Galbert <jviarddegalbert@online.net> Reviewed-on: https://review.coreboot.org/22307 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Patrick Georgi <pgeorgi@google.com>
64 lines
2.6 KiB
Makefile
64 lines
2.6 KiB
Makefile
################################################################################
|
|
## Subdirectories
|
|
################################################################################
|
|
subdirs-y += allwinner
|
|
subdirs-y += amd
|
|
subdirs-y += dmp
|
|
subdirs-y += armltd
|
|
subdirs-y += imgtec
|
|
subdirs-y += intel
|
|
subdirs-y += ti
|
|
subdirs-y += via
|
|
subdirs-$(CONFIG_ARCH_X86) += x86
|
|
subdirs-$(CONFIG_CPU_QEMU_X86) += qemu-x86
|
|
|
|
$(eval $(call create_class_compiler,cpu_microcode,x86_32))
|
|
################################################################################
|
|
## Rules for building the microcode blob in CBFS
|
|
################################################################################
|
|
|
|
ifneq ($(CONFIG_CPU_MICROCODE_MULTIPLE_FILES), y)
|
|
cbfs-files-$(CONFIG_CPU_MICROCODE_CBFS_GENERATE) += cpu_microcode_blob.bin
|
|
endif
|
|
|
|
ifeq ($(CONFIG_CPU_MICROCODE_CBFS_EXTERNAL_HEADER),y)
|
|
cbfs-files-y += cpu_microcode_blob.bin
|
|
cpu_microcode_blob.bin-file = $(objgenerated)/microcode.bin
|
|
|
|
$(objgenerated)/microcode.bin: $(call strip_quotes,$(CONFIG_CPU_MICROCODE_HEADER_FILES))
|
|
echo " util/scripts/ucode_h_to_bin.sh $(objgenerated)/microcode.bin \"$(CONFIG_CPU_MICROCODE_HEADER_FILES)\""
|
|
util/scripts/ucode_h_to_bin.sh $(objgenerated)/microcode.bin $(CONFIG_CPU_MICROCODE_HEADER_FILES)
|
|
endif
|
|
|
|
# We just mash all microcode binaries together into one binary to rule them all.
|
|
# This approach assumes that the microcode binaries are properly padded, and
|
|
# their headers specify the correct size. This works fairly well on isolatied
|
|
# updates, such as Intel and some AMD microcode, but won't work very well if the
|
|
# updates are wrapped in a container, like AMD's microcode update container. If
|
|
# there is only one microcode binary (i.e. one container), then we don't have
|
|
# this issue, and this rule will continue to work.
|
|
$(obj)/cpu_microcode_blob.bin: $$(wildcard $$(cpu_microcode_bins))
|
|
for bin in $(cpu_microcode_bins); do \
|
|
if [ ! -f "$$bin" ]; then \
|
|
echo "Microcode error: $$bin does not exist"; \
|
|
NO_MICROCODE_FILE=1; \
|
|
fi; \
|
|
done; \
|
|
if [ -n "$$NO_MICROCODE_FILE" ]; then \
|
|
if [ -z "$(CONFIG_USE_BLOBS)" ] && [ -n "$(CONFIG_CPU_MICROCODE_CBFS_GENERATE)" ]; then \
|
|
echo "Try enabling binary-only repository in Kconfig 'General setup' menu."; \
|
|
fi; \
|
|
false; \
|
|
fi
|
|
@printf " MICROCODE $(subst $(obj)/,,$(@))\n"
|
|
@echo $(cpu_microcode_bins)
|
|
cat /dev/null $+ > $@
|
|
|
|
cpu_microcode_blob.bin-file ?= $(obj)/cpu_microcode_blob.bin
|
|
cpu_microcode_blob.bin-type := microcode
|
|
|
|
ifneq ($(CONFIG_CPU_MICROCODE_CBFS_LOC),)
|
|
cpu_microcode_blob.bin-COREBOOT-position := $(CONFIG_CPU_MICROCODE_CBFS_LOC)
|
|
else
|
|
cpu_microcode_blob.bin-align := 16
|
|
endif
|