From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 18 Feb 2013 08:51:18 -0500 Subject: [U-Boot] [PATCH v3 9/9] dfu: Support larger than memory transfers. In-Reply-To: <20130218110142.3befd1f8@amdc308.digital.local> References: <1354298472-3605-1-git-send-email-panto@antoniou-consulting.com> <1354298472-3605-10-git-send-email-panto@antoniou-consulting.com> <20130212205150.GA1694@bill-the-cat> <20130218110142.3befd1f8@amdc308.digital.local> Message-ID: <512231D6.605@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 02/18/2013 05:01 AM, Lukasz Majewski wrote: > Hi Tom, > >> On Fri, Nov 30, 2012 at 08:01:12PM +0200, Pantelis Antoniou >> wrote: >> >>> We didn't support upload/download larger than available >>> memory. This is pretty bad when you have to update your root >>> filesystem for example. >>> >>> This patch removes the limitation (and the crashes when you >>> transfered any file larger than 4MB). On top of that reduces >>> the huge dfu buffer from 4MB to just 64K, which was over the >>> top. >>> >>> The sequence number is a 16 bit counter; make sure we handle >>> rollover correctly. This fixes the wrong transfers for large (> >>> 256MB) images. >>> >>> Also utilize a variable to handle initialization, so that we >>> don't rely on just the counter sent by the host. >>> >>> Signed-off-by: Pantelis Antoniou >>> >> >> To be clear, patches 1-8 are good and we should take, but this >> one means we can't use FAT/EXT* partitions without more work. I >> would suggest that we set this part aside for a moment and >> perhaps limit transfers that are larget than RAM to RAW only >> where we can write in chunks today. >> > > As fair as I remember, some additional work needs to be done with > composite.c file (to remove nasty #ifdefs). There was a problem > with newer version of dfu-utils (new handling of descriptors). I thought it ended up being resolved. I'll have to re-read the thread again I guess. - -- Tom -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBAgAGBQJRIjHWAAoJENk4IS6UOR1WEs0P/07uTOJRh+8hBgnpcXwBZ8zD keEtN8vHs3JYOjW1k6styAFNGXy+PBhOJNOIx6ClsdTvCRU8FtGh09SZUAYBrZEj 5WbfqGaeFWY9bpgAhwsNRMXD3mcHq3EGvRm0Ga+ep/EDFd+lgswvfx9EtgxkOjy5 MM0G4BnjwJxWM4DW2Wkk/rXI7Xy8jpVn3abUPLva4iY8X5L6ez9GXp/VNv6nCoNI i+LuGXEnv7BsO9g+x5pvYlnQeMC5BPC7vKNMq9dj8o6MZ/Q7jCQkqz85GIqyDTta UByzr24G4xK5m7V0iFSlV7fnRHjcg7q+uAB6u2YSibssyibIuLoJA2VdiGZqp8oH OUBQ3L2v84QHhcKTQm/yqcQ4FWHaHQ369v4QwnON29yFqtb7Z/M3GEKCqPbPIlge eg+Bb8fymdjELQT4Bo0+EkydlvaQOhkSjxBlVa9GTkRyoPxpE7RNY5lgciseVZO4 hKG/Xfnce7fpQNoE8fJCWRslQp3sOiDE65gFRzNJN/15i+my+xYmN5HiNPWhcgmI 2EVJGx9/LXqZ6yGZh8bQCC3yvNnshG+cm4qAj58ytkLjVSVnsd7yxFYbexUTEJ0q YwOmE/72cgL/3IzpRUmh4o5G+uFJqhFx7zndMQyItdTN09mhZu7dCUtgud66A1Qg wUiQkeF4sWubUMIpYUvz =L9X2 -----END PGP SIGNATURE-----