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 <ag...@suse.de>
>>
>> ---
>>
>> 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("/pcie@%" 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
