From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steve Rae Date: Fri, 23 Jan 2015 14:40:02 -0800 Subject: [U-Boot] [PATCH v4 2/2] fastboot: handle flash write to GPT partitions In-Reply-To: References: <1418428314-9611-1-git-send-email-srae@broadcom.com> <1418428314-9611-2-git-send-email-srae@broadcom.com> Message-ID: <54C2CDC2.5050606@broadcom.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 15-01-23 09:38 AM, Rob Herring wrote: > On Fri, Dec 12, 2014 at 5:51 PM, Steve Rae wrote: >> Implement a feature to allow fastboot to write the downloaded image >> to the space reserved for the Protective MBR and the Primary GUID >> Partition Table. >> Additionally, prepare and write the Backup GUID Partition Table. > > I've been looking at how to do the same thing here. This is an area > that suffers from each vendor doing whatever they want. Using vendor > download/flash tools here is painful. They are all different because > that is where the value add is. ;) What tool do you use on the host > side to create the image? I have seen some vendor code to do it, or > you could use parted plus a disk file and extract the partition table > from it. I find either method a bit fragile and non-standard IMHO. > We use an internal tool -- however, I also note that ALL of the source code for our tool is "GPL-2.0+" (expect for one file which is Public Domain) Is U-Boot (Denx) interested in supporting a "host tool"? > The 2 options I've come up with are 1) enable USB MS and use whatever > host side tool you like or 2) use the existing "gpt write" command in > u-boot and tie that into fastboot "oem format" command. The advantage > and disadvantage of the latter is that it hides the partitioning > details in u-boot from the user, but requires changing the u-boot env > to change partition layout. The partitioning requirements are pretty > SOC specific it seems. > We also have code which creates the GPT tables from a "fastboot oem format" command, and (if I understand correctly) we have code that implements a "gpt" command line, which creates the GPT tables from env variables... If there is interest here, I could investigate further. > I'm not saying we can't support both, but having some standardization > here would be good. > > Rob > I wasn't trying to promote an exclusive solution with this patch (which has been accepted - Thanks!). I was just trying to have an incremental change to the existing "fastboot flash" command (to handle the GPT Tables). Thanks, Steve [... snip ...]