* [U-Boot] [RFC PATCH] usb: dfu: make nand upload working @ 2013-10-12 7:47 Bo Shen 2013-10-12 21:19 ` Lukasz Majewski 0 siblings, 1 reply; 6+ messages in thread From: Bo Shen @ 2013-10-12 7:47 UTC (permalink / raw) To: u-boot If won't pass value to dfu->r_left, it always 0, make no transfer, the dfu-util on host side report failed. So, before starting transfer, pass the value, then nand uploading can work. Signed-off-by: Bo Shen <voice.shen@atmel.com> --- drivers/dfu/dfu.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/dfu/dfu.c b/drivers/dfu/dfu.c index 56b21c7..3c3ad1b 100644 --- a/drivers/dfu/dfu.c +++ b/drivers/dfu/dfu.c @@ -274,6 +274,7 @@ int dfu_read(struct dfu_entity *dfu, void *buf, int size, int blk_seq_num) if (dfu->i_buf_start == NULL) return -ENOMEM; + dfu->r_left = dfu->data.nand.size; ret = dfu->read_medium(dfu, 0, dfu->i_buf_start, &dfu->r_left); if (ret != 0) { debug("%s: failed to get r_left\n", __func__); -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [RFC PATCH] usb: dfu: make nand upload working 2013-10-12 7:47 [U-Boot] [RFC PATCH] usb: dfu: make nand upload working Bo Shen @ 2013-10-12 21:19 ` Lukasz Majewski 2013-10-13 20:50 ` Marek Vasut 2013-10-14 7:25 ` Bo Shen 0 siblings, 2 replies; 6+ messages in thread From: Lukasz Majewski @ 2013-10-12 21:19 UTC (permalink / raw) To: u-boot On Sat, 12 Oct 2013 15:47:14 +0800 Bo Shen <voice.shen@atmel.com> wrote: > If won't pass value to dfu->r_left, it always 0, make no transfer, > the dfu-util on host side report failed. So, before starting transfer, > pass the value, then nand uploading can work. > > Signed-off-by: Bo Shen <voice.shen@atmel.com> > > --- > drivers/dfu/dfu.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/dfu/dfu.c b/drivers/dfu/dfu.c > index 56b21c7..3c3ad1b 100644 > --- a/drivers/dfu/dfu.c > +++ b/drivers/dfu/dfu.c > @@ -274,6 +274,7 @@ int dfu_read(struct dfu_entity *dfu, void *buf, > int size, int blk_seq_num) if (dfu->i_buf_start == NULL) > return -ENOMEM; > > + dfu->r_left = dfu->data.nand.size; Please do not add nand related code to "generic" DFU code. This can damage DFU operation with other memories (like eMMC). Such code shall be added to dfu_nand.c file. > ret = dfu->read_medium(dfu, 0, dfu->i_buf_start, > &dfu->r_left); if (ret != 0) { > debug("%s: failed to get r_left\n", > __func__); Best regards, Lukasz Majewski -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 198 bytes Desc: not available URL: <http://lists.denx.de/pipermail/u-boot/attachments/20131012/bd8b9cf4/attachment.pgp> ^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [RFC PATCH] usb: dfu: make nand upload working 2013-10-12 21:19 ` Lukasz Majewski @ 2013-10-13 20:50 ` Marek Vasut 2013-10-14 7:25 ` Bo Shen 1 sibling, 0 replies; 6+ messages in thread From: Marek Vasut @ 2013-10-13 20:50 UTC (permalink / raw) To: u-boot Dear Lukasz Majewski, > On Sat, 12 Oct 2013 15:47:14 +0800 > > Bo Shen <voice.shen@atmel.com> wrote: > > If won't pass value to dfu->r_left, it always 0, make no transfer, > > the dfu-util on host side report failed. So, before starting transfer, > > pass the value, then nand uploading can work. > > > > Signed-off-by: Bo Shen <voice.shen@atmel.com> > > > > --- > > > > drivers/dfu/dfu.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/dfu/dfu.c b/drivers/dfu/dfu.c > > index 56b21c7..3c3ad1b 100644 > > --- a/drivers/dfu/dfu.c > > +++ b/drivers/dfu/dfu.c > > @@ -274,6 +274,7 @@ int dfu_read(struct dfu_entity *dfu, void *buf, > > int size, int blk_seq_num) if (dfu->i_buf_start == NULL) > > > > return -ENOMEM; > > > > + dfu->r_left = dfu->data.nand.size; > > Please do not add nand related code to "generic" DFU code. This can > damage DFU operation with other memories (like eMMC). > > Such code shall be added to dfu_nand.c file. > ACK, waiting for V2. Lukasz, do you have a suggestion where this should be placed and why is this an actual issue? Best regards, Marek Vasut ^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [RFC PATCH] usb: dfu: make nand upload working 2013-10-12 21:19 ` Lukasz Majewski 2013-10-13 20:50 ` Marek Vasut @ 2013-10-14 7:25 ` Bo Shen 2013-10-16 9:00 ` Bo Shen 1 sibling, 1 reply; 6+ messages in thread From: Bo Shen @ 2013-10-14 7:25 UTC (permalink / raw) To: u-boot Hi Lukasz Majewski, On 10/13/2013 05:19, Lukasz Majewski wrote: > On Sat, 12 Oct 2013 15:47:14 +0800 > Bo Shen <voice.shen@atmel.com> wrote: > >> If won't pass value to dfu->r_left, it always 0, make no transfer, >> the dfu-util on host side report failed. So, before starting transfer, >> pass the value, then nand uploading can work. >> >> Signed-off-by: Bo Shen <voice.shen@atmel.com> >> >> --- >> drivers/dfu/dfu.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/dfu/dfu.c b/drivers/dfu/dfu.c >> index 56b21c7..3c3ad1b 100644 >> --- a/drivers/dfu/dfu.c >> +++ b/drivers/dfu/dfu.c >> @@ -274,6 +274,7 @@ int dfu_read(struct dfu_entity *dfu, void *buf, >> int size, int blk_seq_num) if (dfu->i_buf_start == NULL) >> return -ENOMEM; >> >> + dfu->r_left = dfu->data.nand.size; > > Please do not add nand related code to "generic" DFU code. This can > damage DFU operation with other memories (like eMMC). > > Such code shall be added to dfu_nand.c file. Thanks for point out this. I will check it. >> ret = dfu->read_medium(dfu, 0, dfu->i_buf_start, >> &dfu->r_left); if (ret != 0) { >> debug("%s: failed to get r_left\n", >> __func__); > > Best regards, > > Lukasz Majewski > Best Regards, Bo Shen ^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [RFC PATCH] usb: dfu: make nand upload working 2013-10-14 7:25 ` Bo Shen @ 2013-10-16 9:00 ` Bo Shen 2013-11-01 1:20 ` Bo Shen 0 siblings, 1 reply; 6+ messages in thread From: Bo Shen @ 2013-10-16 9:00 UTC (permalink / raw) To: u-boot Hi Lukasz Majewski, On 10/14/2013 15:25, Bo Shen wrote: > Hi Lukasz Majewski, > > On 10/13/2013 05:19, Lukasz Majewski wrote: >> On Sat, 12 Oct 2013 15:47:14 +0800 >> Bo Shen <voice.shen@atmel.com> wrote: >> >>> If won't pass value to dfu->r_left, it always 0, make no transfer, >>> the dfu-util on host side report failed. So, before starting transfer, >>> pass the value, then nand uploading can work. >>> >>> Signed-off-by: Bo Shen <voice.shen@atmel.com> >>> >>> --- >>> drivers/dfu/dfu.c | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/drivers/dfu/dfu.c b/drivers/dfu/dfu.c >>> index 56b21c7..3c3ad1b 100644 >>> --- a/drivers/dfu/dfu.c >>> +++ b/drivers/dfu/dfu.c >>> @@ -274,6 +274,7 @@ int dfu_read(struct dfu_entity *dfu, void *buf, >>> int size, int blk_seq_num) if (dfu->i_buf_start == NULL) >>> return -ENOMEM; >>> >>> + dfu->r_left = dfu->data.nand.size; >> >> Please do not add nand related code to "generic" DFU code. This can >> damage DFU operation with other memories (like eMMC). >> >> Such code shall be added to dfu_nand.c file. > > Thanks for point out this. > I will check it. After read more deeply into dfu core code, even we move this patch to nand related code as following. I think this only a workaround for NAND uploading, because it the same as mmc uploading, need the dfu buffer size equal or larger than the upload file size or partition size. So, does this acceptable? If acceptable, I will send v2. ---8>--- --- 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: ---<8--- Best Regards, Bo Shen ^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [RFC PATCH] usb: dfu: make nand upload working 2013-10-16 9:00 ` Bo Shen @ 2013-11-01 1:20 ` Bo Shen 0 siblings, 0 replies; 6+ messages in thread From: Bo Shen @ 2013-11-01 1:20 UTC (permalink / raw) To: u-boot Hi Lukasz, On 10/16/2013 17:00, Bo Shen wrote: > Hi Lukasz Majewski, > > On 10/14/2013 15:25, Bo Shen wrote: >> Hi Lukasz Majewski, >> >> On 10/13/2013 05:19, Lukasz Majewski wrote: >>> On Sat, 12 Oct 2013 15:47:14 +0800 >>> Bo Shen <voice.shen@atmel.com> wrote: >>> >>>> If won't pass value to dfu->r_left, it always 0, make no transfer, >>>> the dfu-util on host side report failed. So, before starting transfer, >>>> pass the value, then nand uploading can work. >>>> >>>> Signed-off-by: Bo Shen <voice.shen@atmel.com> >>>> >>>> --- >>>> drivers/dfu/dfu.c | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/drivers/dfu/dfu.c b/drivers/dfu/dfu.c >>>> index 56b21c7..3c3ad1b 100644 >>>> --- a/drivers/dfu/dfu.c >>>> +++ b/drivers/dfu/dfu.c >>>> @@ -274,6 +274,7 @@ int dfu_read(struct dfu_entity *dfu, void *buf, >>>> int size, int blk_seq_num) if (dfu->i_buf_start == NULL) >>>> return -ENOMEM; >>>> >>>> + dfu->r_left = dfu->data.nand.size; >>> >>> Please do not add nand related code to "generic" DFU code. This can >>> damage DFU operation with other memories (like eMMC). >>> >>> Such code shall be added to dfu_nand.c file. >> >> Thanks for point out this. >> I will check it. > > After read more deeply into dfu core code, even we move this patch to > nand related code as following. > I think this only a workaround for NAND uploading, because it the same > as mmc uploading, need the dfu buffer size equal or larger than the > upload file size or partition size. > So, does this acceptable? If acceptable, I will send v2. Would it possible to check this for NAND uploading? > ---8>--- > --- 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: > ---<8--- > > Best Regards, > Bo Shen Best Regards, Bo Shen ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-11-01 1:20 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-10-12 7:47 [U-Boot] [RFC PATCH] usb: dfu: make nand upload working Bo Shen 2013-10-12 21:19 ` Lukasz Majewski 2013-10-13 20:50 ` Marek Vasut 2013-10-14 7:25 ` Bo Shen 2013-10-16 9:00 ` Bo Shen 2013-11-01 1:20 ` Bo Shen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox