[U-Boot] [PATCH v2 1/3] usb, dfu: extract flush code into seperate function

Marek Vasut marex at denx.de
Tue Mar 18 12:32:38 CET 2014


On Tuesday, March 18, 2014 at 07:55:04 AM, Lukasz Majewski wrote:
> Hi Heiko,
> 
> > Hello Marek,
> > 
> > Am 18.03.2014 01:21, schrieb Marek Vasut:
> > > 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<hs at denx.de>
> > >> Cc: Lukasz Majewski<l.majewski at samsung.com>
> > >> Cc: Kyungmin Park<kyungmin.park at samsung.com>
> > >> Cc: Marek Vasut<marex at denx.de>
> > >> Cc: Pantelis Antoniou<panto at 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.
> > 
> > Argh... you are right .... patch 2/3 of this series deletes the
> > complete if ... I can send a new version, but if you want to fix
> > it that would be great!
> 
> In this case, please prepare v3.
> 
> > > Do you want this in 2014.04 or in -next ?
> > 
> > Hmm.. 2014.04 would be nice, but I think this is Lukasz decision,
> > because I changed the dfu state machine, and maybe he want to see
> > more tests?
> 
> When you send the v3, I will test it for regression and then I will
> prepare pull request to Tom for v2014.04.
> 
> Those patches will be added to u-boot-dfu repository.

Feel free to test u-boot-usb/master please.

Best regards,
Marek Vasut


More information about the U-Boot mailing list