On Tuesday, March 18, 2014 at 04:53:30 PM, Lukasz Majewski wrote: > Hi Marek, > > > On Tuesday, March 18, 2014 at 01:35:47 PM, Lukasz Majewski wrote: > > > Hi Marek, > > > > > > > On Tuesday, March 18, 2014 at 07:51:05 AM, Lukasz Majewski wrote: > > > > > Hi Marek, > > > > > > > > > > > On Monday, March 17, 2014 at 11:56:16 AM, Heiko Schocher > > > > > > > > > > > > wrote: > > > > > > > move the flushing code into an extra function dfu_flush(), > > > > > > > so it can be used from other code. > > > > > > > > > > > > > > Signed-off-by: Heiko Schocher <h...@denx.de> > > > > > > > Cc: Lukasz Majewski <l.majew...@samsung.com> > > > > > > > Cc: Kyungmin Park <kyungmin.p...@samsung.com> > > > > > > > Cc: Marek Vasut <ma...@denx.de> > > > > > > > Cc: Pantelis Antoniou <pa...@antoniou-consulting.com> > > > > > > > > > > > > [...] > > > > > > > > > > > > > @@ -199,23 +221,7 @@ int dfu_write(struct dfu_entity *dfu, > > > > > > > void *buf, int size, int blk_seq_num) > > > > > > > > > > > > > > /* end? */ > > > > > > > if (size == 0) { > > > > > > > > > > > > > > - /* Now try and flush to the medium if > > > > > > > needed. */ > > > > > > > - if (dfu->flush_medium) > > > > > > > - ret = dfu->flush_medium(dfu); > > > > > > > - printf("\nDFU complete CRC32: 0x%08x\n", > > > > > > > dfu->crc); - > > > > > > > - /* clear everything */ > > > > > > > - dfu_free_buf(); > > > > > > > - dfu->crc = 0; > > > > > > > - dfu->offset = 0; > > > > > > > - dfu->i_blk_seq_num = 0; > > > > > > > - dfu->i_buf_start = dfu_buf; > > > > > > > - dfu->i_buf_end = dfu_buf; > > > > > > > - dfu->i_buf = dfu->i_buf_start; > > > > > > > - > > > > > > > - dfu->inited = 0; > > > > > > > - > > > > > > > - } > > > > > > > + ret = dfu_flush(dfu, buf, size, > > > > > > > blk_seq_num); > > > > > > > > > > > > This seems broken, at least because you didn't close the > > > > > > opened brace (see 'if (size == 0) {' .... I can fix this up > > > > > > when applying. > > > > > > > > > > I was prepared to pull those patches to u-boot-dfu repo today > > > > > and test them :-). > > > > > > > > > > Afterwards, I would send PR to Tom. > > > > > > > > > > Are you OK with this? > > > > > > > > Doesn't -dfu go through -usb ? ;-) > > > > > > As fair as I remember the dfu code from ./drivers/dfu/* shall go via > > > u-boot-dfu tree directly to Tom. > > > > > > > Or do you want to send to Tom > > > > directly ? Either way WFM. > > > > > > After writing the other message you waited for a response less than > > > one minute :-). > > > > > > > Nonetheless, I picked the patches via > > > > -usb, they're in master now. > > > > > > I've just tested them and they doesn't cause any regression. > > > > > > Would you be so kind and apply one more patch, which was supposed to > > > go with u-boot-dfu PR (this patch waits for submission from > > > 21.02.2014)? > > > > > > http://patchwork.ozlabs.org/patch/322450/ > > > > OK, picked and pushed. I think panto picked it up as well tho, but > > maybe I unleashed too much mental terror unto him with my OvisLink > > WL-1100SD that he is paralyzed by it ;-) Panto, I will pick this > > patch, OK ? > > This patch looks like a little orphan :-). I'm glad that someone pull > it finally.
OK. > > As for the DFU, what do we do about the PRs ? DFU is part of USB, so > > I shall logically pick the PRs from you and move them to trini, but > > if you're unhappy about this, we need to talk about that. Tom, > > Lukasz ... thoughts ? > > I'm fine with sending PRs to you. PR will go out once I do builds. Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot