0fb2e664ce
This patch drops the unnecessary alignment of 64 bytes that was introduced when implementing the split Intel microcode packing logic into CBFS. - The 16-byte alignment that is already used for Intel microcode is sufficient. - Removes unnecessary alignment check of 64 bytes against an AMD platform specific config. TEST=Able to build and boot google/rex without any functional impact. Change-Id: Icc44e9511e321592de7ab8d1346103d0a9951c9b Signed-off-by: Subrata Banik <subratabanik@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/76397 Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
26 lines
1.1 KiB
Makefile
26 lines
1.1 KiB
Makefile
bootblock-$(CONFIG_MICROCODE_UPDATE_PRE_RAM) += microcode_asm.S
|
|
|
|
bootblock-$(CONFIG_SUPPORT_CPU_UCODE_IN_CBFS) += microcode.c
|
|
ramstage-$(CONFIG_SUPPORT_CPU_UCODE_IN_CBFS) += microcode.c
|
|
romstage-$(CONFIG_SUPPORT_CPU_UCODE_IN_CBFS) += microcode.c
|
|
|
|
# Pack individual microcodes per CPUID from CONFIG_CPU_INTEL_UCODE_SPLIT_BINARIES directory into the CBFS.
|
|
ifeq ($(CONFIG_CPU_INTEL_MICROCODE_CBFS_SPLIT_BINS),y)
|
|
microcode-params-dir := $(call strip_quotes,$(CONFIG_CPU_INTEL_UCODE_SPLIT_BINARIES))/
|
|
microcode-params := $(shell find "$(microcode-params-dir)" -type f -exec basename {} \;)
|
|
|
|
# Make "cpu_microcode_$(CPUID).bin" file entry into the FIT table
|
|
$(call add_intermediate, add_mcu_fit, set_fit_ptr $(IFITTOOL))
|
|
$(foreach params, $(microcode-params), $(shell $(IFITTOOL) -f $< -a -n $(params) -t 1 \
|
|
-s $(CONFIG_CPU_INTEL_NUM_FIT_ENTRIES) -r COREBOOT)) true
|
|
|
|
# Add "cpu_microcode_$(CPUID).bin" file into the CBFS
|
|
$(foreach params,$(microcode-params), \
|
|
$(eval cbfs-files-y += $(params)) \
|
|
$(eval $(params)-file := $(microcode-params-dir)/$(params)) \
|
|
$(eval $(params)-type := microcode) \
|
|
$(eval $(params)-align := 16) \
|
|
)
|
|
|
|
endif
|