diff options
author | Maxime Ripard <maxime.ripard@free-electrons.com> | 2018-01-23 21:16:53 +0100 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2018-01-27 09:19:11 -0500 |
commit | 9efac3c8059a8aaf80e68ac34f58c7150ee109cb (patch) | |
tree | 65af1615cb1a7ab6be8a0165b4c2aece933450db /env | |
parent | 8a3a7e2270b39f14c1778a574bb815068d8c74d4 (diff) |
env: Make the env save message a bit more explicit
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.
Reviewed-by: Andre Przywara <andre.przywara@arm.com>
Reviewed-by: Lukasz Majewski <lukma@denx.de>
Reviewed-by: Simon Glass <sjg@chromium.org>
Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
Diffstat (limited to 'env')
-rw-r--r-- | env/env.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/env/env.c b/env/env.c index c186dca478d..157f1e6568c 100644 --- a/env/env.c +++ b/env/env.c @@ -168,8 +168,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; |