Re: [U-Boot] [PATCH 1/1] Revert "env: solve compilation error in SPL"

2019-10-05 Thread Tom Rini
On Fri, Oct 04, 2019 at 04:00:24PM -0400, Tom Rini wrote:

> This reverts commit 7d4776545b0f8a8827e5d061206faf61c9ba6ea9.  The
> changes here break environment validation and furthermore do not seem to
> be required.
> 
> Signed-off-by: Tom Rini 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] [PATCH 1/1] Revert "env: solve compilation error in SPL"

2019-10-04 Thread Tom Rini
This reverts commit 7d4776545b0f8a8827e5d061206faf61c9ba6ea9.  The
changes here break environment validation and furthermore do not seem to
be required.

Signed-off-by: Tom Rini 
---
Here is what, sadly, is going to I feel be the most frustraiting part of
the regression here.  The problem that I'm sure was encountered during
development that needed this change is not present in mainline.  I've
done my usual world build with this revert applied and the only changes
are that the platforms that had a regression have their functionality
back.

 env/common.c | 7 +--
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/env/common.c b/env/common.c
index 3fb60509dd85..4daaa6faea60 100644
--- a/env/common.c
+++ b/env/common.c
@@ -24,10 +24,7 @@ DECLARE_GLOBAL_DATA_PTR;
 #include 
 
 struct hsearch_data env_htab = {
-#if CONFIG_IS_ENABLED(ENV_SUPPORT)
-   /* defined in flags.c, only compile with ENV_SUPPORT */
.change_ok = env_flags_validate,
-#endif
 };
 
 /*
@@ -229,9 +226,7 @@ void env_relocate(void)
 #if defined(CONFIG_NEEDS_MANUAL_RELOC)
env_reloc();
env_fix_drivers();
-
-   if (env_htab.change_ok)
-   env_htab.change_ok += gd->reloc_off;
+   env_htab.change_ok += gd->reloc_off;
 #endif
if (gd->env_valid == ENV_INVALID) {
 #if defined(CONFIG_ENV_IS_NOWHERE) || defined(CONFIG_SPL_BUILD)
-- 
2.17.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot