public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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

  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