From 2821e8e2aee534d8e42fc133d910e4efc625264d Mon Sep 17 00:00:00 2001 From: Subrata Banik Date: Mon, 3 Nov 2025 19:15:08 +0530 Subject: [PATCH] soc/intel/ptl: Remove redundant HAVE_BMP_LOGO_COMPRESS_LZMA Kconfig MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This commit removes the explicit CONFIG_HAVE_BMP_LOGO_COMPRESS_LZMA definition from the Panther Lake Kconfig. This local Kconfig setting, previously defaulted to 'n', is redundant because the Panther Lake build now correctly inherits the intended system-wide default, which is to use LZMA compression for the BMP splash screen. Removing this unnecessary Kconfig option simplifies the configuration and results in a measured ~3ms reduction in boot time during the firmware splash screen rendering phase on Panther Lake platforms. w/o this patch: ``` 963:returning from FspMultiPhaseSiInit 1,096,797 (102,937) 17:starting LZ4 decompress (ignore for x86) 1,111,606 (14,808) 18:finished LZ4 decompress (ignore for x86) 1,111,641 (34) 17:starting LZ4 decompress (ignore for x86) 1,119,857 (8,216) 18:finished LZ4 decompress (ignore for x86) 1,119,879 (21) ``` w/ this patch ``` 963:returning from FspMultiPhaseSiInit 1,097,817 (103,211) 15:starting LZMA decompress (ignore for x86) 1,110,058 (12,241) 16:finished LZMA decompress (ignore for x86) 1,111,096 (1,037) 15:starting LZMA decompress (ignore for x86) 1,117,554 (6,458) 16:finished LZMA decompress (ignore for x86) 1,117,906 (352) ``` Change-Id: I64579e53c7f307d1430767da04a413f80016487f Signed-off-by: Subrata Banik Reviewed-on: https://review.coreboot.org/c/coreboot/+/89876 Tested-by: build bot (Jenkins) Reviewed-by: Jérémy Compostella --- src/soc/intel/pantherlake/Kconfig | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/soc/intel/pantherlake/Kconfig b/src/soc/intel/pantherlake/Kconfig index 7d6e9f7248..a0a39bc765 100644 --- a/src/soc/intel/pantherlake/Kconfig +++ b/src/soc/intel/pantherlake/Kconfig @@ -430,9 +430,6 @@ config PCIE_LTR_MAX_NO_SNOOP_LATENCY help Latency tolerance reporting, max non-snoop latency value defaults to 15.73 ms. -config HAVE_BMP_LOGO_COMPRESS_LZMA - default n - # The default offset to store CSE RW FW version information is at 68. # However, in Intel Panther Lake based systems that use PSR, the additional # size required to keep CSE RW FW version information and PSR back-up status