From 0361e1a865bc5029562f1344dfc7b2d6e0879aa8 Mon Sep 17 00:00:00 2001 From: Matt DeVillier Date: Sun, 9 Nov 2025 18:20:44 -0600 Subject: [PATCH] drivers/intel/mipi_camera: Verify SSDB struct size at build time Add a static assert ensuring `struct ssdb` stays 0x6C bytes, matching the sensor descriptor in ACPI. This guards future edits from drifting away from the documented layout without changing runtime behavior. Change-Id: I2b4dfb86494d13525cbc6e6de4573ceb36f0b482 Signed-off-by: Matt DeVillier Reviewed-on: https://review.coreboot.org/c/coreboot/+/90183 Reviewed-by: Maximilian Brune Tested-by: build bot (Jenkins) --- src/drivers/intel/mipi_camera/ssdb.h | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/drivers/intel/mipi_camera/ssdb.h b/src/drivers/intel/mipi_camera/ssdb.h index 68f234058c..318fb2e8c5 100644 --- a/src/drivers/intel/mipi_camera/ssdb.h +++ b/src/drivers/intel/mipi_camera/ssdb.h @@ -6,6 +6,8 @@ #include #include +#define SSDB_SIZE 108 + enum camera_device_type { DEV_TYPE_SENSOR = 0, DEV_TYPE_VCM, @@ -83,5 +85,7 @@ struct intel_ssdb { uint8_t reserved[13]; /* Pads SSDB out so the binary blob in ACPI is the same size as seen on other firmwares.*/ } __packed; +_Static_assert(sizeof(struct intel_ssdb) == SSDB_SIZE, + "SSDB structure size must match SSDB_SIZE"); #endif