[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