Hi Steve, > > > On 14-08-26 02:14 AM, Lukasz Majewski wrote: > > Hi Steve, > > > >> > >> > >> On 14-08-25 07:57 AM, Lukasz Majewski wrote: > >>> Hi Steve, > >>> > >>>> This series implements the "fastboot flash" command for eMMC > >>>> devices. It supports both raw and sparse images. > >>>> > >>>> NOTES: > >>>> - the support for the "fastboot flash" command is enabled with > >>>> CONFIG_FASTBOOT_FLASH > >>>> - the support for eMMC is enabled with > >>>> CONFIG_FASTBOOT_FLASH_MMC_DEV > >>>> - (future) the support for NAND would be enabled with > >>>> CONFIG_FASTBOOT_FLASH_NAND(???) > >>>> > >>>> This has been tested on ARMv7. > >>>> > >>>> ---- > >>>> > >>>> This series depends on: > >>>> http://patchwork.ozlabs.org/patch/379438/ > >>> > >>> This patch is already applied. > >> Correct > >> (But it wasn't when I was preparing this....) > >> > >>> > >>>> http://patchwork.ozlabs.org/patch/382443/ (to 382446) > >>> > >>> Could you resend the above patch series one more time? > >>> > >>> From the links it seems that in PATCH 1/4 you add > >>> the ./common/aboot.c file and at PATCH 2/4 you remove it. > >>> > >>> Is this done on purpose? > >> yes: done on purpose (in order to see the history of the changes to > >> the file....) -- see the comments in the cover letter to that > >> patch-series!!! > > > > Ok, Now it is clear. > > > > Is fair as I know bsd-3L-clause license is compatible with GPL. > > > > However, to be 100% sure I'd like to ask Tom (as he has much more > > legal experience than me) to confirm that this code can be added to > > u-boot. > > > OK -- note that this "bsd-3L-clause" clause is almost identical to > the existing "bsd-3-clause"....
Tom, could you give your opinion on this? > Thanks, Steve > >> > >>> > >>> (I've also trimmed down the CC list). > >>> > >>>> > >>>> Changes in v5: > >>>> - use the common/aboot.c for the "sparse format" handling > >>>> > >>>> Changes in v4: > >>>> - rearranged this patchset so that "sparse_format.h" can be > >>>> dropped (if we cannot resolve the copyright/licensing issues) > >>>> - update mmc_get_dev(...) to get_dev("mmc",....) > >>>> - update printf() to puts() where applicable > >>>> - update debug string as per feedback > >>>> - rearranged "sparse format" support in this patchset, in order > >>>> to isolate... > >>>> > >>>> Changes in v3: > >>>> - remove most references to 'mmc', > >>>> which leaves only one mmc specific function: mmc_get_dev() > >>>> > >>>> Changes in v2: > >>>> - split large function into three > >>>> - improved handling of response messages > >>>> - additional partition size checking when writing sparse image > >>>> - update README.android-fastboot file > >>>> - new in v2 > >>>> > >>>> Steve Rae (4): > >>>> usb/gadget: fastboot: add eMMC support for flash command > >>>> usb/gadget: fastboot: add support for flash command > >>>> usb/gadget: fastboot: minor cleanup > >>>> usb/gadget: fastboot: implement sparse format > >>>> > >>>> README | 10 +++++ > >>>> common/Makefile | 6 +++ > >>>> common/cmd_fastboot.c | 7 +-- > >>>> common/fb_mmc.c | 95 > >>>> +++++++++++++++++++++++++++++++++++++++++ > >>>> doc/README.android-fastboot | 5 ++- > >>>> drivers/usb/gadget/f_fastboot.c | 44 +++++++++++++++++-- > >>>> include/fb_mmc.h | 8 ++++ 7 files changed, 166 > >>>> insertions(+), 9 deletions(-) create mode 100644 common/fb_mmc.c > >>>> create mode 100644 include/fb_mmc.h > >>>> > >>> > >>> > >>> > >> Thanks, Steve > > > > > > -- Best regards, Lukasz Majewski Samsung R&D Institute Poland (SRPOL) | Linux Platform Group _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot