[U-Boot] [PATCH v2] usb: dfu: make nand upload working
Nowhere pass a value to len, which always 0, make no transfer which cause uploading failed.
This patch make nand upload working. However it needs enough malloc buffer to store read data, that means the buffer at least equal to the upload partition size, or else it doesn't work.
Signed-off-by: Bo Shen voice.shen@atmel.com --- Changes in v2: - Move code to nand part, or else it will break mmc
drivers/dfu/dfu_nand.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/dfu/dfu_nand.c b/drivers/dfu/dfu_nand.c index 1c2e03b..cf082e2 100644 --- a/drivers/dfu/dfu_nand.c +++ b/drivers/dfu/dfu_nand.c @@ -121,6 +121,7 @@ static int dfu_read_medium_nand(struct dfu_entity *dfu, u64 offset, void *buf,
switch (dfu->layout) { case DFU_RAW_ADDR: + *len = dfu->data.nand.size; ret = nand_block_read(dfu, offset, buf, len); break; default:
Hello Bo,
Am 07.11.2013 07:29, schrieb Bo Shen:
Nowhere pass a value to len, which always 0, make no transfer which cause uploading failed.
This patch make nand upload working. However it needs enough malloc buffer to store read data, that means the buffer at least equal to the upload partition size, or else it doesn't work.
Signed-off-by: Bo Shenvoice.shen@atmel.com
Changes in v2:
- Move code to nand part, or else it will break mmc
drivers/dfu/dfu_nand.c | 1 + 1 file changed, 1 insertion(+)
Thanks! Tested on the rut board.
Tested-by: Heiko Schocher hs@denx.de
bye, Heiko
Hi Bo,
Nowhere pass a value to len, which always 0, make no transfer which cause uploading failed.
This patch make nand upload working. However it needs enough malloc buffer to store read data, that means the buffer at least equal to the upload partition size, or else it doesn't work.
Signed-off-by: Bo Shen voice.shen@atmel.com
I would kindly ask some users of NAND part of the DFU (Tom, Heiko) to test if this change doesn't break anything.
Changes in v2:
- Move code to nand part, or else it will break mmc
drivers/dfu/dfu_nand.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/dfu/dfu_nand.c b/drivers/dfu/dfu_nand.c index 1c2e03b..cf082e2 100644 --- a/drivers/dfu/dfu_nand.c +++ b/drivers/dfu/dfu_nand.c @@ -121,6 +121,7 @@ static int dfu_read_medium_nand(struct dfu_entity *dfu, u64 offset, void *buf, switch (dfu->layout) { case DFU_RAW_ADDR:
*len = dfu->data.nand.size;
Best regards, Lukasz
Hello Lukasz,
Am 08.11.2013 16:41, schrieb Lukasz Majewski:
Hi Bo,
Nowhere pass a value to len, which always 0, make no transfer which cause uploading failed.
This patch make nand upload working. However it needs enough malloc buffer to store read data, that means the buffer at least equal to the upload partition size, or else it doesn't work.
Signed-off-by: Bo Shenvoice.shen@atmel.com
I would kindly ask some users of NAND part of the DFU (Tom, Heiko) to test if this change doesn't break anything.
I sent a tested-by for this patch, see:
http://lists.denx.de/pipermail/u-boot/2013-November/166552.html
without this patch nand upload didn;t worked on the rut board, with this patch it worked :-)
bye, Heiko
Hi Heiko,
On 11/11/2013 02:46 PM, Heiko Schocher wrote:
Hello Lukasz,
Am 08.11.2013 16:41, schrieb Lukasz Majewski:
Hi Bo,
Nowhere pass a value to len, which always 0, make no transfer which cause uploading failed.
This patch make nand upload working. However it needs enough malloc buffer to store read data, that means the buffer at least equal to the upload partition size, or else it doesn't work.
Signed-off-by: Bo Shenvoice.shen@atmel.com
I would kindly ask some users of NAND part of the DFU (Tom, Heiko) to test if this change doesn't break anything.
I sent a tested-by for this patch, see:
http://lists.denx.de/pipermail/u-boot/2013-November/166552.html
without this patch nand upload didn;t worked on the rut board, with this patch it worked :-)
Thanks for testing it.
bye, Heiko
Best Regards, Bo Shen
Dear Bo Shen,
Hi Heiko,
On 11/11/2013 02:46 PM, Heiko Schocher wrote:
Hello Lukasz,
Am 08.11.2013 16:41, schrieb Lukasz Majewski:
Hi Bo,
Nowhere pass a value to len, which always 0, make no transfer which cause uploading failed.
This patch make nand upload working. However it needs enough malloc buffer to store read data, that means the buffer at least equal to the upload partition size, or else it doesn't work.
Signed-off-by: Bo Shenvoice.shen@atmel.com
I would kindly ask some users of NAND part of the DFU (Tom, Heiko) to test if this change doesn't break anything.
I sent a tested-by for this patch, see:
http://lists.denx.de/pipermail/u-boot/2013-November/166552.html
without this patch nand upload didn;t worked on the rut board, with this patch it worked :-)
Thanks for testing it.
Thanks guys, PR is already on it's way to mainline.
Best regards, Marek Vasut
Dear Bo Shen,
Nowhere pass a value to len, which always 0, make no transfer which cause uploading failed.
This patch make nand upload working. However it needs enough malloc buffer to store read data, that means the buffer at least equal to the upload partition size, or else it doesn't work.
Signed-off-by: Bo Shen voice.shen@atmel.com
Changes in v2:
- Move code to nand part, or else it will break mmc
drivers/dfu/dfu_nand.c | 1 + 1 file changed, 1 insertion(+)
Applied, thanks.
Best regards, Marek Vasut
participants (4)
-
Bo Shen -
Heiko Schocher -
Lukasz Majewski -
Marek Vasut