public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Lukasz Majewski <l.majewski@samsung.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v5 0/4] Implement "fastboot flash" for eMMC
Date: Mon, 25 Aug 2014 16:57:56 +0200	[thread overview]
Message-ID: <20140825165756.67a1ce84@amdc2363> (raw)
In-Reply-To: <1408823626-12744-1-git-send-email-srae@broadcom.com>

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.

>   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?

(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
> 



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

  parent reply	other threads:[~2014-08-25 14:57 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 [this message]
2014-08-25 17:10   ` Steve Rae
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=20140825165756.67a1ce84@amdc2363 \
    --to=l.majewski@samsung.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