Re: [U-Boot] [PATCH] dfu: fix some issues with reads/uploads

2014-06-20 Thread Lukasz Majewski
Hi Lukasz, > Hi Stephen, > > > On 05/22/2014 04:20 AM, Lukasz Majewski wrote: > > > Hi Stephen, > > > > > >> From: Stephen Warren > > >> > > >> DFU read support appears to rely upon dfu->read_medium() updating > > >> the passed-by-reference len parameter to indicate the remaining > > >> size av

Re: [U-Boot] [PATCH] dfu: fix some issues with reads/uploads

2014-06-12 Thread Stephen Warren
On 06/12/2014 01:17 AM, Lukasz Majewski wrote: > Stephen Warren wrote: ... >> Here's how I'd love to see the dfu command work. It would take a bit >> of refactoring of the code, and a change to the format of the >> dfu_alt_info variable and "dfu" command parameters. Perhaps we could >> have an opti

Re: [U-Boot] [PATCH] dfu: fix some issues with reads/uploads

2014-06-12 Thread Lukasz Majewski
Hi Stephen, > On 05/30/2014 02:28 AM, Lukasz Majewski wrote: > > Stephen Warren wrote: > ... > >> Wouldn't it be better to have the following separate > >> "backends" to DFU: > >> > >> * MMC (raw IO only) > >> * NAND (raw IO only) > >> * RAM (raw IO only) > >> * Filesystem (anything that fs/fat/ex

Re: [U-Boot] [PATCH] dfu: fix some issues with reads/uploads

2014-06-10 Thread Stephen Warren
On 06/10/2014 03:25 PM, Stephen Warren wrote: > From: Stephen Warren > > FIXME: reword after updates to use "*size" command for fs... > > DFU read support appears to rely upon dfu->read_medium() updating the > passed-by-reference len parameter to indicate the remaining size > available for readi

[U-Boot] [PATCH] dfu: fix some issues with reads/uploads

2014-06-10 Thread Stephen Warren
From: Stephen Warren FIXME: reword after updates to use "*size" command for fs... DFU read support appears to rely upon dfu->read_medium() updating the passed-by-reference len parameter to indicate the remaining size available for reading. dfu_read_medium_mmc() never does this, and the implemen

Re: [U-Boot] [PATCH] dfu: fix some issues with reads/uploads

2014-06-10 Thread Stephen Warren
On 05/30/2014 02:28 AM, Lukasz Majewski wrote: > Stephen Warren wrote: ... >> Wouldn't it be better to have the following separate >> "backends" to DFU: >> >> * MMC (raw IO only) >> * NAND (raw IO only) >> * RAM (raw IO only) >> * Filesystem (anything that fs/fat/ext* load/write can support) >> >>

Re: [U-Boot] [PATCH] dfu: fix some issues with reads/uploads

2014-06-03 Thread Lukasz Majewski
Hi Stephen, > On 06/02/2014 12:14 AM, Lukasz Majewski wrote: > > Hi Stephen, > > > >> On 05/30/2014 02:28 AM, Lukasz Majewski wrote: > >> ... > >>> I've tested if raw u-boot can be downloaded and uploaded via DFU. > >>> The u-boot size is 1MiB precisely. > >>> > >>> Corresponding dfu_alt_info en

Re: [U-Boot] [PATCH] dfu: fix some issues with reads/uploads

2014-06-02 Thread Stephen Warren
On 06/02/2014 12:14 AM, Lukasz Majewski wrote: > Hi Stephen, > >> On 05/30/2014 02:28 AM, Lukasz Majewski wrote: >> ... >>> I've tested if raw u-boot can be downloaded and uploaded via DFU. >>> The u-boot size is 1MiB precisely. >>> >>> Corresponding dfu_alt_info entry: >>> "u-boot raw 0x80 0x80

Re: [U-Boot] [PATCH] dfu: fix some issues with reads/uploads

2014-06-01 Thread Lukasz Majewski
Hi Stephen, > On 05/30/2014 02:28 AM, Lukasz Majewski wrote: > ... > > I've tested if raw u-boot can be downloaded and uploaded via DFU. > > The u-boot size is 1MiB precisely. > > > > Corresponding dfu_alt_info entry: > > "u-boot raw 0x80 0x800;" \ > ... > > 2. Upload (raw): > > > > dfu-util -

Re: [U-Boot] [PATCH] dfu: fix some issues with reads/uploads

2014-05-30 Thread Stephen Warren
On 05/30/2014 02:28 AM, Lukasz Majewski wrote: ... > I've tested if raw u-boot can be downloaded and uploaded via DFU. The > u-boot size is 1MiB precisely. > > Corresponding dfu_alt_info entry: > "u-boot raw 0x80 0x800;" \ ... > 2. Upload (raw): > > dfu-util -a0 -U u-boot-mmc.bin_target > It ex

Re: [U-Boot] [PATCH] dfu: fix some issues with reads/uploads

2014-05-30 Thread Lukasz Majewski
Hi Stephen, > On 05/22/2014 04:20 AM, Lukasz Majewski wrote: > > Hi Stephen, > > > >> From: Stephen Warren > >> > >> DFU read support appears to rely upon dfu->read_medium() updating > >> the passed-by-reference len parameter to indicate the remaining > >> size available for reading. > >> > >> d

Re: [U-Boot] [PATCH] dfu: fix some issues with reads/uploads

2014-05-22 Thread Stephen Warren
On 05/22/2014 04:20 AM, Lukasz Majewski wrote: > Hi Stephen, > >> From: Stephen Warren >> >> DFU read support appears to rely upon dfu->read_medium() updating the >> passed-by-reference len parameter to indicate the remaining size >> available for reading. >> >> dfu_read_medium_mmc() never does t

Re: [U-Boot] [PATCH] dfu: fix some issues with reads/uploads

2014-05-22 Thread Lukasz Majewski
Hi Stephen, > From: Stephen Warren > > DFU read support appears to rely upon dfu->read_medium() updating the > passed-by-reference len parameter to indicate the remaining size > available for reading. > > dfu_read_medium_mmc() never does this, and the implementation of > dfu_read_medium_nand()

[U-Boot] [PATCH] dfu: fix some issues with reads/uploads

2014-05-19 Thread Stephen Warren
From: Stephen Warren DFU read support appears to rely upon dfu->read_medium() updating the passed-by-reference len parameter to indicate the remaining size available for reading. dfu_read_medium_mmc() never does this, and the implementation of dfu_read_medium_nand() will only work if called just