From 9efac3c8059a8aaf80e68ac34f58c7150ee109cb Mon Sep 17 00:00:00 2001 From: Maxime Ripard Date: Tue, 23 Jan 2018 21:16:53 +0100 Subject: [PATCH] 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 Reviewed-by: Lukasz Majewski Reviewed-by: Simon Glass Signed-off-by: Maxime Ripard --- env/env.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/env/env.c b/env/env.c index c186dca478..157f1e6568 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; -- 2.25.1