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. Do you want this in 2014.04 or in -next ? [...] Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot