Hi Julien, On 10/11/2023 10:27, Julien Grall wrote: > > > Hi Stefano, > > On 10/11/2023 01:44, Stefano Stabellini wrote: >> Reported-by: Brian Woods <[email protected]> >> Signed-off-by: Stefano Stabellini <[email protected]> >> >> diff --git a/scripts/uboot-script-gen b/scripts/uboot-script-gen >> index b284887..6e52da5 100755 >> --- a/scripts/uboot-script-gen >> +++ b/scripts/uboot-script-gen >> @@ -790,7 +790,7 @@ bitstream_load_and_config() >> check_file_type "$BITSTREAM" "BIT data" >> bitstream_addr=$memaddr >> load_file $BITSTREAM "fpga_bitstream" >> - bitstream_size=$filesize >> + bitstream_size=`printf "0x%X\n" $filesize` > > Looking at [1], there is no indication that the size parameter for "fpga > load" should be hexadecimal. At the contrary, all the example I have > found seems to use $filesize. U-boot expects size to be passed in hex format. You can see it here: https://github.com/u-boot/u-boot/blob/master/cmd/fpga.c#L60C20-L60C27
Also, AFAICT $filesize var that gets updated after image load (e.g. tftpb) is in hex format. ~Michal
