[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 3/8] hw/arm/virt: Use "msi-map" devicetree property for PC
From: |
Eric Auger |
Subject: |
Re: [PATCH v2 3/8] hw/arm/virt: Use "msi-map" devicetree property for PCI |
Date: |
Tue, 27 Sep 2022 14:48:10 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 |
Hi Jean,
On 9/27/22 12:03, Jean-Philippe Brucker wrote:
> The "msi-parent" property can be used on the PCI node when MSIs do not
> contain sideband data (device IDs) [1]. In QEMU, MSI transactions
> contain the requester ID, so the PCI node should use the "msi-map"
> property instead of "msi-parent". In our case the property describes an
> identity map between requester ID and sideband data.
>
> This fixes a warning when passing the DTB generated by QEMU to dtc,
> following a recent change to the GICv3 node:
>
> Warning (msi_parent_property): /pcie@10000000:msi-parent: property size (4)
> too small for cell size 1
>
> [1] linux/Documentation/devicetree/bindings/pci/pci-msi.txt
>
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
Effectively matches example 1 in [1]
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Eric
> ---
> hw/arm/virt.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/arm/virt.c b/hw/arm/virt.c
> index ed6f1ac41e..8605f5058a 100644
> --- a/hw/arm/virt.c
> +++ b/hw/arm/virt.c
> @@ -1489,8 +1489,8 @@ static void create_pcie(VirtMachineState *vms)
> qemu_fdt_setprop(ms->fdt, nodename, "dma-coherent", NULL, 0);
>
> if (vms->msi_phandle) {
> - qemu_fdt_setprop_cells(ms->fdt, nodename, "msi-parent",
> - vms->msi_phandle);
> + qemu_fdt_setprop_cells(ms->fdt, nodename, "msi-map",
> + 0, vms->msi_phandle, 0, 0x10000);
> }
>
> qemu_fdt_setprop_sized_cells(ms->fdt, nodename, "reg",
- [PATCH v2 0/8] hw/arm/virt: Fix dt-schema warnings, Jean-Philippe Brucker, 2022/09/27
- [PATCH v2 5/8] hw/arm/virt: Fix devicetree warnings about the GPIO node, Jean-Philippe Brucker, 2022/09/27
- [PATCH v2 6/8] hw/arm/virt: Fix devicetree warning about the SMMU node, Jean-Philippe Brucker, 2022/09/27
- [PATCH v2 3/8] hw/arm/virt: Use "msi-map" devicetree property for PCI, Jean-Philippe Brucker, 2022/09/27
- [PATCH v2 2/8] hw/arm/virt: Fix devicetree warning about the GIC node, Jean-Philippe Brucker, 2022/09/27
- [PATCH v2 4/8] hw/arm/virt: Fix devicetree warning about the gpio-key node, Jean-Philippe Brucker, 2022/09/27
- [PATCH v2 1/8] hw/arm/virt: Fix devicetree warning about the root node, Jean-Philippe Brucker, 2022/09/27
- [PATCH v2 7/8] hw/arm/virt: Fix devicetree warnings about the virtio-iommu node, Jean-Philippe Brucker, 2022/09/27