From 7896d94c76589c83b9474eb1ba343e37c92f5c0d Mon Sep 17 00:00:00 2001 From: Venkateshwar S Date: Tue, 30 Dec 2025 00:36:39 -0800 Subject: [PATCH] soc/qualcomm/x1p42100: Avoid reserving display buffer region The display buffer was previously reserved as unavailable by coreboot, which prevented the kernel from mapping it. When the splash driver released the buffer, the kernel later crashed on access because the region was never mapped. This patch removes the reservation so the kernel can map the display buffer and reuse it safely. TEST=Create an image.serial.bin and ensure it boots on X1P42100. Check that the display memory region is mapped by kernel in UART logs: [ 0.000000][ T0] node 0: [mem 0x00000000e36a0000- 0x00000000f7bfffff] Change-Id: I507d48713690bac3030f81a29c7e123fd3a03b95 Signed-off-by: Venkateshwar S Reviewed-on: https://review.coreboot.org/c/coreboot/+/90648 Tested-by: build bot (Jenkins) Reviewed-by: Kapil Porwal Reviewed-by: Subrata Banik --- src/soc/qualcomm/x1p42100/soc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/soc/qualcomm/x1p42100/soc.c b/src/soc/qualcomm/x1p42100/soc.c index d1762237d5..78fa43a175 100644 --- a/src/soc/qualcomm/x1p42100/soc.c +++ b/src/soc/qualcomm/x1p42100/soc.c @@ -51,7 +51,6 @@ static void soc_read_resources(struct device *dev) reserved_ram_range(dev, index++, (uintptr_t)_dram_wlan, REGION_SIZE(dram_wlan)); reserved_ram_range(dev, index++, (uintptr_t)_dram_pil, REGION_SIZE(dram_pil)); reserved_ram_range(dev, index++, (uintptr_t)_dram_ta, REGION_SIZE(dram_ta)); - reserved_ram_range(dev, index++, (uintptr_t)_dram_display, REGION_SIZE(dram_display)); /* ACDB carveout region located at 0xFF800000 - (n*5.5 +1+32+3) where n is size of DDR */ reserved_ram_range(dev, index++, (uintptr_t)(_dram_llcc_lpi - calc_acdb_carveout_size()),