diff options
| author | Aristo Chen <[email protected]> | 2026-07-05 03:44:09 +0000 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2026-07-16 12:06:14 -0600 |
| commit | a8fc408cdfcbc3d8a04b493510a54d852f60550d (patch) | |
| tree | 5f27eed38dd9f1401a94849595e32ab1c6475fca | |
| parent | 780084d7c07f511da4a39b43277c573156ed42e0 (diff) | |
sandbox: vbe: size firmware1 area to fit the binman fw-update section
The firmware1 node in test.dts declares area-size = 0xe00000 (14 MiB)
but the binman fw-update section in sandbox_vpl.dtsi is 0x2000000
(32 MiB) and the FIT inside it carries ~16 MiB of external data
(spl + u-boot subimages). The FIT therefore extends past the
declared firmware area, contradicting the documented contract of
vbe_read_fit() that the FIT must fit within @area_size.
The mismatch was tolerated because no caller actually bounded the
external-data load against area_size. Bring the devicetree in line
with the binman section size so the FIT extent stays within the
trusted firmware area, in preparation for vbe_read_fit() enforcing
that bound.
state-offset and version-offset are left as-is; they were already
inside the FIT data region and are not exercised by test_vbe_vpl.
Reviewed-by: Simon Glass <[email protected]>
Signed-off-by: Aristo Chen <[email protected]>
| -rw-r--r-- | arch/sandbox/dts/test.dts | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/sandbox/dts/test.dts b/arch/sandbox/dts/test.dts index 0887de4333b..37a36705b36 100644 --- a/arch/sandbox/dts/test.dts +++ b/arch/sandbox/dts/test.dts @@ -176,7 +176,7 @@ storage = "mmc3"; skip-offset = <0x800000>; area-start = <0>; - area-size = <0xe00000>; + area-size = <0x2000000>; state-offset = <0xdffc00>; state-size = <0x40>; version-offset = <0xdffe00>; |