[U-Boot] [PATCH v2 00/34] env: Move environment code to use location drivers

Tom Rini
2017-07-25T17:49:32Z

On Sun, Jul 23, 2017 at 09:19:35PM -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

I would like to see just this part first, by itself, so we can be sure
it's right. I took a quick stab at trying this, and here's a few
things:
- The freescale SECURE_BOOT boards aren't right.
board/freescale/common/Kconfig must have it select ENV_IS_NOWHERE and
the other options must depend on !CHAIN_OF_TRUST.
- Patch 3 seem to, due to moving things slightly, cause tiny size
changes. On balance, this is fine. But in terms of "did Kconfig
migration happen correctly" it is not, since it makes it easier for
something to slip past.

So, I'm going to grab and update the migration portions of this series
and have them as a zero size (aside from boards with CMD_CONFIG on)
change, hopefully shortly. And then we can still take in the rest of
this series, after comments, this release. Sound good? Thanks!

--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170725/59e8fd2b/attachment.sig>