apb: rename APB functions to use sabre prefix

As hinted in the comment at the top of the file, the naming convention for the
APB types/QOM functions isn't correct. As a starting point we can at least
rename the APB type and related functions to improve the readability of apb.c.

Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Acked-by: Artyom Tarasenko <atar4qemu@gmail.com>
master
Mark Cave-Ayland 2018-01-21 08:59:45 +00:00
parent 90302adaba
commit fe984c7d0c
2 changed files with 54 additions and 56 deletions

View File

@ -70,7 +70,7 @@ do { printf("APB: " fmt , ## __VA_ARGS__); } while (0)
#define NO_IRQ_REQUEST (MAX_IVEC + 1) #define NO_IRQ_REQUEST (MAX_IVEC + 1)
static inline void pbm_set_request(APBState *s, unsigned int irq_num) static inline void sabre_set_request(APBState *s, unsigned int irq_num)
{ {
APB_DPRINTF("%s: request irq %d\n", __func__, irq_num); APB_DPRINTF("%s: request irq %d\n", __func__, irq_num);
@ -78,14 +78,13 @@ static inline void pbm_set_request(APBState *s, unsigned int irq_num)
qemu_set_irq(s->ivec_irqs[irq_num], 1); qemu_set_irq(s->ivec_irqs[irq_num], 1);
} }
static inline void pbm_check_irqs(APBState *s) static inline void sabre_check_irqs(APBState *s)
{ {
unsigned int i; unsigned int i;
/* Previous request is not acknowledged, resubmit */ /* Previous request is not acknowledged, resubmit */
if (s->irq_request != NO_IRQ_REQUEST) { if (s->irq_request != NO_IRQ_REQUEST) {
pbm_set_request(s, s->irq_request); sabre_set_request(s, s->irq_request);
return; return;
} }
/* no request pending */ /* no request pending */
@ -95,7 +94,7 @@ static inline void pbm_check_irqs(APBState *s)
for (i = 0; i < 32; i++) { for (i = 0; i < 32; i++) {
if (s->pci_irq_in & (1ULL << i)) { if (s->pci_irq_in & (1ULL << i)) {
if (s->pci_irq_map[i >> 2] & PBM_PCI_IMR_ENABLED) { if (s->pci_irq_map[i >> 2] & PBM_PCI_IMR_ENABLED) {
pbm_set_request(s, i); sabre_set_request(s, i);
return; return;
} }
} }
@ -103,28 +102,28 @@ static inline void pbm_check_irqs(APBState *s)
for (i = 32; i < 64; i++) { for (i = 32; i < 64; i++) {
if (s->pci_irq_in & (1ULL << i)) { if (s->pci_irq_in & (1ULL << i)) {
if (s->obio_irq_map[i - 32] & PBM_PCI_IMR_ENABLED) { if (s->obio_irq_map[i - 32] & PBM_PCI_IMR_ENABLED) {
pbm_set_request(s, i); sabre_set_request(s, i);
break; break;
} }
} }
} }
} }
static inline void pbm_clear_request(APBState *s, unsigned int irq_num) static inline void sabre_clear_request(APBState *s, unsigned int irq_num)
{ {
APB_DPRINTF("%s: clear request irq %d\n", __func__, irq_num); APB_DPRINTF("%s: clear request irq %d\n", __func__, irq_num);
qemu_set_irq(s->ivec_irqs[irq_num], 0); qemu_set_irq(s->ivec_irqs[irq_num], 0);
s->irq_request = NO_IRQ_REQUEST; s->irq_request = NO_IRQ_REQUEST;
} }
static AddressSpace *pbm_pci_dma_iommu(PCIBus *bus, void *opaque, int devfn) static AddressSpace *sabre_pci_dma_iommu(PCIBus *bus, void *opaque, int devfn)
{ {
IOMMUState *is = opaque; IOMMUState *is = opaque;
return &is->iommu_as; return &is->iommu_as;
} }
static void apb_config_writel (void *opaque, hwaddr addr, static void sabre_config_write(void *opaque, hwaddr addr,
uint64_t val, unsigned size) uint64_t val, unsigned size)
{ {
APBState *s = opaque; APBState *s = opaque;
@ -141,9 +140,9 @@ static void apb_config_writel (void *opaque, hwaddr addr,
s->pci_irq_map[ino] &= PBM_PCI_IMR_MASK; s->pci_irq_map[ino] &= PBM_PCI_IMR_MASK;
s->pci_irq_map[ino] |= val & ~PBM_PCI_IMR_MASK; s->pci_irq_map[ino] |= val & ~PBM_PCI_IMR_MASK;
if ((s->irq_request == ino) && !(val & ~PBM_PCI_IMR_MASK)) { if ((s->irq_request == ino) && !(val & ~PBM_PCI_IMR_MASK)) {
pbm_clear_request(s, ino); sabre_clear_request(s, ino);
} }
pbm_check_irqs(s); sabre_check_irqs(s);
} }
break; break;
case 0x1000 ... 0x107f: /* OBIO interrupt control */ case 0x1000 ... 0x107f: /* OBIO interrupt control */
@ -153,17 +152,17 @@ static void apb_config_writel (void *opaque, hwaddr addr,
s->obio_irq_map[ino] |= val & ~PBM_PCI_IMR_MASK; s->obio_irq_map[ino] |= val & ~PBM_PCI_IMR_MASK;
if ((s->irq_request == (ino | 0x20)) if ((s->irq_request == (ino | 0x20))
&& !(val & ~PBM_PCI_IMR_MASK)) { && !(val & ~PBM_PCI_IMR_MASK)) {
pbm_clear_request(s, ino | 0x20); sabre_clear_request(s, ino | 0x20);
} }
pbm_check_irqs(s); sabre_check_irqs(s);
} }
break; break;
case 0x1400 ... 0x14ff: /* PCI interrupt clear */ case 0x1400 ... 0x14ff: /* PCI interrupt clear */
if (addr & 4) { if (addr & 4) {
unsigned int ino = (addr & 0xff) >> 5; unsigned int ino = (addr & 0xff) >> 5;
if ((s->irq_request / 4) == ino) { if ((s->irq_request / 4) == ino) {
pbm_clear_request(s, s->irq_request); sabre_clear_request(s, s->irq_request);
pbm_check_irqs(s); sabre_check_irqs(s);
} }
} }
break; break;
@ -171,8 +170,8 @@ static void apb_config_writel (void *opaque, hwaddr addr,
if (addr & 4) { if (addr & 4) {
unsigned int ino = ((addr & 0xff) >> 3) | 0x20; unsigned int ino = ((addr & 0xff) >> 3) | 0x20;
if (s->irq_request == ino) { if (s->irq_request == ino) {
pbm_clear_request(s, ino); sabre_clear_request(s, ino);
pbm_check_irqs(s); sabre_check_irqs(s);
} }
} }
break; break;
@ -202,7 +201,7 @@ static void apb_config_writel (void *opaque, hwaddr addr,
} }
} }
static uint64_t apb_config_readl (void *opaque, static uint64_t sabre_config_read(void *opaque,
hwaddr addr, unsigned size) hwaddr addr, unsigned size)
{ {
APBState *s = opaque; APBState *s = opaque;
@ -258,14 +257,14 @@ static uint64_t apb_config_readl (void *opaque,
return val; return val;
} }
static const MemoryRegionOps apb_config_ops = { static const MemoryRegionOps sabre_config_ops = {
.read = apb_config_readl, .read = sabre_config_read,
.write = apb_config_writel, .write = sabre_config_write,
.endianness = DEVICE_BIG_ENDIAN, .endianness = DEVICE_BIG_ENDIAN,
}; };
static void apb_pci_config_write(void *opaque, hwaddr addr, static void sabre_pci_config_write(void *opaque, hwaddr addr,
uint64_t val, unsigned size) uint64_t val, unsigned size)
{ {
APBState *s = opaque; APBState *s = opaque;
PCIHostState *phb = PCI_HOST_BRIDGE(s); PCIHostState *phb = PCI_HOST_BRIDGE(s);
@ -274,8 +273,8 @@ static void apb_pci_config_write(void *opaque, hwaddr addr,
pci_data_write(phb->bus, addr, val, size); pci_data_write(phb->bus, addr, val, size);
} }
static uint64_t apb_pci_config_read(void *opaque, hwaddr addr, static uint64_t sabre_pci_config_read(void *opaque, hwaddr addr,
unsigned size) unsigned size)
{ {
uint32_t ret; uint32_t ret;
APBState *s = opaque; APBState *s = opaque;
@ -286,8 +285,8 @@ static uint64_t apb_pci_config_read(void *opaque, hwaddr addr,
return ret; return ret;
} }
/* The APB host has an IRQ line for each IRQ line of each slot. */ /* The sabre host has an IRQ line for each IRQ line of each slot. */
static int pci_apb_map_irq(PCIDevice *pci_dev, int irq_num) static int pci_sabre_map_irq(PCIDevice *pci_dev, int irq_num)
{ {
/* Return the irq as swizzled by the PBM */ /* Return the irq as swizzled by the PBM */
return irq_num; return irq_num;
@ -316,7 +315,7 @@ static int pci_simbaB_map_irq(PCIDevice *pci_dev, int irq_num)
return (0x10 + (PCI_SLOT(pci_dev->devfn) << 2) + irq_num) & 0x1f; return (0x10 + (PCI_SLOT(pci_dev->devfn) << 2) + irq_num) & 0x1f;
} }
static void pci_apb_set_irq(void *opaque, int irq_num, int level) static void pci_sabre_set_irq(void *opaque, int irq_num, int level)
{ {
APBState *s = opaque; APBState *s = opaque;
@ -326,7 +325,7 @@ static void pci_apb_set_irq(void *opaque, int irq_num, int level)
if (level) { if (level) {
s->pci_irq_in |= 1ULL << irq_num; s->pci_irq_in |= 1ULL << irq_num;
if (s->pci_irq_map[irq_num >> 2] & PBM_PCI_IMR_ENABLED) { if (s->pci_irq_map[irq_num >> 2] & PBM_PCI_IMR_ENABLED) {
pbm_set_request(s, irq_num); sabre_set_request(s, irq_num);
} }
} else { } else {
s->pci_irq_in &= ~(1ULL << irq_num); s->pci_irq_in &= ~(1ULL << irq_num);
@ -338,7 +337,7 @@ static void pci_apb_set_irq(void *opaque, int irq_num, int level)
s->pci_irq_in |= 1ULL << irq_num; s->pci_irq_in |= 1ULL << irq_num;
if ((s->irq_request == NO_IRQ_REQUEST) if ((s->irq_request == NO_IRQ_REQUEST)
&& (s->obio_irq_map[irq_num - 32] & PBM_PCI_IMR_ENABLED)) { && (s->obio_irq_map[irq_num - 32] & PBM_PCI_IMR_ENABLED)) {
pbm_set_request(s, irq_num); sabre_set_request(s, irq_num);
} }
} else { } else {
s->pci_irq_in &= ~(1ULL << irq_num); s->pci_irq_in &= ~(1ULL << irq_num);
@ -346,7 +345,7 @@ static void pci_apb_set_irq(void *opaque, int irq_num, int level)
} }
} }
static void pci_pbm_reset(DeviceState *d) static void sabre_reset(DeviceState *d)
{ {
APBState *s = APB_DEVICE(d); APBState *s = APB_DEVICE(d);
PCIDevice *pci_dev; PCIDevice *pci_dev;
@ -379,12 +378,12 @@ static void pci_pbm_reset(DeviceState *d)
} }
static const MemoryRegionOps pci_config_ops = { static const MemoryRegionOps pci_config_ops = {
.read = apb_pci_config_read, .read = sabre_pci_config_read,
.write = apb_pci_config_write, .write = sabre_pci_config_write,
.endianness = DEVICE_LITTLE_ENDIAN, .endianness = DEVICE_LITTLE_ENDIAN,
}; };
static void pci_pbm_realize(DeviceState *dev, Error **errp) static void sabre_realize(DeviceState *dev, Error **errp)
{ {
APBState *s = APB_DEVICE(dev); APBState *s = APB_DEVICE(dev);
PCIHostState *phb = PCI_HOST_BRIDGE(dev); PCIHostState *phb = PCI_HOST_BRIDGE(dev);
@ -403,17 +402,17 @@ static void pci_pbm_realize(DeviceState *dev, Error **errp)
&s->pci_mmio); &s->pci_mmio);
phb->bus = pci_register_root_bus(dev, "pci", phb->bus = pci_register_root_bus(dev, "pci",
pci_apb_set_irq, pci_apb_map_irq, s, pci_sabre_set_irq, pci_sabre_map_irq, s,
&s->pci_mmio, &s->pci_mmio,
&s->pci_ioport, &s->pci_ioport,
0, 32, TYPE_PCI_BUS); 0, 32, TYPE_PCI_BUS);
pci_create_simple(phb->bus, 0, "pbm-pci"); pci_create_simple(phb->bus, 0, "pbm-pci");
/* APB IOMMU */ /* IOMMU */
memory_region_add_subregion_overlap(&s->apb_config, 0x200, memory_region_add_subregion_overlap(&s->apb_config, 0x200,
sysbus_mmio_get_region(SYS_BUS_DEVICE(s->iommu), 0), 1); sysbus_mmio_get_region(SYS_BUS_DEVICE(s->iommu), 0), 1);
pci_setup_iommu(phb->bus, pbm_pci_dma_iommu, s->iommu); pci_setup_iommu(phb->bus, sabre_pci_dma_iommu, s->iommu);
/* APB secondary busses */ /* APB secondary busses */
pci_dev = pci_create_multifunction(phb->bus, PCI_DEVFN(1, 0), true, pci_dev = pci_create_multifunction(phb->bus, PCI_DEVFN(1, 0), true,
@ -429,7 +428,7 @@ static void pci_pbm_realize(DeviceState *dev, Error **errp)
qdev_init_nofail(&pci_dev->qdev); qdev_init_nofail(&pci_dev->qdev);
} }
static void pci_pbm_init(Object *obj) static void sabre_init(Object *obj)
{ {
APBState *s = APB_DEVICE(obj); APBState *s = APB_DEVICE(obj);
SysBusDevice *sbd = SYS_BUS_DEVICE(obj); SysBusDevice *sbd = SYS_BUS_DEVICE(obj);
@ -444,7 +443,7 @@ static void pci_pbm_init(Object *obj)
for (i = 0; i < 32; i++) { for (i = 0; i < 32; i++) {
s->obio_irq_map[i] = ((0x1f << 6) | 0x20) + i; s->obio_irq_map[i] = ((0x1f << 6) | 0x20) + i;
} }
qdev_init_gpio_in_named(DEVICE(s), pci_apb_set_irq, "pbm-irq", MAX_IVEC); qdev_init_gpio_in_named(DEVICE(s), pci_sabre_set_irq, "pbm-irq", MAX_IVEC);
qdev_init_gpio_out_named(DEVICE(s), s->ivec_irqs, "ivec-irq", MAX_IVEC); qdev_init_gpio_out_named(DEVICE(s), s->ivec_irqs, "ivec-irq", MAX_IVEC);
s->irq_request = NO_IRQ_REQUEST; s->irq_request = NO_IRQ_REQUEST;
s->pci_irq_in = 0ULL; s->pci_irq_in = 0ULL;
@ -456,7 +455,7 @@ static void pci_pbm_init(Object *obj)
0, NULL); 0, NULL);
/* apb_config */ /* apb_config */
memory_region_init_io(&s->apb_config, OBJECT(s), &apb_config_ops, s, memory_region_init_io(&s->apb_config, OBJECT(s), &sabre_config_ops, s,
"apb-config", 0x10000); "apb-config", 0x10000);
/* at region 0 */ /* at region 0 */
sysbus_init_mmio(sbd, &s->apb_config); sysbus_init_mmio(sbd, &s->apb_config);
@ -473,7 +472,7 @@ static void pci_pbm_init(Object *obj)
sysbus_init_mmio(sbd, &s->pci_ioport); sysbus_init_mmio(sbd, &s->pci_ioport);
} }
static void pbm_pci_host_realize(PCIDevice *d, Error **errp) static void sabre_pci_host_realize(PCIDevice *d, Error **errp)
{ {
pci_set_word(d->config + PCI_COMMAND, pci_set_word(d->config + PCI_COMMAND,
PCI_COMMAND_MEMORY | PCI_COMMAND_MASTER); PCI_COMMAND_MEMORY | PCI_COMMAND_MASTER);
@ -482,12 +481,12 @@ static void pbm_pci_host_realize(PCIDevice *d, Error **errp)
PCI_STATUS_DEVSEL_MEDIUM); PCI_STATUS_DEVSEL_MEDIUM);
} }
static void pbm_pci_host_class_init(ObjectClass *klass, void *data) static void sabre_pci_host_class_init(ObjectClass *klass, void *data)
{ {
PCIDeviceClass *k = PCI_DEVICE_CLASS(klass); PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
DeviceClass *dc = DEVICE_CLASS(klass); DeviceClass *dc = DEVICE_CLASS(klass);
k->realize = pbm_pci_host_realize; k->realize = sabre_pci_host_realize;
k->vendor_id = PCI_VENDOR_ID_SUN; k->vendor_id = PCI_VENDOR_ID_SUN;
k->device_id = PCI_DEVICE_ID_SUN_SABRE; k->device_id = PCI_DEVICE_ID_SUN_SABRE;
k->class_id = PCI_CLASS_BRIDGE_HOST; k->class_id = PCI_CLASS_BRIDGE_HOST;
@ -502,41 +501,41 @@ static const TypeInfo pbm_pci_host_info = {
.name = "pbm-pci", .name = "pbm-pci",
.parent = TYPE_PCI_DEVICE, .parent = TYPE_PCI_DEVICE,
.instance_size = sizeof(PCIDevice), .instance_size = sizeof(PCIDevice),
.class_init = pbm_pci_host_class_init, .class_init = sabre_pci_host_class_init,
.interfaces = (InterfaceInfo[]) { .interfaces = (InterfaceInfo[]) {
{ INTERFACE_CONVENTIONAL_PCI_DEVICE }, { INTERFACE_CONVENTIONAL_PCI_DEVICE },
{ }, { },
}, },
}; };
static Property pbm_pci_host_properties[] = { static Property sabre_properties[] = {
DEFINE_PROP_UINT64("special-base", APBState, special_base, 0), DEFINE_PROP_UINT64("special-base", APBState, special_base, 0),
DEFINE_PROP_UINT64("mem-base", APBState, mem_base, 0), DEFINE_PROP_UINT64("mem-base", APBState, mem_base, 0),
DEFINE_PROP_END_OF_LIST(), DEFINE_PROP_END_OF_LIST(),
}; };
static void pbm_host_class_init(ObjectClass *klass, void *data) static void sabre_class_init(ObjectClass *klass, void *data)
{ {
DeviceClass *dc = DEVICE_CLASS(klass); DeviceClass *dc = DEVICE_CLASS(klass);
dc->realize = pci_pbm_realize; dc->realize = sabre_realize;
dc->reset = pci_pbm_reset; dc->reset = sabre_reset;
dc->props = pbm_pci_host_properties; dc->props = sabre_properties;
set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories); set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
} }
static const TypeInfo pbm_host_info = { static const TypeInfo sabre_info = {
.name = TYPE_APB, .name = TYPE_APB,
.parent = TYPE_PCI_HOST_BRIDGE, .parent = TYPE_PCI_HOST_BRIDGE,
.instance_size = sizeof(APBState), .instance_size = sizeof(APBState),
.instance_init = pci_pbm_init, .instance_init = sabre_init,
.class_init = pbm_host_class_init, .class_init = sabre_class_init,
}; };
static void pbm_register_types(void) static void sabre_register_types(void)
{ {
type_register_static(&pbm_host_info); type_register_static(&sabre_info);
type_register_static(&pbm_pci_host_info); type_register_static(&pbm_pci_host_info);
} }
type_init(pbm_register_types) type_init(sabre_register_types)

View File

@ -15,7 +15,6 @@
#define OBIO_SER_IRQ 0x2b #define OBIO_SER_IRQ 0x2b
#define TYPE_APB "pbm" #define TYPE_APB "pbm"
#define APB_DEVICE(obj) \ #define APB_DEVICE(obj) \
OBJECT_CHECK(APBState, (obj), TYPE_APB) OBJECT_CHECK(APBState, (obj), TYPE_APB)