[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/4] arm: Add PCIe host bridge in virt machine
From: |
Alexander Graf |
Subject: |
Re: [Qemu-devel] [PATCH 3/4] arm: Add PCIe host bridge in virt machine |
Date: |
Mon, 12 Jan 2015 17:36:20 +0100 |
User-agent: |
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 |
On 12.01.15 17:20, Claudio Fontana wrote:
> Just adding a nit here below:
>
> On 06.01.2015 17:03, Alexander Graf wrote:
>> Now that we have a working "generic" PCIe host bridge driver, we can plug
>> it into ARMs virt machine to always have PCIe available to normal ARM VMs.
>>
>> I've successfully managed to expose a Bochs VGA device, XHCI and an e1000
>> into an AArch64 VM with this and they all lived happily ever after.
>>
>> Signed-off-by: Alexander Graf <address@hidden>
>>
>> ---
>>
>> Linux 3.19 only supports the generic PCIe host bridge driver for 32bit ARM
>> systems. If you want to use it with AArch64 guests, please apply the
>> following
>> patch or wait until upstream cleaned up the code properly:
>>
>> http://csgraf.de/agraf/pci/pci-3.19.patch
>> ---
[...]
>> + nodename = g_strdup_printf("/address@hidden" PRIx64, base);
>> + qemu_fdt_add_subnode(vbi->fdt, nodename);
>> + qemu_fdt_setprop_string(vbi->fdt, nodename,
>> + "compatible", "pci-host-ecam-generic");
>> + qemu_fdt_setprop_string(vbi->fdt, nodename, "device_type", "pci");
>> + qemu_fdt_setprop_cell(vbi->fdt, nodename, "#address-cells", 3);
>> + qemu_fdt_setprop_cell(vbi->fdt, nodename, "#size-cells", 2);
>> + qemu_fdt_setprop_cells(vbi->fdt, nodename, "bus-range", 0, 1);
>> +
>> + qemu_fdt_setprop_sized_cells(vbi->fdt, nodename, "reg",
>> + 2, base_ecam, 2, size_ecam);
>> + qemu_fdt_setprop_sized_cells(vbi->fdt, nodename, "ranges",
>> + 1, 0x01000000, 2, 0,
>> + 2, base_ioport, 2, size_ioport,
>> +
>> + 1, 0x02000000, 2, base_mmio,
>> + 2, base_mmio, 2, size_mmio);
>> +
>> + qemu_fdt_setprop_cell(vbi->fdt, nodename, "#interrupt-cells", 1);
>> + qemu_fdt_setprop_cells(vbi->fdt, nodename, "interrupt-map",
>> + 0, 0, 0, /* device */
>> + 0, /* PCI irq */
>> + gic_phandle, GIC_FDT_IRQ_TYPE_SPI, irq,
>> + GIC_FDT_IRQ_FLAGS_LEVEL_HI /* system irq */);
>
>
> nit: are there two extra spaces here? (alignment)
Yes, because the attribute spans 2 lines ;)
Alex
- Re: [Qemu-devel] [PATCH 3/4] arm: Add PCIe host bridge in virt machine, (continued)
- Re: [Qemu-devel] [PATCH 3/4] arm: Add PCIe host bridge in virt machine, Alexander Graf, 2015/01/07
- Re: [Qemu-devel] [PATCH 3/4] arm: Add PCIe host bridge in virt machine, Claudio Fontana, 2015/01/08
- Re: [Qemu-devel] [PATCH 3/4] arm: Add PCIe host bridge in virt machine, Alexander Graf, 2015/01/08
- Re: [Qemu-devel] [PATCH 3/4] arm: Add PCIe host bridge in virt machine, Claudio Fontana, 2015/01/08
- Re: [Qemu-devel] [PATCH 3/4] arm: Add PCIe host bridge in virt machine, Claudio Fontana, 2015/01/12
- Re: [Qemu-devel] [PATCH 3/4] arm: Add PCIe host bridge in virt machine, Alexander Graf, 2015/01/12
- Re: [Qemu-devel] [PATCH 3/4] arm: Add PCIe host bridge in virt machine, alvise rigo, 2015/01/08
- Re: [Qemu-devel] [PATCH 3/4] arm: Add PCIe host bridge in virt machine, Peter Maydell, 2015/01/08
- Re: [Qemu-devel] [PATCH 3/4] arm: Add PCIe host bridge in virt machine, Claudio Fontana, 2015/01/08
Re: [Qemu-devel] [PATCH 3/4] arm: Add PCIe host bridge in virt machine, Claudio Fontana, 2015/01/12
- Re: [Qemu-devel] [PATCH 3/4] arm: Add PCIe host bridge in virt machine,
Alexander Graf <=
Re: [Qemu-devel] [PATCH 3/4] arm: Add PCIe host bridge in virt machine, alvise rigo, 2015/01/12
[Qemu-devel] [PATCH 1/4] pci: Split pcie_host_mmcfg_map(), Alexander Graf, 2015/01/06
[Qemu-devel] [PATCH 2/4] pci: Add generic PCIe host bridge, Alexander Graf, 2015/01/06