diff mbox

[U-Boot,1/4,v2] common/console.c: Drop sandbox special-case console code

Message ID 1447683994-25930-1-git-send-email-sr@denx.de
State Accepted
Delegated to: Tom Rini
Headers show

Commit Message

Stefan Roese Nov. 16, 2015, 2:26 p.m. UTC
As done in commit da229e4e [sandbox: Drop special-case sandbox console code],
this patch drops the sandbox special-case code in vprintf() that was
missed by Simon at that time.

Signed-off-by: Stefan Roese <sr@denx.de>
Cc: Simon Glass <sjg@chromium.org>
---
v2:
- new patch

 common/console.c | 5 -----
 1 file changed, 5 deletions(-)

Comments

Simon Glass Nov. 16, 2015, 9:08 p.m. UTC | #1
On 16 November 2015 at 07:26, Stefan Roese <sr@denx.de> wrote:
> As done in commit da229e4e [sandbox: Drop special-case sandbox console code],
> this patch drops the sandbox special-case code in vprintf() that was
> missed by Simon at that time.
>
> Signed-off-by: Stefan Roese <sr@denx.de>
> Cc: Simon Glass <sjg@chromium.org>
> ---
> v2:
> - new patch
>
>  common/console.c | 5 -----
>  1 file changed, 5 deletions(-)

Acked-by: Simon Glass <sjg@chromium.org>
Tom Rini Nov. 23, 2015, 10:44 p.m. UTC | #2
On Mon, Nov 16, 2015 at 03:26:31PM +0100, Stefan Roese wrote:

> As done in commit da229e4e [sandbox: Drop special-case sandbox console code],
> this patch drops the sandbox special-case code in vprintf() that was
> missed by Simon at that time.
> 
> Signed-off-by: Stefan Roese <sr@denx.de>
> Cc: Simon Glass <sjg@chromium.org>
> Acked-by: Simon Glass <sjg@chromium.org>

Applied to u-boot/master, thanks!
diff mbox

Patch

diff --git a/common/console.c b/common/console.c
index ace206c..6fd7a10 100644
--- a/common/console.c
+++ b/common/console.c
@@ -559,11 +559,6 @@  int vprintf(const char *fmt, va_list args)
 	uint i;
 	char printbuffer[CONFIG_SYS_PBSIZE];
 
-#if defined(CONFIG_PRE_CONSOLE_BUFFER) && !defined(CONFIG_SANDBOX)
-	if (!gd->have_console)
-		return 0;
-#endif
-
 	/* For this to work, printbuffer must be larger than
 	 * anything we ever want to print.
 	 */