From: Steve Rae <srae@broadcom.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] fastboot: check for alias when looking up partition by name
Date: Fri, 20 Mar 2015 09:30:23 -0700 [thread overview]
Message-ID: <550C4B1F.4070700@broadcom.com> (raw)
In-Reply-To: <20150320085010.69159f40@amdc2363>
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>
next prev parent reply other threads:[~2015-03-20 16:30 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 [this message]
2015-03-23 9:13 ` Lukasz Majewski
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=550C4B1F.4070700@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