From: Steve Rae <srae@broadcom.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v5 0/4] Implement "fastboot flash" for eMMC
Date: Mon, 25 Aug 2014 10:10:28 -0700 [thread overview]
Message-ID: <53FB6E04.20708@broadcom.com> (raw)
In-Reply-To: <20140825165756.67a1ce84@amdc2363>
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!!!
>
> (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
next prev parent reply other threads:[~2014-08-25 17:10 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-23 19:53 [U-Boot] [PATCH v5 0/4] Implement "fastboot flash" for eMMC Steve Rae
2014-08-23 19:53 ` [U-Boot] [PATCH v5 1/4] usb/gadget: fastboot: add eMMC support for flash command Steve Rae
2014-08-26 9:28 ` Lukasz Majewski
2014-08-26 9:41 ` Marek Vasut
2014-08-26 18:54 ` Steve Rae
2014-08-23 19:53 ` [U-Boot] [PATCH v5 2/4] usb/gadget: fastboot: add " Steve Rae
2014-08-26 11:53 ` Lukasz Majewski
2014-08-23 19:53 ` [U-Boot] [PATCH v5 3/4] usb/gadget: fastboot: minor cleanup Steve Rae
2014-08-26 11:55 ` Lukasz Majewski
2014-08-23 19:53 ` [U-Boot] [PATCH v5 4/4] usb/gadget: fastboot: implement sparse format Steve Rae
2014-08-26 11:58 ` Lukasz Majewski
2014-08-23 20:12 ` [U-Boot] [PATCH v5 0/4] Implement "fastboot flash" for eMMC Marek Vasut
2014-08-25 7:19 ` Lukasz Majewski
2014-08-25 7:52 ` Marek Vasut
2014-08-25 14:57 ` Lukasz Majewski
2014-08-25 17:10 ` Steve Rae [this message]
2014-08-26 9:14 ` Lukasz Majewski
2014-08-26 16:27 ` Steve Rae
2014-08-27 7:04 ` Lukasz Majewski
2014-08-28 13:24 ` Tom Rini
2014-08-28 17:06 ` Steve Rae
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=53FB6E04.20708@broadcom.com \
--to=srae@broadcom.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox