From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Wed, 22 Jun 2016 07:51:59 +0200 Subject: [U-Boot] [PATCH 5/6] autoboot: move CONFIG_SILENT_CONSOLE handling In-Reply-To: <1466487167-18987-6-git-send-email-yamada.masahiro@socionext.com> References: <1466487167-18987-1-git-send-email-yamada.masahiro@socionext.com> <1466487167-18987-6-git-send-email-yamada.masahiro@socionext.com> Message-ID: <576A277F.1060107@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Masahiro, Am 21.06.2016 um 07:32 schrieb Masahiro Yamada: > Factor out the same code from the callees to the caller. > > Signed-off-by: Masahiro Yamada > --- > > common/autoboot.c | 21 ++++++++++----------- > 1 file changed, 10 insertions(+), 11 deletions(-) Thanks! Reviewed-by: Heiko Schocher bye, Heiko > > diff --git a/common/autoboot.c b/common/autoboot.c > index eb31c88..2d19104 100644 > --- a/common/autoboot.c > +++ b/common/autoboot.c > @@ -202,11 +202,6 @@ static int __abortboot(int bootdelay) > if (!abort) > debug_bootkeys("key timeout\n"); > > -#ifdef CONFIG_SILENT_CONSOLE > - if (abort) > - gd->flags &= ~GD_FLG_SILENT; > -#endif > - > return abort; > } > > @@ -263,18 +258,22 @@ static int __abortboot(int bootdelay) > > putc('\n'); > > -#ifdef CONFIG_SILENT_CONSOLE > - if (abort) > - gd->flags &= ~GD_FLG_SILENT; > -#endif > - > return abort; > } > # endif /* CONFIG_AUTOBOOT_KEYED */ > > static int abortboot(int bootdelay) > { > - return __abortboot(bootdelay); > + int ret; > + > + ret = __abortboot(bootdelay); > + > +#ifdef CONFIG_SILENT_CONSOLE > + if (abort) > + gd->flags &= ~GD_FLG_SILENT; > +#endif > + > + return ret; > } > > static void process_fdt_options(const void *blob) > -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany