From 0fb2e664ce4c49193f08bc0700c859ff78fdc744 Mon Sep 17 00:00:00 2001 From: Subrata Banik Date: Mon, 10 Jul 2023 22:14:32 +0530 Subject: [PATCH] cpu/intel/microcode: Drop unnecessary alignment for split microcode 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 Reviewed-on: https://review.coreboot.org/c/coreboot/+/76397 Reviewed-by: Eric Lai Tested-by: build bot (Jenkins) Reviewed-by: Felix Held --- src/cpu/intel/microcode/Makefile.inc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/cpu/intel/microcode/Makefile.inc b/src/cpu/intel/microcode/Makefile.inc index 4ea8227e13..a2b0a7b190 100644 --- a/src/cpu/intel/microcode/Makefile.inc +++ b/src/cpu/intel/microcode/Makefile.inc @@ -19,7 +19,7 @@ $(foreach params,$(microcode-params), \ $(eval cbfs-files-y += $(params)) \ $(eval $(params)-file := $(microcode-params-dir)/$(params)) \ $(eval $(params)-type := microcode) \ - $(eval $(params)-align := $(if $(filter y,$(CONFIG_SOC_AMD_COMMON_BLOCK_LPC_SPI_DMA)),64,16)) \ + $(eval $(params)-align := 16) \ ) endif