Raul Rangel has submitted this change. ( https://review.coreboot.org/c/coreboot/+/55973 )
Change subject: util/cbfstool: Allow setting alignment for payload ......................................................................
util/cbfstool: Allow setting alignment for payload
The -a flag was already implemented, it just wasn't exposed for the add-payload command.
Setting the alignment of the payload will enable using the SPI DMA controller to read the payload on AMD devices.
BUG=b:179699789 TEST=cbfstool foo.bin add-payload -a 64 ...
Signed-off-by: Raul E Rangel rrangel@chromium.org Change-Id: I9f4aea5f0cbeaa8e761212041099b37f4718ac39 Reviewed-on: https://review.coreboot.org/c/coreboot/+/55973 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Martin Roth martinroth@google.com --- M util/cbfstool/cbfstool.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Martin Roth: Looks good to me, approved
diff --git a/util/cbfstool/cbfstool.c b/util/cbfstool/cbfstool.c index 48c9387..8fd81fe 100644 --- a/util/cbfstool/cbfstool.c +++ b/util/cbfstool/cbfstool.c @@ -1723,7 +1723,7 @@ {"add", "H:r:f:n:t:c:b:a:p:yvA:j:gh?", cbfs_add, true, true}, {"add-flat-binary", "H:r:f:n:l:e:c:b:p:vA:gh?", cbfs_add_flat_binary, true, true}, - {"add-payload", "H:r:f:n:c:b:C:I:p:vA:gh?", cbfs_add_payload, + {"add-payload", "H:r:f:n:c:b:a:C:I:p:vA:gh?", cbfs_add_payload, true, true}, {"add-stage", "a:H:r:f:n:t:c:b:P:QS:p:yvA:gh?", cbfs_add_stage, true, true},