hw/char: Consistent function names for sifive_uart

This cleans up function names in the SiFive UART model.

Signed-off-by: Lukas Jünger <lukas.juenger@greensocs.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
Message-id: 20210616092326.59639-2-lukas.juenger@greensocs.com
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
This commit is contained in:
Lukas Jünger 2021-06-16 11:23:25 +02:00 committed by Alistair Francis
parent 79a412891f
commit 244a9fcb31
1 changed files with 24 additions and 22 deletions

View File

@ -31,7 +31,7 @@
*/ */
/* Returns the state of the IP (interrupt pending) register */ /* Returns the state of the IP (interrupt pending) register */
static uint64_t uart_ip(SiFiveUARTState *s) static uint64_t sifive_uart_ip(SiFiveUARTState *s)
{ {
uint64_t ret = 0; uint64_t ret = 0;
@ -48,7 +48,7 @@ static uint64_t uart_ip(SiFiveUARTState *s)
return ret; return ret;
} }
static void update_irq(SiFiveUARTState *s) static void sifive_uart_update_irq(SiFiveUARTState *s)
{ {
int cond = 0; int cond = 0;
if ((s->ie & SIFIVE_UART_IE_TXWM) || if ((s->ie & SIFIVE_UART_IE_TXWM) ||
@ -63,7 +63,7 @@ static void update_irq(SiFiveUARTState *s)
} }
static uint64_t static uint64_t
uart_read(void *opaque, hwaddr addr, unsigned int size) sifive_uart_read(void *opaque, hwaddr addr, unsigned int size)
{ {
SiFiveUARTState *s = opaque; SiFiveUARTState *s = opaque;
unsigned char r; unsigned char r;
@ -74,7 +74,7 @@ uart_read(void *opaque, hwaddr addr, unsigned int size)
memmove(s->rx_fifo, s->rx_fifo + 1, s->rx_fifo_len - 1); memmove(s->rx_fifo, s->rx_fifo + 1, s->rx_fifo_len - 1);
s->rx_fifo_len--; s->rx_fifo_len--;
qemu_chr_fe_accept_input(&s->chr); qemu_chr_fe_accept_input(&s->chr);
update_irq(s); sifive_uart_update_irq(s);
return r; return r;
} }
return 0x80000000; return 0x80000000;
@ -84,7 +84,7 @@ uart_read(void *opaque, hwaddr addr, unsigned int size)
case SIFIVE_UART_IE: case SIFIVE_UART_IE:
return s->ie; return s->ie;
case SIFIVE_UART_IP: case SIFIVE_UART_IP:
return uart_ip(s); return sifive_uart_ip(s);
case SIFIVE_UART_TXCTRL: case SIFIVE_UART_TXCTRL:
return s->txctrl; return s->txctrl;
case SIFIVE_UART_RXCTRL: case SIFIVE_UART_RXCTRL:
@ -99,7 +99,7 @@ uart_read(void *opaque, hwaddr addr, unsigned int size)
} }
static void static void
uart_write(void *opaque, hwaddr addr, sifive_uart_write(void *opaque, hwaddr addr,
uint64_t val64, unsigned int size) uint64_t val64, unsigned int size)
{ {
SiFiveUARTState *s = opaque; SiFiveUARTState *s = opaque;
@ -109,11 +109,11 @@ uart_write(void *opaque, hwaddr addr,
switch (addr) { switch (addr) {
case SIFIVE_UART_TXFIFO: case SIFIVE_UART_TXFIFO:
qemu_chr_fe_write(&s->chr, &ch, 1); qemu_chr_fe_write(&s->chr, &ch, 1);
update_irq(s); sifive_uart_update_irq(s);
return; return;
case SIFIVE_UART_IE: case SIFIVE_UART_IE:
s->ie = val64; s->ie = val64;
update_irq(s); sifive_uart_update_irq(s);
return; return;
case SIFIVE_UART_TXCTRL: case SIFIVE_UART_TXCTRL:
s->txctrl = val64; s->txctrl = val64;
@ -129,9 +129,9 @@ uart_write(void *opaque, hwaddr addr,
__func__, (int)addr, (int)value); __func__, (int)addr, (int)value);
} }
static const MemoryRegionOps uart_ops = { static const MemoryRegionOps sifive_uart_ops = {
.read = uart_read, .read = sifive_uart_read,
.write = uart_write, .write = sifive_uart_write,
.endianness = DEVICE_NATIVE_ENDIAN, .endianness = DEVICE_NATIVE_ENDIAN,
.valid = { .valid = {
.min_access_size = 4, .min_access_size = 4,
@ -139,7 +139,7 @@ static const MemoryRegionOps uart_ops = {
} }
}; };
static void uart_rx(void *opaque, const uint8_t *buf, int size) static void sifive_uart_rx(void *opaque, const uint8_t *buf, int size)
{ {
SiFiveUARTState *s = opaque; SiFiveUARTState *s = opaque;
@ -150,26 +150,27 @@ static void uart_rx(void *opaque, const uint8_t *buf, int size)
} }
s->rx_fifo[s->rx_fifo_len++] = *buf; s->rx_fifo[s->rx_fifo_len++] = *buf;
update_irq(s); sifive_uart_update_irq(s);
} }
static int uart_can_rx(void *opaque) static int sifive_uart_can_rx(void *opaque)
{ {
SiFiveUARTState *s = opaque; SiFiveUARTState *s = opaque;
return s->rx_fifo_len < sizeof(s->rx_fifo); return s->rx_fifo_len < sizeof(s->rx_fifo);
} }
static void uart_event(void *opaque, QEMUChrEvent event) static void sifive_uart_event(void *opaque, QEMUChrEvent event)
{ {
} }
static int uart_be_change(void *opaque) static int sifive_uart_be_change(void *opaque)
{ {
SiFiveUARTState *s = opaque; SiFiveUARTState *s = opaque;
qemu_chr_fe_set_handlers(&s->chr, uart_can_rx, uart_rx, uart_event, qemu_chr_fe_set_handlers(&s->chr, sifive_uart_can_rx, sifive_uart_rx,
uart_be_change, s, NULL, true); sifive_uart_event, sifive_uart_be_change, s,
NULL, true);
return 0; return 0;
} }
@ -183,9 +184,10 @@ SiFiveUARTState *sifive_uart_create(MemoryRegion *address_space, hwaddr base,
SiFiveUARTState *s = g_malloc0(sizeof(SiFiveUARTState)); SiFiveUARTState *s = g_malloc0(sizeof(SiFiveUARTState));
s->irq = irq; s->irq = irq;
qemu_chr_fe_init(&s->chr, chr, &error_abort); qemu_chr_fe_init(&s->chr, chr, &error_abort);
qemu_chr_fe_set_handlers(&s->chr, uart_can_rx, uart_rx, uart_event, qemu_chr_fe_set_handlers(&s->chr, sifive_uart_can_rx, sifive_uart_rx,
uart_be_change, s, NULL, true); sifive_uart_event, sifive_uart_be_change, s,
memory_region_init_io(&s->mmio, NULL, &uart_ops, s, NULL, true);
memory_region_init_io(&s->mmio, NULL, &sifive_uart_ops, s,
TYPE_SIFIVE_UART, SIFIVE_UART_MAX); TYPE_SIFIVE_UART, SIFIVE_UART_MAX);
memory_region_add_subregion(address_space, base, &s->mmio); memory_region_add_subregion(address_space, base, &s->mmio);
return s; return s;