Since we'll soon have support for multiple environments, the environment
saving message might end up being printed multiple times if the higher
priority environment cannot be used.

That might confuse the user, so let's make it explicit if the operation
failed or not.

Signed-off-by: Maxime Ripard <maxime.rip...@free-electrons.com>
---
 env/env.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/env/env.c b/env/env.c
index 673bfa6ba41b..1d13220aa79b 100644
--- a/env/env.c
+++ b/env/env.c
@@ -131,8 +131,9 @@ int env_save(void)
                if (!drv->save)
                        continue;
 
-               printf("Saving Environment to %s...\n", drv->name);
+               printf("Saving Environment to %s... ", drv->name);
                ret = drv->save();
+               printf("%s\n", ret ? "Failed" : "OK");
                if (!ret)
                        return 0;
 
-- 
2.14.3

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

Reply via email to