From 790b5cf442b1d8d9312652f17cf7b16d48e2e6a8 Mon Sep 17 00:00:00 2001 From: Subrata Banik Date: Tue, 3 Oct 2023 14:51:26 +0000 Subject: {commonlib, libpayload}: Add "has_external_display" in coreboot table This patch introduces a new coreboot table entry named "has_external_display" to understand if external display is attached. This information is useful to prevent graceful shutdown by payload when the LID is closed but an external display is present. This piece of the information will be gathered by coreboot and passed into the payload using this new entry aka external_display because payload (i.e., deptcharge) doesn't have any other way to determine if external display is available. BUG=b:299137940 TEST=Able to build and boot google/rex. w/o this patch: LID closed and external display attached (HDMI) in developer mode (GBB 0x39): > System is powered off by depthcharge w/ this patch: LID closed and external display attached (HDMI) in developer mode (GBB 0x39): > Booted to OS and device is alive/usable Change-Id: I0fa7eee4c5a50371a7a66c6ca1ac2c7d046d010b Signed-off-by: Subrata Banik Reviewed-on: https://review.coreboot.org/c/coreboot/+/77796 Reviewed-by: Eric Lai Reviewed-by: Nick Vaccaro Reviewed-by: Julius Werner Tested-by: build bot (Jenkins) --- src/drivers/intel/fsp2_0/graphics.c | 10 ++++++++++ src/drivers/intel/fsp2_0/include/fsp/graphics.h | 10 ++++++++++ 2 files changed, 20 insertions(+) (limited to 'src/drivers/intel/fsp2_0') diff --git a/src/drivers/intel/fsp2_0/graphics.c b/src/drivers/intel/fsp2_0/graphics.c index 6514209d04..a98f3bbba2 100644 --- a/src/drivers/intel/fsp2_0/graphics.c +++ b/src/drivers/intel/fsp2_0/graphics.c @@ -54,6 +54,13 @@ enum fw_splash_screen_status { FW_SPLASH_SCREEN_ENABLED, }; +/* Check and report if an external display is attached */ +__weak int fsp_soc_report_external_display(void) +{ + /* Default implementation, on-board display enabled */ + return 0; +} + /* * Update elog with Firmware Splash Screen related information * based on enum fw_splash_screen_status. @@ -123,6 +130,9 @@ void fsp_report_framebuffer_info(const uintptr_t framebuffer_bar, .reserved_mask_pos = fbinfo->rsvd.pos, .reserved_mask_size = fbinfo->rsvd.size, .orientation = orientation, + .flags = { + .has_external_display = fsp_soc_report_external_display(), + }, }; fb_add_framebuffer_info_ex(&fb); diff --git a/src/drivers/intel/fsp2_0/include/fsp/graphics.h b/src/drivers/intel/fsp2_0/include/fsp/graphics.h index dfd7b4e65b..a5f781f87f 100644 --- a/src/drivers/intel/fsp2_0/include/fsp/graphics.h +++ b/src/drivers/intel/fsp2_0/include/fsp/graphics.h @@ -14,4 +14,14 @@ void fsp_report_framebuffer_info(const uintptr_t framebuffer_bar, enum lb_fb_orientation orientation); +/* SoC Overrides */ +/* + * Check and report if an external display is attached + * + * Possible return values: + * 1 - An external device is attached. + * 0 - On-board display alone. + */ +int fsp_soc_report_external_display(void); + #endif /* _FSP2_0_GRAPHICS_H_ */ -- cgit v1.2.3