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] fastboot: check for alias when looking up partition by name
Date: Mon, 23 Mar 2015 10:13:17 +0100	[thread overview]
Message-ID: <20150323101317.033b4654@amdc2363> (raw)
In-Reply-To: <550C4B1F.4070700@broadcom.com>

Hi Steve,

> Hi Lukasz,
> 
> On 15-03-20 12:50 AM, Lukasz Majewski wrote:
> > Hi Steve,
> >
> >>
> >>
> >> On 15-03-12 10:17 AM, Michael Scott wrote:
> >>>
> >>> On 03/12/2015 09:23 AM, Steve Rae wrote:
> 
> [... snip ...]
> 
> >>>> An interesting feature (which seems unnecessary to me...)
> >>>> However,
> >>> A bit of background:
> >>>
> >>> We are using fastboot support on Nvidia Jetson-TK1 platform where
> >>> a GPT limitation sets partition names to 3 letters.
> >>>
> >>> IE: LNX = boot, APP = system and UDA = userdata.
> >> OK -- then this patch makes much more sense!
> >> - so when the user performs "mmc part", (I'm assuming it will list
> >> the "3 letter names"), then they perform "fastboot flash LNX
> >> boot.bin" (which makes sense to me....)
> >> - or they can setup these aliases and perform "fastboot flash boot
> >> boot.bin" (I would probably stick with the former myself...)
> >> Thanks, Steve
> >>>
> >>> To present a "normal" fastboot experience to users, we use this
> >>> patch.
> >>>
> >>>> Acked-by: Steve Rae <srae@broadcom.com>
> >>>
> >>> Thank you for the Ack!
> >
> > Are there more comments for this patch?
> >
> 
> No - the feature is OK
> (personally - I would use the names that match what "mmc part"
> displays; so I don't think that I would create any aliases....)
> 
> Acked-by: Steve Rae <srae@broadcom.com>

Applied to u-boot-dfu tree. Thanks!

-- 
Best regards,

Lukasz Majewski

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

      reply	other threads:[~2015-03-23  9:13 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-11 17:02 [U-Boot] [PATCH] fastboot: check for alias when looking up partition by name Michael Scott
2015-03-12 16:23 ` Steve Rae
2015-03-12 17:17   ` Michael Scott
2015-03-12 18:14     ` Steve Rae
2015-03-20  7:50       ` Lukasz Majewski
2015-03-20 16:30         ` Steve Rae
2015-03-23  9:13           ` Lukasz Majewski [this message]

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=20150323101317.033b4654@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