From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Kocialkowski Date: Thu, 13 Nov 2014 23:16:09 +0100 Subject: [U-Boot] [PATCH 0/2] spl: MMC U-Boot image load from raw partition In-Reply-To: <20141113121601.7177bdb4@lilith> References: <1415484896-28928-1-git-send-email-contact@paulk.fr> <20141108231923.243bf07a@lilith> <20141110184609.GX24724@bill-the-cat> <20141113121601.7177bdb4@lilith> Message-ID: <1415916969.2372.10.camel@collins> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Le jeudi 13 novembre 2014 ? 12:16 +0100, Albert ARIBAUD a ?crit : > Hello Tom, > > On Mon, 10 Nov 2014 13:46:09 -0500, Tom Rini wrote: > > On Sat, Nov 08, 2014 at 11:19:23PM +0100, Albert ARIBAUD wrote: > > > Hello Paul, > > > > > > On Sat, 8 Nov 2014 23:14:54 +0100, Paul Kocialkowski > > > wrote: > > > > This is a first attempt at adding support for U-Boot image load from raw > > > > partitions. It does not support OS boot as I cannot test it on my current > > > > setup. > > > > > > > > This is going to be useful for the Optimus Black port (please do not consider > > > > this as dead code because no board is using it right now, there will be one > > > > soon)! > > > > > > Well... Why don't you just post these two patches a little later, as > > > part of the upcoming series which will add support for the Optimus > > > Black? > > > > So to me the "dead code" thing is starting to get a lot more ambiguous > > since with Kconfig we'll need need every single possible choice enabled > > in some defconfig, just some way to turn it on, on a possibly relevant > > board. In this case, any OMAP3+ board would be a fine place to try this > > out, IFF it's done as a Kconfig choice. > > Not sure I'm understanding you right, but it seems to me we're in sync: > as long as the code is enabled somewhere on some target, it is not dead > code. I'm precisely asking that the code here be submitted along with > the target that uses it. Or did I miss something? Well I think it makes sense to not call this dead code as long as it *can be* enabled and used on another supported board (for that matter, any OMAP3+ board will indeed do). This is very different from e.g. the regulator code that I submitted, which is only relevant for devices with that particular piece of hardware (so far, none supported by U-Boot). So it makes sense to submit that regulator patch only along with support for a board that uses it. -- Paul Kocialkowski, Replicant developer Replicant is a fully free Android distribution Website: http://www.replicant.us/ Blog: http://blog.replicant.us/ Wiki/tracker/forums: http://redmine.replicant.us/ -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: This is a digitally signed message part URL: