From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Wed, 29 Jan 2014 08:07:01 +0100 Subject: [U-Boot] [PATCH] ventana: Add Gateworks Ventana family support In-Reply-To: References: <1390329660-2448-1-git-send-email-tharvey@gateworks.com> <52E66F88.9010002@denx.de> <52E7846D.50601@denx.de> Message-ID: <52E8A895.2060208@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Tim, On 29.01.2014 02:26, Tim Harvey wrote: >>> Regardless of what is loading the next level (u-boot.bin) the initial >>> flashing of NAND is still currently handled only by kobs-ng so I'm not >>> sure how this differs in this respect. I plan to look at adding the >>> functionality of kobs-ng to u-boot at some point. >> >> If as I think kobs-ng is only a flasher, it does not take part to the >> build of U-Boot binaries. IMHO it should be not part of U-Boot sources, >> but maybe there are some features that can be interesting. >> > > I was not referring to making the code a part of uboot sources but > adding the functionality that it provides such that one could use > uboot to update itself on an IMX NAND device. I'm actually surprised > nobody has done this before for IMX in general as its a bit > inconvenient to boot to a linux based OS in order to run kobs-ng to > flash the bootloader. Regardless, this would be functionality added > later. That would be really convenient. Not having to boot Linux and to rely on kobs-ng for flashing U-Boot for NAND booting imx6. > It appears the only other mainlined IMX6 bootloader to support NAND is > the Titanium and there is no README for it at all. If there were, I > would expect it to say pretty much the same thing that my proposed > Ventana README states. There was a comment by Fabio on original > titanium patch to include a README explaining how to flash and boot > from NAND but it apparently didn't make it in: > http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/158436/focus=158441 > > I've added Stefan Roese and Fabio to the cc to hear their thoughts. Yes, your README seems correct to me. So for this README: Reviewed-by: Stefan Roese Thanks, Stefan