From mboxrd@z Thu Jan  1 00:00:00 1970
From: Lukasz Majewski <l.majewski@samsung.com>
Date: Mon, 23 Mar 2015 10:13:17 +0100
Subject: [U-Boot] [PATCH] fastboot: check for alias when looking up
	partition by name
In-Reply-To: <550C4B1F.4070700@broadcom.com>
References: <1426093351-8998-1-git-send-email-michael.scott@linaro.org>
 <5501BD75.4020006@broadcom.com> <5501CA1F.5000903@linaro.org>
 <5501D782.3000503@broadcom.com> <20150320085010.69159f40@amdc2363>
 <550C4B1F.4070700@broadcom.com>
Message-ID: <20150323101317.033b4654@amdc2363>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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