Martin Roth has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/34224 )
Change subject: fit_payload: Always set DT size ......................................................................
fit_payload: Always set DT size
CB:32870 changed FIT loading code to make an FDT mandatory (because the platforms that can use FIT images always need an FDT). Remove one left-over conditional that is now dead code.
Found by Coverity.
Change-Id: Ia7765d45f068ab4bdc720ea7ae87dcc62a4b7d3d Signed-off-by: Julius Werner jwerner@chromium.org Reviewed-on: https://review.coreboot.org/c/coreboot/+/34224 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Patrick Rudolph siro@das-labor.org --- M src/lib/fit_payload.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Patrick Rudolph: Looks good to me, approved
diff --git a/src/lib/fit_payload.c b/src/lib/fit_payload.c index 8e75915..a4d3705 100644 --- a/src/lib/fit_payload.c +++ b/src/lib/fit_payload.c @@ -228,7 +228,7 @@
/* Collect infos for fit_payload_arch */ kernel.size = config->kernel->size; - fdt.size = dt ? dt_flat_size(dt) : 0; + fdt.size = dt_flat_size(dt); initrd.size = config->ramdisk ? config->ramdisk->size : 0;
/* Invoke arch specific payload placement and fixups */