[U-Boot] [PATCH] test: dfu: script: wrong md5sum on nand partitions

Stephen Warren swarren at wwwdotorg.org
Fri Sep 12 16:53:42 CEST 2014


On 09/12/2014 12:27 AM, Heiko Schocher wrote:
> when uplaoding a file, at least from a nand partition, the complete
> mtd nand partition size is transferred. This leads in a wrong md5sum
> as the filesize is different between the downloaded file and the
> uploaded file. Limit the uploaded filesize to the downloaded
> fixes this.

I was going to say that it'd be better to fix U-Boot's NAND code to 
transfer the correct amount of data. However, I suppose the correct 
amount *is* the whole partition for anything other than a filesystem 
file. As such, I'd suggest replacing "nand" in the patch description 
with something else, since presumably the exact same issue applies to 
partitions on eMMC. The issue applies to any partition.

I wonder if there's a way to distinguish between file tests and 
partition tests in dfu_gadget_test.sh, so that the $N_FILE_FULL -> 
$N_FILE conversion can be applied only for partitions. Otherwise, a file 
upload/download test could end up changing (increasing) the file length 
and this bug wouldn't be detected.



More information about the U-Boot mailing list