[U-Boot] [PATCH 4/4] fastboot: Enable flashing by default on sunxi

[Maxime Ripard]

On Thu, Nov 30, 2017 at 09:56:18AM +0000, Andre Przywara wrote:
> Hi,
> 
> On 30/11/17 07:58, Maxime Ripard wrote:
> > On Thu, Nov 30, 2017 at 12:23:28AM +0000, André Przywara wrote:
> >> On 28/11/17 10:34, Maxime Ripard wrote:
> >>> Now that more and more devices are built using eMMC, providing a way to
> >>> easily flash the system without too much hassle seems like a right thing to
> >>> do.
> >>>
> >>> Since fastboot is the most deployed tool to do that these days, we can just
> >>> rely on it to provide a way to flash the various components in the system
> >>> (SPL, U-Boot and the system itself) easily, especially since you can upload
> >>> the U-Boot hosting the fastboot "server" through FEL.
> >>>
> >>> Signed-off-by: Maxime Ripard <maxime.ripard at free-electrons.com>
> >>
> >> Reviewed-by: Andre Przywara <andre.przywara at arm.com>
> >>
> >> The binary size for 32-bit targets is still reasonable with this
> >> addition (thanks to Thumb2).
> >>
> >> But if I enable MUSB for the Pine64 it generates a 537KB u-boot.itb. And
> >> the build process does not complain, I guess because I don't generate a
> >> combined binary (glueing SPL in front). But this isn't really caused by
> >> this patch, so just a heads up.
> > 
> > I guess we would need to have kind of the same check at compile time
> > for ARM64 then. I'm not sure exactly how that would be doable
> > though...
> 
> Me neither without digging deeply into the build system ;-)
> 
> By the way, can you point me to the part which triggers the generation
> of u-boot-sunxi-with-spl.bin? This does not get generated for ARM64 at
> the moment, though it should. And this is probably the place where the
> check lives for ARM.

I *think* this is in arch/arm/cpu/armv7/sunxi/config.mk

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171130/01493611/attachment.sig>