From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Tue, 20 Oct 2015 15:32:27 -0400 Subject: [U-Boot] [PATCH v2 00/14] Implement fastboot over NAND and sparse transfers In-Reply-To: <20151019081746.GT2711@lukather> References: <1444912462-3949-1-git-send-email-maxime.ripard@free-electrons.com> <56225424.3020009@redhat.com> <56225E41.2060301@redhat.com> <20151019081746.GT2711@lukather> Message-ID: <20151020193227.GX23893@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Mon, Oct 19, 2015 at 10:17:46AM +0200, Maxime Ripard wrote: > On Sat, Oct 17, 2015 at 04:42:09PM +0200, Hans de Goede wrote: > > Hi, > > > > On 17-10-15 15:59, Hans de Goede wrote: > > >Hi, > > > > > >On 15-10-15 14:34, Maxime Ripard wrote: > > >>Hi everyone, > > >> > > >>Here is the second attempt at getting fastboot flashing functions > > >>working on top of a NAND, for arbitraly large images. > > >> > > >>While the NAND support itself was quite easy to do, the support for > > >>the Android sparse images was quite difficult to add, and ended up > > >>being a quite huge refactoring of the sparse parse already in tree, > > >>that was tied to the MMC layer. > > >> > > >>This serie has been tested on a CHIP and an A13-Olinuxino, two > > >>Allwinner devices, the CHIP for the NAND, and the Cubietruck to test > > >>the MMC. > > >> > > >>Let me know what you think, > > >>Maxime > > > > > >Thanks, I've applied the 3 sunxi patches (12 - 14) to u-boot-sunxi/next, > > >with the commit msg fixup suggested by Ian. > > > > Looks like I've spoken too soon, the patch for enabling otg on the cubietruck > > causes build errors: > > > > arm-linux-gnu-ld.bfd: error: /usr/lib/gcc/arm-linux-gnueabi/5.2.1/libgcc.a(_udivmoddi4.o) uses VFP register arguments, u-boot does not > > arm-linux-gnu-ld.bfd: failed to merge target specific data of file /usr/lib/gcc/arm-linux-gnueabi/5.2.1/libgcc.a(_udivmoddi4.o) > > Makefile:1173: recipe for target 'u-boot' failed > > make: *** [u-boot] Error 1 > > > > Usually this means that some code is using 64 bit int division > > without using the proper macros for this. > > > > For some reason I'm not seeing the same on the A13-OlinuxIno, that patch > > for that seems to be missing the id pin, which could have something to do > > with this. > > > > I think this is caused by some 64 bit math in the fastboot code somewhere ... > > > > For now I've dropped these 2 patches from u-boot-sunxi/next > > Which 2 patches then? Cubietruck and this one? > > Do you have a toolchain tarball for this so that I can test easily? The arm-none-eabi toolchain that's apt-get'able should show this. You could even fudge this by (I _think_) forcing U-Boot to build with -march=armv5te rather than armv7. I _might_ even be able to spot it in the patch review :) -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: