Skip to content

usb: gadget: fastboot miscellaneous patches

Steve Rae u-boot 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest3 patches 0 repliesNew

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3usb: gadget: fastboot: add max-download-size variable+0 -01 reply
1/3usb: gadget: fastboot: add max-download-size variable+0 -00 replies
2/3usb: gadget: fastboot: explicitly set radix of maximum download size+0 -00 replies
3/3usb: gadget: fastboot: terminate commands with NULL+0 -02 replies

Reviews trailers per patch

None of the 4 patches has a Reviewed-by or Acked-by.

PatchSteve Rae
1/3T-b
1/3T-b
2/3T-b
3/3T-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [U-Boot] [PATCH 0/3] usb: gadget: fastboot miscellaneous patches
Quoted 48 lines · Marek Vasut
Quoted 47 lines · Eric Nelson

Hi Marek,

Quoted 14 lines · Marek Vasut
Quoted 13 lines · Eric Nelson

Hi Marek,

Quoted 10 lines · Marek Vasut
Quoted 8 lines · Eric Nelson

While trying to configure Nitrogen6X boards to use Android Fastboot, I found a number of places where the implementation doesn't appear to match the latest host tools on AOSP.

Eric Nelson (3):
usb: gadget: fastboot: add max-download-size variable
usb: gadget: fastboot: explicitly set radix of maximum download size
usb: gadget: fastboot: terminate commands with NULL

Quoted 6 lines
Quoted 4 lines
Quoted 1 line

Just to make sure, are those fixes for 2014.10 or new stuff for next ?

These patches are against master, but should apply against usb/next and dfu/next and "next" is fine for us.

3/3 looks like a bugfix though. Is that a bugfix ?

I wasn't able to get fastboot to do much of anything without all three.

-- lack of max-download-size simply stopped downloads
-- the missing radix caused my host to think that the 0x07000000 size (copied from Beagle) was 1.8 MiB instead of 100+ MiB. -- the lack of termination showed up as a request to download a **huge** image when I tried to send u-boot.imx. I think I got lucky that the next characters in the buffer looked like hex digits.

I suspect that others are either holding on to some local patches or are perhaps using old versions of the Fastboot host program.

After applying all three, I was able to configure for flashing on an MMC device, but I don't have anything configured to use EFI partitions, so there's no immediate route to usage for us.

I'd really like to be able to "fastboot flash bootloader u-boot.imx" and program SPI-NOR and also be able to boot a kernel and RAM disk using "fastboot boot uImage uramdisk.img", but neither of them seems very close. The first needs some more structure, and the latter seemed to decide its' own address for the kernel and simply ignore the RAM disk.

Quoted 8 lines
Quoted 2 lines

I have the sense that this code is pretty much a work in progress, but I'd like to hear otherwise from those who have used it.

OK, so let's wait for the others' opinions.

Best regards,
Marek Vasut

I would recommend 1/3 & 2/3 for 2014.10 (I'm not certain about 3/3 because I don't think that it can actually occur on my boards....) Thanks, Steve