erebion/coreboot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

soc/intel/ptl: Remove redundant HAVE_BMP_LOGO_COMPRESS_LZMA Kconfig

Browse source 
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 <subratabanik@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/89876
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Jérémy Compostella <jeremy.compostella@intel.com>
This commit is contained in:
Subrata Banik 2025-11-03 19:15:08 +05:30 • committed by Matt DeVillier
commit 2821e8e2ae
1 changed files with 0 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

3
src/soc/intel/pantherlake/Kconfig
View file

@ -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