Skip to content

Commit

Permalink
target/unicore32: Prefer qemu_semihosting_log_out() over curses
Browse files Browse the repository at this point in the history
Use the common API for semihosting logging.

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20200603123754.19059-4-f4bug@amsat.org>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
  • Loading branch information
philmd authored and vivier committed Jun 9, 2020
1 parent 7a7b663 commit c7a856b
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 52 deletions.
1 change: 1 addition & 0 deletions default-configs/unicore32-softmmu.mak
Expand Up @@ -3,3 +3,4 @@
# Boards:
#
CONFIG_PUV3=y
CONFIG_SEMIHOSTING=y
57 changes: 5 additions & 52 deletions target/unicore32/helper.c
Expand Up @@ -14,9 +14,7 @@
#include "cpu.h"
#include "exec/exec-all.h"
#include "exec/helper-proto.h"
#ifndef CONFIG_USER_ONLY
#include "ui/console.h"
#endif
#include "hw/semihosting/console.h"

#undef DEBUG_UC32

Expand Down Expand Up @@ -161,58 +159,13 @@ uint32_t helper_cp0_get(CPUUniCore32State *env, uint32_t creg, uint32_t cop)
return 0;
}

#ifdef CONFIG_CURSES

/* KEY_EVENT is defined in wincon.h and in curses.h. Avoid redefinition. */
#undef KEY_EVENT
#include <curses.h>
#undef KEY_EVENT

/*
* FIXME:
* 1. curses windows will be blank when switching back
* 2. backspace is not handled yet
*/
static void putc_on_screen(unsigned char ch)
void helper_cp1_putc(target_ulong regval)
{
static WINDOW *localwin;
static int init;

if (!init) {
/* Assume 80 * 30 screen to minimize the implementation */
localwin = newwin(30, 80, 0, 0);
scrollok(localwin, TRUE);
init = TRUE;
}
const char c = regval;

if (isprint(ch)) {
wprintw(localwin, "%c", ch);
} else {
switch (ch) {
case '\n':
wprintw(localwin, "%c", ch);
break;
case '\r':
/* If '\r' is put before '\n', the curses window will destroy the
* last print line. And meanwhile, '\n' implifies '\r' inside. */
break;
default: /* Not handled, so just print it hex code */
wprintw(localwin, "-- 0x%x --", ch);
}
}

wrefresh(localwin);
}
#else
#define putc_on_screen(c) do { } while (0)
#endif

void helper_cp1_putc(target_ulong x)
{
putc_on_screen((unsigned char)x); /* Output to screen */
DPRINTF("%c", x); /* Output to stdout */
qemu_semihosting_log_out(&c, sizeof(c));
}
#endif
#endif /* !CONFIG_USER_ONLY */

bool uc32_cpu_exec_interrupt(CPUState *cs, int interrupt_request)
{
Expand Down

0 comments on commit c7a856b

Please sign in to comment.