From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 17 Jul 2017 20:56:35 -0400 Subject: [U-Boot] [PATCH 00/30] env: Move environment code to use location drivers In-Reply-To: References: <20170709205313.116174-1-sjg@chromium.org> <20170712122445.GB14320@bill-the-cat> Message-ID: <20170718005635.GB14320@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thu, Jul 13, 2017 at 10:45:10AM -0600, Simon Glass wrote: > Hi Tom, > > On 12 July 2017 at 06:24, Tom Rini wrote: > > > > On Sun, Jul 09, 2017 at 02:52:43PM -0600, Simon Glass wrote: > > > > > The environment code is ripe for improvement in various ways. It has lots > > > of duplication and inconsistencies between how things work with different > > > environment locations. It does not properly use Kconfig. Error checking > > > and handling is patchy. > > > > > > This series makes a start at improving things: > > > > > > - moves the rest of the CONFIG_ENV_IS_IN_... options to Kconfig > > > > After this, did you do some checks to make sure that everyone was > > migrated correctly? I ask since I had made an attempt a while back to > > convert and I found a lot of cases that didn't happen right and ended up > > hacking up something to have every target print all of the env related > > options to a file (and have buildman save it). But it might have been.. > > I need to do another pass with this series. I noticed that it doesn't > bisect properly (i.e. some interim commits mess things up that later > ones fix). OK. > > > - introduces the concept of a location driver for the environment > > > - drops some of the duplicated code > > > - improves error handling a little (more to be done here) > > > - moves code into a new env/ directory > > > > > > There is still much more to do: > > > - complete Kconfig conversion (e.g. CONFIG_ENV_SIZE) > > > > Because I was converting these parts as well. > > I think that is much harder because this option often uses other > options in its expression, but perhaps it is not as bad as I think. It's pretty awful, in some respects. Once I gave in and hacked buildman and U-Boot to have a "host tool" that would have the calculated values, spit that out and save the output file it wasn't too bad to convert. Best to save it for a follow up series :) -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: