mirrors/qemu
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

hw/display/exynos4210_fimd: Use qemu_log_mask(GUEST_ERROR)

Browse Source 
Replace DPRINT_ERROR() by qemu_log_mask(GUEST_ERROR).

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-id: 20200526062252.19852-13-f4bug@amsat.org
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
This commit is contained in:
Philippe Mathieu-Daudé 2020-05-26 08:22:50 +02:00 committed by Gerd Hoffmann
parent becce5e90a
commit b3caeaf2c8
1 changed files with 29 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

44
hw/display/exynos4210_fimd.c
View File

@ -31,6 +31,7 @@
#include "ui/pixel_ops.h" #include "ui/pixel_ops.h"
#include "qemu/bswap.h" #include "qemu/bswap.h"
#include "qemu/module.h" #include "qemu/module.h"
#include "qemu/log.h"
/* Debug messages configuration */ /* Debug messages configuration */
#define EXYNOS4210_FIMD_DEBUG 0 #define EXYNOS4210_FIMD_DEBUG 0
@ -39,20 +40,15 @@
#if EXYNOS4210_FIMD_DEBUG == 0 #if EXYNOS4210_FIMD_DEBUG == 0
#define DPRINT_L1(fmt, args...) do { } while (0) #define DPRINT_L1(fmt, args...) do { } while (0)
#define DPRINT_L2(fmt, args...) do { } while (0) #define DPRINT_L2(fmt, args...) do { } while (0)
#define DPRINT_ERROR(fmt, args...) do { } while (0)
#elif EXYNOS4210_FIMD_DEBUG == 1 #elif EXYNOS4210_FIMD_DEBUG == 1
#define DPRINT_L1(fmt, args...) \ #define DPRINT_L1(fmt, args...) \
do {fprintf(stderr, "QEMU FIMD: "fmt, ## args); } while (0) do {fprintf(stderr, "QEMU FIMD: "fmt, ## args); } while (0)
#define DPRINT_L2(fmt, args...) do { } while (0) #define DPRINT_L2(fmt, args...) do { } while (0)
#define DPRINT_ERROR(fmt, args...) \
do {fprintf(stderr, "QEMU FIMD ERROR: "fmt, ## args); } while (0)
#else #else
#define DPRINT_L1(fmt, args...) \ #define DPRINT_L1(fmt, args...) \
do {fprintf(stderr, "QEMU FIMD: "fmt, ## args); } while (0) do {fprintf(stderr, "QEMU FIMD: "fmt, ## args); } while (0)
#define DPRINT_L2(fmt, args...) \ #define DPRINT_L2(fmt, args...) \
do {fprintf(stderr, "QEMU FIMD: "fmt, ## args); } while (0) do {fprintf(stderr, "QEMU FIMD: "fmt, ## args); } while (0)
#define DPRINT_ERROR(fmt, args...) \
do {fprintf(stderr, "QEMU FIMD ERROR: "fmt, ## args); } while (0)
#endif #endif
#if EXYNOS4210_FIMD_MODE_TRACE == 0 #if EXYNOS4210_FIMD_MODE_TRACE == 0
@ -1108,7 +1104,7 @@ static inline int fimd_get_buffer_id(Exynos4210fimdWindow *w)
case FIMD_WINCON_BUF2_STAT: case FIMD_WINCON_BUF2_STAT:
return 2; return 2;
default: default:
DPRINT_ERROR("Non-existent buffer index\n"); qemu_log_mask(LOG_GUEST_ERROR, "FIMD: Non-existent buffer index\n");
return 0; return 0;
} }
} }
@ -1160,19 +1156,23 @@ static void fimd_update_memory_section(Exynos4210fimdState *s, unsigned win)
if (int128_get64(w->mem_section.size) != w->fb_len || if (int128_get64(w->mem_section.size) != w->fb_len ||
!memory_region_is_ram(w->mem_section.mr)) { !memory_region_is_ram(w->mem_section.mr)) {
DPRINT_ERROR("Failed to find window %u framebuffer region\n", win); qemu_log_mask(LOG_GUEST_ERROR,
"FIMD: Failed to find window %u framebuffer region\n",
win);
goto error_return; goto error_return;
} }
w->host_fb_addr = cpu_physical_memory_map(fb_start_addr, &fb_mapped_len, w->host_fb_addr = cpu_physical_memory_map(fb_start_addr, &fb_mapped_len,
false); false);
if (!w->host_fb_addr) { if (!w->host_fb_addr) {
DPRINT_ERROR("Failed to map window %u framebuffer\n", win); qemu_log_mask(LOG_GUEST_ERROR,
"FIMD: Failed to map window %u framebuffer\n", win);
goto error_return; goto error_return;
} }
if (fb_mapped_len != w->fb_len) { if (fb_mapped_len != w->fb_len) {
DPRINT_ERROR("Window %u mapped framebuffer length is less then " qemu_log_mask(LOG_GUEST_ERROR,
"FIMD: Window %u mapped framebuffer length is less than "
"expected\n", win); "expected\n", win);
cpu_physical_memory_unmap(w->host_fb_addr, fb_mapped_len, 0, 0); cpu_physical_memory_unmap(w->host_fb_addr, fb_mapped_len, 0, 0);
goto error_return; goto error_return;
@ -1490,7 +1490,9 @@ static void exynos4210_fimd_write(void *opaque, hwaddr offset,
break; break;
case 3: case 3:
if (w != 1 && w != 2) { if (w != 1 && w != 2) {
DPRINT_ERROR("Bad write offset 0x%08x\n", offset); qemu_log_mask(LOG_GUEST_ERROR,
"FIMD: Bad write offset 0x%08"HWADDR_PRIx"\n",
offset);
return; return;
} }
s->window[w].osdsize = val; s->window[w].osdsize = val;
@ -1624,7 +1626,9 @@ static void exynos4210_fimd_write(void *opaque, hwaddr offset,
break; break;
case FIMD_VIDW0ADD0_B2 ... FIMD_VIDW4ADD0_B2: case FIMD_VIDW0ADD0_B2 ... FIMD_VIDW4ADD0_B2:
if (offset & 0x0004) { if (offset & 0x0004) {
DPRINT_ERROR("bad write offset 0x%08x\n", offset); qemu_log_mask(LOG_GUEST_ERROR,
"FIMD: bad write offset 0x%08"HWADDR_PRIx"\n",
offset);
break; break;
} }
w = (offset - FIMD_VIDW0ADD0_B2) >> 3; w = (offset - FIMD_VIDW0ADD0_B2) >> 3;
@ -1638,14 +1642,18 @@ static void exynos4210_fimd_write(void *opaque, hwaddr offset,
break; break;
case FIMD_SHD_ADD0_START ... FIMD_SHD_ADD0_END: case FIMD_SHD_ADD0_START ... FIMD_SHD_ADD0_END:
if (offset & 0x0004) { if (offset & 0x0004) {
DPRINT_ERROR("bad write offset 0x%08x\n", offset); qemu_log_mask(LOG_GUEST_ERROR,
"FIMD: bad write offset 0x%08"HWADDR_PRIx"\n",
offset);
break; break;
} }
s->window[(offset - FIMD_SHD_ADD0_START) >> 3].shadow_buf_start = val; s->window[(offset - FIMD_SHD_ADD0_START) >> 3].shadow_buf_start = val;
break; break;
case FIMD_SHD_ADD1_START ... FIMD_SHD_ADD1_END: case FIMD_SHD_ADD1_START ... FIMD_SHD_ADD1_END:
if (offset & 0x0004) { if (offset & 0x0004) {
DPRINT_ERROR("bad write offset 0x%08x\n", offset); qemu_log_mask(LOG_GUEST_ERROR,
"FIMD: bad write offset 0x%08"HWADDR_PRIx"\n",
offset);
break; break;
} }
s->window[(offset - FIMD_SHD_ADD1_START) >> 3].shadow_buf_end = val; s->window[(offset - FIMD_SHD_ADD1_START) >> 3].shadow_buf_end = val;
@ -1665,7 +1673,8 @@ static void exynos4210_fimd_write(void *opaque, hwaddr offset,
s->window[w].palette[i] = val; s->window[w].palette[i] = val;
break; break;
default: default:
DPRINT_ERROR("bad write offset 0x%08x\n", offset); qemu_log_mask(LOG_GUEST_ERROR,
"FIMD: bad write offset 0x%08"HWADDR_PRIx"\n", offset);
break; break;
} }
} }
@ -1715,7 +1724,9 @@ static uint64_t exynos4210_fimd_read(void *opaque, hwaddr offset,
break; break;
case 3: case 3:
if (w != 1 && w != 2) { if (w != 1 && w != 2) {
DPRINT_ERROR("bad read offset 0x%08x\n", offset); qemu_log_mask(LOG_GUEST_ERROR,
"FIMD: bad read offset 0x%08"HWADDR_PRIx"\n",
offset);
return 0xBAADBAAD; return 0xBAADBAAD;
} }
ret = s->window[w].osdsize; ret = s->window[w].osdsize;
@ -1809,7 +1820,8 @@ static uint64_t exynos4210_fimd_read(void *opaque, hwaddr offset,
return s->window[w].palette[i]; return s->window[w].palette[i];
} }
DPRINT_ERROR("bad read offset 0x%08x\n", offset); qemu_log_mask(LOG_GUEST_ERROR,
"FIMD: bad read offset 0x%08"HWADDR_PRIx"\n", offset);
return 0xBAADBAAD; return 0xBAADBAAD;
} }