hw/pl181: Use LOG_UNIMP and LOG_GUEST_ERROR

Rather than a mix of direct printing to stderr and aborting
via hw_error(), use LOG_UNIMP and LOG_GUEST_ERROR.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
This commit is contained in:
Peter Maydell 2012-10-18 14:11:37 +01:00 committed by Blue Swirl
parent 051c02b6c9
commit 9351d70829

View File

@ -352,7 +352,7 @@ static uint64_t pl181_read(void *opaque, target_phys_addr_t offset,
case 0xa0: case 0xa4: case 0xa8: case 0xac: case 0xa0: case 0xa4: case 0xa8: case 0xac:
case 0xb0: case 0xb4: case 0xb8: case 0xbc: case 0xb0: case 0xb4: case 0xb8: case 0xbc:
if (s->fifo_len == 0) { if (s->fifo_len == 0) {
fprintf(stderr, "pl181: Unexpected FIFO read\n"); qemu_log_mask(LOG_GUEST_ERROR, "pl181: Unexpected FIFO read\n");
return 0; return 0;
} else { } else {
uint32_t value; uint32_t value;
@ -363,7 +363,8 @@ static uint64_t pl181_read(void *opaque, target_phys_addr_t offset,
return value; return value;
} }
default: default:
hw_error("pl181_read: Bad offset %x\n", (int)offset); qemu_log_mask(LOG_GUEST_ERROR,
"pl181_read: Bad offset %x\n", (int)offset);
return 0; return 0;
} }
} }
@ -387,11 +388,11 @@ static void pl181_write(void *opaque, target_phys_addr_t offset,
s->cmd = value; s->cmd = value;
if (s->cmd & PL181_CMD_ENABLE) { if (s->cmd & PL181_CMD_ENABLE) {
if (s->cmd & PL181_CMD_INTERRUPT) { if (s->cmd & PL181_CMD_INTERRUPT) {
fprintf(stderr, "pl181: Interrupt mode not implemented\n"); qemu_log_mask(LOG_UNIMP,
abort(); "pl181: Interrupt mode not implemented\n");
} if (s->cmd & PL181_CMD_PENDING) { } if (s->cmd & PL181_CMD_PENDING) {
fprintf(stderr, "pl181: Pending commands not implemented\n"); qemu_log_mask(LOG_UNIMP,
abort(); "pl181: Pending commands not implemented\n");
} else { } else {
pl181_send_command(s); pl181_send_command(s);
pl181_fifo_run(s); pl181_fifo_run(s);
@ -427,14 +428,15 @@ static void pl181_write(void *opaque, target_phys_addr_t offset,
case 0xa0: case 0xa4: case 0xa8: case 0xac: case 0xa0: case 0xa4: case 0xa8: case 0xac:
case 0xb0: case 0xb4: case 0xb8: case 0xbc: case 0xb0: case 0xb4: case 0xb8: case 0xbc:
if (s->datacnt == 0) { if (s->datacnt == 0) {
fprintf(stderr, "pl181: Unexpected FIFO write\n"); qemu_log_mask(LOG_GUEST_ERROR, "pl181: Unexpected FIFO write\n");
} else { } else {
pl181_fifo_push(s, value); pl181_fifo_push(s, value);
pl181_fifo_run(s); pl181_fifo_run(s);
} }
break; break;
default: default:
hw_error("pl181_write: Bad offset %x\n", (int)offset); qemu_log_mask(LOG_GUEST_ERROR,
"pl181_write: Bad offset %x\n", (int)offset);
} }
pl181_update(s); pl181_update(s);
} }