Message ID | 1439661901-9848-2-git-send-email-hdegoede@redhat.com |
---|---|
State | Accepted |
Delegated to: | Tom Rini |
Headers | show |
On Sat, Aug 15, 2015 at 08:05:01PM +0200, Hans de Goede wrote: > The set_default_env() function from env_common.c expects either > a fully formatted error msg, e.g.: "## Resetting to default environment\n" > or an error msg prefixed with an !, in which case it will format it. > > Fix the init_mmc_for_env() error messages to be prefixed with a ! > this changes the bootup-log on sunxi when no mmc card is found from: > > MMC: SUNXI SD/MMC: 0 > No MMC card foundIn: serial > Out: serial > > To: > > MMC: SUNXI SD/MMC: 0 > *** Warning - No MMC card found, using default environment > > In: serial > Out: serial > > Which clearly is how things should look. > > Signed-off-by: Hans de Goede <hdegoede@redhat.com> Applied to u-boot/master, thanks!
diff --git a/common/env_mmc.c b/common/env_mmc.c index 6c4ce2f..51e7707 100644 --- a/common/env_mmc.c +++ b/common/env_mmc.c @@ -93,13 +93,13 @@ static inline int mmc_set_env_part(struct mmc *mmc) {return 0; }; static const char *init_mmc_for_env(struct mmc *mmc) { if (!mmc) - return "No MMC card found"; + return "!No MMC card found"; if (mmc_init(mmc)) - return "MMC init failed"; + return "!MMC init failed"; if (mmc_set_env_part(mmc)) - return "MMC partition switch failed"; + return "!MMC partition switch failed"; return NULL; }
The set_default_env() function from env_common.c expects either a fully formatted error msg, e.g.: "## Resetting to default environment\n" or an error msg prefixed with an !, in which case it will format it. Fix the init_mmc_for_env() error messages to be prefixed with a ! this changes the bootup-log on sunxi when no mmc card is found from: MMC: SUNXI SD/MMC: 0 No MMC card foundIn: serial Out: serial To: MMC: SUNXI SD/MMC: 0 *** Warning - No MMC card found, using default environment In: serial Out: serial Which clearly is how things should look. Signed-off-by: Hans de Goede <hdegoede@redhat.com> --- common/env_mmc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)