summaryrefslogtreecommitdiff
path: root/payloads/libpayload
diff options
context:
space:
mode:
authorSubrata Banik <subratabanik@google.com>2023-10-03 14:51:26 +0000
committerSubrata Banik <subratabanik@google.com>2023-10-04 18:50:49 +0000
commit790b5cf442b1d8d9312652f17cf7b16d48e2e6a8 (patch)
tree85ac3fa28467767730b57b0b050e739e59154df7 /payloads/libpayload
parent913ea97fbe0907a86e221a9553b21fcc7aecff57 (diff)
{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 <subratabanik@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/77796 Reviewed-by: Eric Lai <ericllai@google.com> Reviewed-by: Nick Vaccaro <nvaccaro@google.com> Reviewed-by: Julius Werner <jwerner@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'payloads/libpayload')
-rw-r--r--payloads/libpayload/include/coreboot_tables.h7
1 files changed, 7 insertions, 0 deletions
diff --git a/payloads/libpayload/include/coreboot_tables.h b/payloads/libpayload/include/coreboot_tables.h
index 4502e34484..5c3f0c47ec 100644
--- a/payloads/libpayload/include/coreboot_tables.h
+++ b/payloads/libpayload/include/coreboot_tables.h
@@ -220,6 +220,11 @@ enum cb_fb_orientation {
CB_FB_ORIENTATION_RIGHT_UP = 3,
};
+struct cb_framebuffer_flags {
+ u8 has_external_display : 1;
+ u8 reserved : 7;
+};
+
struct cb_framebuffer {
u32 tag;
u32 size;
@@ -238,6 +243,8 @@ struct cb_framebuffer {
u8 reserved_mask_pos;
u8 reserved_mask_size;
u8 orientation;
+ struct cb_framebuffer_flags flags;
+ u8 pad;
};
#define CB_GPIO_ACTIVE_LOW 0