[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv4 04/13] sun4m: move DMA device wiring from spar
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCHv4 04/13] sun4m: move DMA device wiring from sparc32_dma_init() to sun4m_hw_init() |
Date: |
Fri, 27 Oct 2017 12:46:19 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 |
On 10/25/2017 12:59 PM, Mark Cave-Ayland wrote:
> By using the sysbus interface it is possible to wire up the esp/le devices
> to the sun4m DMA controller directly during sun4m_hw_init() instead of
> passing qemu_irqs into the sparc32_dma_init() function.
>
> This is an intermediate step to allow further reorganisation as more logic
> is moved into the relevant SPARC32 DMA devices; there will be a final
> refactoring of sparc32_dma_init() once this work is complete.
>
> Signed-off-by: Mark Cave-Ayland <address@hidden>
> Reviewed-by: Artyom Tarasenko <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> hw/sparc/sun4m.c | 29 ++++++++++++++++-------------
> 1 file changed, 16 insertions(+), 13 deletions(-)
>
> diff --git a/hw/sparc/sun4m.c b/hw/sparc/sun4m.c
> index 88a9752..4f2ed4b 100644
> --- a/hw/sparc/sun4m.c
> +++ b/hw/sparc/sun4m.c
> @@ -307,8 +307,7 @@ static void *iommu_init(hwaddr addr, uint32_t version,
> qemu_irq irq)
> return s;
> }
>
> -static void *sparc32_dma_init(hwaddr daddr, qemu_irq parent_irq,
> - void *iommu, qemu_irq *dev_irq, int is_ledma)
> +static void *sparc32_dma_init(hwaddr daddr, void *iommu, int is_ledma)
> {
> DeviceState *dev;
> SysBusDevice *s;
> @@ -317,8 +316,6 @@ static void *sparc32_dma_init(hwaddr daddr, qemu_irq
> parent_irq,
> qdev_prop_set_ptr(dev, "iommu_opaque", iommu);
> qdev_init_nofail(dev);
> s = SYS_BUS_DEVICE(dev);
> - sysbus_connect_irq(s, 0, parent_irq);
> - *dev_irq = qdev_get_gpio_in(dev, 0);
> sysbus_mmio_map(s, 0, daddr);
>
> return s;
> @@ -821,9 +818,10 @@ static void sun4m_hw_init(const struct sun4m_hwdef
> *hwdef,
> DeviceState *slavio_intctl;
> const char *cpu_model = machine->cpu_model;
> unsigned int i;
> - void *iommu, *espdma, *ledma, *nvram;
> - qemu_irq *cpu_irqs[MAX_CPUS], slavio_irq[32], slavio_cpu_irq[MAX_CPUS],
> - espdma_irq, ledma_irq;
> + void *iommu, *nvram;
> + DeviceState *espdma, *ledma;
> + SysBusDevice *sbd;
> + qemu_irq *cpu_irqs[MAX_CPUS], slavio_irq[32], slavio_cpu_irq[MAX_CPUS];
> qemu_irq esp_reset, dma_enable;
> qemu_irq fdc_tc;
> unsigned long kernel_size;
> @@ -882,11 +880,13 @@ static void sun4m_hw_init(const struct sun4m_hwdef
> *hwdef,
> empty_slot_init(hwdef->iommu_pad_base,hwdef->iommu_pad_len);
> }
>
> - espdma = sparc32_dma_init(hwdef->dma_base, slavio_irq[18],
> - iommu, &espdma_irq, 0);
> + espdma = sparc32_dma_init(hwdef->dma_base, iommu, 0);
> + sbd = SYS_BUS_DEVICE(espdma);
> + sysbus_connect_irq(sbd, 0, slavio_irq[18]);
>
> - ledma = sparc32_dma_init(hwdef->dma_base + 16ULL,
> - slavio_irq[16], iommu, &ledma_irq, 1);
> + ledma = sparc32_dma_init(hwdef->dma_base + 16ULL, iommu, 1);
> + sbd = SYS_BUS_DEVICE(ledma);
> + sysbus_connect_irq(sbd, 0, slavio_irq[16]);
>
> if (graphic_depth != 8 && graphic_depth != 24) {
> error_report("Unsupported depth: %d", graphic_depth);
> @@ -939,7 +939,8 @@ static void sun4m_hw_init(const struct sun4m_hwdef *hwdef,
> empty_slot_init(hwdef->sx_base, 0x2000);
> }
>
> - lance_init(&nd_table[0], hwdef->le_base, ledma, ledma_irq);
> + lance_init(&nd_table[0], hwdef->le_base, ledma,
> + qdev_get_gpio_in(ledma, 0));
>
> nvram = m48t59_init(slavio_irq[0], hwdef->nvram_base, 0, 0x2000, 1968,
> 8);
>
> @@ -971,7 +972,9 @@ static void sun4m_hw_init(const struct sun4m_hwdef *hwdef,
>
> esp_init(hwdef->esp_base, 2,
> espdma_memory_read, espdma_memory_write,
> - espdma, espdma_irq, &esp_reset, &dma_enable);
> + espdma,
> + qdev_get_gpio_in(espdma, 0),
> + &esp_reset, &dma_enable);
>
> qdev_connect_gpio_out(espdma, 0, esp_reset);
> qdev_connect_gpio_out(espdma, 1, dma_enable);
>
- [Qemu-devel] [PATCHv4 00/13] sun4m: sparc32_dma tidy-ups, Mark Cave-Ayland, 2017/10/25
- [Qemu-devel] [PATCHv4 05/13] sun4m_iommu: move TYPE_SUN4M_IOMMU declaration to sun4m.h, Mark Cave-Ayland, 2017/10/25
- [Qemu-devel] [PATCHv4 02/13] sparc32_dma: split esp and le into separate DMA devices, Mark Cave-Ayland, 2017/10/25
- [Qemu-devel] [PATCHv4 07/13] esp: move TYPE_ESP and SysBusESPState from esp.c to esp.h, Mark Cave-Ayland, 2017/10/25
- [Qemu-devel] [PATCHv4 01/13] sparc32_dma: rename SPARC32_DMA type to SPARC32_DMA_DEVICE, Mark Cave-Ayland, 2017/10/25
- [Qemu-devel] [PATCHv4 04/13] sun4m: move DMA device wiring from sparc32_dma_init() to sun4m_hw_init(), Mark Cave-Ayland, 2017/10/25
- Re: [Qemu-devel] [PATCHv4 04/13] sun4m: move DMA device wiring from sparc32_dma_init() to sun4m_hw_init(),
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PATCHv4 09/13] lance: move TYPE_LANCE and SysBusPCNetState from lance.c to lance.h, Mark Cave-Ayland, 2017/10/25
- Re: [Qemu-devel] [PATCHv4 09/13] lance: move TYPE_LANCE and SysBusPCNetState from lance.c to lance.h, Peter Maydell, 2017/10/25
- Re: [Qemu-devel] [PATCHv4 09/13] lance: move TYPE_LANCE and SysBusPCNetState from lance.c to lance.h, Mark Cave-Ayland, 2017/10/26
- Re: [Qemu-devel] [PATCHv4 09/13] lance: move TYPE_LANCE and SysBusPCNetState from lance.c to lance.h, Mark Cave-Ayland, 2017/10/30
- Re: [Qemu-devel] [PATCHv4 09/13] lance: move TYPE_LANCE and SysBusPCNetState from lance.c to lance.h, Peter Maydell, 2017/10/30
- Re: [Qemu-devel] [PATCHv4 09/13] lance: move TYPE_LANCE and SysBusPCNetState from lance.c to lance.h, Mark Cave-Ayland, 2017/10/30
- Re: [Qemu-devel] [PATCHv4 09/13] lance: move TYPE_LANCE and SysBusPCNetState from lance.c to lance.h, Philippe Mathieu-Daudé, 2017/10/30
- Re: [Qemu-devel] [PATCHv4 09/13] lance: move TYPE_LANCE and SysBusPCNetState from lance.c to lance.h, Mark Cave-Ayland, 2017/10/30
Re: [Qemu-devel] [PATCHv4 09/13] lance: move TYPE_LANCE and SysBusPCNetState from lance.c to lance.h, Philippe Mathieu-Daudé, 2017/10/27
[Qemu-devel] [PATCHv4 06/13] sparc32_dma: use object link instead of qdev property to pass IOMMU reference, Mark Cave-Ayland, 2017/10/25